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

K8SPXC-926: increase MaxConcurrentReconciles #1131

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pooknull
Copy link
Contributor

@pooknull pooknull commented Apr 15, 2022

K8SPXC-926 Powered by Pull Request Badge

https://jira.percona.com/browse/K8SPXC-926

If we use an operator for two or more clusters, and one of them is stuck, we can't update the other clusters. That's why we need concurrent reconciles

@pull-request-size pull-request-size bot added size/XS 0-9 lines size/M 30-99 lines and removed size/XS 0-9 lines labels Apr 15, 2022
@hors hors requested a review from egegunes April 19, 2022 17:35
@@ -115,7 +137,7 @@ type ReconcilePerconaXtraDBCluster struct {
}

func (r *ReconcilePerconaXtraDBCluster) logger(name, namespace string) logr.Logger {
return log.NewDelegatingLogger(r.log).WithName("perconaxtradbcluster").
return r.log.WithName("perconaxtradbcluster").
Copy link
Contributor

Choose a reason for hiding this comment

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

what was the reason behind these changes?

https://jira.percona.com/browse/K8SPXC-926

If we use an operator for two or more clusters, and one of them is stuck, we can't update the other clusters. That's why we need concurrent reconciles
@pooknull pooknull force-pushed the dev/K8SPXC-926-concurrent-reconciles branch from 685eb60 to d64e70d Compare April 20, 2023 10:16
@pull-request-size pull-request-size bot added size/XXL 1000+ lines and removed size/M 30-99 lines labels Apr 20, 2023
@JNKPercona
Copy link
Collaborator

Test name Status
affinity-8-0 passed
auto-tuning-8-0 passed
cross-site-8-0 passed
demand-backup-cloud-8-0 passed
demand-backup-encrypted-with-tls-8-0 passed
demand-backup-8-0 passed
haproxy-5-7 passed
haproxy-8-0 failure
ignore-labels-annotations-8-0 passed
init-deploy-5-7 passed
init-deploy-8-0 passed
limits-8-0 passed
monitoring-2-0-8-0 passed
one-pod-5-7 passed
one-pod-8-0 passed
pitr-8-0 passed
proxy-protocol-8-0 passed
proxysql-sidecar-res-limits-8-0 passed
recreate-8-0 passed
restore-to-encrypted-cluster-8-0 passed
scaling-proxysql-8-0 passed
scaling-8-0 passed
scheduled-backup-5-7 passed
scheduled-backup-8-0 passed
security-context-8-0 passed
smart-update1-8-0 passed
smart-update2-8-0 passed
storage-8-0 passed
tls-issue-cert-manager-ref-8-0 passed
tls-issue-cert-manager-8-0 passed
tls-issue-self-8-0 passed
upgrade-consistency-8-0 passed
upgrade-haproxy-8-0 failure
upgrade-proxysql-8-0 failure
users-5-7 passed
users-8-0 passed
validation-hook-8-0 passed
We run 37 out of 37

commit: d64e70d
image: perconalab/percona-xtradb-cluster-operator:PR-1131-d64e70d4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XXL 1000+ lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants