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 latest CRDs to helm chart #167

Merged
merged 1 commit into from
Aug 4, 2021
Merged

Conversation

e0ne
Copy link
Collaborator

@e0ne e0ne commented Jul 23, 2021

No description provided.

@e0ne e0ne marked this pull request as draft July 27, 2021 06:34
@e0ne e0ne marked this pull request as ready for review July 27, 2021 14:42
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 27, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 27, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
@@ -0,0 +1,41 @@
# permissions to do leader election.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i dont think this is used today when running make deploy-k8s

IMO you need to update the role.yaml template

perhaps other .yaml files from deploy folder are out of sync

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, I'll update my PR with it

@e0ne e0ne requested a review from moshe010 July 27, 2021 15:58
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 28, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 28, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 28, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 28, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 28, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 28, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
e0ne added a commit to e0ne/network-operator that referenced this pull request Jul 29, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
Copy link
Collaborator

@adrianchiris adrianchiris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

e0ne added a commit to e0ne/network-operator that referenced this pull request Aug 1, 2021
We can't use SR-IOV Network Operator chart from the official repo until
k8snetworkplumbingwg/sriov-network-operator#167
will be merged and new chart will be punlished.

This patch syncs the latest changes from the upstrem to Network Operator
repo to deploy the lates version of the SR-IOV Network Operator.

Signed-off-by: Ivan Kolodiazhnyi <[email protected]>
@adrianchiris
Copy link
Collaborator

@pliurh @zshi-redhat can this be merged ?

@pliurh pliurh merged commit 1980c0b into k8snetworkplumbingwg:master Aug 4, 2021
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.

4 participants