Bug 2040793: Fix ordering issues with snapshot e2e by gnufied · Pull Request #1119 · openshift/kubernetes

gnufied · 2022-01-14T17:27:20Z

Fix snapshot e2e failures

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=2040793

…umes

openshift-ci-robot · 2022-01-14T17:27:29Z

@gnufied: the contents of this pull request could be automatically validated.

The following commits are valid:

7d63874|UPSTREAM: 107173: Fix order of commands in the snapshot tests for volumes: the upstream PR kubernetes/kubernetes#107173 has merged
ebd3a85|UPSTREAM: 107312: Execute sync before taking the snapshot: the upstream PR kubernetes/kubernetes#107312 has merged

Comment /validate-backports to re-evaluate validity of the upstream PRs, for example when they are merged upstream.

openshift-ci · 2022-01-14T17:28:46Z

@gnufied: This pull request references Bugzilla bug 2040793, 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 (4.10.0) matches configured target release for branch (4.10.0)
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 (wduan@redhat.com), skipping review request.

Details

In response to this:

Bug 2040793: Fix ordering issues with snapshot e2e

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.

gnufied · 2022-01-14T17:28:47Z

/bugzilla refresh

openshift-ci · 2022-01-14T17:28:50Z

@gnufied: This pull request references Bugzilla bug 2040793, which is valid.

3 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target release (4.10.0) matches configured target release for branch (4.10.0)
bug is in the state POST, 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 (wduan@redhat.com), skipping review request.

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.

openshift-ci · 2022-01-14T17:31:18Z

@gnufied: An error was encountered querying GitHub for users with public email (wduan@redhat.com) for bug 2040793 on the Bugzilla server at https://bugzilla.redhat.com. No known errors were detected, please see the full error message for details.

Full error message.


non-200 OK status code: 403 Forbidden body: "{\n  \"documentation_url\": \"https://docs.github.com/en/free-pro-team@latest/rest/overview/resources-in-the-rest-api#secondary-rate-limits\",\n  \"message\": \"You have exceeded a secondary rate limit. Please wait a few minutes before you try again.\"\n}\n"

Please contact an administrator to resolve this issue, then request a bug refresh with /bugzilla refresh.

Details

In response to this:

Bug 2040793: Fix ordering issues with snapshot e2e

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.

dobsonj · 2022-01-14T18:06:13Z

/lgtm
/bugzilla refresh

/assign @mfojtik
for approval

openshift-ci · 2022-01-14T18:06:17Z

@dobsonj: This pull request references Bugzilla bug 2040793, which is valid.

3 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target release (4.10.0) matches configured target release for branch (4.10.0)
bug is in the state POST, 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 (wduan@redhat.com), skipping review request.

Details

In response to this:

/lgtm
/bugzilla refresh

/assign @mfojtik
for approval

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.

gnufied · 2022-01-14T18:13:58Z

/assign @mfojtik @deads2k

gnufied · 2022-01-14T19:34:48Z

/retest

gnufied · 2022-01-14T20:07:28Z

One upstream PR is already backported - kubernetes#107341 and other one is pending kubernetes#107566

soltysh

/approve
/retest

openshift-ci · 2022-01-17T12:21:04Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dobsonj, gnufied, soltysh

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:

~~DOWNSTREAM_OWNERS~~ [soltysh]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

openshift-bot · 2022-01-17T14:41:51Z

/retest-required

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

openshift-bot · 2022-01-17T15:02:05Z

/retest-required

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

openshift-bot · 2022-01-17T15:14:05Z

/retest-required

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

openshift-bot · 2022-01-17T16:02:03Z

/retest-required

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

openshift-bot · 2022-01-17T16:50:14Z

/retest-required

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

openshift-bot · 2022-01-20T06:01:52Z

/retest-required

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

openshift-bot · 2022-01-20T09:51:52Z

/retest-required

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

openshift-bot · 2022-01-20T10:03:51Z

/retest-required

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

openshift-bot · 2022-01-20T13:27:52Z

/retest-required

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

openshift-bot · 2022-01-20T13:39:51Z

/retest-required

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

openshift-bot · 2022-01-20T18:29:53Z

/retest-required

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

openshift-bot · 2022-01-20T23:37:39Z

/retest-required

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

openshift-bot · 2022-01-21T03:08:33Z

/retest-required

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

openshift-bot · 2022-01-21T03:21:31Z

/retest-required

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

openshift-bot · 2022-01-21T07:15:31Z

/retest-required

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

openshift-bot · 2022-01-21T10:52:34Z

/retest-required

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

openshift-bot · 2022-01-21T11:40:34Z

/retest-required

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

openshift-bot · 2022-01-21T14:04:32Z

/retest-required

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

openshift-bot · 2022-01-21T16:04:36Z

/retest-required

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

openshift-bot · 2022-01-21T18:17:32Z

/retest-required

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

deads2k · 2022-01-21T19:40:03Z

/override ci/prow/k8s-e2e-gcp-serial
/skip

openshift-ci · 2022-01-21T19:40:05Z

@deads2k: Overrode contexts on behalf of deads2k: ci/prow/k8s-e2e-gcp-serial

Details

In response to this:

/override ci/prow/k8s-e2e-gcp-serial
/skip

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.

gnufied · 2022-01-21T19:55:18Z

The failure in serial tests have nothing to do with change as far as I can tell.

For instance:

E0121 17:14:30.598203      75 portforward.go:340] error creating error stream for port 3100 -> 3100: Timeout occured
{"component":"entrypoint","file":"prow/entrypoint/run.go:165","func":"k8s.io/test-infra/prow/entrypoint.Options.ExecuteProcess","level":"error","msg":"Process did not finish before 10m0s timeout","severity":"error","time":"2022-01-21T17:23:55Z"}
{"component":"entrypoint","error":"os: process already finished","file":"prow/entrypoint/run.go:243","func":"k8s.io/test-infra/prow/entrypoint.gracefullyTerminate","level":"error","msg":"Could not interrupt process after timeout","severity":"error","time":"2022-01-21T17:23:55Z"}
{"component":"entrypoint","file":"prow/entrypoint/run.go:255","func":"k8s.io/test-infra/prow/entrypoint.gracefullyTerminate","level":"error","msg":"Process did not exit before 15s grace period","severity":"error","time":"2022-01-21T17:24:10Z"}
{"component":"entrypoint","error":"os: process already finished","file":"prow/entrypoint/run.go:257","func":"k8s.io/test-infra/prow/entrypoint.gracefullyTerminate","level":"error","msg":"Could not kill process after grace period","severity":"error","time":"2022-01-21T17:24:10Z"}
{"component":"entrypoint","error":"process timed out","file":"prow/entrypoint/run.go:80","func":"k8s.io/test-infra/prow/entrypoint.Options.Run","level":"error","msg":"Error executing test process","severity":"error","time":"2022-01-21T17:24:10Z"}
error: failed to execute wrapped command: exit status 127

And the second one is scheduler e2e failure. Both failures can not be caused by snapshot ordering changes proposed here. These are just e2e change which are reordering snapshot, restore and check file content to ensure we have a valid snapshot.

openshift-bot · 2022-01-21T23:07:32Z

/retest-required

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

openshift-bot · 2022-01-21T23:32:35Z

/retest-required

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

openshift-bot · 2022-01-22T02:32:07Z

/retest-required

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

openshift-bot · 2022-01-22T03:44:08Z

/retest-required

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

openshift-bot · 2022-01-22T04:32:06Z

/retest-required

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

openshift-bot · 2022-01-22T06:08:05Z

/retest-required

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

openshift-bot · 2022-01-22T07:20:06Z

/retest-required

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

openshift-ci · 2022-01-22T08:29:33Z

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

openshift-ci · 2022-01-22T12:38:31Z

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

openshift/kubernetes#1119

Bugzilla bug 2040793 has been moved to the MODIFIED state.

Details

In response to this:

Bug 2040793: Fix ordering issues with snapshot e2e

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.

gnufied added 2 commits January 14, 2022 12:25

UPSTREAM: 107173: Fix order of commands in the snapshot tests for vol…

7d63874

…umes

UPSTREAM: 107312: Execute sync before taking the snapshot

ebd3a85

openshift-ci-robot added the backports/validated-commits Indicates that all commits come to merged upstream PRs. label Jan 14, 2022

gnufied changed the title ~~Fix ordering issues with snapshot e2e~~ Bug 2040793: Fix ordering issues with snapshot e2e Jan 14, 2022

openshift-ci bot added bugzilla/severity-medium Referenced Bugzilla bug's severity is medium 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 Jan 14, 2022

openshift-ci bot requested review from soltysh and sttts January 14, 2022 17:28

openshift-ci bot assigned mfojtik Jan 14, 2022

openshift-ci bot assigned dobsonj Jan 14, 2022

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 14, 2022

openshift-ci bot assigned deads2k Jan 14, 2022

soltysh approved these changes Jan 17, 2022

View reviewed changes

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 17, 2022

jsafrane mentioned this pull request Jan 21, 2022

Bug 1927478: UPSTREAM: <carry>: Fix conformance and serial tests by stopping node cordoning #1126

Closed

openshift-merge-robot merged commit 59c6e59 into openshift:master Jan 22, 2022

openshift-ci bot mentioned this pull request Jan 24, 2022

Bug 2040793: Fix ordering issues with snapshot e2e openshift/origin#26778

Merged

Conversation

gnufied commented Jan 14, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openshift-ci-robot commented Jan 14, 2022

Uh oh!

openshift-ci bot commented Jan 14, 2022

Uh oh!

gnufied commented Jan 14, 2022

Uh oh!

openshift-ci bot commented Jan 14, 2022

Uh oh!

openshift-ci bot commented Jan 14, 2022

Uh oh!

dobsonj commented Jan 14, 2022

Uh oh!

openshift-ci bot commented Jan 14, 2022

Uh oh!

gnufied commented Jan 14, 2022

Uh oh!

gnufied commented Jan 14, 2022

Uh oh!

gnufied commented Jan 14, 2022

Uh oh!

soltysh left a comment

Choose a reason for hiding this comment

Uh oh!

openshift-ci bot commented Jan 17, 2022

Uh oh!

openshift-bot commented Jan 17, 2022

Uh oh!

openshift-bot commented Jan 17, 2022

Uh oh!

openshift-bot commented Jan 17, 2022

Uh oh!

openshift-bot commented Jan 17, 2022

Uh oh!

openshift-bot commented Jan 17, 2022

Uh oh!

openshift-bot commented Jan 20, 2022

Uh oh!

openshift-bot commented Jan 20, 2022

Uh oh!

openshift-bot commented Jan 20, 2022

Uh oh!

openshift-bot commented Jan 20, 2022

Uh oh!

openshift-bot commented Jan 20, 2022

Uh oh!

openshift-bot commented Jan 20, 2022

Uh oh!

openshift-bot commented Jan 20, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

deads2k commented Jan 21, 2022

Uh oh!

openshift-ci bot commented Jan 21, 2022

Uh oh!

gnufied commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 21, 2022

Uh oh!

openshift-bot commented Jan 22, 2022

gnufied commented Jan 14, 2022 •

edited

Loading