Skip to content

During upgrade plan do not stop the pe-postgresql service on the Primary Master #55

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
mristok opened this issue Jan 23, 2020 · 1 comment

Comments

@mristok
Copy link

mristok commented Jan 23, 2020

Describe the Bug

Currently the peadm::upgrade plan stops the 'pe-postgresql' service on the Primary Master before executing the puppet-enterprise-installer. However, during the upgrade one of the first steps is to backup the Postgresql instance which requires that the service is running.

# Shut down pe-* services on the master. Only shutting down the ones
# that have failover pairs on the master replica.
['pe-console-services', 'pe-nginx', 'pe-puppetserver', 'pe-puppetdb', 'pe-postgresql'].each |$service| {
  run_task('service', $master_target,
    action => 'stop',
    name   => $service,
  )
}

Expected Behavior

Remove 'pe-postgresql' from the list of services that are being stopped before the upgrade is initiated.

Steps to Reproduce

Steps to reproduce the behavior:

  1. Start with a deployed instance of PE, running at >=2019.0
  2. Execute upgrade plan
  3. Failure during puppet-enterprise-installer task, due to pe-postgresql service being stopped
run_task('peadm::pe_install', $master_target,
  tarball => $upload_tarball_path,
)

Environment

  • Version = 2019.0
  • Platform Ubuntu 18.04

Additional Context

@mristok mristok added the bug label Jan 23, 2020
@reidmv
Copy link
Contributor

reidmv commented Mar 24, 2020

I believe this was resolved by #56.

@reidmv reidmv closed this as completed Mar 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants