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 num_queues parameter for virtual machine NICs #324

Closed
wants to merge 7 commits into from

Conversation

ckhordiasma
Copy link

implemented num_queues option for nics in nutanix_virtual_machine
objects. If num_queues is changed for an existing nic, the VM will be
restarted. If a new nic is added to a VM with a queue number specified,
the VM will not be restarted.

If any nic parameters besides num_queues are changed, the VM should not
restart.

Also added some lines in the test file, I did not put in much because the tests were not working for me (tests would pass no matter what I added)

implemented num_queues option for nics in nutanix_virtual_machine
objects. If num_queues is changed for an existing nic, the VM will be
restarted. If a new nic is added to a VM with a queue number specified,
the VM will not be restarted.

If any nic parameters besides num_queues are changed, the VM should not
restart.
@siddharth-nutanix
Copy link
Collaborator

@ckhordiasma Can you pull latest master and resolve conflicts ?

@siddharth-nutanix
Copy link
Collaborator

@ckhordiasma
Please resolve the conflicts and rebase with master.

@ckhordiasma
Copy link
Author

ckhordiasma commented Feb 22, 2022 via email

@siddharth-nutanix
Copy link
Collaborator

siddharth-nutanix commented Feb 28, 2022

/ok-to-test

Acceptance test run status: failure

@siddharth-nutanix
Copy link
Collaborator

@ckhordiasma Please check
https://github.com/nutanix/terraform-provider-nutanix/runs/5354524381?check_suite_focus=true

TestAccNutanixVirtualMachine_WithSubnet test case has failed.

@siddharth-nutanix
Copy link
Collaborator

siddharth-nutanix commented Mar 7, 2022

/ok-to-test

Acceptance test run status: failure

@ckhordiasma
Copy link
Author

when I run make fmt or make fmtcheck locally, it doesn't flag me with formatting issues of my code. Is there some other dependency I need?

@siddharth-kulshrestha
Copy link
Contributor

when I run make fmt or make fmtcheck locally, it doesn't flag me with formatting issues of my code. Is there some other dependency I need?

make lint

@siddharth-nutanix
Copy link
Collaborator

siddharth-nutanix commented Mar 9, 2022

/ok-to-test

Acceptance test run status: failure

@abhimutant abhimutant changed the base branch from master to task/m-PR-324 March 10, 2022 06:42
Copy link
Collaborator

@abhimutant abhimutant left a comment

Choose a reason for hiding this comment

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

Merging it to a different branch to check

@siddharth-nutanix
Copy link
Collaborator

siddharth-nutanix commented Mar 12, 2022

/ok-to-test

Acceptance test run status: failure

@bhati-pradeep
Copy link
Collaborator

Closing this PR, as #385 merged with this above commits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants