Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

EVEREST-458 Restart everest operator once a new version of operator i… #172

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

gen1us2k
Copy link
Contributor

…s installed

CHANGE DESCRIPTION

Problem:
EVEREST-458

Short explanation of the problem.

During the installation of a database engine operator, CLI does not ensure the deployment has been rolled out. Hence for MongoDB installation, since it's too long and the rollout is slower than others the everest operator gets restarted and after the restart, a psmdb operator is completely deployed.

Also, during the QA process and after a discussion we decided to restart the everest operator only if a new version of an operator was deployed

Solution:

Wait for the deployment rollout during the installation of an operator and restart the everest operator once it a new version of a database engine operator is deployed.

CHECKLIST

Jira

  • Is the Jira ticket created and referenced properly?

Tests

  • Is an Integration test/test case added for the new feature/change?
  • Are unit tests added where appropriate?

@gen1us2k gen1us2k marked this pull request as ready for review October 11, 2023 11:59
@gen1us2k gen1us2k requested a review from a user October 11, 2023 11:59
Copy link
Contributor

@oksana-grishchenko oksana-grishchenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So the "new version of a database engine operator" means a new type of operator and it has nothing to do with the versioning of an operator itself. Was a bit confused at first by the description.
Looks good, should work 👍

Copy link
Collaborator

@recharte recharte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The DoRolloutWait was an important piece, well done 💪

@gen1us2k gen1us2k merged commit cd623f3 into main Oct 12, 2023
8 checks passed
@gen1us2k gen1us2k deleted the EVEREST-458 branch October 12, 2023 09:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants