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

[MAIN] Image injection#1191

Merged
openshift-merge-robot merged 6 commits into
openshift:mainfrom
skonto:image_injection_main
Aug 2, 2022
Merged

[MAIN] Image injection#1191
openshift-merge-robot merged 6 commits into
openshift:mainfrom
skonto:image_injection_main

Conversation

@skonto
Copy link
Copy Markdown

@skonto skonto commented Jul 27, 2022

  • Similar to [RELEASE-1.5] Image injection #1189.
  • When the 1.7 release is cut the ci job that does the cut should create the right release manifests to be used by the S-O as usual.

skonto and others added 4 commits July 27, 2022 16:45
* inject images, generate manifests (knative#1150)

* Revert temoprary branch for image injection (knative#1159)

Co-authored-by: Kenjiro Nakayama <nakayamakenjiro@gmail.com>
@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 Jul 27, 2022
@openshift-ci openshift-ci Bot requested review from mgencur and mvinkler July 27, 2022 14:06
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 27, 2022
@skonto skonto changed the title [wip] [MAIN] Image injection [MAIN] Image injection Jul 28, 2022
@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 Jul 28, 2022
@skonto
Copy link
Copy Markdown
Author

skonto commented Jul 29, 2022

/hold to add the pdb fix

@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 Jul 29, 2022
@skonto
Copy link
Copy Markdown
Author

skonto commented Jul 29, 2022

/unhold

@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 Jul 29, 2022
@nak3
Copy link
Copy Markdown

nak3 commented Aug 2, 2022

Could you separate or drop the PDB patch? With the same reason as #1195, it should not patch it for now.

This reverts commit 1790632.
@nak3
Copy link
Copy Markdown

nak3 commented Aug 2, 2022

/lgtm

Thank you!

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

openshift-ci Bot commented Aug 2, 2022

[APPROVALNOTIFIER] This PR is APPROVED

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

3 participants