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.

Sign up for GitHub

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

Jump to bottom

karmada: ignore sync errors since they are temporary #4091

Merged
merged 1 commit into from
Feb 13, 2024
Merged

karmada: ignore sync errors since they are temporary #4091

merged 1 commit into from
Feb 13, 2024

Conversation

grosser
Copy link
Contributor

@grosser grosser commented Feb 9, 2024 

[20:32:39]   Normal SyncSucceed: Successfully applied resource(kube-system/kube-service-watcher) to cluster eks-sandbox x3

[20:32:39]   Warning SyncFailed: Failed to create or update resource(kube-system/kube-service-watcher) in member cluster(eks-sandbox): Operation cannot be fulfilled on deployments.apps "kube-service-watcher": the object has been modified; please apply your changes to the latest version and try again x2

but after a few reconciles it's just fine, so need to ignore these

Risks

  • Low

@grosser grosser closed this Feb 12, 2024
@grosser grosser reopened this Feb 12, 2024
@grosser grosser marked this pull request as ready for review February 12, 2024 20:36
@grosser grosser requested a review from a team as a code owner February 12, 2024 20:36
a7i
a7i approved these changes Feb 13, 2024
View reviewed changes
Copy link
Contributor

@a7i a7i left a comment

Choose a reason for hiding this comment

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

🤞🏼

@grosser grosser merged commit caf3c54 into master Feb 13, 2024
17 of 18 checks passed
@grosser grosser deleted the grosser/karamada branch February 13, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@a7i a7i a7i approved these changes

@SerenaTiede-Zen SerenaTiede-Zen SerenaTiede-Zen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@grosser @a7i @SerenaTiede-Zen