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
Add nmstatectl for the unit tests #38290 #38290
Add nmstatectl for the unit tests #38290 #38290
Conversation
@andfasano: the
Prior to this PR being merged, you will need to either run and acknowledge or opt to skip these rehearsals. Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
/pj-rehearse |
63ff083
to
4f4219c
Compare
/pj-rehearse |
…-based installer tests
4f4219c
to
21693a1
Compare
/pj-rehearse |
/pj-rehearse pull-ci-openshift-installer-master-unit |
@andfasano: This pull request references AGENT-598 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@andfasano: This pull request references AGENT-598 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Please make the same changes to the release-4.14 branch config, so that we don't lose this config after branching. |
Isn't the CI bot copying it already from master -> 4.14?
Good point, but it will require a backport of #38290 to 4.13 once landed (we could add it later eventually) |
/pj-rehearse |
/pj-rehearse pull-ci-openshift-installer-master-unit |
@andfasano: This pull request references Jira Issue OCPBUGS-12869, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/pj-rehearse ack |
/lgtm |
@andfasano: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/test ci/prow/yamllint |
@patrickdillon: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andfasano, bfournie, patrickdillon The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retitle Add nmstatectl for the unit tests #38290 |
@andfasano: No Jira issue is referenced in the title of this pull request. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/jira refresh |
@patrickdillon: No Jira issue is referenced in the title of this pull request. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test yamllint |
d3ec68d
into
openshift:master
@andfasano: Updated the
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
…-based installer tests (openshift#38290)
Some of the agent-based installer unit tests require the presence of nmstatectl, otherwise they will be skipped silently.
Note: this PR could be merged only after openshift/installer#7089