Skip to content
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

Fixes infinite job generation after ForceSyncGeneration changed #2477

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

0xavi0
Copy link
Contributor

@0xavi0 0xavi0 commented May 31, 2024

gitrepos Status.UpdateGeneration value was changed but not updated, which made the controller to enter in an infinite loop regenerating the job.

Refers to: #2474

@0xavi0 0xavi0 self-assigned this May 31, 2024
@0xavi0 0xavi0 requested a review from a team as a code owner May 31, 2024 15:13
@0xavi0 0xavi0 added this to the v2.9.0 milestone May 31, 2024
`gitrepo`s `Status.UpdateGeneration` value was changed but not updated, which made
the controller to enter in an infinite loop regenerating the job.

Refers to: rancher#2474

Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
@0xavi0 0xavi0 force-pushed the 2474-infinite-job-regeneration branch from bb9fc77 to eb6688a Compare May 31, 2024 15:15
Copy link
Contributor

@weyfonk weyfonk left a comment

Choose a reason for hiding this comment

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

Nice one! LGTM

@manno manno merged commit 290fc61 into rancher:main Jun 4, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants