Skip to content
This repository has been archived by the owner on Mar 5, 2018. It is now read-only.

Configration steps report #6

Closed
rwngwn opened this issue Jun 1, 2015 · 6 comments
Closed

Configration steps report #6

rwngwn opened this issue Jun 1, 2015 · 6 comments
Assignees

Comments

@rwngwn
Copy link
Contributor

rwngwn commented Jun 1, 2015

After running the module we should create some easy to read and parse report of steps taken, and their statuses. This can be stored also in a Docker image for audit purposes.

@vpavlin
Copy link

vpavlin commented Jun 3, 2015

s/Docker image/Docker container/ ... correct?

@rwngwn
Copy link
Contributor Author

rwngwn commented Jun 3, 2015

actually both. in container for run-time configuration and in image for build-time. Because if you have cct inside you can run in Dockerfile too.

@vpavlin
Copy link

vpavlin commented Jun 3, 2015

Ah, correct..good point. How does this map to the module definition? Are you going to distinguish between build vs. runtime module types?

@rwngwn
Copy link
Contributor Author

rwngwn commented Jun 3, 2015

No. modules are completely same. Because you typically need similar stuff. Only real difference is that in run-time you will just use new --command parameter to run your desired program in container in a way like:

cct --comand "/opt/eap/bin/standalone.sh -b 0.0.0.0" -- eap.yaml

so this will just process all changes in eap.yaml file and then it will just start eap.
now its implemented like: https://github.com/containers-tools/cct/blob/master/cct/cli/main.py#L48.
Will document it tomorrow :)

@vpavlin
Copy link

vpavlin commented Jun 3, 2015

Not that it matters much, but from UX point of view, I 'd probably more expect something like:

cct eap.yaml -- /opt/eap/bin/standalone.sh -b 0.0.0.0

So that I don't have to care about escaping issues with the command arguments..although I am not entirely sure how hard it would be to define this in ArgParser

@rwngwn
Copy link
Contributor Author

rwngwn commented Jun 3, 2015

yep, will look more at ArgParser.. but it maybe hard because you can do stuff like

cct foo.yaml bar.yaml baz.yaml

so you can process multiple yaml files. But wil try to make better syntax. This was just implement to us to be able to really start using it in our images.

@rwngwn rwngwn added the ready label Jul 14, 2015
@rwngwn rwngwn self-assigned this Jul 14, 2015
@rwngwn rwngwn closed this as completed Dec 7, 2015
@rwngwn rwngwn removed the ready label Dec 7, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants