Skip to content

repos: add RHEL 8.3 repo for use in openshift/os Prow jobs#18664

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
miabbott:addrhel8.3
May 20, 2021
Merged

repos: add RHEL 8.3 repo for use in openshift/os Prow jobs#18664
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
miabbott:addrhel8.3

Conversation

@miabbott
Copy link
Copy Markdown
Member

RHCOs 4.7 will be using RHEL 8.3 in the short term, until after OCP
4.8 GAs. While changes to 4.7 are unlikely during this time, it
would be useful to have the ability to run Prow jobs on the
release-4.7 branch of openshift/os with the correct content.

RHCOs 4.7 will be using RHEL 8.3 in the short term, until after OCP
4.8 GAs.  While changes to 4.7 are unlikely during this time, it
would be useful to have the ability to run Prow jobs on the
`release-4.7` branch of `openshift/os` with the correct content.
@openshift-ci openshift-ci Bot requested review from droslean and smarterclayton May 18, 2021 15:31
@miabbott
Copy link
Copy Markdown
Member Author

I think I did this correctly; followed the path blazed by Colin + Timothée

/assign travier
/assign cgwalters

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.

Looks sane to me

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

travier commented May 19, 2021

Shouldn't we switch the OCP-4.7-rhel8 to 8.3 instead of adding a new one? (and we will update it to 8.4 soon)

@miabbott
Copy link
Copy Markdown
Member Author

Shouldn't we switch the OCP-4.7-rhel8 to 8.3 instead of adding a new one? (and we will update it to 8.4 soon)

I opted for a new set of repos to minimize the impact on any other component/repo that might be relying on ocp-4.7-rhel8

This way we only have to update the release-4.7 Prow job on openshift/os and no where else.

@miabbott
Copy link
Copy Markdown
Member Author

/assign @bradmwilliams

Copy link
Copy Markdown
Contributor

@bradmwilliams bradmwilliams left a comment

Choose a reason for hiding this comment

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

/lgtm

@miabbott
Copy link
Copy Markdown
Member Author

Looks like we still need approval from DPTP, gonna try...

/assign @stevekuznetsov

@miabbott
Copy link
Copy Markdown
Member Author

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

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 20, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alvaroaleman, bradmwilliams, cgwalters, miabbott

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 openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 20, 2021
@openshift-merge-robot openshift-merge-robot merged commit 4ba3d87 into openshift:master May 20, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 20, 2021

@miabbott: Updated the following 4 configmaps:

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

In response to this:

RHCOs 4.7 will be using RHEL 8.3 in the short term, until after OCP
4.8 GAs. While changes to 4.7 are unlikely during this time, it
would be useful to have the ability to run Prow jobs on the
release-4.7 branch of openshift/os with the correct content.

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 added a commit to miabbott/os that referenced this pull request May 21, 2021
Update the CI test to fetch RHEL 8.3 repos.

This should be reverted once we intend to start using RHEL 8.3 in
RHCOS 4.7.

See:  openshift/release#18664
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