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

pkg/scaffold/ansible; Add k8s_status module #972

Merged
merged 3 commits into from
Jan 22, 2019

Conversation

dymurray
Copy link
Contributor

Move k8s_status rendering all inside of scaffold
Add tests for k8s status

Description of the change:
Adds k8s_status module into Ansible Operator scaffolding

Motivation for the change:
k8s_status module is missing in 0.4.0 release.

Move k8s_status rendering all inside of scaffold
Add tests for k8s status
@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 21, 2019
@AlexNPavel
Copy link
Contributor

/hold

We need to update the SDK version in Gopkg.toml to use the branch v0.4.x so the CI runs against the branch and not the last tag v0.4.0
/pkg/scaffold/ansible/gopkgtoml.go@v0.4.x#L39

Wait for this PR to be merged: #974

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 21, 2019
@AlexNPavel
Copy link
Contributor

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 21, 2019
@shawn-hurley shawn-hurley added language/ansible Issue is related to an Ansible operator project kind/bug Categorizes issue or PR as related to a bug. labels Jan 22, 2019
Copy link
Member

@shawn-hurley shawn-hurley left a comment

Choose a reason for hiding this comment

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

LGTM

@shawn-hurley shawn-hurley merged commit 3d92c95 into operator-framework:v0.4.x Jan 22, 2019
shawn-hurley pushed a commit to shawn-hurley/operator-sdk that referenced this pull request Apr 23, 2019
Move k8s_status rendering all inside of scaffold
Add tests for k8s status

<!--

Before making a PR, please read our contributing guidelines https://github.com/operator-framework/operator-sdk/blob/master/CONTRIBUTING.MD

Note: Make sure your branch is rebased to the latest upstream master.

-->

**Description of the change:**
Adds k8s_status module into Ansible Operator scaffolding

**Motivation for the change:**
k8s_status module is missing in 0.4.0 release.
<!--

Note: If this PR is fixing an issue make sure to add a note saying:
Closes #<ISSUE_NUMBER>

-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. language/ansible Issue is related to an Ansible operator project size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants