Upgrade to Operator SDK 1.41.1#384
Conversation
|
/retest |
1 similar comment
|
/retest |
stuggi
force-pushed
the
operator_sdk_1.41.1
branch
from
5631d72 to
c072849
Compare
|
/retest |
|
We need to update the local webhook script ( |
good catch, thanks! |
stuggi
force-pushed
the
operator_sdk_1.41.1
branch
from
c072849 to
85086d1
Compare
|
/retest |
stuggi
force-pushed
the
operator_sdk_1.41.1
branch
3 times, most recently
from
9dc47e7 to
8943e05
Compare
This commit upgrades the mariadb-operator to use Operator SDK version 1.41.1, with the following major changes: Project Structure: - Move pkg/ packages to internal/ following Go best practices - Move controllers/ to internal/controller/ per new operator-sdk layout - Replace main.go with cmd/main.go - Add controller and webhook test suites Configuration Updates: - Split certificate management into separate metrics and webhook certs - Add network policies for metrics and webhook traffic - Rename auth_proxy resources to metrics for clarity - Add admin RBAC roles for all CRD types - Update manager deployment configuration and patches - Modernize kustomization configurations Webhook Changes: - Add dedicated webhook implementation in internal/webhook/v1beta1/ - Remove old CRD webhook and CA injection patches - Update webhook manifests and service configuration Cleanup: - Remove obsolete config patches and overlays Dependencies: - Update go.mod and go.sum for operator-sdk 1.41.1 dependencies Jira: OSPRH-22041 Assisted-by: claude-4-sonnet Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
stuggi
force-pushed
the
operator_sdk_1.41.1
branch
from
8943e05 to
03bf512
Compare
| for mod in $(shell find . -name go.mod -exec dirname {} \;); do \ | ||
| pushd ./$$mod ; \ | ||
| if [ -f test/functional/suite_test.go ]; then \ | ||
| KUBEBUILDER_ASSETS="$(shell $(ENVTEST) -v debug --bin-dir $(LOCALBIN) use $(ENVTEST_K8S_VERSION) -p path)" $(GINKGO) --trace --cover --coverprofile cover.out --covermode=atomic --coverpkg=../../pkg/mariadb,../../controllers,../../api/v1beta1 ${PROC_CMD} $(GINKGO_ARGS) ./test/... || exit 1; \ |
There was a problem hiding this comment.
Do we need to update any of the paths here?
There was a problem hiding this comment.
The respective dirs that would be a problem (the ones that changed locations) have no tests in them. So I don't think these really matter. We should still clean it up, but we can do it in a follow-up.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abays, stuggi The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-merge-bot
Bot
merged commit f08eaf8
into
openstack-k8s-operators:main
2 participants
This commit upgrades the mariadb-operator to use Operator SDK version 1.41.1, with the following major changes:
Project Structure:
Configuration Updates:
Webhook Changes:
Cleanup:
Dependencies:
Jira: OSPRH-22041
Assisted-by: claude-4-sonnet
Depends-On: openstack-k8s-operators/openstack-operator#1683
Depends-On: openstack-k8s-operators/install_yamls#1110