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.

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

Sync upstream: 2021-6-23 #526

Merged
merged 17 commits into from
Jun 25, 2021
Merged

Conversation

pliurh
Copy link
Contributor

@pliurh pliurh commented Jun 23, 2021

No description provided.

@pliurh pliurh requested a review from zshi-redhat June 23, 2021 06:15
@openshift-ci openshift-ci bot requested review from fedepaol and fepan June 23, 2021 06:16
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 23, 2021
@pliurh
Copy link
Contributor Author

pliurh commented Jun 23, 2021

/hold

@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 Jun 23, 2021
@pliurh
Copy link
Contributor Author

pliurh commented Jun 23, 2021

/retest

1 similar comment
@pliurh
Copy link
Contributor Author

pliurh commented Jun 23, 2021

/retest

manuelbuil and others added 17 commits June 23, 2021 22:07
Signed-off-by: Manuel Buil <[email protected]>
Signed-off-by: Kennelly, Martin <[email protected]>
Upgrade sprig to ver.3 which updates CWE-330
vulnerable goutils to 1.1.1.
Sprig ver.3 retains the same Go API as ver.2.

Signed-off-by: Martin Kennelly <[email protected]>
* Golang code to ensure VFs created during testing remain
  in test network namespace (KinD worker network namespace)

* Shell script to fetch test tools, execute E2E tests using
  KinD and teardown KinD test environment

Signed-off-by: Kennelly, Martin <[email protected]>
Signed-off-by: Patryk Strusiewicz-Surmacki <[email protected]>
Move PF/VF netns monitoring to test script to prevent
escape processes during testing.

Add logging to PF/VF monitoring. Increase in test log level
required to view its output.

PF/VF monitor does not exist when PF not found, assumes it is
in correct netns and continues.

Rename execution of KinD cluster and testing file to be
consistent with existing files.

Move e2e doc to docs.

Signed-off-by: Kennelly, Martin <[email protected]>
Signed-off-by: Martin Kennelly <[email protected]>
Label node-role.kubernetes.io/master has traditionally no value
associated inherited from kubeadm deplyments but some platforms use
"true" as value. Use node affinity instead of node selector ignoring the
value of the label.
Use a ConfigMap 'supported-nic-ids' to store the NIC model list
to replace the current hardcoded list. Deprecate the ConfigMap
'unsupported-nic-ids'.

Also deprecate the machanism which allow users can change the
content of ConfigMap on the fly. Now users have to restart the
webhook and config daemon pods after changing the ConfigMap.
Sync content of MutatingWebhookConfiguration and ValidatingWebhookConfiguration
CRs when needed.
@pliurh
Copy link
Contributor Author

pliurh commented Jun 24, 2021

/unhold

@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 Jun 24, 2021
@zshi-redhat
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 25, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pliurh, zshi-redhat

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-merge-robot openshift-merge-robot merged commit f9e3d84 into openshift:master Jun 25, 2021
zeeke added a commit to zeeke/sriov-network-operator that referenced this pull request Nov 23, 2023
…oller_runtime

Update controller-runtime to v0.16.3
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. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants