-
Notifications
You must be signed in to change notification settings - Fork 77
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
molecule-vagrant: Improve network handling #99
Labels
Comments
apatard
added a commit
to apatard/molecule-plugins
that referenced
this issue
Feb 15, 2023
Current code doesn't ensure that the network name is valid. According to current Vagrant documentation, it can only be: - public_network - private_network - forwarded_port Moreover, the same documentation is saying that public network configuration may be reduced to: config.vm.network "public_network" So add needed code to support interfaces without any options. On the test side: - modify the network test to check that the generated Vagrantfile is correct when no option is specified - add a test to ensure invalid network name are caught. Fixes: ansible-community#99 Signed-off-by: Arnaud Patard <[email protected]>
apatard
added a commit
to apatard/molecule-plugins
that referenced
this issue
Mar 21, 2023
Current code doesn't ensure that the network name is valid. According to current Vagrant documentation, it can only be: - public_network - private_network - forwarded_port Moreover, the same documentation is saying that public network configuration may be reduced to: config.vm.network "public_network" So add needed code to support interfaces without any options. On the test side: - modify the network test to check that the generated Vagrantfile is correct when no option is specified - add a test to ensure invalid network name are caught. Fixes: ansible-community#99 Signed-off-by: Arnaud Patard <[email protected]>
apatard
added a commit
to apatard/molecule-plugins
that referenced
this issue
Mar 21, 2023
Current code doesn't ensure that the network name is valid. According to current Vagrant documentation, it can only be: - public_network - private_network - forwarded_port Moreover, the same documentation is saying that public network configuration may be reduced to: config.vm.network "public_network" So add needed code to support interfaces without any options. On the test side: - modify the network test to check that the generated Vagrantfile is correct when no option is specified - add a test to ensure invalid network name are caught. Fixes: ansible-community#99 Signed-off-by: Arnaud Patard <[email protected]>
apatard
added a commit
to apatard/molecule-plugins
that referenced
this issue
Apr 12, 2023
Current code doesn't ensure that the network name is valid. According to current Vagrant documentation, it can only be: - public_network - private_network - forwarded_port Moreover, the same documentation is saying that public network configuration may be reduced to: config.vm.network "public_network" So add needed code to support interfaces without any options. On the test side: - modify the network test to check that the generated Vagrantfile is correct when no option is specified - add a test to ensure invalid network name are caught. Fixes: ansible-community#99 Signed-off-by: Arnaud Patard <[email protected]>
apatard
added a commit
to apatard/molecule-plugins
that referenced
this issue
Apr 28, 2023
Current code doesn't ensure that the network name is valid. According to current Vagrant documentation, it can only be: - public_network - private_network - forwarded_port Moreover, the same documentation is saying that public network configuration may be reduced to: config.vm.network "public_network" So add needed code to support interfaces without any options. On the test side: - modify the network test to check that the generated Vagrantfile is correct when no option is specified - add a test to ensure invalid network name are caught. Fixes: ansible-community#99 Signed-off-by: Arnaud Patard <[email protected]>
apatard
added a commit
to apatard/molecule-plugins
that referenced
this issue
Jun 26, 2023
Current code doesn't ensure that the network name is valid. According to current Vagrant documentation, it can only be: - public_network - private_network - forwarded_port Moreover, the same documentation is saying that public network configuration may be reduced to: config.vm.network "public_network" So add needed code to support interfaces without any options. On the test side: - modify the network test to check that the generated Vagrantfile is correct when no option is specified - add a test to ensure invalid network name are caught. Fixes: ansible-community#99 Signed-off-by: Arnaud Patard <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From a irc discussion, we should:
The text was updated successfully, but these errors were encountered: