-
Notifications
You must be signed in to change notification settings - Fork 84
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
Fix warnings in kustomization file #2073
base: main
Are you sure you want to change the base?
Fix warnings in kustomization file #2073
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: priyanshikhetwani The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi @priyanshikhetwani. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
✅ Deploy Preview for kubernetes-sigs-cluster-api-ibmcloud ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
/ok-to-test |
dcbdb46
to
61f0361
Compare
61f0361
to
506478e
Compare
506478e
to
b591a65
Compare
What this PR does / why we need it: Fix Warnings in Kustomization file
make release
Description:
Diff:
bases (deprecated): This field was used to reference other directories or resources (e.g., CRDs, namespaces, etc.) as "bases" from which resources would be pulled.
resources (recommended): The resources field is now used to include external files or resources directly. Kustomize now uses resources instead of bases for the same purpose.
Migration Approach:
Replacing the bases field with resources, using the same paths.
Before:
After:
Diff:
commonLabels (deprecated): The commonLabels field was used to apply common labels to all resources within the kustomization.
labels (recommended): The labels field now serves the same purpose as commonLabels, but with more flexibility and consistent behavior.
Migration Approach:
Replacing the commonLabels field with labels, ensuring that the structure matches the new format for labels.
Before:
After:
Diff:
patchesStrategicMerge (deprecated): This field was used to apply strategic merge patches to the resources defined in the Kustomization.
patches (recommended): The patches field is now used to apply patches (both strategic and JSON patch) to resources. The main difference is that patches supports both strategicMerge and jsonPatch types.
Migration Approach:
Replacing the patchesStrategicMerge field with the patches field. The structure of the patches will stay mostly the same, but now it supports a wider variety of patch types.
Before:
[patchesStrategicMerge:
After:
Diff:
vars (deprecated): The vars field was used to define variables that could be used for substitution throughout the Kustomization file.
replacements (recommended): The replacements field provides a more explicit way to define variable substitutions. It allows you to specify where the values are coming from (source) and where they should be applied (target).
Migration Approach:
For each variable defined in vars, create a replacements entry. This involves specifying the source field and the target field for the substitution.
Before:
After:
Test:
kustomize build . --stack-trace > sample.yaml
Which issue(s) this PR fixes:
Fixes #2061
Special notes for your reviewer:
/area provider/ibmcloud
Release note: