Skip to content

Conversation

@mayankshah1607
Copy link
Member

@mayankshah1607 mayankshah1607 commented Nov 23, 2024

  • Earlier it was not possible to add more operator installations on existing DB namespaces. This is because the installplan hook ran only once, during installation (i.e, in the post-install step)
  • Since you'd have to run helm update .. to modify an existing helm release (to add more operator installations), this PR updates the chart hook to also run on post-upgrade
  • Adds an additional check to not handle the InstallPlans of installed Subscriptions
  • Also uses a more generic name for the chart hook
  • Revert the operatorgroup name - use the one originally used in Everest, otherwise upgrading on existing installation breaks

@mayankshah1607 mayankshah1607 marked this pull request as ready for review November 24, 2024 08:41
@mayankshah1607 mayankshah1607 requested a review from a team as a code owner November 24, 2024 08:41
Signed-off-by: Mayank Shah <[email protected]>
Signed-off-by: Mayank Shah <[email protected]>
@mayankshah1607 mayankshah1607 changed the title EVEREST-1511 | Allow adding operators to existing DB namespaces. EVEREST-1511 | Allow adding operators to existing DB namespaces Nov 24, 2024
@mayankshah1607 mayankshah1607 merged commit 4f40679 into main Nov 25, 2024
2 checks passed
@mayankshah1607 mayankshah1607 deleted the EVEREST-1511-update-dbnamespace branch November 25, 2024 10:15
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.

2 participants