Skip to content

Conversation

@jkatz
Copy link
Contributor

@jkatz jkatz commented Dec 9, 2020

Checklist:

  • Have you added an explanation of what your changes do and why you'd like them to be included?
  • Have you updated or added documentation for the change, as applicable?
  • Have you tested your changes on all related environments with successful results, as applicable?

Type of Changes:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

What is the current behavior? (link to any open issues here)

  • pgo df will fail on clusters created using pgo create cluster --restore-from
  • The cluster bootstrap Job persists even after it successfully runs

What is the new behavior (if this is a feature change)?

  • pgo df works on clusters with lingering bootstrap jobs
  • The cluster bootstrap Job is removed after it successfully completes

Other information:

Issue: [ch9919]
Fixes #2029

jkatz added 2 commits December 8, 2020 22:19
The bootstrap Pod, a remnaint of a cluster restore, gets caught
up in the `pgo df` search, but unfortunately this is not a valid
Pod. This exlcude this Pod from being considered.

Issue: [ch2029]
Issue: CrunchyData#2029
When the bootstrap Job completes successfully after a restore,
it contains information that ends up being consumed by other
parts of the Operator system, such as Patroni. As the logs
from the Job do not provide much, if any, helpful information
after a restore succeeds, it's best to have the Operator
eliminate the job.

As such, this changes the behavior so that the bootstrap Job
is removed.

As this has lead to some buggy behavior, this is being
considered as a bug fix, as regular operational work would
dictate that the Job is removed anyway.

Issue: [ch9919]
@jkatz jkatz requested a review from andrewlecuyer December 9, 2020 03:48
@jkatz jkatz merged commit 23c935d into CrunchyData:master Dec 9, 2020
@jkatz jkatz deleted the df-bootstrap branch December 9, 2020 17:08
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

Successfully merging this pull request may close these issues.

After a create cluster with --restore-from pgo df for that cluster gives an error..

3 participants