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

Molecule override cmd #1131

Merged
merged 2 commits into from
Feb 25, 2019

Conversation

fabianvf
Copy link
Member

Description of the change:
Molecule now requires you to set the override_command option to false in order to prevent it from overriding the CMD directive on a docker image you're using. This PR brings the generated molecule.yml files up to date. See ansible/molecule#1771

Motivation for the change:
We're going to break without it.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 25, 2019
Copy link
Contributor

@hasbro17 hasbro17 left a comment

Choose a reason for hiding this comment

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

LGTM

@fabianvf fabianvf merged commit 9afbe83 into operator-framework:master Feb 25, 2019
@fabianvf
Copy link
Member Author

fabianvf commented Mar 1, 2019

/cherrypick v0.4.x

fabianvf added a commit to fabianvf/operator-sdk that referenced this pull request Mar 1, 2019
* use new override_command option

* Update tests

(cherry picked from commit 9afbe83)
shawn-hurley pushed a commit to shawn-hurley/operator-sdk that referenced this pull request Mar 1, 2019
* use new override_command option

* Update tests
shawn-hurley pushed a commit that referenced this pull request Mar 1, 2019
* use new override_command option

* Update tests

(cherry picked from commit 9afbe83)
AlexNPavel pushed a commit to AlexNPavel/operator-sdk that referenced this pull request Mar 26, 2019
* use new override_command option

* Update tests
shawn-hurley pushed a commit to shawn-hurley/operator-sdk that referenced this pull request Apr 23, 2019
)

* use new override_command option

* Update tests

(cherry picked from commit 9afbe83)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants