Skip to content

Conversation

@jottofar
Copy link
Contributor

library-go manifest and verify packages were created since both CVO and the oc client use them.

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 25, 2020
@jottofar jottofar force-pushed the ota-190-library-go-reuse branch from c583ff1 to f0004e9 Compare August 25, 2020 16:02
@jottofar
Copy link
Contributor Author

/test unit

@jottofar jottofar force-pushed the ota-190-library-go-reuse branch from f0004e9 to e159773 Compare August 25, 2020 18:58
@jottofar
Copy link
Contributor Author

/retest

@jottofar jottofar force-pushed the ota-190-library-go-reuse branch 2 times, most recently from e159773 to 491e499 Compare August 26, 2020 18:59
@jottofar jottofar force-pushed the ota-190-library-go-reuse branch from 491e499 to 34cc37c Compare August 31, 2020 13:46
@jottofar
Copy link
Contributor Author

/retest

1 similar comment
@jottofar
Copy link
Contributor Author

jottofar commented Sep 7, 2020

/retest

@jottofar
Copy link
Contributor Author

jottofar commented Sep 7, 2020

/bugzilla refresh

@openshift-ci-robot
Copy link
Contributor

@jottofar: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

/bugzilla refresh

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.

@jottofar
Copy link
Contributor Author

/bugzilla refresh

@openshift-ci-robot
Copy link
Contributor

@jottofar: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

/bugzilla refresh

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.

@jottofar
Copy link
Contributor Author

/bugzilla refresh

@openshift-ci-robot
Copy link
Contributor

@jottofar: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

/bugzilla refresh

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.

@jottofar jottofar force-pushed the ota-190-library-go-reuse branch from 34cc37c to f23ecf6 Compare October 6, 2020 13:30
@openshift-ci-robot
Copy link
Contributor

@jottofar: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/unit f23ecf6 link /test unit
ci/prow/gofmt f23ecf6 link /test gofmt
ci/prow/integration f23ecf6 link /test integration
ci/prow/e2e f23ecf6 link /test e2e
ci/prow/images f23ecf6 link /test images
ci/prow/e2e-upgrade f23ecf6 link /test e2e-upgrade

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.

@jottofar jottofar force-pushed the ota-190-library-go-reuse branch 3 times, most recently from 415528b to a2ae4fe Compare October 19, 2020 16:09
@jottofar
Copy link
Contributor Author

/retest

1 similar comment
@jottofar
Copy link
Contributor Author

/retest

@jottofar jottofar force-pushed the ota-190-library-go-reuse branch from a2ae4fe to b3c36da Compare October 27, 2020 14:11
Steps:

  $ go get github.com/openshift/library-go
  $ go mod tidy
  $ go mod vendor

using:

  $ go version
  go version go1.15.3 linux/amd64
library-go manifest and verify packages were created since both CVO and
the oc client use them.
@jottofar jottofar force-pushed the ota-190-library-go-reuse branch from b3c36da to ce2874f Compare November 18, 2020 14:11
@jottofar
Copy link
Contributor Author

/retest

@jottofar
Copy link
Contributor Author

/test integration

@jottofar
Copy link
Contributor Author

/retest

1 similar comment
@jottofar
Copy link
Contributor Author

/retest

@wking
Copy link
Member

wking commented Dec 5, 2020

/test e2e-agnostic

@wking
Copy link
Member

wking commented Dec 6, 2020

Integration is still blocked on openshift/release#13745 or similar.

/override ci/prow/integration

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Dec 6, 2020
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jottofar, wking

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-robot
Copy link
Contributor

@wking: Overrode contexts on behalf of wking: ci/prow/integration

Details

In response to this:

Integration is still blocked on openshift/release#13745 or similar.

/override ci/prow/integration

/lgtm

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-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

9 similar comments
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot
Copy link
Contributor

openshift-merge-robot commented Dec 6, 2020

@jottofar: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e a2ae4fe link /test e2e

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-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit 84b3884 into openshift:master Dec 6, 2020
wking added a commit to wking/cluster-version-operator that referenced this pull request May 4, 2021
Catching the generator up with ce2874f (Modify to use library-go
manifest and verify packages, 2020-08-25, openshift#443).
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.

5 participants