Bug 2027983: [4.9.z] Make config parsing more resilient for unknown fields by creydr · Pull Request #860 · openshift/ovn-kubernetes

creydr · 2021-12-01T08:17:07Z

Cherry-pick 4392857 into 4.9

Signed-off-by: Christoph Stäbler <cstabler@redhat.com> (cherry picked from commit 4392857)

openshift-ci · 2021-12-01T08:17:14Z

@creydr: This pull request references Bugzilla bug 2027983, 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.

6 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target release (4.9.z) matches configured target release for branch (4.9.z)
bug is in the state NEW, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)
dependent bug Bugzilla bug 1988440 is in the state VERIFIED, which is one of the valid states (VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE))
dependent Bugzilla bug 1988440 targets the "4.10.0" release, which is one of the valid target releases: 4.10.0
bug has dependents

Requesting review from QA contact:
/cc @anuragthehatter

Details

In response to this:

Bug 2027983: Make config parsing more resilient for unknown fields

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 · 2021-12-01T08:22:18Z

@creydr: 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:

Bug 2027983 [4.9.z]: Make config parsing more resilient for unknown fields

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 · 2021-12-01T08:23:05Z

@creydr: This pull request references Bugzilla bug 2027983, which is valid.

6 validation(s) were run on this bug

bug is open, matching expected state (open)
bug target release (4.9.z) matches configured target release for branch (4.9.z)
bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)
dependent bug Bugzilla bug 1988440 is in the state VERIFIED, which is one of the valid states (VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE))
dependent Bugzilla bug 1988440 targets the "4.10.0" release, which is one of the valid target releases: 4.10.0
bug has dependents

Requesting review from QA contact:
/cc @anuragthehatter

Details

In response to this:

Bug 2027983: [4.9.z] Make config parsing more resilient for unknown fields

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.

creydr · 2021-12-01T11:19:05Z

/retest-required

creydr · 2021-12-01T19:37:56Z

/retest

creydr · 2021-12-02T15:46:59Z

/retest

trozet · 2021-12-02T16:21:04Z

/label backport-risk-assessed

trozet · 2021-12-02T16:21:28Z

/lgtm

openshift-ci · 2021-12-02T16:27:18Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: creydr, trozet

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:

~~OWNERS~~ [trozet]

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

openshift-bot · 2021-12-02T19:34:16Z

/retest-required

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

openshift-bot · 2021-12-02T20:00:21Z

/retest-required

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

openshift-bot · 2021-12-02T20:39:13Z

/retest-required

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

openshift-bot · 2021-12-02T21:44:14Z

/retest-required

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

openshift-bot · 2021-12-02T23:02:14Z

/retest-required

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

openshift-bot · 2021-12-03T01:25:16Z

/retest-required

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

openshift-bot · 2021-12-03T03:22:15Z

/retest-required

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

openshift-bot · 2021-12-03T04:01:14Z

/retest-required

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

openshift-bot · 2021-12-03T05:38:15Z

/retest-required

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

openshift-bot · 2021-12-03T06:38:15Z

/retest-required

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

openshift-bot · 2021-12-03T07:14:19Z

/retest-required

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

openshift-bot · 2021-12-03T08:14:34Z

/retest-required

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

openshift-bot · 2021-12-03T08:26:13Z

/retest-required

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

openshift-bot · 2021-12-03T08:38:12Z

/retest-required

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

openshift-bot · 2021-12-03T09:03:15Z

/retest-required

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

openshift-bot · 2021-12-03T09:39:16Z

/retest-required

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

openshift-bot · 2021-12-03T09:51:15Z

/retest-required

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

openshift-bot · 2021-12-03T10:27:15Z

/retest-required

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

openshift-bot · 2021-12-03T10:39:19Z

/retest-required

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

creydr · 2021-12-03T10:51:52Z

/skip
as openshift-bot keeps triggering retest-required even no required job is failing

zhaozhanqi · 2021-12-06T01:18:45Z

/label cherry-pick-approved

openshift-ci · 2021-12-06T03:14:16Z

@creydr: The following test 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/4.9-upgrade-from-stable-4.8-e2e-aws-ovn-upgrade	`d8033d0`	link	false	`/test 4.9-upgrade-from-stable-4.8-e2e-aws-ovn-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.

openshift-bot · 2021-12-06T03:20:28Z

/retest-required

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

openshift-bot · 2021-12-06T03:44:32Z

/retest-required

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

openshift-ci · 2021-12-06T06:21:28Z

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

openshift/ovn-kubernetes#860

Bugzilla bug 2027983 has been moved to the MODIFIED state.

Details

In response to this:

Bug 2027983: [4.9.z] Make config parsing more resilient for unknown fields

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.

creydr · 2021-12-08T20:24:56Z

/cherry-pick release-4.8

openshift-cherrypick-robot · 2021-12-08T20:25:56Z

@creydr: new pull request created: #868

Details

In response to this:

/cherry-pick release-4.8

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.

Make config parsing more resilient for unknown fields

d8033d0

Signed-off-by: Christoph Stäbler <cstabler@redhat.com> (cherry picked from commit 4392857)

openshift-ci bot added bugzilla/severity-high Referenced Bugzilla bug's severity is high 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 Dec 1, 2021

openshift-ci bot requested review from abhat, anuragthehatter and danwinship December 1, 2021 08:17

creydr changed the title ~~Bug 2027983: Make config parsing more resilient for unknown fields~~ Bug 2027983 [4.9.z]: Make config parsing more resilient for unknown fields Dec 1, 2021

openshift-ci bot removed bugzilla/severity-high Referenced Bugzilla bug's severity is high 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 Dec 1, 2021

creydr changed the title ~~Bug 2027983 [4.9.z]: Make config parsing more resilient for unknown fields~~ Bug 2027983: [4.9.z] Make config parsing more resilient for unknown fields Dec 1, 2021

openshift-ci bot added bugzilla/severity-high Referenced Bugzilla bug's severity is high 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 Dec 1, 2021

openshift-ci bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label Dec 2, 2021

openshift-ci bot assigned trozet Dec 2, 2021

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 2, 2021

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 2, 2021

openshift-ci bot added the cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. label Dec 6, 2021

openshift-merge-robot merged commit 22159e2 into openshift:release-4.9 Dec 6, 2021

openshift-cherrypick-robot mentioned this pull request Dec 8, 2021

[release-4.8] Bug 2030465: Make config parsing more resilient for unknown fields #868

Merged

Comments

Conversation

creydr commented Dec 1, 2021

Uh oh!

openshift-ci bot commented Dec 1, 2021

Uh oh!

openshift-ci bot commented Dec 1, 2021

Uh oh!

openshift-ci bot commented Dec 1, 2021

Uh oh!

creydr commented Dec 1, 2021

Uh oh!

creydr commented Dec 1, 2021

Uh oh!

creydr commented Dec 2, 2021

Uh oh!

trozet commented Dec 2, 2021

Uh oh!

trozet commented Dec 2, 2021

Uh oh!

openshift-ci bot commented Dec 2, 2021

Uh oh!

openshift-bot commented Dec 2, 2021

Uh oh!

openshift-bot commented Dec 2, 2021

Uh oh!

openshift-bot commented Dec 2, 2021

Uh oh!

openshift-bot commented Dec 2, 2021

Uh oh!

openshift-bot commented Dec 2, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

openshift-bot commented Dec 3, 2021

Uh oh!

creydr commented Dec 3, 2021

Uh oh!

zhaozhanqi commented Dec 6, 2021

Uh oh!

openshift-ci bot commented Dec 6, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openshift-bot commented Dec 6, 2021

Uh oh!

openshift-bot commented Dec 6, 2021

Uh oh!

openshift-ci bot commented Dec 6, 2021

Uh oh!

creydr commented Dec 8, 2021

Uh oh!

openshift-cherrypick-robot commented Dec 8, 2021

Uh oh!

Reviewers

Assignees

Labels

openshift-ci bot commented Dec 6, 2021 •

edited

Loading