Skip to content

loki moved namespaces, so we have to move to match#26769

Merged
deads2k merged 2 commits intoopenshift:masterfrom
deads2k:loki-again
Jan 19, 2022
Merged

loki moved namespaces, so we have to move to match#26769
deads2k merged 2 commits intoopenshift:masterfrom
deads2k:loki-again

Conversation

@deads2k
Copy link
Contributor

@deads2k deads2k commented Jan 19, 2022

No description provided.

@deads2k deads2k added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged. labels Jan 19, 2022
@deads2k
Copy link
Contributor Author

deads2k commented Jan 19, 2022

I plan to force merge on a good build.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 19, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k

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 Jan 19, 2022
@openshift-ci openshift-ci bot requested review from bparees and spadgett January 19, 2022 21:51
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jan 19, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 19, 2022

New changes are detected. LGTM label has been removed.

@deads2k deads2k added the lgtm Indicates that a PR is ready to be merged. label Jan 19, 2022
@openshift-bot
Copy link
Contributor

/retest-required

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

2 similar comments
@openshift-bot
Copy link
Contributor

/retest-required

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

@openshift-bot
Copy link
Contributor

/retest-required

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

@deads2k deads2k merged commit b6f466c into openshift:master Jan 19, 2022
@deads2k
Copy link
Contributor Author

deads2k commented Jan 19, 2022

built, worked in the upgrade run. merging to fix signal overnight.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 19, 2022

@deads2k: The following tests 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/e2e-aws-cgroupsv2 943e0bb link false /test e2e-aws-cgroupsv2
ci/prow/e2e-aws-csi 943e0bb link false /test e2e-aws-csi

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.

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. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments