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

Restore the return of 'public_ipv4_address' and fix the bug that the VM does not have a public IP address #1379

Conversation

Fred-sun
Copy link
Collaborator

SUMMARY

Restore the return of 'public_ipv4_address' and fix the bug that the VM does not have a public IP address

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

azure_rm.yml

ADDITIONAL INFORMATION

@donjuanmon
Copy link

+1, this bug is also affecting our environment and we are unable to sync our inventory.

@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged high_priority High priority inventory plugin/Inventory/azure_rm.py related issues labels Dec 18, 2023
@Fred-sun
Copy link
Collaborator Author

@donjuanmon We are very sorry for the trouble caused to you. We will push forward the merger as soon as possible. Thank you!

@donjuanmon
Copy link

@donjuanmon We are very sorry for the trouble caused to you. We will push forward the merger as soon as possible. Thank you!

No worries, I recreated our execution environment with your PR fork. Can confirm inventory sync is working for hosts without a public IP. Looking forward to the merge!

@xuzhang3 xuzhang3 merged commit 6277820 into ansible-collections:dev Dec 19, 2023
@em-
Copy link

em- commented Dec 19, 2023

With version 2.1.1 ansible_host is being set to an empty list ([]) for my VMs that do not have a public IP address but only a private one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high_priority High priority inventory plugin/Inventory/azure_rm.py related issues ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants