We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
kudo init --dry-run -o yaml has some serialization issues.
kudo init --dry-run -o yaml
What happened: Three things I encountered
helm templating not happy with {{ .Params.COUNT }} in comment, best remove
{{ .Params.COUNT }}
file for example, if `name: COUNT` then using the variable in a spec like: spec: replicas: {{ .Params.COUNT }}'
1st character lower case serviceAccount, should be ServiceAccount
serviceAccount
ServiceAccount
apiVersion: v1 kind: serviceAccount metadata: creationTimestamp: null labels: app: kudo-manager name: kudo-manager namespace: kudo-system
1st character lower case namespace, should be Namespace
namespace
Namespace
kind: namespace metadata: creationTimestamp: null labels: app: kudo-manager controller-tools.k8s.io: "1.0" name: kudo-system spec: {} status: {}
What you expected to happen: see before
How to reproduce it (as minimally and precisely as possible): run kubectl kudo init --dry-run -o yaml
kubectl kudo init --dry-run -o yaml
Anything else we need to know?:
Environment:
kubectl version
kubectl kudo version
uname -a
The text was updated successfully, but these errors were encountered:
I love you man!
Sorry, something went wrong.
@realmbgl I don't understand this comment...
ah.. the output of the CRD YAML has this as part of the name description... hmmm
could you help me understand how you are using this with helm?
did a simple helm smoke test using the following commands
kubectl kudo init --dry-run -o yaml > kudo/templates/kudo.yaml helm install -n kudo ./kudo
with the 3 things patched the install works
you ask and you shall receive!!
kensipe
Successfully merging a pull request may close this issue.
kudo init --dry-run -o yaml
has some serialization issues.What happened:
Three things I encountered
helm templating not happy with
{{ .Params.COUNT }}
in comment, best remove1st character lower case
serviceAccount
, should beServiceAccount
1st character lower case
namespace
, should beNamespace
What you expected to happen:
see before
How to reproduce it (as minimally and precisely as possible):
run
kubectl kudo init --dry-run -o yaml
Anything else we need to know?:
Environment:
kubectl version
):kubectl kudo version
): 0.6.0uname -a
):The text was updated successfully, but these errors were encountered: