VirtualMachinesOperations.get() takes 3 positional arguments but 4 were given

[RyanMesser]

• Package Name: azure-mgmt-compute
• Package Version: 30.0.0
• Operating System: Ubuntu 22.04 LTS on Azure
• Python Version: 3.10.6

Describe the bug
A clear and concise description of what the bug is.

When running fence_azure_arm --msi --plug=host --action=status (or any other action) the following error is recieved
VirtualMachinesOperations.get() takes 3 positional arguments but 4 were given

If I install version 29.0.0 this issue isn't present, not sure if bug or missing information that is needed.

To Reproduce
Steps to reproduce the behavior:

1. ssh/cli to Ubuntu host
2. Run above fence_azure_arm command and error is seen
3. Other than status the action doesn't complete

Expected behavior
A clear and concise description of what you expected to happen.
For actions such as "off" and "reboot" the host to reboot or stop

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

[kashifkhan]

Thank you for the feedback @RyanMesser . We will investigate and get back to you asap.

[msyyc]

Hi @RyanMesser I think your error is related with expand that is changed from positional argument to keyword-only argument since expand is query parameter. It can be fixed by:

...
client.virtual_machines.get(..., expand="xxx")

[RyanMesser]

Hi @RyanMesser I think your error is related with expand that is changed from positional argument to keyword-only argument since expand is query parameter. It can be fixed by:

...
client.virtual_machines.get(..., expand="xxx")

Is this a fix you're going to release or is this something i need to do manually on each host?

[msyyc]

You need to do it manually.

[github-actions]

Hi @RyanMesser. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text "/unresolve" to remove the "issue-addressed" label and continue the conversation.

[github-actions]

Hi @RyanMesser, since you haven’t asked that we /unresolve the issue, we’ll close this out. If you believe further discussion is needed, please add a comment /unresolve to reopen the issue.

[RyanMesser]

/unresolve

I think more information is needed for this as I don't feel it really awnsers the question or provides enough context for the awnser.

Hi @RyanMesser, since you haven’t asked that we /unresolve the issue, we’ll close this out. If you believe further discussion is needed, please add a comment /unresolve to reopen the issue.

[msyyc]

@RyanMesser, Is your source code here? https://github.com/ClusterLabs/fence-agents/tree/main
If so, you just need to change this line to

vmStatus = compute_client.virtual_machines.get(rgName, vmName, expand="instanceView")

or open issue for the tool developer to fix it.

[github-actions]

Hi @RyanMesser. Thank you for opening this issue and giving us the opportunity to assist. To help our team better understand your issue and the details of your scenario please provide a response to the question asked above or the information requested above. This will help us more accurately address your issue.

[github-actions]

Hi @RyanMesser, we're sending this friendly reminder because we haven't heard back from you in 7 days. We need more information about this issue to help address it. Please be sure to give us your input. If we don't hear back from you within 14 days of this comment the issue will be automatically closed. Thank you!