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

Use PCI addr in veth alias, fallback to scanning #42

Merged
merged 1 commit into from
Apr 23, 2020
Merged

Conversation

krsna1729
Copy link
Member

This will be primarily for k8s setup. To test locally

docker exec bess ip link set ens803f2 alias '0000:07:02.0'
docker exec bess ip link set ens803f3 alias '0000:07:02.1'
docker exec bess ./bessctl run spgwu

Signed-off-by: Saikrishna Edupuganti saikrishna.edupuganti@intel.com

This will be primarily for k8s setup. To test locally

docker exec bess ip link set ens803f2 alias '0000:07:02.0'
docker exec bess ip link set ens803f3 alias '0000:07:02.1'
docker exec bess ./bessctl run spgwu

Signed-off-by: Saikrishna Edupuganti <saikrishna.edupuganti@intel.com>
@krsna1729 krsna1729 requested a review from ajamshed April 17, 2020 03:17
@onf-cla-manager
Copy link

Hi @krsna1729, this is the ONF bot 🤖 I'm glad you want to contribute to our projects! However, before accepting your contribution, we need to ask you to sign a Contributor License Agreement (CLA). You can do it online, it will take only a few minutes:

✒️ 👉 https://cla.opennetworking.org

After signing, make sure to add your Github user ID krsna1729 to the agreement.

@krsna1729
Copy link
Member Author

/cc @hyunsun

Copy link
Contributor

@ajamshed ajamshed left a comment

Choose a reason for hiding this comment

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

Will test this once topics/enhancements is merged.

@ajamshed ajamshed merged commit 326cf42 into master Apr 23, 2020
@ajamshed ajamshed deleted the use-ifalias branch April 23, 2020 05:25
This pull request was closed.
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.

2 participants