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

Encapsulate the host interface name into the VM pool and tap manager #811

Conversation

CuriousGeorgiy
Copy link
Member

@CuriousGeorgiy CuriousGeorgiy commented Sep 11, 2023

Fix typo in getEnvVal error message

Encapsulate the host interface name into the VM pool and tap manager

Currently, the host interface name is separated from the VM pool and tap
manager, and as a result, it needs to be passed to all VM pool methods.

This is inconvenient, and so encapsulate the host interface name into the
VM pool and tap manager.

Closes #810
Part of #794

Implementation Notes ⚒️

  • The host interface name is encapsulated into the VM pool and tap manager.

External Dependencies 🍀

  • N/A.

Breaking API Changes ⚠️

  • N/A.

Simply specify none (N/A) if not applicable.

@CuriousGeorgiy CuriousGeorgiy force-pushed the encapsulate-host-interface-into-vm-pool-and-tap-manager branch 3 times, most recently from 24f3340 to 254965a Compare September 11, 2023 10:03
Copy link
Member

@ustiugov ustiugov left a comment

Choose a reason for hiding this comment

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

thx. please merge when tests pass.

Signed-off-by: Georgiy Lebedev <lebedev.gk@phystech.edu>
Currently, the host interface name is separated from the VM pool and tap
manager, and as a result, it needs to be passed to all VM pool methods.

This is inconvenient, so encapsulate the host interface name into the VM
pool and tap manager.

Closes vhive-serverless#810
Part of vhive-serverless#794

Signed-off-by: Georgiy Lebedev <lebedev.gk@phystech.edu>
@CuriousGeorgiy CuriousGeorgiy force-pushed the encapsulate-host-interface-into-vm-pool-and-tap-manager branch from 254965a to c27b6df Compare September 12, 2023 06:31
@CuriousGeorgiy CuriousGeorgiy enabled auto-merge (rebase) September 12, 2023 09:30
@ustiugov ustiugov merged commit 3a22b77 into vhive-serverless:main Sep 14, 2023
23 checks passed
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.

Encapsulate the host interface name into the VM pool and tap manager
2 participants