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-1047 - fix: percona operator image identification #146

Merged
merged 2 commits into from
Jun 30, 2022

Conversation

liogate
Copy link
Contributor

@liogate liogate commented Jun 21, 2022

Issue with percona cluster pxdb

The following line requires operator pods container name to equal "percona-xtradb-cluster-operator".

https://github.com/percona/percona-xtradb-cluster-operator/blob/9f096f007e2d53b74a4752c354c456a31c7fdd4f/pkg/controller/pxc/controller.go#L1401

Otherwise, you'll have this issue :

operator image not found","errorVerbose":"operator image not found
github.com/percona/percona-xtradb-cluster-operator/pkg/controller/pxc.operatorImageName
	/go/src/github.com/percona/percona-xtradb-cluster-operator/pkg/controller/pxc/controller.go:1405
github.com/percona/percona-xtradb-cluster-operator/pkg/controller/pxc.(*ReconcilePerconaXtraDBCluster).deploy
	/go/src/github.com/percona/percona-xtradb-cluster-operator/pkg/controller/pxc/controller.go:545
github.com/percona/percona-xtradb-cluster-operator/pkg/controller/pxc.(*ReconcilePerconaXtraDBCluster).Reconcile
	/go/src/github.com/percona/percona-xtradb-cluster-operator/pkg/controller/pxc/controller.go:307
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
	/go/src/github.com/percona/percona-xtradb-cluster-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:114
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	/go/src/github.com/percona/percona-xtradb-cluster-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:311
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	/go/src/github.com/percona/percona-xtradb-cluster-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	/go/src/github.com/percona/percona-xtradb-cluster-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:227
runtime.goexit
	/usr/local/go/src/runtime/asm_amd64.s:1571

@it-percona-cla
Copy link

it-percona-cla commented Jun 21, 2022

CLA assistant check
All committers have signed the CLA.

@tplavcic tplavcic changed the title fix: percona operator image identification K8SPXC-1047 - fix: percona operator image identification Jun 21, 2022
@tplavcic
Copy link
Member

Hi @liogate ! Thanks for the contribution! Could you please bump the chart version (it should fix the failed test).

@hors hors requested review from tplavcic and cap1984 June 21, 2022 11:32
@liogate
Copy link
Contributor Author

liogate commented Jun 27, 2022

Hi @tplavcic, bumped to version 1.11.1 ✅

@liogate
Copy link
Contributor Author

liogate commented Jun 30, 2022

Hello @tplavcic @nmarukovich @cap1984 can we merge this fix please ? 🙏

@tplavcic tplavcic merged commit 706e109 into percona:main Jun 30, 2022
@tplavcic
Copy link
Member

@liogate Thank you for contribution and sorry for waiting!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants