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

Openshift rebase #89

Merged
merged 10 commits into from Nov 27, 2019
Merged

Conversation

dulek
Copy link
Contributor

@dulek dulek commented Nov 27, 2019

No description provided.

Danil Golov and others added 10 commits November 21, 2019 16:03
This commit contains spec file for nested-dpdk
case in kuryr-kubernetes. It describes how to setup
and use it.

Change-Id: I4cb83d49b56b16382b00edacf8cfbdd814f0d504
Implements: blueprint nested-dpdk-support
Signed-off-by: Danil Golov <d.golov@samsung.com>
Default vif should use the ifname supplied by CNI ADD request, instead
of using a fixed value (This helps to work with multus cni as well)

This is kind of a hack, as the vifs dict still uses the DEFAULT_IFNAME
for keeping track of the default vif, but changing that would require
some refactoring (as we do not know the requested ifname in controller,
it is only known when kuryr-cni-daemon tries to plug the vif)

Closes-Bug: 1852380
Change-Id: I4b185b8ca811793a7dfd459d2d89346119709acc
Signed-off-by: Yash Gupta <y.gupta@samsung.com>
Allowing user to configure the addtional_ifname_prefix allows
kuryr-kubernetes to work more smoothly with other CNI plugins (when
using multus or multiple chained CNIs for example).

Change-Id: I4ce35e939c1564c1ba0bd2ae55c09fa24a35c9ed
Signed-off-by: Yash Gupta <y.gupta@samsung.com>
Change-Id: I6c847a67468d2ed5e2b9efb76fd3c35842c71191
@openshift-ci-robot openshift-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Nov 27, 2019
@MaysaMacedo
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 27, 2019
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dulek, MaysaMacedo

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 7800803 into openshift:master Nov 27, 2019
MaysaMacedo pushed a commit to MaysaMacedo/kuryr-kubernetes-1 that referenced this pull request Jul 1, 2021
configure endpoints to expose openshift-sdn metrics to Prometheus
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. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants