-
Notifications
You must be signed in to change notification settings - Fork 210
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
deployments: Rework to be more idiomatic jsonnet structure #28
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sadly found one bug.
I like the idea of this dev.jsonnet :)
Otherwise LGTM!
@@ -3,6 +3,9 @@ data: | |||
conprof.yaml: InNjcmFwZV9jb25maWdzIjoKLSAiam9iX25hbWUiOiAiY29ucHJvZiIKICAia3ViZXJuZXRlc19zZF9jb25maWdzIjoKICAtICJuYW1lc3BhY2VzIjoKICAgICAgIm5hbWVzIjoKICAgICAgLSAiY29ucHJvZiIKICAgICJyb2xlIjogInBvZCIKICAicmVsYWJlbF9jb25maWdzIjoKICAtICJhY3Rpb24iOiAia2VlcCIKICAgICJyZWdleCI6ICJjb25wcm9mLioiCiAgICAic291cmNlX2xhYmVscyI6CiAgICAtICJfX21ldGFfa3ViZXJuZXRlc19wb2RfbmFtZSIKICAtICJzb3VyY2VfbGFiZWxzIjoKICAgIC0gIl9fbWV0YV9rdWJlcm5ldGVzX25hbWVzcGFjZSIKICAgICJ0YXJnZXRfbGFiZWwiOiAibmFtZXNwYWNlIgogIC0gInNvdXJjZV9sYWJlbHMiOgogICAgLSAiX19tZXRhX2t1YmVybmV0ZXNfcG9kX25hbWUiCiAgICAidGFyZ2V0X2xhYmVsIjogInBvZCIKICAtICJzb3VyY2VfbGFiZWxzIjoKICAgIC0gIl9fbWV0YV9rdWJlcm5ldGVzX3BvZF9jb250YWluZXJfbmFtZSIKICAgICJ0YXJnZXRfbGFiZWwiOiAiY29udGFpbmVyIgogICJzY3JhcGVfaW50ZXJ2YWwiOiAiMW0iCiAgInNjcmFwZV90aW1lb3V0IjogIjFtIg== | |||
kind: Secret | |||
metadata: | |||
labels: | |||
app.kubernetes.io/name: conprof | |||
app.kubernetes.io/version: vv0.1.0-dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems like a bug
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch!
name: conprof-main | ||
labels: | ||
app.kubernetes.io/name: conprof | ||
app.kubernetes.io/version: vv0.1.0-dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
34b1e76
to
cfc99d9
Compare
storage: Implement head.getByID to use with indexReader.LabelValueFor
This PR reworks the jsonnet code to be more idiomatic, and don't work on global, but local state instead. Additionally this adds the recommended kubernetes application labels.
@metalmatze