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

Enable static network configuration via vmware guestinfo #2 #107

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sidaf
Copy link

@sidaf sidaf commented Feb 1, 2023

Based on pull request #83 opened on Nov 22, 2021 by kimbernator.

Apply fixes for "pep8: H306: imports not in alphabetical order" and "pep8: W292 no newline at end of file", which hopefully should make this mergeable.

@sidaf
Copy link
Author

sidaf commented Feb 1, 2023

@nwmcsween
Copy link

Is there anything needed to push this past the finish line?

@pbesret
Copy link

pbesret commented Nov 13, 2023

Could someone merge if this PR is ok ?

@ader1990
Copy link
Member

@pbesret I gave it a recheck on the opendev board, as we merged support for the 3.11 Python. Let's continue this discussion on opendev.

Thanks!

@FlorianLaunay
Copy link
Contributor

Hi folks.
Great job! This feature will finally bring VMware datasource to its expected level.
Can we expect a merge soon?

@FlorianLaunay
Copy link
Contributor

Hi @ader1990. This feature will be awesome! Can we expect a merge soon?

@FlorianLaunay
Copy link
Contributor

Hi @ader1990
Could you please close this PR ? This feature was finally brought by f0fe66b in release 1.1.5.

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.

5 participants