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 "mail" parameter to UserUpdateParameters in graphrbac #12127

Merged
merged 1 commit into from
Dec 30, 2020

Conversation

dcepelik
Copy link
Contributor

This patch adds a single optional field "mail" to UserUpdateParameters schema. This makes it possible to update user's e-mail address using the (Go) Azure SDK. Currently this is not possible.

I have curled the graphrbac API with the extra body parameter and it works as expected. It seems that this field was omitted from the OpenAPI spec by accident rather than on purpose, since the field is available in the UserCreateParameters (so not including it in UserUpdateParameters creates a strange asymmetry where I can configure a user's e-mail address, but cannot change it later on).

The curl proving that the field is in fact respected by the UserUpdate endpoint is

curl \
    -i \
    -X PATCH \
    -H "Authorization: Bearer <token>" \
    -H "Content-Type: application/json" \
    -d '{"mail": "foo@bar.com"}' \
    "https://graph.windows.net/<tenantID>/users/<objectID>?api-version=1.6"

Addition of the field will make it possible to support the mail property on the azuread_user resource in terraform-provider-azuread. Currently it's not possible to configure e-mail addresses for users onboarded via Terraform.

Sorry for not sticking with the original pull request checklist, it seemed to me that it applies to Microsoft employees only.

This patch adds a single optional field "mail" to UserUpdateParameters
schema. This makes it possible to update user's e-mail address using
the (Go) Azure SDK. Currently this is not possible.

I have curled the graphrbac API with the extra body parameter and it
works as expected.

Addition of the field will make it possible to support the mail property
on the azuread_user resource in terraform-provider-azuread.
@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Dec 15, 2020

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.

️️✔️LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.

️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️[Staging] Cross Version BreakingChange (Base on preview version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️[Staging] Cross Version BreakingChange (Base on stable version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
Posted by Swagger Pipeline | How to fix these errors?

@ghost ghost added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Dec 15, 2020
@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Dec 15, 2020

Swagger Generation Artifacts

️️✔️ azure-sdk-for-js succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs]Release - Generate from e91876b. SDK Automation 14.0.0
    warn	Skip initScript due to not configured
    command	autorest --version=V2 --typescript --license-header=MICROSOFT_MIT_NO_VERSION --use=@microsoft.azure/autorest.typescript@4.4.4 --typescript-sdks-folder=/home/vsts/work/1/s/azure-sdk-for-js/azure-sdk-for-js ../../azure-rest-api-specs/specification/graphrbac/data-plane/readme.md
  • ️✔️@azure/graph [View full logs]  [Release SDK Changes]
    cmderr	[npmPack] node-resolve: setting options.module is deprecated, please override options.mainFields instead
    cmderr	[npmPack] ./esm/graphRbacManagementClient.js → ./dist/graph.js...
    cmderr	[npmPack] created ./dist/graph.js in 328ms
️️✔️ azure-sdk-for-go succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs]Release - Generate from e91876b. SDK Automation 14.0.0
    command	sh ./initScript.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
    command	go run ./tools/generator/main.go ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json
  • ️✔️graphrbac/1.6/graphrbac [View full logs]  [Release SDK Changes]
    info	[Changelog] ### New Constants
    info	[Changelog]
    info	[Changelog] 1. ObjectType.ObjectTypeAppRoleAssignment
    info	[Changelog]
    info	[Changelog] ### New Funcs
    info	[Changelog]
    info	[Changelog] 1. *AppRoleAssignment.UnmarshalJSON([]byte) error
    info	[Changelog] 1. *AppRoleAssignmentListResultIterator.Next() error
    info	[Changelog] 1. *AppRoleAssignmentListResultIterator.NextWithContext(context.Context) error
    info	[Changelog] 1. *AppRoleAssignmentListResultPage.Next() error
    info	[Changelog] 1. *AppRoleAssignmentListResultPage.NextWithContext(context.Context) error
    info	[Changelog] 1. ADGroup.AsAppRoleAssignment() (*AppRoleAssignment, bool)
    info	[Changelog] 1. AppRoleAssignment.AsADGroup() (*ADGroup, bool)
    info	[Changelog] 1. AppRoleAssignment.AsAppRoleAssignment() (*AppRoleAssignment, bool)
    info	[Changelog] 1. AppRoleAssignment.AsApplication() (*Application, bool)
    info	[Changelog] 1. AppRoleAssignment.AsBasicDirectoryObject() (BasicDirectoryObject, bool)
    info	[Changelog] 1. AppRoleAssignment.AsDirectoryObject() (*DirectoryObject, bool)
    info	[Changelog] 1. AppRoleAssignment.AsServicePrincipal() (*ServicePrincipal, bool)
    info	[Changelog] 1. AppRoleAssignment.AsUser() (*User, bool)
    info	[Changelog] 1. AppRoleAssignment.MarshalJSON() ([]byte, error)
    info	[Changelog] 1. AppRoleAssignmentListResult.IsEmpty() bool
    info	[Changelog] 1. AppRoleAssignmentListResultIterator.NotDone() bool
    info	[Changelog] 1. AppRoleAssignmentListResultIterator.Response() AppRoleAssignmentListResult
    info	[Changelog] 1. AppRoleAssignmentListResultIterator.Value() AppRoleAssignment
    info	[Changelog] 1. AppRoleAssignmentListResultPage.NotDone() bool
    info	[Changelog] 1. AppRoleAssignmentListResultPage.Response() AppRoleAssignmentListResult
    info	[Changelog] 1. AppRoleAssignmentListResultPage.Values() []AppRoleAssignment
    info	[Changelog] 1. Application.AsAppRoleAssignment() (*AppRoleAssignment, bool)
    info	[Changelog] 1. DirectoryObject.AsAppRoleAssignment() (*AppRoleAssignment, bool)
    info	[Changelog] 1. NewAppRoleAssignmentListResultIterator(AppRoleAssignmentListResultPage) AppRoleAssignmentListResultIterator
    info	[Changelog] 1. NewAppRoleAssignmentListResultPage(AppRoleAssignmentListResult, func(context.Context, AppRoleAssignmentListResult) (AppRoleAssignmentListResult, error)) AppRoleAssignmentListResultPage
    info	[Changelog] 1. ServicePrincipal.AsAppRoleAssignment() (*AppRoleAssignment, bool)
    info	[Changelog] 1. ServicePrincipalsClient.AddOwner(context.Context, string, AddOwnerParameters) (autorest.Response, error)
    info	[Changelog] 1. ServicePrincipalsClient.AddOwnerPreparer(context.Context, string, AddOwnerParameters) (*http.Request, error)
    info	[Changelog] 1. ServicePrincipalsClient.AddOwnerResponder(*http.Response) (autorest.Response, error)
    info	[Changelog] 1. ServicePrincipalsClient.AddOwnerSender(*http.Request) (*http.Response, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignedTo(context.Context, string) (AppRoleAssignmentListResultPage, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignedToComplete(context.Context, string) (AppRoleAssignmentListResultIterator, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignedToPreparer(context.Context, string) (*http.Request, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignedToResponder(*http.Response) (AppRoleAssignmentListResult, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignedToSender(*http.Request) (*http.Response, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignments(context.Context, string) (AppRoleAssignmentListResultPage, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignmentsComplete(context.Context, string) (AppRoleAssignmentListResultIterator, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignmentsPreparer(context.Context, string) (*http.Request, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignmentsResponder(*http.Response) (AppRoleAssignmentListResult, error)
    info	[Changelog] 1. ServicePrincipalsClient.ListAppRoleAssignmentsSender(*http.Request) (*http.Response, error)
    info	[Changelog] 1. ServicePrincipalsClient.RemoveOwner(context.Context, string, string) (autorest.Response, error)
    info	[Changelog] 1. ServicePrincipalsClient.RemoveOwnerPreparer(context.Context, string, string) (*http.Request, error)
    info	[Changelog] 1. ServicePrincipalsClient.RemoveOwnerResponder(*http.Response) (autorest.Response, error)
    info	[Changelog] 1. ServicePrincipalsClient.RemoveOwnerSender(*http.Request) (*http.Response, error)
    info	[Changelog] 1. User.AsAppRoleAssignment() (*AppRoleAssignment, bool)
    info	[Changelog]
    info	[Changelog] ## Struct Changes
    info	[Changelog]
    info	[Changelog] ### New Structs
    info	[Changelog]
    info	[Changelog] 1. AppRoleAssignment
    info	[Changelog] 1. AppRoleAssignmentListResult
    info	[Changelog] 1. AppRoleAssignmentListResultIterator
    info	[Changelog] 1. AppRoleAssignmentListResultPage
    info	[Changelog]
    info	[Changelog] ### New Struct Fields
    info	[Changelog]
    info	[Changelog] 1. UserUpdateParameters.Mail
    info	[Changelog]
️⚠️ azure-sdk-for-java warning [Detail]
  • ⚠️Warning [Logs]Release - Generate from e91876b. SDK Automation 14.0.0
    command	./eng/mgmt/automation/init.sh ../azure-sdk-for-java_tmp/initInput.json ../azure-sdk-for-java_tmp/initOutput.json
    command	./eng/mgmt/automation/generate.py ../azure-sdk-for-java_tmp/generateInput.json ../azure-sdk-for-java_tmp/generateOutput.json
    cmderr	[generate.py] 2020-12-30 19:31:11 ERROR [Skip] readme path does not format as specification/*/resource-manager/readme.md
    warn	No file changes detected after generation
    warn	No package detected after generation
️️✔️ azure-sdk-for-python succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs]Release - Generate from e91876b. SDK Automation 14.0.0
    warn	Skip initScript due to not configured
    command	npx n 12
    command	python3 -m venv venv
    command	pip install --upgrade setuptools wheel pip
    command	pip install pathlib jinja2 msrestazure
    command	autorest --version=V2 --use=@microsoft.azure/autorest.python@~4.0.71 --python --python-mode=update --multiapi --keep-version-file --no-async --python-sdks-folder=/home/vsts/work/1/s/azure-sdk-for-python/sdk ../azure-rest-api-specs/specification/graphrbac/data-plane/readme.md
  • ️✔️azure-graphrbac [View full logs]  [Release SDK Changes] Breaking Change Detected
    cmderr	[buildConf] INFO:packaging_tools:Building template azure-graphrbac
    cmderr	[buildConf] INFO:packaging_tools.conf:Skipping default conf since the file exists
    cmderr	[buildConf] INFO:packaging_tools:Skipping CHANGELOG.md template, since a previous one was found
    cmderr	[buildConf] INFO:packaging_tools:Template done azure-graphrbac
    cmderr	[changelogSetup] WARNING: Skipping azure-nspkg as it is not installed.
    cmderr	[changelogReport] INFO:__main__:Looking for Autorest generated package in azure.graphrbac
    cmderr	[changelogReport] INFO:__main__:Found azure.graphrbac
    cmderr	[changelogReport] INFO:__main__:Working on azure.graphrbac
    cmderr	[changelogReport] INFO:__main__:Report written to sdk/graphrbac/azure-graphrbac/code_reports/latest/report.json
    cmderr	[changelogReportLatest] INFO:__main__:Download versions of azure-graphrbac on PyPI
    cmderr	[changelogReportLatest] INFO:__main__:Got ['0.30.0a1', '0.30.0rc1', '0.30.0rc2', '0.30.0rc3', '0.30.0rc4', '0.30.0rc5', '0.30.0rc6', '0.30.0', '0.31.0', '0.32.0', '0.33.0', '0.40.0', '0.50.0', '0.51.0', '0.51.1', '0.52.0', '0.53.0', '0.60.0', '0.61.0', '0.61.1']
    cmderr	[changelogReportLatest] INFO:__main__:Only keep last PyPI version
    cmderr	[changelogReportLatest] INFO:__main__:Installing version 0.61.1 of azure-graphrbac in a venv
    cmderr	[changelogReportLatest] Cache entry deserialization failed, entry ignored
    cmderr	[changelogReportLatest] INFO:__main__:Looking for Autorest generated package in azure.graphrbac
    cmderr	[changelogReportLatest] INFO:__main__:Found azure.graphrbac
    cmderr	[changelogReportLatest] INFO:__main__:Working on azure.graphrbac
    cmderr	[changelogReportLatest] INFO:__main__:Report written to sdk/graphrbac/azure-graphrbac/code_reports/0.61.1/report.json
    cmdout	[Changelog] Size of delta 6.134% size of original (original: 53488 chars, delta: 3281 chars)
    cmdout	[Changelog] **Features**
    cmdout	[Changelog]   - Model UserUpdateParameters has a new parameter mail
    cmdout	[Changelog]   - Added operation ServicePrincipalsOperations.list_app_role_assignments
    cmdout	[Changelog]   - Added operation ServicePrincipalsOperations.list_app_role_assigned_to
    cmdout	[Changelog]   - Added operation ServicePrincipalsOperations.add_owner
    cmdout	[Changelog]   - Added operation ServicePrincipalsOperations.remove_owner
    cmdout	[Changelog] **Breaking changes**
    cmdout	[Changelog]   - Operation UsersOperations.list has a new signature
    warn	Breaking change found in changelog
️️✔️[Staging] ApiDocPreview succeeded [Detail] [Expand]
 Please click here to preview with your @microsoft account. 
Posted by Swagger Pipeline | How to fix these errors?

@ghost
Copy link

ghost commented Dec 15, 2020

Thank you for your contribution dcepelik! We will review the pull request and get back to you soon.

@lmazuel lmazuel merged commit e91876b into Azure:master Dec 30, 2020
xiangyan99 added a commit that referenced this pull request Jan 5, 2021
* [Hub Generated] Review request for Microsoft.AlertsManagement to add version stable/2019-06-01 (#11833)

* Changing Swagger file

* Changing examples

* Fixing tags example

* Fixing tags example

* Fixing tags example

* prettier fix for white spaces

* Enable azure-sdk-for-net-track2 (#12169)

* Updating existing CRR APIs with zone restore feature (#12157)

* - Added change for support of cross zone restores.

* - updating example json

* - Fixing prettier

* lastUpdatedDate for templates (#11900)

* lastUpdatedDate for templates

* fix

* [Hub Generated] Review request for Microsoft.AppPlatform to add version preview/2020-11-01-preview (#11823)

* Adds base for updating Microsoft.AppPlatform from version stable/2020-07-01 to version 2020-11-01-preview

* Updates readme

* Updates API version in new specs and examples

* Introduce additional change to 2020-11-01-preview.

* Additional properties to monitoringSettings.
* Readonly properties to requiredTraffics.

Signed-off-by: Pan Li <panli@microsoft.com>

* Make credscan happy.

Signed-off-by: Pan Li <panli@microsoft.com>

* [Hub Generated] Review request for Microsoft.Consumption to add version stable/2019-10-01 (#12113)

* Fix linter and swagger warnings

* Fix errors

* Removed unneeded change

* [ASC.Automations] Add new data type "RegulatoryComplianceAssessment" (#12185)

* Add SubAssessment event source type and add more automations examples

* Change Location isReadOnlu to false

* Test

* Add new read/write location and add new type TrackedResourceLocation

* Minor

* minor

* prettier fixes

* Change variable name to TrackedResourceLocation

* Change type name TrackedResourceLocation to AzureTrackedResourceLocation

* Update automations example

* Add SecureScores & SecureScoreControls as new supported datatypes in Automations resource

* [ASC.Automations] add new data type RegulatoryComplianceAssessment

* add datalake store track2 config (#12186)

* cleanup old pipeline dependencies (#11889)

* cleanup pipeline deps

* add rest-api-specs-scripts

* Extending recovery network input in replication intent API to support new network creation (#12191)

* [Hub Generated] Review request for Microsoft.Maps to add version preview/2020-02-01-preview (#12172)

* Adding operations for Maps Creator resource.

* Fixing typo.

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Adds suppression to readme

* Resolving linter errors.

* Minor update.

* Updating all reference to v2 schema.

* Marked private atlas as deprecated in description.

* add operationsmanagement track2 config (#12141)

* Update comment.yml (#12202)

Add ARM traffic query link. When add breakingChangeReviewRequired

* Adding new properties to response body (#12201)

* adding new properties

* updating examples

* prettier fix

* [Hub Generated] Review request for Microsoft.ContainerService to add version stable/2020-12-01 (#12064)

* Add autorest.az configurations for azure monitor control service (#12090)

* add AMCS config for codegen

* reorganize command group

* rename command group

* Codegen modify try to rename DataCollectionRules create parameters

* hide DataCollectionRules Create & Update commands and DataCollectionRuleAssociations Create command

* use alias instand of rename in code-gen

* use monitor-control-service as the extension name

* Update readme.python.md

* add logic track2 config (#12166)

* add alertsmanagement track2 config (#11759)

* Update comment.yml (#12213)

* add mixedreality track2 confi (#12070)

* add signalr track2 config (#11892)

* add cognitiveservice track2 config (#11498)

* add machinglearningservices track2 config (#12183)

* add apimanagement track2 config (#12187)

* add serialconsole track2 config (#12046)

* Update TransparentDataEncryption Group Name (#12160)

* update TransparentDataEncryption Group name

* update reference of TransparentDataEncryption

* Update readme.python.md (#12216)

* Required changes for Azure HealthBot swagger (#12124)

* Fixes for SDK generation files.

* Fixes for SDK generation files.

* Added GO.

* Change Healthcare bot to Healthbot.

* 1. Remove unused properties of SKU.
2. Removed unused CheckNameAvailability.json
3. Remove subgroup.

* Sku is required.

* [Hub Generated] Review request for Microsoft.IoTCentral to add version stable/2018-09-01 (#12224)

* update to name

* update

* add api-version of guest configuration to go SDK (#12217)

* add api-version for templatespecs (#12197)

* Update specificationRepositoryConfiguration.json (#12218)

* Adding new api version to devops RP (#11585)

* Initial commit as per PR review

* Actual changes to api spec for new version

* Switching to multi-api build for python

* Fix python spec to correct namespaces in multiapi mode

* Revert changes to python generation

* Add systemdata

* Fix the path for sys data reference

* Fix path once more

* remove additional properties after including systemdata

* Remove more addtional properties

* Move system data to root

* Fix wrong output-folder (#12229)

* Remove swagger and examples because the controller will be used for private preview. (#12177)

* Update pull_request_assignment.yml (#12230)

* [Hub Generated] Review request for Microsoft.StorageCache to add version stable/2020-10-01 (#11407)

* Init the next version so diffs can work better.

* Updates readme

* Updates API version in new specs and examples

* Updates to StorageTargetProperties and added examples of cmk, mtu

Add 202 to the storage cache and storage target create/update and update examples

Fix ST no junction example's 202 response.

add properties for nfs extended groups

fix issues with extended groups properties found with autorest

add properties and objects for Active Directory username download

undo unintended change

changes from comments in pull request

AccessPolicy support.

Example fixes for Access Policies.

Fix attribute names to match latest RP.

update to credential properties for LDAP and Active Directory

marking password properties with x-ms-secret tag

minor changes on extended groups and add examples

Added blob NFS and some other validation fixes.

Update required property for domainName from dnsName

Updated blobNfs examples and some kpi fixes.

Correct validation errors in examples.

Added systemdata to resources.

Remove x-ms-secret in keyvault reference due to linter error and common types not using it.

Remove blobNfs from this version.

Remove blobNfs from spec file.

Remove x-ms-secret due to linter errors.

Fix certificate spelling.

Updating prettier and spell check errors.

Used prettier on main spec file.

Readded x-ms-secret that open api hub failed on but the PR pipeline allows.

* Add prettier fix after rebase

* Remove 202 bodies and add systemData to examples.

* Update spec with prettier.

* Address comments on spec for descriptions, readmes, and extendedGroupsEnabled.

* Updating to address addition ldap and pattern comments.

* Update version tag to include 01

* Restore changes that would be considered breaking changes matching 2020-03-01

* Address a few other comments.

* Update netbios field names and some descriptions.

* Fix s360 for missing debugInfo operation.

* Fix credscan error in example file.

* Update required fields, new validator failure on debugInfo return codes, and bindPassword example value.

* Update debug info example with new return codes.

* Update other credscan password errors.

* Update Semantic-and-Model-Violations-Reference.md (#12199)

Adding secret_property code, update some other codes.

* ADP - add system metadata to dataPool (#12179)

* Swagger Linting Fix (#12162)

* Fix Linting Issuing

* no message

* fix description

* add description and object back

* test to resolve model validation test

(cherry picked from commit ab273df)

* [SQL][V20180601] Updating LocationCapabilities spec (#12032)

* [SQL][V20180601] Updating LocationCapabilities spec

* Fill 2020 versions

* Rollback V2018 changes and bump LocationCapabilities in V4 to 2020-08-01-preview

* update order for CI-FixRequiredOnFailure (#12227)

* update rdbms track2 config (#12245)

* ADT: refix path (#12161)

* python track2 configure (#12150)

* Add and remove owner for service principals (#12081)

* [Hub Generated] Review request for Microsoft.MixedReality to add version stable/2020-05-01 (#11810)

* fix spec

* fix prettier

* [Hub Generated] Review request for Microsoft.MixedReality to add version preview/2019-12-02-preview (#11769)

* fix for swagger completeness

* forgot curly brace

* fix example

* Api Management - make /tenant endpoints ARM compliant in 2020-06-01-preview version (#11549)

* Adds base for updating Microsoft.ApiManagement from version stable/2019-12-01 to version 2020-06-01-preview

* Updates readme

* Updates API version in new specs and examples

* Add support in API Management for Availability Zones (#10284)

* apim in azs

* fix prettier check

* PATCH should return 200 OK (#10328)

* add support for PATCH returning 200 OK

* CI fixes

prettier fix

CI fixes part 2

* Password no longer a mandatory property when uploading Certificates

* add missing x-ms-odata extension for filter support

* +gatewayhostnameconfiguration protocol changes (#10292)

* [2020-06-01-preview] Update Oauth Server secrets Contract (#10602)

* Oauth server secrets contract

* fix azureMonitor enum

* API Management Service Deleted Services Resource (#10607)

* API Management Service Deleted Services Resource

* Path fix

* Lint + custom-words fixes

* Location URI parameter for deletedservices Resource

* GET for deletedservices by service name

* Remove resourceGroupName from resource path

* fixes

* schema for purge operation

* perttier applied

* 204 response code added

Co-authored-by: REDMOND\glfeokti <glfeokti@microsoft.com>

* OperationNameFormat property added to Diagnostic contract (#10641)

* OperationNameFormat property added to Diagnostic contract

* add azuremonitor to update contract

Co-authored-by: REDMOND\glfeokti <glfeokti@microsoft.com>

* [Microsoft.ApiManagement][2020-06-01-preview] Change Network Status response contract (#10331)

* Change Network Status response contract

* Update examples for network status contract

* ApiManagement - tenant/settings endpoints

* ApiManagement - tenant/settings endpoints fix

* ApiManagement - tenant/settings endpoints fix prettier

* ApiManagement - tenant/settings endpoints fix 3

* ApiManagement - tenant/settings endpoints fix 4

* ApiManagement - tenant/settings endpoints fix 5

Co-authored-by: Samir Solanki <samirsolanki@outlook.com>
Co-authored-by: maksimkim <maksim.kim@gmail.com>
Co-authored-by: promoisha <feoktistovgg@gmail.com>
Co-authored-by: REDMOND\glfeokti <glfeokti@microsoft.com>
Co-authored-by: RupengLiu <rliu1211@terpmail.umd.edu>
Co-authored-by: vfedonkin <vifedo@microsoft.com>

* Add "mail" parameter to UserUpdateParameters in graphrbac (#12127)

This patch adds a single optional field "mail" to UserUpdateParameters
schema. This makes it possible to update user's e-mail address using
the (Go) Azure SDK. Currently this is not possible.

I have curled the graphrbac API with the extra body parameter and it
works as expected.

Addition of the field will make it possible to support the mail property
on the azuread_user resource in terraform-provider-azuread.

Co-authored-by: David Čepelík <david.cepelik@showmax.com>

* [Hub Generated] Review request for Microsoft.Advisor to add version stable/2020-01-01 (#12262)

* add resourcegraph track2 config (#12122)

* add reservations track2 config (#12027)

* Removing a readonly tag from a property (#12254)

* Fix swagger correctness errors (#12246)

* Fix swagger correctness issues in Security for Iot service

* changed int format from 32 to 64

* Add systemData to iotSecuritySolutions

Co-authored-by: Liran Chen <lirche@microsoft.com>

* [Hub Generated] Review request for Microsoft.Consumption to add version stable/2019-10-01 (#12196)

* AAS - Swagger Linting and Correctness Fix (#12176)

* Fix R4013 IntegerTypeMustHaveFormat

* Fix R4010 RequiredDefaultResponse

* Fix R4007 DefaultErrorResponseSchema

* removed extra bracket

* fix semantic issue

* add type error

* Fix linting issue

* test to resolve model validation test

* Revert "test to resolve model validation test"

This reverts commit ab273df.

* Resolve Model Validation Issue

* Resolve Model Validation Issue v2

* test - add missing properties

(cherry picked from commit 48ec27c)

* fix format

* set default value

* add x-ms-enum

* fix nit

* Add missing property

* add missing properties

* add missing bracket

* change GatewayListStatusLive type

* add missing "origin" property

* fix model validation

* Revert "change GatewayListStatusLive type"

This reverts commit 4f3fa74.

* Revert "fix model validation"

This reverts commit 945bc23.

* add redhatopenshift track2 config (#12045)

* [Hub Generated] Review request for Microsoft.CostManagement to add version stable/2019-10-01 (#12194)

* Adding nextLink and previousLink to Tags endpoint properties

* moving properties out

* prettier check fix

Co-authored-by: Jorge Chavez Nieto <jochav@microsoft.com>

* add relay track2 config (#11495)

* update healthbot t2 config (#12269)

* update healthbot t2 config

* fix readme

* add automation track2 config (#11628)

* add automation track2 config

* update config

* Azure Remote Rendering REST API swagger file - for review (#12015)

* Azure Remote Rendering API specifcation

* fix schema issue

* fix examples

* fix request bodies integer values, :stop path

* pointing readme.md to open api spec for ARR

* fix typo, add custom words

* add msvc to custom words

* fix readme.md

* ran prettifier

* fix reference to json

* reference sts from file as well

* fix readme.md

* changes to readme.md

* remove example from error to fix error

* add 200 OK status for retried PUT for conversions and sessiosn

* ran prettier

* make autorest linter not crash

* fix casing of nextLinkName paramter

* factor out conversion list response

* add examples for conversions

* adding examples for sessions, prettify files

* [Hub Generated] Review request for Microsoft.Maps/Microsoft.Maps to add version preview/2.0 (#12175)

* microsoft.imagery 512 not supported

Adding micorosft.imagery to 512 tile not supported list

* Updating Raster Tilesets to Have Max Zoom 22

Certain raster tilesets now support up to zoom level 22.

* minor changes to swagger (#12253)

* minor changes to swagger

* reverting sdk definitions file

* Add addons to AVS 2020-07-17-preview (#12236)

* found missing AVS/privateclouds/addons commit, moving it to 2020-07-17-preview API from PR repo

* fixed prettier errors

* Fixed vladation warnings for x-ms-enum and missing description/title, related to addons

* fixed validation error for missing systemData for addons responses

* fixed prettier errors

* fixed validation error for missing systemData for addons responses

* systemData for any Resource

* make sure systemData is readOnly

* systemData needs to go in a new api version

* suppress systemData error

Co-authored-by: Cameron Taggart <cameron.taggart@microsoft.com>

* Azs api bug fix (#12133)

* Old api version for ComputeOperationResult

* Api Bugfix for Azure Stack Fabric Admin

* Fix for typo in readme

* Add default reponse

* Fix example issue for default error

* Update comment.yml (#12273)

* [Hub Generated] Review request for Microsoft.CostManagement to add version stable/2019-11-01 (#12031)

* fix dataSet typo

* fix dimensions and tags typo

* more dimensions and tags to fix

* one more miss on dimensions and tags

* Add the includeMonetaryCommitment missing field

* fix spell check error

* make property read only

* remove the not operation from query filters

* New backup service api version (#12171)

* Initial commit: adding new api version

* - new api 2020-12-01

* updating the readme.md package reference

* - adding the zone information for new api

* - updating api-version in examples
- added 'zone' in Get recoverypoint api example json

* Fixing prettier issue

* Revert "- updating api-version in examples"

This reverts commit 80baa9f.

* - adding api-version in examples

* adding new api version in package mds

* [IotHub] Introducing networkRuleSets in IotHubProperties [2020-08-31 : preview and stable ] (#10853)

* Adds base for updating Microsoft.Devices from version preview/2020-07-10-preview to version 2020-08-31-preview

* Updates readme

* Updates API version in new specs and examples

* Adds base for updating Microsoft.Devices from version stable/2020-08-01 to version 2020-08-31

* Updates readme

* Updates API version in new specs and examples

* Adding networkRuleSets support for iothub

* Adding networkRuleSets Support in preview version

* Adding Examples, correcting order in iothub.json

* Updating Readme to have stable and previe versions

* Modifying Tag to package-2020-08-31

* Adding Arm identity properties from 2020-07-010-preview version

* Removing creds

* minor fix

* swagger correctness fix (#12101)

* swagger correctness fix

* update preview version too

* Revert change and add defaults (#12243)

* Remove gallery preview version 2020-09-30 from stable release (#12222)

* add managementpartner track2 config (#12043)

* [Hub Generated] Review request for Microsoft.Insights to add version stable/2019-03-01 (#12232)

* Fixed swagger validations for api version 2019

* reverted error response change and suppress

* removed redundant error details

* renamed suppression rule

* fixed suppression

Co-authored-by: napolish <67951636+napolish@users.noreply.github.com>
Co-authored-by: Phoenix He <tih@microsoft.com>
Co-authored-by: arpja <46751982+arpja@users.noreply.github.com>
Co-authored-by: ShaniFelig <74960756+ShaniFelig@users.noreply.github.com>
Co-authored-by: Pan Li <Incarnation-p-lee@users.noreply.github.com>
Co-authored-by: Zach Rathbun <zachra@microsoft.com>
Co-authored-by: surashed <55134940+surashed@users.noreply.github.com>
Co-authored-by: Kaihui (Kerwin) Sun <sunkaihuisos@gmail.com>
Co-authored-by: Zhenglai Zhang <darinzh@microsoft.com>
Co-authored-by: om-nishant <50401171+om-nishant@users.noreply.github.com>
Co-authored-by: Ihar Voitka <ihvo@users.noreply.github.com>
Co-authored-by: Ruoxuan Wang <52271048+ruowan@users.noreply.github.com>
Co-authored-by: Ju Hee Lee <juhee0202@gmail.com>
Co-authored-by: xiazhan <allan.zhang76@gmail.com>
Co-authored-by: kai ru <69238381+kairu-ms@users.noreply.github.com>
Co-authored-by: Ji Wang <jiwang5@microsoft.com>
Co-authored-by: Andy Zhang <xiazhang@microsoft.com>
Co-authored-by: guy-microsoft <50947884+guy-microsoft@users.noreply.github.com>
Co-authored-by: PoAn (Baron) Chen <chen.baron@hotmail.com>
Co-authored-by: Arcturus <dapzhang@microsoft.com>
Co-authored-by: suyash691 <suyash691@github.com>
Co-authored-by: nemijato <68876730+nemijato@users.noreply.github.com>
Co-authored-by: brpanask <64860651+brpanask@users.noreply.github.com>
Co-authored-by: Ray Chen <raychen@microsoft.com>
Co-authored-by: Eli Arbel <aelij@arbel.net>
Co-authored-by: David Han <76010854+david-msft@users.noreply.github.com>
Co-authored-by: Roman Khotsyn <rokhot@microsoft.com>
Co-authored-by: David R. Williamson <drwill@microsoft.com>
Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>
Co-authored-by: Xinyi Joffre <xiou@microsoft.com>
Co-authored-by: roytan-microsoft <73565668+roytan-microsoft@users.noreply.github.com>
Co-authored-by: Vitaliy Fedonkin <vitaliy.fedonkin@gmail.com>
Co-authored-by: Samir Solanki <samirsolanki@outlook.com>
Co-authored-by: maksimkim <maksim.kim@gmail.com>
Co-authored-by: promoisha <feoktistovgg@gmail.com>
Co-authored-by: REDMOND\glfeokti <glfeokti@microsoft.com>
Co-authored-by: RupengLiu <rliu1211@terpmail.umd.edu>
Co-authored-by: vfedonkin <vifedo@microsoft.com>
Co-authored-by: David Čepelík <d@dcepelik.cz>
Co-authored-by: David Čepelík <david.cepelik@showmax.com>
Co-authored-by: mudit794 <42189950+mudit794@users.noreply.github.com>
Co-authored-by: Dhawal Jain <41853104+dhawal777@users.noreply.github.com>
Co-authored-by: MichalHel <51286657+MichalHel@users.noreply.github.com>
Co-authored-by: Liran Chen <lirche@microsoft.com>
Co-authored-by: jochav <75458295+jochav@users.noreply.github.com>
Co-authored-by: Jorge Chavez Nieto <jochav@microsoft.com>
Co-authored-by: rikogeln <40666347+rikogeln@users.noreply.github.com>
Co-authored-by: chgennar <56695749+chgennar@users.noreply.github.com>
Co-authored-by: hivyas <61890270+hivyas@users.noreply.github.com>
Co-authored-by: Douglas Lee <65295939+leedouglas@users.noreply.github.com>
Co-authored-by: Cameron Taggart <cameron.taggart@microsoft.com>
Co-authored-by: LingyunSu <33764806+LingyunSu@users.noreply.github.com>
Co-authored-by: elabicha <61994320+elabicha@users.noreply.github.com>
Co-authored-by: Nikhil Kumar Mengani <62704762+nimengan@users.noreply.github.com>
Co-authored-by: Junbo Wang <juniwang@microsoft.com>
Co-authored-by: Daniel Orozco <64658732+dagoroz@users.noreply.github.com>
Co-authored-by: Adam Sandor <adsandor@microsoft.com>
Co-authored-by: yashmuel <63585146+yashmuel@users.noreply.github.com>
josuhazure pushed a commit to josuhazure/azure-rest-api-specs that referenced this pull request Jan 6, 2021
This patch adds a single optional field "mail" to UserUpdateParameters
schema. This makes it possible to update user's e-mail address using
the (Go) Azure SDK. Currently this is not possible.

I have curled the graphrbac API with the extra body parameter and it
works as expected.

Addition of the field will make it possible to support the mail property
on the azuread_user resource in terraform-provider-azuread.

Co-authored-by: David Čepelík <david.cepelik@showmax.com>
@ghostinushanka
Copy link

Hello @lmazuel
I see that the pull request been accepted and merged, but the resulting state confuses me, as I cannot find the actual changes in the latest versions of the SDK.
It seems that (in the example of GO SDK) the #13208 did contain the code changes to the SDK for the mail parameter, but the #13960, listed after the acceptance, actually does not have those changes.

Could you please clarify whether it is meant to land into the SDK in the end?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-BreakingChange-Python customer-reported Issues that are reported by GitHub users external to the Azure organization.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants