Add cluster machine approver manifests #21
Conversation
openshift-ci
bot
added
the
do-not-merge/hold
|
/unhold |
openshift-ci
bot
removed
the
do-not-merge/hold
|
/retest |
alexander-demicev
force-pushed
the
cmamanifests
branch
2 times, most recently
from
1fae5a5 to
c9969f7
Compare
elmiko
left a comment
elmiko
left a comment
There was a problem hiding this comment.
mostly makes sense to me, just a question about one of the fields.
| name: config | ||
| env: | ||
| - name: RELEASE_VERSION | ||
| value: "0.0.1-snapshot" |
There was a problem hiding this comment.
will this need to be updated in future releases?
There was a problem hiding this comment.
This is generally substituted by the build system for the correct value IIUC
There was a problem hiding this comment.
ok, cool. just wanted to make sure we didn't need to update it frequently.
| name: config | ||
| env: | ||
| - name: RELEASE_VERSION | ||
| value: "0.0.1-snapshot" |
There was a problem hiding this comment.
This is generally substituted by the build system for the correct value IIUC
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoelSpeed The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
the
approved
alexander-demicev
force-pushed
the
cmamanifests
branch
2 times, most recently
from
5ba77a0 to
858fbdb
Compare
|
/retest |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
10 similar comments
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
@alexander-demichev Looking at the logs of the approver added by this PR, it can't find the CRDs, is this something you're aware of? /hold |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
20 similar comments
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
/hold |
openshift-ci
bot
added
the
do-not-merge/hold
|
@alexander-demichev: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
5 participants
Add cluster machine approver manifests, all manifests are annotated with
release.openshift.io/feature-gate: "TechPreviewNoUpgrade"and cluster-machine-approver flags are set to work with CAPI machines./hold
depends on openshift/cluster-version-operator#694