-
Notifications
You must be signed in to change notification settings - Fork 308
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
check timestamps before waiting for agent rollout #5698
check timestamps before waiting for agent rollout #5698
Conversation
should address: #5690 |
@starbops The fix avoid the failure of wait_rollout and https://github.com/harvester/harvester/blob/8958ebee30387c8591eeb17ddf60e178517e9de3/package/upgrade/upgrade_manifests.sh#L1310C1-L1311C1 will be called. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The upgrade went well on a 2-node cluster. LGTM, thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
@Mergifyio backport v1.2 v1.3 |
🛑 Command
|
@Mergifyio backport v1.2 v1.3 |
✅ Backports have been created
|
(cherry picked from commit cc1c2ad) # Conflicts: # package/upgrade/upgrade_manifests.sh
(cherry picked from commit cc1c2ad)
(cherry picked from commit cc1c2ad) fixed rebase conflicts
(cherry picked from commit cc1c2ad)
(cherry picked from commit cc1c2ad) fixed rebase conflicts
(cherry picked from commit cc1c2ad) fixed rebase conflicts
(cherry picked from commit cc1c2ad) fixed rebase conflicts
(cherry picked from commit cc1c2ad)
(cherry picked from commit cc1c2ad)
IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:
PR attempts to address random upgrade fails where the upgrade fails due to the following error:
This is caused by a delay in rollout of fleet-agent, where the rollout happens when the check for fleet-agent rollout status is running, and deployment gets recreated.
Solution:
The fix attempts to use original fleet-agent creation timestamp before fleet cluster object is patched, and ensures new fleet-agent deployment has a newer creation timestamp before rollout status is checked.
Related Issue:
Test plan: