-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Change the azure release periodic jobs to run on the 2nd subscription #21632
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
Change the azure release periodic jobs to run on the 2nd subscription #21632
Conversation
61c2b9b to
36137a1
Compare
|
/hold It needs #21633 |
|
/test pj-rehearse |
3 similar comments
|
/test pj-rehearse |
|
/test pj-rehearse |
|
/test pj-rehearse |
|
/test pj-rehearse |
36137a1 to
9749dfb
Compare
|
/test pj-rehearse |
1 similar comment
|
/test pj-rehearse |
9749dfb to
da0a936
Compare
da0a936 to
7a8f773
Compare
Signed-off-by: Nikolaos Moraitis <[email protected]>
7a8f773 to
f7f3bd7
Compare
|
/test pj-rehearse |
2 similar comments
|
/test pj-rehearse |
|
/test pj-rehearse |
|
/test pj-rehease |
|
@droslean: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
DetailsIn 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 pj-rehearse |
1 similar comment
|
/test pj-rehearse |
|
@droslean: The following tests 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. |
| cluster_profile: azure4 | ||
| cluster_profile: azure-2 | ||
| env: | ||
| BASE_DOMAIN: ci2.azure.devcluster.openshift.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
qq: is there a reason i don't see this env change being reflected on any job objects in the other file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be injected in the install-config.yaml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abhinavdahiya If you mean the job yaml file, this env var doesn't have to, because it will be consumed by ci-operator itself.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abhinavdahiya If you mean the job yaml file, this env var doesn't have to, because it will be consumed by ci-operator itself.
Ah, that makes sense then..
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinavdahiya, droslean 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 |
|
/hold cancel |
|
/retest-required Please review the full test history for this PR and help us cut down flakes. |
|
@droslean: Updated the following 2 configmaps:
DetailsIn 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. |
/cc @openshift/test-platform
Signed-off-by: Nikolaos Moraitis [email protected]