这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@devsunb
Copy link

@devsunb devsunb commented Jun 2, 2024

@CLAassistant
Copy link

CLAassistant commented Jun 2, 2024

CLA assistant check
All committers have signed the CLA.

@edigaryev
Copy link
Collaborator

Softnet's primary goal is VM network isolation.

Why use it if you don't need that property? 🤔

@devsunb
Copy link
Author

devsunb commented Jun 2, 2024

I haven't found a way to avoid the PRIVATE flag when a VM network interface is added as a member to a bridge network interface while using the default shared NAT network, so I tried using softnet. I don't have a lot of knowledge about this, so I think I may have used the wrong method. Is there any way to use the default shared NAT network and allow VM to VM communication?

@edigaryev
Copy link
Collaborator

I haven't found a way to avoid the PRIVATE flag when a VM network interface is added as a member to a bridge network interface while using the default shared NAT network

Interesting. When we were developing Softnet that wasn't the case on macOS 12 (Monterey), but that seems to be changed since macOS Ventura (13)/Sonoma (14).

Have you tried using --net-bridged instead? It doesn't have such limitation.

@devsunb
Copy link
Author

devsunb commented Jun 2, 2024

Yes. Using --net-bridged works fine, but I need an independent NAT network within my macOS host.

@edigaryev
Copy link
Collaborator

It feels to me that the naming of --net-softnet-disable-isolation is an oxymoron and that it'll make lots of people scratch their head trying to figure out what Softnet even offers in terms of security after all.

What I propose instead is to disable the interface isolation automatically when --net-softnet-allow=0.0.0.0/0 is passed to Tart.

This way no additional command-line arguments needs to be introduced. What do you think?

@edigaryev
Copy link
Collaborator

Superseded by #853.

@edigaryev edigaryev closed this Jul 2, 2024
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.

3 participants