Skip to content

release-controller: fix RHEL AV repo baseurl#26334

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
miabbott:rhelav_85_fix
Feb 22, 2022
Merged

release-controller: fix RHEL AV repo baseurl#26334
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
miabbott:rhelav_85_fix

Conversation

@miabbott
Copy link
Copy Markdown
Member

Forgot an important part of the URL in #26305

@miabbott
Copy link
Copy Markdown
Member Author

/assign jupierce

Copy link
Copy Markdown
Member

@cgwalters cgwalters left a comment

Choose a reason for hiding this comment

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

The content of this PR LGTM

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Feb 17, 2022
@petr-muller
Copy link
Copy Markdown
Member

/test ordered-prow-config

Copy link
Copy Markdown
Member

@petr-muller petr-muller left a comment

Choose a reason for hiding this comment

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

/hold

Holding if Justin's nod is needed, feel free to unhold otherwise.

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 21, 2022
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 21, 2022
@jupierce
Copy link
Copy Markdown
Contributor

/lgtm

@jupierce
Copy link
Copy Markdown
Contributor

/unhold

@openshift-ci openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 21, 2022
@openshift-ci openshift-ci Bot removed the lgtm Indicates that a PR is ready to be merged. label Feb 21, 2022
@miabbott
Copy link
Copy Markdown
Member Author

Rebased and pushed a new commit based on make update-ci-build-clusters ⬆️

@miabbott
Copy link
Copy Markdown
Member Author

/retest

Forgot an important part of the URL in openshift#26305
@HuijingHei
Copy link
Copy Markdown
Contributor

HuijingHei commented Feb 22, 2022

I did not see any failed tests, trigger test to clarify the results

@HuijingHei
Copy link
Copy Markdown
Contributor

/test all

@HuijingHei
Copy link
Copy Markdown
Contributor

Get error log: {"component":"applyconfig","error":"failed to apply config","file":"/go/src/github.com/openshift/ci-tools/cmd/applyconfig/applyconfig.go:555","func":"main.main","level":"error","msg":"There were failures while applying config","severity":"error","time":"2022-02-21T17:28:48Z"}, this is not required test and maybe can be ignored?

@petr-muller
Copy link
Copy Markdown
Member

petr-muller commented Feb 22, 2022

/test build01-dry
/test build02-dry

The failed tests are not triggered on this revision of the PR, the comment just informs that they were triggered on some previous revision and they failed. The risk of conditionally-triggered CI jobs /shrug - they are not necessary for this PR to merge even if they failed in the past.

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Feb 22, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Feb 22, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, jupierce, miabbott, petr-muller

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Feb 22, 2022

@miabbott: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/rehearse/openshift/release/master/build01-dry fbb3ab1b355e48c04f8c950fb8784054ee58df6b link unknown /test pj-rehearse
ci/rehearse/openshift/release/master/build02-dry fbb3ab1b355e48c04f8c950fb8784054ee58df6b link unknown /test pj-rehearse

Full PR test history. Your PR dashboard.

Details

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.

@openshift-merge-robot openshift-merge-robot merged commit a0b16cb into openshift:master Feb 22, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Feb 22, 2022

@miabbott: Updated the following 6 configmaps:

  • base-repos configmap in namespace ocp at cluster app.ci using the following files:
    • key ocp-4.11-rhel85.repo using file core-services/release-controller/_repos/ocp-4.11-rhel85.repo
  • base-repos configmap in namespace ocp at cluster build01 using the following files:
    • key ocp-4.11-rhel85.repo using file core-services/release-controller/_repos/ocp-4.11-rhel85.repo
  • base-repos configmap in namespace ocp at cluster build02 using the following files:
    • key ocp-4.11-rhel85.repo using file core-services/release-controller/_repos/ocp-4.11-rhel85.repo
  • base-repos configmap in namespace ocp at cluster build03 using the following files:
    • key ocp-4.11-rhel85.repo using file core-services/release-controller/_repos/ocp-4.11-rhel85.repo
  • base-repos configmap in namespace ocp at cluster build04 using the following files:
    • key ocp-4.11-rhel85.repo using file core-services/release-controller/_repos/ocp-4.11-rhel85.repo
  • base-repos configmap in namespace ocp at cluster vsphere using the following files:
    • key ocp-4.11-rhel85.repo using file core-services/release-controller/_repos/ocp-4.11-rhel85.repo
Details

In response to this:

Forgot an important part of the URL in #26305

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants