Skip to content
This repository was archived by the owner on Dec 1, 2022. It is now read-only.

[WIP][TEST-ONLY]Release next test pdb#1043

Closed
skonto wants to merge 3 commits into
openshift:release-nextfrom
skonto:release-next-test-pdb
Closed

[WIP][TEST-ONLY]Release next test pdb#1043
skonto wants to merge 3 commits into
openshift:release-nextfrom
skonto:release-next-test-pdb

Conversation

@skonto
Copy link
Copy Markdown

@skonto skonto commented Feb 2, 2022

No description provided.

@openshift-ci openshift-ci Bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Feb 2, 2022
@openshift-ci openshift-ci Bot requested a review from mgencur February 2, 2022 13:26
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Feb 2, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: skonto

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 requested a review from mvinkler February 2, 2022 13:26
@openshift-ci openshift-ci Bot added area/test-and-release Issues or PRs related to test and release approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Feb 2, 2022
@skonto skonto force-pushed the release-next-test-pdb branch from c0202c4 to d8379c7 Compare February 2, 2022 13:31
@openshift-ci openshift-ci Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 2, 2022
@skonto skonto removed the request for review from mvinkler February 2, 2022 13:31
@skonto
Copy link
Copy Markdown
Author

skonto commented Feb 2, 2022 •
edited
Loading

@mgencur I guess this only change is just the api version in the serving-ci.yaml file. I guess this should pass but verification happens locally assuming the modified script can produce the required serving-ci.yaml file.

@mgencur
Copy link
Copy Markdown

mgencur commented Feb 2, 2022

Alright

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Feb 2, 2022

@skonto: 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.

@skonto
Copy link
Copy Markdown
Author

skonto commented Feb 2, 2022

Ok will patch main then. All tests pass.

@skonto skonto mentioned this pull request Feb 2, 2022
Merged
@skonto skonto closed this Feb 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@mgencur mgencur Awaiting requested review from mgencur

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/test-and-release Issues or PRs related to test and release do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@skonto @mgencur @serverless-qe