Skip to content

make/targets/openshift/yaml-patch: fix missing repository#58

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
s-urbaniak:fix-yaml-patch
Dec 13, 2021
Merged

make/targets/openshift/yaml-patch: fix missing repository#58
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
s-urbaniak:fix-yaml-patch

Conversation

@s-urbaniak
Copy link
Copy Markdown
Contributor

yaml-patch seems to have disappeared.
This fixes it by moving to the pivotal repository.
@openshift-ci openshift-ci Bot requested review from soltysh and sttts December 13, 2021 09:25
Copy link
Copy Markdown

@soltysh soltysh left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@sttts
Copy link
Copy Markdown
Contributor

sttts commented Dec 13, 2021

/lgtm
/approve
/hold

worth a proof PR ?

@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 Dec 13, 2021
@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Dec 13, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 13, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: s-urbaniak, soltysh, sttts

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 Dec 13, 2021
@s-urbaniak
Copy link
Copy Markdown
Contributor Author

s-urbaniak commented Dec 13, 2021

worth a proof PR ?

indeed, i proofed it locally, let me open one against cert-manager-operator (the victim)

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 13, 2021

@s-urbaniak: 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.

@s-urbaniak
Copy link
Copy Markdown
Contributor Author

proof PR: openshift/api#1083

@sttts
Copy link
Copy Markdown
Contributor

sttts commented Dec 13, 2021

/cancel hold

@soltysh
Copy link
Copy Markdown

soltysh commented Dec 13, 2021

/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 Dec 13, 2021
@openshift-merge-robot openshift-merge-robot merged commit 7e33a7e into openshift:master Dec 13, 2021
@s-urbaniak s-urbaniak deleted the fix-yaml-patch branch December 13, 2021 09:43
florkbr added a commit to florkbr/console-operator that referenced this pull request Dec 14, 2021
This will need to be ported to our master-multi-cluster branch

See: openshift/build-machinery-go#58

CC @TheRealJon @spadgett
eggfoobar added a commit to eggfoobar/console-operator that referenced this pull request Dec 14, 2021
updated to using latest library-go
updated build images to use go 1.17
updated method signatures for library-go methods where context is passed in
update test to work with go 1.17 crypto message change

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>
eggfoobar added a commit to eggfoobar/console-operator that referenced this pull request Dec 14, 2021
updated to using latest library-go
updated build images to use go 1.17
updated method signatures for library-go methods where context is passed in
update test to work with go 1.17 crypto message change

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>
eggfoobar added a commit to eggfoobar/cluster-image-registry-operator that referenced this pull request Dec 14, 2021
fetched latest library-go and vendored dep
updated handlers to use updated sctruct names
updated okd builder image to use go 1.17

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>
eggfoobar added a commit to eggfoobar/cluster-image-registry-operator that referenced this pull request Dec 15, 2021
fetched latest library-go and vendored dep
updated handlers to use updated sctruct names
updated okd builder image to use go 1.17

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>
eggfoobar added a commit to eggfoobar/cluster-config-operator that referenced this pull request Dec 15, 2021
updated library go to latest

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>

feat: updated logs to explicitly add flags

logs requires explict calls to add flags in latest version

Signed-off-by: ehila <ehila@redhat.com>
eggfoobar added a commit to eggfoobar/cluster-config-operator that referenced this pull request Dec 21, 2021
updated library go to latest

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>

feat: updated logs to explicitly add flags

logs requires explict calls to add flags in latest version

Signed-off-by: ehila <ehila@redhat.com>
@jsafrane
Copy link
Copy Markdown
Contributor

/cherry-pick release-4.9

@openshift-cherrypick-robot
Copy link
Copy Markdown

@jsafrane: new pull request created: #59

Details

In response to this:

/cherry-pick release-4.9

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.

eggfoobar added a commit to eggfoobar/cluster-image-registry-operator that referenced this pull request Dec 24, 2021
fetched latest library-go and vendored dep
updated handlers to use updated sctruct names
updated okd builder image to use go 1.17

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>
eggfoobar added a commit to eggfoobar/cluster-image-registry-operator that referenced this pull request Jan 7, 2022
fetched latest library-go and vendored dep
updated handlers to use updated sctruct names
updated okd builder image to use go 1.17

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>
eggfoobar added a commit to eggfoobar/cluster-config-operator that referenced this pull request Jan 10, 2022
updated library go to latest

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>

feat: updated logs to explicitly add flags

logs requires explict calls to add flags in latest version

Signed-off-by: ehila <ehila@redhat.com>

bump: library-go

updated library-go to latest for fix in pr openshift/library-go#1273

Signed-off-by: ehila <ehila@redhat.com>
eggfoobar added a commit to eggfoobar/cluster-image-registry-operator that referenced this pull request Jan 25, 2022
fetched latest library-go and vendored dep
updated handlers to use updated sctruct names
updated okd builder image to use go 1.17

Signed-off-by: ehila <ehila@redhat.com>

fix: updated build-machinery to fix yaml-patch verify

updated build-machinery
- more info: openshift/build-machinery-go#58

Signed-off-by: ehila <ehila@redhat.com>

fix: pull in latest library-go

pull in to grab fix described here openshift/library-go#1272

Signed-off-by: ehila <ehila@redhat.com>

bump: library-go

updated library-go to latest for fix in pr openshift/library-go#1273

Signed-off-by: ehila <ehila@redhat.com>
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.

6 participants