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

Add support for existing networks #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kompiz
Copy link

@kompiz kompiz commented Apr 26, 2023

This PR is for adding support to existing networks. This is done by checking the value of the boolean assign_floating_ip - if it's true new networks be created, if false existing networks will be used instead.

I don't have access to a provider for testing if everything works as intended when the var assign_floating_ip is true. Could someone else try this?

If there is any code that is unclear or badly written, don't hesitate to let me know - I appreciate learning how to improve.

@remche
Copy link
Owner

remche commented Apr 27, 2023

@kompiz thanks for your work in this PR. I will review/test asap.

@kompiz
Copy link
Author

kompiz commented May 8, 2023

Is the code very messy? 😅
Let me know if I can help explain what's going on.

@remche
Copy link
Owner

remche commented May 12, 2023

I didn't have time to dive into it but it's very important to me not to introduce breaking changes (i.e. examples/minimal test). I will be really happy to merge the PR once it's ok.
Thanks again for you work on this features !

@kompiz
Copy link
Author

kompiz commented May 15, 2023

Absolutely. There are a few places I'm slightly worried about, I can't test them myself as my provider is a bit different. I'll put comments on them later today.

@stale
Copy link

stale bot commented Jul 14, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Jul 14, 2023
@remche remche added the enhancement New feature or request label Jul 15, 2023
@stale stale bot removed the wontfix This will not be worked on label Jul 15, 2023
@pratik705
Copy link

Hello @kompiz @remche

Is this feature ready for the use? I have a use case where I would like to use existing provider network in the openstack environment.

@remche
Copy link
Owner

remche commented Jul 20, 2023

@pratik705 this PR is not ready to merge now because it would break existing deployment.

I bet you can use the PR branch for your module source and it will use just work though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants