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

Extend waiting time for updating Condition in installation CRD. #12696

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

cnvergence
Copy link
Contributor

@cnvergence cnvergence commented Nov 24, 2021

Description
Sometimes kyma-operator can't update the installation status, with the following error being reported: Operation cannot be fulfilled on installations.installer.kyma-project.io "kyma-installation": the object has been modified; please apply your changes to the latest version and try again.
This PR will extend the waiting time between steps and increase the number of steps to retry updating the CR.

Related issue(s)

@kyma-bot kyma-bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 24, 2021
@mjakobczyk mjakobczyk added area/installation Issues or PRs related to installation do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Nov 24, 2021
@kyma-bot kyma-bot added the lgtm Looks good to me! label Nov 24, 2021
@mjakobczyk mjakobczyk removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 25, 2021
@kyma-bot kyma-bot merged commit 56650d2 into kyma-project:release-1.25 Nov 25, 2021
@cnvergence cnvergence deleted the kyma-installer-fix branch November 25, 2021 08:30
@k15r k15r mentioned this pull request Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/installation Issues or PRs related to installation lgtm Looks good to me! size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants