Skip to content

Bump to 1.24#1588

Merged
openshift-merge-robot merged 2 commits into
openshift-knative:mainfrom
skonto:bump_to_1.24
May 31, 2022
Merged

Bump to 1.24#1588
openshift-merge-robot merged 2 commits into
openshift-knative:mainfrom
skonto:bump_to_1.24

Conversation

@skonto
Copy link
Copy Markdown
Collaborator

@skonto skonto commented May 26, 2022

Fixes JIRA #

Proposed Changes

@skonto skonto requested review from mgencur and nak3 May 26, 2022 12:05
@openshift-ci openshift-ci Bot requested review from matzew and rhuss and removed request for nak3 May 26, 2022 12:05
skonto
skonto commented May 26, 2022
View reviewed changes
Comment thread olm-catalog/serverless-operator/index/Dockerfile

RUN /bin/opm init serverless-operator --default-channel=stable --output yaml >> /configs/index.yaml
RUN /bin/opm --skip-tls-verify render -o yaml registry.ci.openshift.org/knative/openshift-serverless-v1.22.0:serverless-stop-bundle \
RUN /bin/opm --skip-tls-verify render -o yaml registry.ci.openshift.org/knative/openshift-serverless-v1.23.0:serverless-stop-bundle \
Copy link
Copy Markdown
Collaborator Author

@skonto skonto May 26, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

Right now the image is not available, @mgencur should not this be available due to the release cut? Images have not been created yet due to: openshift/release#28924?

Copy link
Copy Markdown
Contributor

@mgencur mgencur May 26, 2022

Choose a reason for hiding this comment

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

The image will be available once any post-submit job runs (after any PR against 1.23 is merged).

@skonto skonto requested review from dsimansk and nak3 and removed request for matzew and rhuss May 26, 2022 12:13
@skonto
Copy link
Copy Markdown
Collaborator Author

skonto commented May 26, 2022 

{  the build serverless-operator-src failed after 20m20s with reason DockerBuildFailed: Dockerfile build strategy has failed.

ing-kafka": error during bulk transfer for copier.request{...b.com/prometheus/procfs/proc_io.go: no space left on device}

@skonto
Copy link
Copy Markdown
Collaborator Author

skonto commented May 26, 2022

/retest

4 similar comments
@droslean
Copy link
Copy Markdown

droslean commented May 27, 2022

/retest

@skonto
Copy link
Copy Markdown
Collaborator Author

skonto commented May 27, 2022

/retest

@skonto
Copy link
Copy Markdown
Collaborator Author

skonto commented May 30, 2022

/retest

@skonto
Copy link
Copy Markdown
Collaborator Author

skonto commented May 30, 2022

/retest

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 30, 2022

@skonto: The following test 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/prow/4.10-upgrade-tests-aws-ocp-410 855cdc9 link false /test 4.10-upgrade-tests-aws-ocp-410

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
Collaborator Author

skonto commented May 31, 2022

/retest

matzew
matzew approved these changes May 31, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@matzew matzew left a comment

Choose a reason for hiding this comment

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

/test all
/lgtm
/approve

/hold
(for tests)

@openshift-ci openshift-ci Bot added the lgtm label May 31, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 31, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: matzew, 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

@matzew
Copy link
Copy Markdown
Member

matzew commented May 31, 2022

/unhold
all required tests have run 👍

@openshift-merge-robot openshift-merge-robot merged commit a96f9a3 into openshift-knative:main May 31, 2022
@skonto skonto mentioned this pull request Jul 15, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@matzew matzew matzew approved these changes

@mgencur mgencur Awaiting requested review from mgencur

@dsimansk dsimansk Awaiting requested review from dsimansk

@nak3 nak3 Awaiting requested review from nak3

Assignees

@matzew matzew

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@skonto @droslean @matzew @mgencur @openshift-merge-robot