-
Notifications
You must be signed in to change notification settings - Fork 179
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
[Flake] Scylla Manager integration [It] should register cluster and sync repair tasks #1694
Comments
Since I have the questionable pleasure of working on the manager controller now, I volunteer. /assign |
@rzetelskik have you managed to take a look? Do we know what needs to be fixed / does this depend on the manager annotation support? |
It's generally caused by an entirely broken reconciliation logic. See e.g. #1752. Most of the recent flakes were due to the deletions not being propagated to status, which is not surprising if you look into the code - reconciliation loop iterates over the state taken from the manager and schedules actions based on the differences between it and the spec/status. So if it successfully deletes a task in manager, but hits a conflict when updating the status, well, that's just that, the task is not going to be removed from status any time later, because it's now gone from manager. #1850 should address most of the flakiness. It's now waiting for #1851, #1853.
Maybe, but the above needs to be fixed first anyway. I was told to touch as little of the existing logic as possible, so I didn't touch anything related to cluster creation/deletion logic e.g., or updating the tasks for that matter. |
no worries and thanks. (I primarily asked to see have a list of PR to prioritize.) |
Link to the job that flaked.
https://prow.scylla-operator.scylladb.com/view/gs/scylla-operator-prow/pr-logs/pull/scylladb_scylla-operator/1691/pull-scylla-operator-master-e2e-gke-parallel-clusterip/1747926847733108736
Snippet of what failed.
The text was updated successfully, but these errors were encountered: