chore: update config format to fix deprecation warning#9
Open
larkro wants to merge 1 commit intoreclaim-the-stack:masterfrom
Open
chore: update config format to fix deprecation warning#9larkro wants to merge 1 commit intoreclaim-the-stack:masterfrom
larkro wants to merge 1 commit intoreclaim-the-stack:masterfrom
Conversation
Contributor
|
Thanks for the PR. Did you also confirm that the patch was correctly applied? Not sure when I'll have time to verify the stack but I'll definitely verify this at some point. |
Author
|
Thank you for the swift reply. No stress :-) $ k k get configmap -n argocd argocd-cm -o yaml apiVersion: v1
data:
application.instanceLabelKey: argocd.argoproj.io/instance
kustomize.buildOptions: --load-restrictor LoadRestrictionsNone --enable-helm
resource.customizations.health.argoproj.io_Application: |
hs = {}
hs.status = "Progressing"
hs.message = ""
if obj.status ~= nil then
if obj.status.health ~= nil then
hs.status = obj.status.health.status
if obj.status.health.message ~= nil then
hs.message = obj.status.health.message
end
end
end
return hs
resource.customizations.ignoreDifferences.admissionregistration.k8s.io_MutatingWebhookConfiguration: |
jqPathExpressions:
- ".webhooks[]?.clientConfig.caBundle"
- ".webhooks[]?.failurePolicy"
resource.customizations.ignoreDifferences.admissionregistration.k8s.io_ValidatingWebhookConfiguration: |
jqPathExpressions:
- ".webhooks[]?.clientConfig.caBundle"
- ".webhooks[]?.failurePolicy"
resource.customizations.ignoreDifferences.apiextensions.k8s.io_CustomResourceDefinition: |-
jsonPointers:
- /spec/names/shortNames
kind: ConfigMap
metadata:
creationTimestamp: "2024-09-22T07:59:52Z"
labels:
app.kubernetes.io/name: argocd-cm
app.kubernetes.io/part-of: argocd
name: argocd-cm
namespace: argocd
resourceVersion: "9805"
uid: bb96a3f0-29ac-473e-88ad-7c885b25e6ca |
bram-pkg
reviewed
Sep 26, 2024
Comment on lines
+8
to
+9
| apiVersion: kustomize.config.k8s.io/v1beta1 | ||
| kind: Kustomization |
There was a problem hiding this comment.
Suggested change
| apiVersion: kustomize.config.k8s.io/v1beta1 | |
| kind: Kustomization |
bram-pkg
reviewed
Sep 26, 2024
| @@ -1,10 +1,11 @@ | |||
| namespace: argocd | |||
There was a problem hiding this comment.
Suggested change
| namespace: argocd | |
| apiVersion: kustomize.config.k8s.io/v1beta1 | |
| kind: Kustomization | |
| namespace: argocd |
bram-pkg
reviewed
Sep 26, 2024
Comment on lines
6
to
7
| # NOTE: patchesStrategicMerge is deprecated but still works. Converting to the suggested | ||
| # patches: { file: ./config.yaml } syntax results in a validation error. |
There was a problem hiding this comment.
Suggested change
| # NOTE: patchesStrategicMerge is deprecated but still works. Converting to the suggested | |
| # patches: { file: ./config.yaml } syntax results in a validation error. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
First heard out about reclaim-the-stack at BalticRuby. Thank you for making this.
Saw the patchesStrategicMerge deprecation warning and did:
$ cd platform/argocd/
$ kustomize edit fix
$ cd -
Read the note/comment, could the noted validation error be because of file: vs path: ?
I tried to edit using file: as in the note, and got error: invalid Kustomization, but the automated kustomize edit fix is without warnings.
Tested creating/deleting resources:
$ kubectl create -k platform/argocd
$ kubectl delete -k platform/argocd
No warning for me anymore using:
$ kustomize version
v5.4.3
$ kubectl version
Client Version: v1.31.1
Kustomize Version: v5.4.2
Server Version: v1.31.1