projectatomic / ContainerApplicationGenericLabels

Default Containerized Application Labels used to document the application and/or image
68 stars 29 forks source link

Support for Intents #1

Open sghosh151 opened 9 years ago

sghosh151 commented 9 years ago

Having a set of labels be available to support intents as described in https://github.com/docker/docker/issues/11187.

Do we have to worry about datatypes for labels? - string vs json blob?

rhatdan commented 9 years ago

@sghosh151 During the LABEL discussion at docker, they really wanted to avoid JSON data in the labels, and wanted to stick to strings. Feeling that JSON data in locations like Dockerfiles would quickly become difficult to use for non expert users.

cgwalters commented 9 years ago

Strings only forces e.g. the RUN label down the path of shell script for command execution. It's convenient but as soon as you have arguments that need to be quoted you're into shell quoting hell. Docker learned that lesson and changed RUN to support an array syntax too.