Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[release-3.11] Bug 1930100: Pod status patches #26237

Merged
merged 2 commits into from
Jul 6, 2021
Merged

[release-3.11] Bug 1930100: Pod status patches #26237

merged 2 commits into from
Jul 6, 2021

Conversation

rphillips
Copy link
Contributor

@rphillips rphillips commented Jun 16, 2021 •
edited
Loading

Backports two upstream fixes into 3.11 for pod statuses: 86320 and 88591.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 16, 2021

@rphillips: 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.

In response to this:

[release-3.11] Pod status patches

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-ci openshift-ci bot requested review from bparees and smarterclayton June 16, 2021 17:16
@rphillips rphillips force-pushed the pod_status_patches_3.11 branch from 54818b1 to a365637 Compare June 16, 2021 17:19
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 16, 2021

@rphillips: 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.

In response to this:

[release-3.11] Pod status patches

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.

@nee1esh
Copy link

nee1esh commented Jun 21, 2021

/retitle [release-3.11] Bug 1930100: Pod status patches

@openshift-ci openshift-ci bot changed the title [release-3.11] Pod status patches [release-3.11] Bug 1930100: Pod status patches Jun 21, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 21, 2021

@rphillips: This pull request references Bugzilla bug 1930100, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target release (3.11.z) matches configured target release for branch (3.11.z)
  • bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)

No GitHub users were found matching the public email listed for the QA contact in Bugzilla (schoudha@redhat.com), skipping review request.

In response to this:

[release-3.11] Bug 1930100: Pod status patches

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-ci openshift-ci bot added bugzilla/severity-urgent Referenced Bugzilla bug's severity is urgent for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. labels Jun 21, 2021
@nee1esh
Copy link

nee1esh commented Jun 21, 2021

/retest

1 similar comment
@brenton
Copy link
Contributor

brenton commented Jun 24, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 24, 2021

please forgive the noise, I'm trying configure the jobs to use a dockerhub account that won't hit the current rate limits.

@brenton
Copy link
Contributor

brenton commented Jun 24, 2021

/retest

14 similar comments
@brenton
Copy link
Contributor

brenton commented Jun 24, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 24, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 25, 2021

/retest

@brenton
Copy link
Contributor

brenton commented Jun 26, 2021

/retest

@nee1esh
Copy link

nee1esh commented Jul 2, 2021

/retest

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 2, 2021 •
edited
Loading

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

Test name Commit Details Rerun command
ci/prow/e2e-gcp-crio a365637 link /test e2e-gcp-crio

Full PR test history. Your PR dashboard.

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.

@mrunalp
Copy link
Member

mrunalp commented Jul 6, 2021

/override cmd

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 6, 2021

@mrunalp: mrunalp unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file.

In response to this:

/override cmd

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.

@mrunalp
Copy link
Member

mrunalp commented Jul 6, 2021

/lgtm

@mrunalp mrunalp added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 6, 2021
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 6, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 6, 2021

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: mrunalp, rphillips

The full list of commands accepted by this bot can be found here.

The pull request process is described here

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

@sjenning
Copy link
Contributor

sjenning commented Jul 6, 2021

/override cmd

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 6, 2021

@sjenning: /override requires a failed status context to operate on.
The following unknown contexts were given:

  • cmd

Only the following contexts were expected:

  • ci/openshift-jenkins/cmd
  • ci/openshift-jenkins/end_to_end
  • ci/openshift-jenkins/extended_clusterup
  • ci/openshift-jenkins/extended_conformance_install
  • ci/prow/e2e-gcp
  • ci/prow/e2e-gcp-crio
  • ci/prow/images
  • ci/prow/integration
  • ci/prow/unit
  • ci/prow/verify
  • tide

In response to this:

/override cmd

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.

@sjenning
Copy link
Contributor

sjenning commented Jul 6, 2021

/override ci/openshift-jenkins/cmd

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 6, 2021

@sjenning: Overrode contexts on behalf of sjenning: ci/openshift-jenkins/cmd

In response to this:

/override ci/openshift-jenkins/cmd

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.

@openshift-merge-robot openshift-merge-robot merged commit fa67f7b into openshift:release-3.11 Jul 6, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 6, 2021

@rphillips: All pull requests linked via external trackers have merged:

Bugzilla bug 1930100 has been moved to the MODIFIED state.

In response to this:

[release-3.11] Bug 1930100: Pod status patches

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
Reviewers

@bparees bparees Awaiting requested review from bparees

@smarterclayton smarterclayton Awaiting requested review from smarterclayton

Assignees

@mrunalp mrunalp

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. bugzilla/severity-urgent Referenced Bugzilla bug's severity is urgent for the branch this PR is targeting. 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@rphillips @nee1esh @brenton @mrunalp @sjenning @openshift-bot @openshift-merge-robot