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 vm_agent_version to output of azure_rm_virtualmachine_info #1289

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

Klaas-
Copy link
Contributor

@Klaas- Klaas- commented Oct 16, 2023

SUMMARY

Also return vm_agent_version in azure_rm_virtualmachine_info

This is only returned by API for turned on VMs, so if it's undefined also return "Unknown". "Unknown" + powered on VM means something is wrong with the agent inside the VM.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

azure_rm_virtualmachine_info

Copy link
Collaborator

@Fred-sun Fred-sun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small changes request!

plugins/modules/azure_rm_virtualmachine_info.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_virtualmachine_info.py Outdated Show resolved Hide resolved
plugins/modules/azure_rm_virtualmachine_info.py Outdated Show resolved Hide resolved
@Fred-sun Fred-sun added enhancement New feature or request medium_priority Medium priority work in In trying to solve, or in working with contributors labels Oct 31, 2023
@Fred-sun
Copy link
Collaborator

Fred-sun commented Nov 1, 2023

kingdly ping!

Co-authored-by: Fred-sun <37327967+Fred-sun@users.noreply.github.com>
@Klaas-
Copy link
Contributor Author

Klaas- commented Nov 1, 2023

kingdly ping!

done -- thanks for the review @Fred-sun

@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged and removed work in In trying to solve, or in working with contributors labels Nov 1, 2023
@xuzhang3
Copy link
Collaborator

@klaas LGTM

@xuzhang3 xuzhang3 merged commit ee30130 into ansible-collections:dev Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request medium_priority Medium priority 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.

3 participants