Skip to content

Conversation

akshith-gunasheelan
Copy link
Collaborator

Description

Starting from OneView version 8.40, the HostNVMeQualifiedNameType field has been made mandatory during server profile update operations. If this field is not explicitly included in the update request, OneView returns an error.
Currently, the Golang and Terraform SDK does not include the HostNVMeQualifiedNameType attribute, which leads to failure when attempting to update an existing server profile.
This PR addresses the issue

Issues Resolved

[List any issues this PR will resolve. e.g., Fixes #1]

Check List

  • New functionality includes testing.
    • All tests pass for go 1.11 + gofmt checks.
  • New functionality has been documented in the README if applicable.
    • New functionality has been thoroughly documented in the examples (please include helpful comments).
  • Changes are documented in the CHANGELOG.

Copy link
Collaborator

@nabhajit-ray nabhajit-ray left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@AvijitOS AvijitOS left a comment

Choose a reason for hiding this comment

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

What will happen if from golang only i try to create SP but did not pass the HostNVMeQualifiedNameType property? where in golang we will consider default value as autogenerated
I mean don't we need the validation here
also not seeing change in example test case

@akshith-gunasheelan akshith-gunasheelan merged commit 9363c32 into master May 27, 2025
1 check passed
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.

3 participants