Skip to content

Conversation

@vareti
Copy link
Contributor

@vareti vareti commented Nov 5, 2020

This PR temporarily increases memory limits for e2e-serial to unblock release payload promotion. This change will be reverted once the root cause is diagnosed.

@openshift-merge-robot
Copy link
Contributor

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

Test name Commit Details Rerun command
ci/prow/ci-operator-registry b2c3f8a link /test ci-operator-registry
ci/prow/ci-operator-config-metadata b2c3f8a link /test ci-operator-config-metadata

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.

Copy link

@damemi damemi left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Nov 5, 2020
@damemi
Copy link

damemi commented Nov 5, 2020

@vareti actually it looks like you need to run make ci-operator-config
/lgtm cancel

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Nov 5, 2020
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: vareti
To complete the pull request process, please assign damemi after the PR has been reviewed.
You can assign the PR to them by writing /assign @damemi in a comment when ready.

The full list of commands accepted by this bot can be found 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 openshift-ci-robot removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 5, 2020
@vareti
Copy link
Contributor Author

vareti commented Nov 5, 2020

@damemi thanks for reviewing this. we probably would not need this as the issue seems to be fixed in openshift/machine-config-operator#2204. closing the pr

/close

@openshift-ci-robot
Copy link
Contributor

@vareti: Closed this PR.

Details

In response to this:

@damemi thanks for reviewing this. we probably would not need this as the issue seems to be fixed in openshift/machine-config-operator#2204. closing the pr

/close

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.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants