Skip to content

Conversation

simonferquel
Copy link
Contributor

Custom actions lacked a description field. This fixed that. Also introduces stateless custom actions to implement things like dry-run, help etc.

The docker-app tool currently abuse the spec to do this kind of things: inspect and render actions are executed without passing credentials, and are used to visualize the effect of parameter values on the generated stack. We think it should be part of the standard as we see similar requirements for many other generic CNAB (Helm, Terraform, ...)

Custom actions lacked a description field. This fixed that.
Also introduces stateless custom actions to implement things like
`dry-run`, `help` etc.
The docker-app tool currently abuse the spec to do this kind of things
for the `inspect` and `render` actions, that are executed without
passing credentials, and are used to visualize the effect of parameter
values on the generated stack.

Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
@simonferquel
Copy link
Contributor Author

Thankis @radu-matei, fixed your comments, but mot sure yet if it is proper english :)

Copy link
Member

@technosophos technosophos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@technosophos technosophos merged commit be9ee1b into cnabio:master Feb 11, 2019
radu-matei pushed a commit to cnabio/duffle that referenced this pull request Mar 14, 2019
This implements stateless actions as defined in cnabio/cnab-spec#85.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants