runtime icon indicating copy to clipboard operation
runtime copied to clipboard

Default to dash separated?

Open ibuildthecloud opened this issue 3 years ago • 3 comments

container, job, secrets, acorns, etc keys have to be a-z0-9- because that is what k8s expects. In general we were going to default all keys to camelCase but it's weird to save one key must be foo-bar and everything else is fooBar. We should just ensure all keys are maybe foo-bar. The crappy thing about dash separated is that you have to quote the key in cue.

ibuildthecloud avatar May 26 '22 23:05 ibuildthecloud

sticking with camel case.

need a validation error on the keys in the acorn file to ensure theyre dns names

simple fix is changing regex for the keys

cjellick avatar Jul 18 '22 20:07 cjellick

@cjellick were any fixes or changes to the code base done in this issue, or is it just to document the decission.

cloudnautique avatar Aug 08 '22 16:08 cloudnautique

I'm going to put this under the AML umbrella that @ibuildthecloud will be doing

cjellick avatar Aug 08 '22 18:08 cjellick

Moving out of v0.4.0, because I dont think @ibuildthecloud is going to get to it, given the kubecon release

cjellick avatar Oct 05 '22 16:10 cjellick