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

Add hello work flow #2

Open
wants to merge 55 commits into
base: master
Choose a base branch
from
Open

Add hello work flow #2

wants to merge 55 commits into from

Conversation

abdallahyas
Copy link
Owner

No description provided.

pliurh and others added 30 commits July 5, 2021 02:26
…ction

Network Resource Injector (NRI) has switches to config map to enable
optional features such as enableHugePageDownApi,
enableHonorExistingResources etc. This commit allows Operator to
create controlSwitch configmap for NRI and removes injectHugepageDownApi
from webhook-server container arguments.
Bump controller-gen to v0.4.1, kustomize to v3.8.7
1. use json format for config file
2. split VF creation and switchdev configuration in configure-switchdev.sh
…rol-switch

NRI: use control switch configmap to enable hugepage injection
ghcr.io/k8snetworkplumbingwg is an official image repository for
k8snetworkplumbingwg organization witch is updated after each merged
pull request.
Use images from ghcr.io/k8snetworkplumbingwg repository
mstflint in CentOS repository is too old and doesn't support latest
Nvidia Nics like BlueField 2.

This patch makes mstflint package location configurable so we can
build image with required version and location of it.
…g#171)

After moving sriovnetworkv1.NicIdMap into ConfigMap we need to
initialize it's data on operator startup.

Signed-off-by: Ivan Kolodyazhny <[email protected]>
Until k8snetworkplumbingwg/sriov-network-device-plugin#276
wiill be fixed we need to restart device plugin pod each time after
SR-IOV Network Operator plugin applied. It's needed because plugin
could change a number of VF resources even if config is not changed.
…tflint

Make mstflint configurable for config daemon
Using 2 managers for the same controllers causes inconsistency in
objects reconcilation. For example we saw a problem with some SriovNetwork
update events not reaching the controller (Reconcile was not triggered).
Using the same Mgr as the specified client for the controller solves these kind of problems.

Signed-off-by: Ori Braunshtein <[email protected]>
Use GlobalMgr for SriovNetwork controllers
…k_finalizers

Remove SriovNetwork finalizers on controller shutdown
Use CRD v1 in SriovNetworkPoolConfig patches
…lure_policy_upate

Set webhook failurePolicy to Ignore on controller pod shutdown
Don't check MCP state when node in Draining_MCP_Paused state
github.com/cespare/xxhash/v2 v2.1.1 is incompatible with Go 1.17 (see cespare/xxhash#54)
Here we bump up the dependency to solve the issue.

Signed-off-by: Federico Paolinelli <[email protected]>
Update github.com/cespare/xxhash/v2 to v2.1.2
This commit add a support to the sriov-network-operator
to request from the sriov-device-plugin to mount the /dev/vhost-net device into the container.

Signed-off-by: Sebastian Sch <[email protected]>
SchSeba and others added 8 commits November 23, 2021 12:12
…z1875338-workaround

Revert "Add a workaround for BZ 1875338"
This commit extend the sriov-network-config-daemon to try and load the tun device on start.

Signed-off-by: Sebastian Sch <[email protected]>
This patch adds a Jenkins file example, it also modifies the E2E
kind script to support a new mode of operation: the bash service.
The modification was done to support running the scripts in a limited
privilege shell.
To set the GUID to VF, we need to set the admin MAC and reload the
VF driver. It will take a long time if users set big 'numVFs'. This
PR changes the logic to only reload the VF driver when isRdma is
set. So that, users not using ROCE will no longer need to wait for
the VF driver reload.
Reload VF driver only when rdma is enabled
@github-actions
Copy link

github-actions bot commented Dec 2, 2021

Thanks for your PR,
To run vendors CIs use one of:

  • test-all: To run all tests for all vendors.
  • test-e2e-all: To run all E2E tests for all vendors.
  • test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.
    To skip the vendors CIs use one of:
  • skip-all: To skip all tests for all vendors.
  • skip-e2e-all: To skip all E2E tests for all vendors.
  • skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@abdallahyas abdallahyas force-pushed the add-hello-work-flow branch 19 times, most recently from 2b3f2e1 to 628c3c5 Compare December 7, 2021 13:40
Added the pr-ci-triggers github workflow to print out available
vendors CIs trigger phrases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants