Kuadrant / multicluster-gateway-controller

multi-cluster gateway controller, manages multi-cluster gateways based on gateway api and policy attachment
Apache License 2.0
10 stars 23 forks source link

Suggestion for change to kustomize setup #662

Closed mikenairn closed 10 months ago

mikenairn commented 10 months ago

Rename everything in config/policy-controller to be consistent with normal kubebuilder/operatorsdk controller naming conventions.

Update the default mgc overlay to allow specifying a different nameprefix for mgc (gateway) and policy components.

./bin/kustomize build config/default/ | yq .metadata.name
multicluster-gateway-controller-system
dnshealthcheckprobes.kuadrant.io
dnspolicies.kuadrant.io
dnsrecords.kuadrant.io
managedzones.kuadrant.io
tlspolicies.kuadrant.io
mgc-add-on-manager
mgc-controller-manager
mgc-policy-controller-manager
mgc-leader-election-role
mgc-policy-leader-election-role
mgc-kuadrant-addon
mgc-manager-role
mgc-metrics-reader
mgc-policy-dnsrecord-editor-role
mgc-policy-dnsrecord-viewer-role
mgc-policy-manager-role
mgc-proxy-role
mgc-leader-election-rolebinding
mgc-policy-leader-election-rolebinding
mgc-kuadrant-addon
mgc-manager-rolebinding
mgc-policy-manager-rolebinding
mgc-proxy-rolebinding
mgc-controller-manager-metrics-service
mgc-controller-manager
mgc-kuadrant-add-on-manager
mgc-policy-controller-manager
mgc-kuadrant-addon
mgc-policy-glbc-ca
openshift-ci[bot] commented 10 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mikenairn

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/Kuadrant/multicluster-gateway-controller/blob/separate-controller-deploy/OWNERS)~~ [mikenairn] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment