Skip to content

[AutoPR azure-resourcemanager-netapp]-generated-from-SDK Generation - Java-5986909#48350

Open
azure-sdk wants to merge 1 commit intoAzure:mainfrom
azure-sdk:sdkauto/azure-resourcemanager-netapp-5986909
Open

[AutoPR azure-resourcemanager-netapp]-generated-from-SDK Generation - Java-5986909#48350
azure-sdk wants to merge 1 commit intoAzure:mainfrom
azure-sdk:sdkauto/azure-resourcemanager-netapp-5986909

Conversation

@azure-sdk
Copy link
Collaborator

@azure-sdk azure-sdk commented Mar 10, 2026

Configurations: 'specification/netapp/resource-manager/Microsoft.NetApp/NetApp/tspconfig.yaml', API Version: 2025-12-15-preview, SDK Release Type: beta, and CommitSHA: 'ffdc765830f1ece01d260e4b026f7e4f9389a038' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5986909 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release. Release plan link: https://web.powerapps.com/apps/821ab569-ae60-420d-8264-d7b5d5ef734c?release-plan-id=df108cf4-e116-f111-8341-000d3a597030 Submitted by: b-aubald@microsoft.com

Release Plan Details

…pp/NetApp/tspconfig.yaml', API Version: 2025-12-15-preview, SDK Release Type: beta, and CommitSHA: 'ffdc765830f1ece01d260e4b026f7e4f9389a038' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5986909 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
@github-actions github-actions bot added the Mgmt This issue is related to a management-plane library. label Mar 10, 2026
@audunn audunn marked this pull request as ready for review March 11, 2026 13:06
Copilot AI review requested due to automatic review settings March 11, 2026 13:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

* `provisioningState()` was removed
* `withNfsV4IdDomain(java.lang.String)` was removed
* `disableShowmount()` was removed
* `nfsV4IdDomain()` was removed
Copy link
Member

@weidongxu-microsoft weidongxu-microsoft Mar 11, 2026

Choose a reason for hiding this comment

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

I don't know what happened to the patch model.

You can see that another nfsV4IDDomain is added to it.

#### `models.NetAppAccountPatch` was modified

* `nfsV4IDDomain()` was added
* `withNfsV4IDDomain(java.lang.String)` was added

class https://github.com/Azure/azure-sdk-for-java/pull/48350/changes#diff-77a4e050070290f77d3cd7ca90e6e275c6bd7dcb2016e2aede6d6d3c43442e33

I think you should do a clientName to change nfsV4IDDomain to nfsV4IdDomain on Java, to avoid this unnecessary break.

Copy link
Member

@audunn audunn Mar 11, 2026

Choose a reason for hiding this comment

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

Possibly due to AccountPropertiesPatch was added this verison.
Will try to fix with Azure/azure-rest-api-specs#41250:

// Fix property name case changes
@@clientName(Microsoft.NetApp.AccountPropertiesPatch.nfsV4IDDomain,
  "nfsV4IdDomain",
  "java"
);

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

Labels

Mgmt This issue is related to a management-plane library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants