Skip to content
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

Allow to override config values with CLI options #1

Open
martbab opened this issue Oct 10, 2016 · 1 comment
Open

Allow to override config values with CLI options #1

martbab opened this issue Oct 10, 2016 · 1 comment
Assignees

Comments

@martbab
Copy link
Contributor

martbab commented Oct 10, 2016

ipa-docker-test-runner should accept CLI options specifying alternative git repo path, image name, and other options.

@martbab martbab self-assigned this Oct 11, 2016
@martbab martbab modified the milestone: v0.2 release Nov 9, 2016
martbab referenced this issue Nov 11, 2016
The default ChainMap implementation does not handle nested
dictionaries very
well. Implement a deep version of mapping that should handle this
case.

https://github.com/martbab/ipa-docker-test-runner/issues/1
martbab referenced this issue Nov 11, 2016
This allows for overriding the config options from CLI or additional config
files.

https://github.com/martbab/ipa-docker-test-runner/issues/1
martbab referenced this issue Nov 11, 2016
A function was added which sets the value of config override based on the long
option name. E.g. `--container-image test` results in setting
override['container']['image'] to 'test'.

A reverse function (create CLI options from config constants) will require
more substantial work and will be implemented separately.

https://github.com/martbab/ipa-docker-test-runner/issues/1
martbab referenced this issue Nov 11, 2016
This does not implement generic overriding of config options from CLI but
allows for using ipa-docker-test-runner in Travis CI.

https://github.com/martbab/ipa-docker-test-runner/issues/1
@martbab martbab removed this from the v0.2 release milestone Nov 16, 2016
@martbab
Copy link
Contributor Author

martbab commented Nov 16, 2016

Removing 0.2 milestone since it is not required there.

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

No branches or pull requests

1 participant