Skip to content

release-controller/repos: Updates for RHEL 8.4 GA#18691

Merged
openshift-merge-robot merged 3 commits intoopenshift:masterfrom
travier:rhel84ga
May 20, 2021
Merged

release-controller/repos: Updates for RHEL 8.4 GA#18691
openshift-merge-robot merged 3 commits intoopenshift:masterfrom
travier:rhel84ga

Conversation

@travier
Copy link
Copy Markdown
Member

@travier travier commented May 19, 2021

release-controller/repos: Use Beta repos for advanced-virt in 4.8+

This is 488c9ba repos: Point advanced-virt repos to the "beta" CDN


release-controller/repos: Remove 8.4 Beta repos


Revert "release-controller: Add generated deployments for RHEL 8.4 repos"

This reverts commit 89dc983.

@cgwalters
Copy link
Copy Markdown
Member

Why not keep the 8-4 pinned repos? Won't we potentially need them later for at least testing?

@travier
Copy link
Copy Markdown
Member Author

travier commented May 19, 2021

8 & 8.4 should have the same content going forward until 8.5, and at this point we will want to move all active OCP branches to "forced" 8.4 to not move to 8.5.

@travier
Copy link
Copy Markdown
Member Author

travier commented May 19, 2021

I'm not opposed to keeping them but I don't think there will be a case where we need them.

@travier
Copy link
Copy Markdown
Member Author

travier commented May 19, 2021

Given #18664 (comment), I'll update this one to keep the pinned repos. I'm not sure it's that simple as we might want to force 8.4 for EUS.

@miabbott
Copy link
Copy Markdown
Member

/lgtm
/assign @bradmwilliams

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label May 19, 2021
@miabbott
Copy link
Copy Markdown
Member

And someone from DPTP...wonder if this will work

/assign dptp

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 19, 2021

@miabbott: GitHub didn't allow me to assign the following users: dptp.

Note that only openshift members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

Details

In response to this:

And someone from DPTP...wonder if this will work

/assign dptp

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.

@miabbott
Copy link
Copy Markdown
Member

/assign @alvaroaleman

@miabbott
Copy link
Copy Markdown
Member

/unassign @alvaroaleman
/assign @openshift/openshift-team-developer-productivity-test-platform

@alvaroaleman
Copy link
Copy Markdown
Contributor

/assign @jupierce
I don't know anything about this, sorry

@jupierce jupierce added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 20, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 20, 2021

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: miabbott, travier

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-merge-robot openshift-merge-robot merged commit 04c24f4 into openshift:master May 20, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 20, 2021

@travier: Updated the following 4 configmaps:

  • base-repos configmap in namespace ocp at cluster app.ci using the following files:
    • key ocp-4.8-rhel8.repo using file core-services/release-controller/_repos/ocp-4.8-rhel8.repo
    • key ocp-4.8-rhel84.repo using file ``
    • key ocp-4.9-rhel8.repo using file core-services/release-controller/_repos/ocp-4.9-rhel8.repo
    • key ocp-4.9-rhel84.repo using file ``
  • base-repos configmap in namespace ocp at cluster build01 using the following files:
    • key ocp-4.8-rhel8.repo using file core-services/release-controller/_repos/ocp-4.8-rhel8.repo
    • key ocp-4.8-rhel84.repo using file ``
    • key ocp-4.9-rhel8.repo using file core-services/release-controller/_repos/ocp-4.9-rhel8.repo
    • key ocp-4.9-rhel84.repo using file ``
  • base-repos configmap in namespace ocp at cluster build02 using the following files:
    • key ocp-4.8-rhel8.repo using file core-services/release-controller/_repos/ocp-4.8-rhel8.repo
    • key ocp-4.8-rhel84.repo using file ``
    • key ocp-4.9-rhel8.repo using file core-services/release-controller/_repos/ocp-4.9-rhel8.repo
    • key ocp-4.9-rhel84.repo using file ``
  • base-repos configmap in namespace ocp at cluster vsphere using the following files:
    • key ocp-4.8-rhel8.repo using file core-services/release-controller/_repos/ocp-4.8-rhel8.repo
    • key ocp-4.8-rhel84.repo using file ``
    • key ocp-4.9-rhel8.repo using file core-services/release-controller/_repos/ocp-4.9-rhel8.repo
    • key ocp-4.9-rhel84.repo using file ``
Details

In response to this:

release-controller/repos: Use Beta repos for advanced-virt in 4.8+

This is 488c9ba repos: Point advanced-virt repos to the "beta" CDN


release-controller/repos: Remove 8.4 Beta repos


Revert "release-controller: Add generated deployments for RHEL 8.4 repos"

This reverts commit 89dc983.

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.

@travier travier deleted the rhel84ga branch May 20, 2021 21:59
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.

7 participants