Skip to content

Added ExpirationTime field to Role Assignment response DTO for version 2026-07-01-preview#49769

Open
denyspanov-public wants to merge 1 commit into
Azure:mainfrom
denyspanov-public:main
Open

Added ExpirationTime field to Role Assignment response DTO for version 2026-07-01-preview#49769
denyspanov-public wants to merge 1 commit into
Azure:mainfrom
denyspanov-public:main

Conversation

@denyspanov-public

Copy link
Copy Markdown
Member

Description

Added ExpirationTime field to Role Assignment response DTO for version 2026-07-01-preview
Azure/azure-rest-api-specs#44421

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

Copilot AI review requested due to automatic review settings July 9, 2026 13:48
@github-actions github-actions Bot added the Mgmt This issue is related to a management-plane library. label Jul 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the azure-resourcemanager-authorization management library model layer to include an expirationTime value on role assignment properties (per the 2026-07-01-preview spec update), and documents the feature in the changelog.

Changes:

  • Added an expirationTime field to RoleAssignmentProperties and deserialization support for it.
  • Added forwarding accessors for expirationTime on RoleAssignmentInner and RoleAssignmentCreateParameters.
  • Added a changelog entry describing the new property.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

File Description
sdk/authorization/azure-resourcemanager-authorization/src/main/java/com/azure/resourcemanager/authorization/models/RoleAssignmentCreateParameters.java Adds an expirationTime() accessor forwarding to inner properties.
sdk/authorization/azure-resourcemanager-authorization/src/main/java/com/azure/resourcemanager/authorization/fluent/models/RoleAssignmentProperties.java Introduces expirationTime field + getter and deserializes it from JSON.
sdk/authorization/azure-resourcemanager-authorization/src/main/java/com/azure/resourcemanager/authorization/fluent/models/RoleAssignmentInner.java Adds an expirationTime() accessor forwarding to inner properties.
sdk/authorization/azure-resourcemanager-authorization/CHANGELOG.md Documents the newly added expirationTime property.

/*
* Time the role assignment expires
*/
private String expirationTime;
Comment on lines +290 to +292
public String expirationTime() {
return this.expirationTime;
}
Comment on lines +372 to 374
} else if ("expirationTime".equals(fieldName)) {
deserializedRoleAssignmentProperties.expirationTime = reader.getString();
} else {
Comment on lines +258 to +260
public String expirationTime() {
return this.innerProperties() == null ? null : this.innerProperties().expirationTime();
}
Comment on lines +316 to +318
public String expirationTime() {
return this.innerProperties() == null ? null : this.innerProperties().expirationTime();
}

### Features Added

- Added read-only `expirationTime` property to `RoleAssignment` returned by role assignment get and list operations.
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.

2 participants