Skip to content

[K8S-537] bind operator and operator chart versions#599

Merged
chrisseto merged 1 commit intomainfrom
chris/p/operator-chart-operator
Apr 4, 2025
Merged

[K8S-537] bind operator and operator chart versions#599
chrisseto merged 1 commit intomainfrom
chris/p/operator-chart-operator

Conversation

@chrisseto
Copy link
Contributor

Prior to this commit the operator helm chart was managed as a standalone module/project. Doing so resulted in needless confusion and compatibility issues.

This commit moves charts/operator to operator/chart and combines the go modules into one, effectively making the release and development process of the operator and its chart one in the same.

@chrisseto
Copy link
Contributor Author

Ah, I should also mention the RBAC config of the operator was impressively out of date. I also updated as part of the move as I changed our tests to point at the local kustomize configurations instead of a static version.

@chrisseto chrisseto force-pushed the chris/p/operator-chart-operator branch 4 times, most recently from 8e7a097 to b55a397 Compare March 28, 2025 17:57
@RafalKorepta
Copy link
Contributor

Currently I see problems with RBAC

deploying *v1.Role: "rp-connectors-sidecar-controllers": roles.rbac.authorization.k8s.io "rp-connectors-sidecar-controllers" is forbidden: 
user "system:serviceaccount:redpanda:redpanda-operator" (groups=["system:serviceaccounts" "system:serviceaccounts:redpanda" "system:authenticated"]) 
is attempting to grant RBAC permissions not currently held:
{APIGroups:[""], Resources:["persistentvolumeclaims"], Verbs:["delete" "patch" "update" "watch"]}
{APIGroups:["apps"], Resources:["statefulsets/status"], Verbs:["patch" "update"]}

Copy link
Contributor

@RafalKorepta RafalKorepta left a comment

Choose a reason for hiding this comment

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

I found one Role didn't have RoleBinding. After my flight I will push fix

Prior to this commit the operator helm chart was managed as a standalone
module/project. Doing so resulted in needless confusion and compatibility issues.

This commit moves `charts/operator` to `operator/chart` and combines the go
modules into one, effectively making the release and development process of the
operator and its chart one in the same.
@chrisseto chrisseto force-pushed the chris/p/operator-chart-operator branch from d5e3574 to 931cd12 Compare April 3, 2025 18:24
@chrisseto
Copy link
Contributor Author

I found one Role didn't have RoleBinding. After my flight I will push fix

I fixed up the role bindings. Acceptance and integration tests are passing locally for me.

@chrisseto chrisseto enabled auto-merge (rebase) April 3, 2025 21:59
@chrisseto chrisseto merged commit 4ac62a0 into main Apr 4, 2025
12 checks passed
@chrisseto chrisseto deleted the chris/p/operator-chart-operator branch April 4, 2025 13:23
@chrisseto
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
release/v2.4.x
release/v2.3.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants