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

OCPBUGS-2085: CARRY: Dockerfile.base: bump to openvswitch2.17.0-37.4.el8fdp #1298

Merged
merged 1 commit into from Oct 13, 2022
Merged

OCPBUGS-2085: CARRY: Dockerfile.base: bump to openvswitch2.17.0-37.4.el8fdp #1298

merged 1 commit into from Oct 13, 2022

Conversation

flavio-fernandes
Copy link
Contributor

This is a continuation of commit
2c797c4

In particular, this bump includes fixes for AVX512 runtime checks:

  • dpif-netdev: Refactor AVX512 runtime checks. (#2100393)
  • dpif-netdev: Fix leak of AVX512 DPIF scratch pad
  • dpif-netdev-extract-avx512: Protect GCC builtin usage

All other changes can be obtained here:
https://access.redhat.com/downloads/content/rhel---8/x86_64/7816/openvswitch2.17/2.17.0-37.4.el8fdp/x86_64/fd431d51/package-changelog

Signed-off-by: Flavio Fernandes flaviof@redhat.com

@flavio-fernandes
CARRY: Dockerfile.base: bump to openvswitch2.17.0-37.4.el8fdp
0f02928 
This is a continuation of commit
openshift@2c797c4

In particular, this bump includes fixes for AVX512 runtime checks:
- dpif-netdev: Refactor AVX512 runtime checks. (#2100393)
- dpif-netdev: Fix leak of AVX512 DPIF scratch pad
- dpif-netdev-extract-avx512: Protect GCC builtin usage

All other changes can be obtained here:
https://access.redhat.com/downloads/content/rhel---8/x86_64/7816/openvswitch2.17/2.17.0-37.4.el8fdp/x86_64/fd431d51/package-changelog

Signed-off-by: Flavio Fernandes <flaviof@redhat.com>
@openshift-ci-robot openshift-ci-robot added jira/severity-critical Referenced Jira bug's severity is critical for the branch this PR is targeting. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Oct 6, 2022
@openshift-ci-robot
Copy link
Contributor

@flavio-fernandes: This pull request references Jira Issue OCPBUGS-2085, which is valid. The bug has been moved to the POST state.

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

Requesting review from QA contact:
/cc @zhiqiangf

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

This is a continuation of commit
2c797c4

In particular, this bump includes fixes for AVX512 runtime checks:

  • dpif-netdev: Refactor AVX512 runtime checks. (#2100393)
  • dpif-netdev: Fix leak of AVX512 DPIF scratch pad
  • dpif-netdev-extract-avx512: Protect GCC builtin usage

All other changes can be obtained here:
https://access.redhat.com/downloads/content/rhel---8/x86_64/7816/openvswitch2.17/2.17.0-37.4.el8fdp/x86_64/fd431d51/package-changelog

Signed-off-by: Flavio Fernandes flaviof@redhat.com

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-robot openshift-ci-robot added the bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. label Oct 6, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 6, 2022

@openshift-ci-robot: GitHub didn't allow me to request PR reviews from the following users: zhiqiangf.

Note that only openshift members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

@flavio-fernandes: This pull request references Jira Issue OCPBUGS-2085, which is valid. The bug has been moved to the POST state.

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

Requesting review from QA contact:
/cc @zhiqiangf

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

This is a continuation of commit
2c797c4

In particular, this bump includes fixes for AVX512 runtime checks:

  • dpif-netdev: Refactor AVX512 runtime checks. (#2100393)
  • dpif-netdev: Fix leak of AVX512 DPIF scratch pad
  • dpif-netdev-extract-avx512: Protect GCC builtin usage

All other changes can be obtained here:
https://access.redhat.com/downloads/content/rhel---8/x86_64/7816/openvswitch2.17/2.17.0-37.4.el8fdp/x86_64/fd431d51/package-changelog

Signed-off-by: Flavio Fernandes flaviof@redhat.com

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.

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 trozet and tssurya October 6, 2022 18:36
@flavio-fernandes
Copy link
Contributor Author

@zshi-redhat PTAL!

@flavio-fernandes
Copy link
Contributor Author

/assign @zshi-redhat
/assign @dcbw

@abhat
Copy link
Contributor

abhat commented Oct 6, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 6, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 6, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abhat, flavio-fernandes

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 6, 2022
@tssurya
Copy link
Contributor

tssurya commented Oct 6, 2022

/hold
downstream merge is more important if this goes in first it will churn the merge pool. This can get in after FF also I think?

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 6, 2022
@flavio-fernandes
Copy link
Contributor Author

/hold downstream merge is more important if this goes in first it will churn the merge pool. This can get in after FF also I think?

Yes. I agree.

@zshi-redhat
Copy link
Contributor

@tssurya shall we unhold given the downstream merge pr is closed?

@tssurya
Copy link
Contributor

tssurya commented Oct 11, 2022

/hold cancel
got #1300 in

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 11, 2022
@tssurya
Copy link
Contributor

tssurya commented Oct 11, 2022

/retest

@flavio-fernandes
Copy link
Contributor Author

/test e2e-metal-ipi-ovn-dualstack

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 945ffe2 and 2 for PR HEAD 0f02928 in total

@flavio-fernandes
Copy link
Contributor Author

/test e2e-metal-ipi-ovn-dualstack

1 similar comment
@dcbw
Copy link
Member

dcbw commented Oct 12, 2022

/test e2e-metal-ipi-ovn-dualstack

@dcbw
Copy link
Member

dcbw commented Oct 13, 2022

/test e2e-metal-ipi-ovn-ipv6
devscripts setup failures long before networking happens

+(utils.sh:400): setup_release_mirror(): [[ -z '' ]]
+(utils.sh:401): setup_release_mirror(): setup_legacy_release_mirror
+(utils.sh:413): setup_legacy_release_mirror(): oc adm release mirror --insecure=true -a /opt/dev-scripts/pull_secret.json --from registry.build05.ci.openshift.org/ci-op-f19xty7v/release@sha256:c47861ddd91b369b861b56c39087bc6161f9d1bde829d987f5b8c9148174157c --to-release-image virthost.ostest.test.metalkube.org:5000/localimages/local-release-image:c47861ddd91b369b861b56c39087bc6161f9d1bde829d987f5b8c9148174157c --to virthost.ostest.test.metalkube.org:5000/localimages/local-release-image
+(utils.sh:418): setup_legacy_release_mirror(): tee /opt/dev-scripts/registry/ostest-image_mirror-c47861ddd91b369b861b56c39087bc6161f9d1bde829d987f5b8c9148174157c.log
error: unable to load image-references from release payload: invalid character 'E' after top-level value
+(utils.sh:1): setup_legacy_release_mirror(): removetmp
+(utils.sh:660): removetmp(): '[' -n ' /tmp/release-update--4JP5NyFTyv' ']'
+(utils.sh:660): removetmp(): rm -rf /tmp/release-update--4JP5NyFTyv
make: *** [Makefile:42: ironic] Error 1
Fetching kubeconfig, other credentials...
scp: /root/dev-scripts/ocp/*/auth/kubeconfig: No such file or directory
{"component":"entrypoint","error":"wrapped process failed: exit status 1","file":"k8s.io/test-infra/prow/entrypoint/run.go:79","func":"k8s.io/test-infra/prow/entrypoint.Options.Run","level":"error","msg":"Error executing test process","severity":"error","time":"2022-10-12T23:38:48Z"}
error: failed to execute wrapped command: exit status 1

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 13, 2022 •
edited

@flavio-fernandes: 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-ovn-windows 0f02928 link false /test e2e-aws-ovn-windows
ci/prow/e2e-aws-ovn-hypershift 0f02928 link false /test e2e-aws-ovn-hypershift
ci/prow/e2e-vsphere-windows 0f02928 link false /test e2e-vsphere-windows
ci/prow/okd-e2e-gcp-ovn 0f02928 link false /test okd-e2e-gcp-ovn
ci/prow/e2e-ovn-hybrid-step-registry 0f02928 link false /test e2e-ovn-hybrid-step-registry
ci/prow/e2e-openstack-ovn 0f02928 link false /test e2e-openstack-ovn

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.

@dcbw
Copy link
Member

dcbw commented Oct 13, 2022

/override ci/prow/e2e-metal-ipi-ovn-ipv6
Currently permafailing https://coreos.slack.com/archives/CBN38N3MW/p1665659723466089?thread_ts=1665659723.263569&cid=CBN38N3MW

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 13, 2022

@dcbw: Overrode contexts on behalf of dcbw: ci/prow/e2e-metal-ipi-ovn-ipv6

In response to this:

/override ci/prow/e2e-metal-ipi-ovn-ipv6
Currently permafailing https://coreos.slack.com/archives/CBN38N3MW/p1665659723466089?thread_ts=1665659723.263569&cid=CBN38N3MW

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-merge-robot openshift-merge-robot merged commit ecd74ad into openshift:master Oct 13, 2022
@openshift-ci-robot
Copy link
Contributor

@flavio-fernandes: All pull requests linked via external trackers have merged:

Jira Issue OCPBUGS-2085 has been moved to the MODIFIED state.

In response to this:

This is a continuation of commit
2c797c4

In particular, this bump includes fixes for AVX512 runtime checks:

  • dpif-netdev: Refactor AVX512 runtime checks. (#2100393)
  • dpif-netdev: Fix leak of AVX512 DPIF scratch pad
  • dpif-netdev-extract-avx512: Protect GCC builtin usage

All other changes can be obtained here:
https://access.redhat.com/downloads/content/rhel---8/x86_64/7816/openvswitch2.17/2.17.0-37.4.el8fdp/x86_64/fd431d51/package-changelog

Signed-off-by: Flavio Fernandes flaviof@redhat.com

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

@trozet trozet Awaiting requested review from trozet

@tssurya tssurya Awaiting requested review from tssurya

Assignees

@abhat abhat

@zshi-redhat zshi-redhat

@dcbw dcbw

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. jira/severity-critical Referenced Jira bug's severity is critical for the branch this PR is targeting. jira/valid-bug Indicates that a referenced Jira 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.

None yet
7 participants
@flavio-fernandes @openshift-ci-robot @abhat @tssurya @zshi-redhat @dcbw @openshift-merge-robot