Skip to content

Conversation

djzager
Copy link
Contributor

@djzager djzager commented Jul 10, 2019

This makes it possible for ansible based operators to interact with CRDs
that didn't exist when the operator started.

fixes #1330

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

@fabianvf fabianvf left a comment

Choose a reason for hiding this comment

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

/lgtm

djzager added 2 commits July 11, 2019 15:34
This makes it possible for ansible based operators to interact with CRDs
that didn't exist when the operator started.

fixes operator-framework#1330
Before this change a `make test` would leave around testdata you
wouldn't want to put in version control. This change just updates the
watches test to defer os.Remove(...).
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jul 11, 2019
@openshift-ci-robot
Copy link

New changes are detected. LGTM label has been removed.

@shawn-hurley shawn-hurley merged commit f7f02de into operator-framework:master Jul 12, 2019
@djzager djzager deleted the ao-restmapper branch August 20, 2019 10:30
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.

Bad request (400) - could not determine if we should add owner ref

4 participants