Skip to content

Commit

Permalink
fix: resolve Kustomize deprecations (rackerlabs#584)
Browse files Browse the repository at this point in the history
Kustomize has deprecated the use of a couple feature flags and we need to
make adjustments to ensure that we're maintaining the environment in
expected ways.

* remove the CLI option from `kustomize.sh` to disable automatic reordering
* add the `sortOptions` keyword to the base kustomization files to ensure
  resource sorting is handled as fifo.
* replace the `bases` keyword with the `resources` keyword

Signed-off-by: Kevin Carter <[email protected]>
  • Loading branch information
cloudnull authored and mnaghavi committed Dec 9, 2024
1 parent a1b9f22 commit 51d9ebe
Show file tree
Hide file tree
Showing 69 changed files with 155 additions and 19 deletions.
2 changes: 2 additions & 0 deletions base-kustomize/argocd/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- namespace.yaml

Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/backups/etcd/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- etcd-backup.yaml
4 changes: 3 additions & 1 deletion base-kustomize/barbican/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
bases:
sortOptions:
order: fifo
resources:
- ../base
2 changes: 2 additions & 0 deletions base-kustomize/barbican/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- barbican-mariadb-database.yaml
- barbican-rabbitmq-queue.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/ceilometer/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/ceilometer/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- all.yaml
- hpa-ceilometer-notification.yaml
4 changes: 3 additions & 1 deletion base-kustomize/cinder/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/cinder/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- cinder-mariadb-database.yaml
- cinder-rabbitmq-queue.yaml
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/cinder/netapp/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ images:
newName: ghcr.io/rackerlabs/genestack/cinder-volume-rxt
newTag: 2024.1-ubuntu_jammy

sortOptions:
order: fifo
resources:
- configmap-etc.yaml
- deploy-volume-netapp.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/designate/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/designate/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- designate-mariadb-database.yaml
- designate-rabbitmq-queue.yaml
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/envoyproxy-gateway/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- './namespace.yaml'
- './gatewayclass.yaml'
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/gateway/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- all.yaml
patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/gateway/envoyproxy/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- './gateway.yaml' # namespace: envoy-gateway-system (common gateway)
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- internal-gateway-api.yaml # namespace: nginx-gateway (common gateway)
- internal-gateway-issuer.yaml #namespace: nginx-gateway
4 changes: 3 additions & 1 deletion base-kustomize/glance/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/glance/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- glance-mariadb-database.yaml
- glance-rabbitmq-queue.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/gnocchi/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/gnocchi/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- gnocchi-temp-keyring.yaml
- all.yaml
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/grafana/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- ns-grafana.yaml
- azure-client-secret.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/heat/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/heat/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- heat-mariadb-database.yaml
- heat-rabbitmq-queue.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/horizon/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/horizon/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- horizon-mariadb-database.yaml
- all.yaml
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/k8s-dashboard/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- dashboard-rbac-default.yaml
4 changes: 3 additions & 1 deletion base-kustomize/keystone/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/keystone/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- keystone-mariadb-database.yaml
- keystone-rabbitmq-queue.yaml
Expand Down
2 changes: 1 addition & 1 deletion base-kustomize/kustomize.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,5 @@ set -e
KUSTOMIZE_DIR=${1:-$GENESTACK_KUSTOMIZE_ARG}
pushd "$(dirname "${BASH_SOURCE[0]}")" &>/dev/null
cat <&0 > "${KUSTOMIZE_DIR}"/../base/all.yaml
kubectl kustomize --reorder='none' "${KUSTOMIZE_DIR}"
kubectl kustomize "${KUSTOMIZE_DIR}"
popd &>/dev/null
4 changes: 3 additions & 1 deletion base-kustomize/magnum/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
bases:
sortOptions:
order: fifo
resources:
- ../base
2 changes: 2 additions & 0 deletions base-kustomize/magnum/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- magnum-mariadb-database.yaml
- magnum-rabbitmq-queue.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/mariadb-cluster/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- "../base"

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/mariadb-cluster/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- mariadb-configmap.yaml
- mariadb-replication.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/mariadb-cluster/galera/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- "../base"

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/memcached/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

sortOptions:
order: fifo
resources:
- ../base

Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/memcached/base-monitoring/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

sortOptions:
order: fifo
resources:
- ../base

Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/memcached/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

sortOptions:
order: fifo
resources:
- all.yaml

Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/neutron/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/neutron/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- neutron-mariadb-database.yaml
- neutron-rabbitmq-queue.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/nova/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/nova/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- nova-mariadb-database.yaml
- nova-rabbitmq-queue.yaml
Expand Down
4 changes: 3 additions & 1 deletion base-kustomize/octavia/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/octavia/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- octavia-mariadb-database.yaml
- octavia-rabbitmq-queue.yaml
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/openstack/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- ns-openstack.yaml
- issuer-kube-system-selfsigned.yaml
2 changes: 2 additions & 0 deletions base-kustomize/ovn-backup/base/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,7 @@ configMapGenerator:
- name: ovn-backup-config
envs:
- ovn-backup.config
sortOptions:
order: fifo
resources:
- ovn-backup.yaml
2 changes: 2 additions & 0 deletions base-kustomize/ovn/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- ovn-setup.yaml
4 changes: 3 additions & 1 deletion base-kustomize/placement/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/placement/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- placement-mariadb-database.yaml
- all.yaml
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/postgres-cluster/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- postgres-cluster.yaml
2 changes: 2 additions & 0 deletions base-kustomize/postgres-cluster/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- postgres-cluster.yaml
2 changes: 2 additions & 0 deletions base-kustomize/postgres-operator/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- ns-postgres.yaml

Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/prometheus-mysql-exporter/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- monitoring_user_create.yaml
- monitoring_user_grant.yaml
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/prometheus/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- all.yaml
4 changes: 3 additions & 1 deletion base-kustomize/rabbitmq-cluster/aio/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
bases:
sortOptions:
order: fifo
resources:
- ../base

patches:
Expand Down
2 changes: 2 additions & 0 deletions base-kustomize/rabbitmq-cluster/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- rabbitmq-cluster.yaml
2 changes: 2 additions & 0 deletions base-kustomize/rabbitmq-operator/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- cluster-operator.yaml
2 changes: 2 additions & 0 deletions base-kustomize/rabbitmq-topology-operator/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
sortOptions:
order: fifo
resources:
- messaging-topology-operator-with-certmanager.yaml
2 changes: 2 additions & 0 deletions base-kustomize/rook-cluster-external-pvc/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- cluster-on-pvc.yaml
- toolbox.yaml
2 changes: 2 additions & 0 deletions base-kustomize/rook-cluster/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- rook-cluster.yaml
- toolbox.yaml
2 changes: 2 additions & 0 deletions base-kustomize/rook-defaults-external-pvc/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- filesystem.yaml
- storageclass-cephfs.yaml
2 changes: 2 additions & 0 deletions base-kustomize/rook-defaults/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
sortOptions:
order: fifo
resources:
- filesystem.yaml
- storageclass-cephfs.yaml
Expand Down
Loading

0 comments on commit 51d9ebe

Please sign in to comment.