Skip to content

OWLS-86552 - Multiple pod restarts during rolling update fix. #2109

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

Merged
merged 1 commit into from
Dec 17, 2020

Conversation

ankedia
Copy link
Member

@ankedia ankedia commented Dec 17, 2020

Fix for multiple pod restarts during rolling update. In a domain with multiple clusters, previously it was launching multiple roll commands (one for each cluster) during rolling update. With this change, the roll command gets launched only once after all managed servers are started. This change executes the ManagedServerUpAfter step after the servers in all the clusters get started instead of executing it after servers start in each cluster .

…agedServerUpAfterStep is executed after servers in all clusters started.
@rjeberhard rjeberhard merged commit 46a67d7 into release/3.1.1 Dec 17, 2020
rjeberhard added a commit that referenced this pull request Dec 17, 2020
* Prepare for 3.1.1

* OWLS-86552 - Fix for multiple pod restarts during rolling update. ManagedServerUpAfterStep is executed after servers in all clusters started. (#2109)

* Add PR reference

Co-authored-by: Anil Kedia <37935279+ankedia@users.noreply.github.com>
@ankedia ankedia deleted the owls-86552-multi-pod-restarts branch December 17, 2020 20:45
rjeberhard added a commit that referenced this pull request Jan 22, 2021
* Prepare for 3.1.1

* OWLS-86552 - Fix for multiple pod restarts during rolling update. ManagedServerUpAfterStep is executed after servers in all clusters started. (#2109)

* Add PR reference

* patch annotation with slashes fix - from PR 2089

* Update versions and release notes

* Missing files

* Restore 3.1.1 chart

* Clarify Docker Hub, Docker Store and remove unnecessary references to Docker

* Build schema

* Fix broken link

* Adjust wording

* Review comments

Co-authored-by: Anil Kedia <37935279+ankedia@users.noreply.github.com>
Co-authored-by: anthony_lai <anthony.lai@oracle.com>
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.

3 participants