Skip to content

WINC-858: Prep for 7.0.0 release#1237

Merged
openshift-merge-robot merged 1 commit into
openshift:masterfrom
jrvaldes:prep-for-release-7
Sep 15, 2022
Merged

WINC-858: Prep for 7.0.0 release#1237
openshift-merge-robot merged 1 commit into
openshift:masterfrom
jrvaldes:prep-for-release-7

Conversation

@jrvaldes
Copy link
Copy Markdown
Contributor

@jrvaldes jrvaldes commented Sep 6, 2022

This change bumps the operator and the OCP version
according to WMCO 7.0.0 release.

Ran:

  make bundle

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 6, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Sep 6, 2022

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@jrvaldes jrvaldes mentioned this pull request Sep 6, 2022
Merged
@jrvaldes jrvaldes marked this pull request as ready for review September 6, 2022 15:12
@openshift-ci openshift-ci Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 6, 2022
@jrvaldes jrvaldes marked this pull request as draft September 6, 2022 15:12
@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 6, 2022
@jrvaldes
[bundle] Prep for 7.0.0 release
d794d1d 
This change bumps the operator and the OCP version
according to WMCO 7.0.0 release.

Ran:
```
  make bundle
```
@jrvaldes jrvaldes marked this pull request as ready for review September 7, 2022 01:15
@openshift-ci openshift-ci Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 7, 2022
@openshift-ci openshift-ci Bot requested a review from sebsoto September 7, 2022 01:15
@jrvaldes
Copy link
Copy Markdown
Contributor Author

jrvaldes commented Sep 7, 2022

/retest-required

saifshaikh48
saifshaikh48 reviewed Sep 7, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@saifshaikh48 saifshaikh48 left a comment

Choose a reason for hiding this comment

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

/lgtm

Thanks for doing this Jose

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

openshift-ci Bot commented Sep 7, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jrvaldes, sebsoto

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 Sep 7, 2022
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Sep 7, 2022

/retest-required

Remaining retests: 0 against base HEAD 9484ad1 and 2 for PR HEAD d794d1d in total

aravindhp
aravindhp reviewed Sep 7, 2022
View reviewed changes
Comment thread bundle/manifests/windows-machine-config-operator.clusterserviceversion.yaml
createdAt: REPLACE_DATE
description: An operator that enables Windows container workloads on OCP
olm.skipRange: '>=5.0.0 <6.0.0'
olm.skipRange: '>=6.0.0 <7.0.0'
Copy link
Copy Markdown
Contributor

@aravindhp aravindhp Sep 7, 2022

Choose a reason for hiding this comment

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

@mansikulkarni96 @saifshaikh48 I wonder if we can get render-templates to update all places where version is being used. Is that part of release improvement epic?

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Sep 8, 2022

/retest-required

Remaining retests: 0 against base HEAD 5928b62 and 1 for PR HEAD d794d1d in total

@jrvaldes
Copy link
Copy Markdown
Contributor Author

jrvaldes commented Sep 12, 2022

/test azure-e2e-operator

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Sep 13, 2022

/retest-required

Remaining retests: 0 against base HEAD 1a1f09b and 0 for PR HEAD d794d1d in total

@jrvaldes
Copy link
Copy Markdown
Contributor Author

jrvaldes commented Sep 13, 2022

/hold

To follow up on Azure CI issue

@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 Sep 13, 2022
@aravindhp
Copy link
Copy Markdown
Contributor

aravindhp commented Sep 13, 2022

/retitle Prep for 7.0.0 release

Lets be consistent with our naming. @jrvaldes does this also need to be associated with WINC-858?

@openshift-ci openshift-ci Bot changed the title Bump versions in prep for 7.0.0 Prep for 7.0.0 release Sep 13, 2022
@jrvaldes jrvaldes changed the title Prep for 7.0.0 release WINC-858: Prep for 7.0.0 release Sep 14, 2022
@jrvaldes
Copy link
Copy Markdown
Contributor Author

jrvaldes commented Sep 14, 2022 •
edited by openshift-ci Bot
Loading

/retitle Prep for 7.0.0 release

Lets be consistent with our naming. @jrvaldes does this also need to be associated with WINC-858?

It doesn't hurt. Done.

@jrvaldes
Copy link
Copy Markdown
Contributor Author

jrvaldes commented Sep 14, 2022

/test azure-e2e-operator

@jrvaldes
Copy link
Copy Markdown
Contributor Author

jrvaldes commented Sep 14, 2022

/hold cancel

@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 Sep 14, 2022
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Sep 14, 2022

/hold

Revision d794d1d was retested 3 times: holding

@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 Sep 14, 2022
@aravindhp
Copy link
Copy Markdown
Contributor

aravindhp commented Sep 14, 2022 •
edited
Loading

/override ci/prow/azure-e2e-operator

PR is provider agnostic and we are having Azure installer issues. Ref: OCPBUGS-927

@aravindhp
Copy link
Copy Markdown
Contributor

aravindhp commented Sep 14, 2022

/hold cancel

@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 Sep 14, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Sep 14, 2022

@aravindhp: Overrode contexts on behalf of aravindhp: ci/prow/azure-e2e-operator

Details

In response to this:

/override ci/prow/azure-e2e-operator

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.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Sep 15, 2022

@jrvaldes: all tests passed!

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 435fa06 into openshift:master Sep 15, 2022
@saifshaikh48 saifshaikh48 mentioned this pull request Mar 16, 2023
Merged
@jrvaldes jrvaldes deleted the prep-for-release-7 branch May 19, 2025 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mansikulkarni96 mansikulkarni96 Awaiting requested review from mansikulkarni96

3 more reviewers

@aravindhp aravindhp aravindhp left review comments

@saifshaikh48 saifshaikh48 saifshaikh48 left review comments

@sebsoto sebsoto sebsoto approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@saifshaikh48 saifshaikh48

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@jrvaldes @openshift-ci-robot @aravindhp @sebsoto @saifshaikh48 @openshift-merge-robot