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 OT and AI config #25350

Merged
merged 2 commits into from
Sep 12, 2023
Merged

Add OT and AI config #25350

merged 2 commits into from
Sep 12, 2023

Conversation

michaelkira
Copy link
Contributor

@michaelkira michaelkira commented Aug 17, 2023

ARM (Control Plane) API Specification Update Pull Request

PR review workflow diagram

Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

diagram

[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])

If you need further help with anything, see Getting help section below.

Purpose of this PR

What's the purpose of this PR? Check all that apply. This is mandatory!

  • New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
  • Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
  • Update existing version to fix swagger quality issues in S360.
  • Other, please clarify:
    • edit this with your clarification

Due diligence checklist

To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

Breaking changes review (Step 1)

  • If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
    you must follow the breaking changes process.
    IMPORTANT This applies even if:
    • The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
    • You believe there is no need for you to request breaking change approval, for any reason.
      Such claims must be reviewed, and the process is the same.

ARM API changes review (Step 2)

  • If this PR is in purview of ARM review then automation will add the ARMReview label.
  • If you want to force ARM review, add the label yourself.
  • Proceed according to the diagram at the top of this comment.

Getting help

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Aug 17, 2023

Next Steps to Merge

✔️ All automated merging requirements have been met! Refer to step 4 in the PR workflow diagram (even if your PR is for data plane, not ARM).

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Aug 17, 2023

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
compared swaggers (via Oad v0.10.4)] new version base version
ManagedEnvironments.json 2023-08-01-preview(7c1f0fa) 2023-05-01(main)
ManagedEnvironments.json 2023-08-01-preview(7c1f0fa) 2023-05-02-preview(main)
️️✔️LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
compared tags (via openapi-validator v2.1.4) new version base version
package-preview-2023-08 package-preview-2023-08(7c1f0fa) package-preview-2023-08(release-app-Microsoft.App-2023-08-01-preview)
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Aug 17, 2023

Swagger Generation Artifacts

️️✔️ApiDocPreview succeeded [Detail] [Expand]
 Please click here to preview with your @microsoft account. 
️️✔️SDK Breaking Change Tracking succeeded [Detail] [Expand]

Breaking Changes Tracking




️⚠️ azure-sdk-for-python-track2 warning [Detail]
  • ⚠️Warning [Logs] Generate from 429352f7a75ab3af64a164d7bf01b898d166d6dd. SDK Automation 14.0.0
    command	sh scripts/automation_init.sh ../azure-sdk-for-python_tmp/initInput.json ../azure-sdk-for-python_tmp/initOutput.json
    cmderr	[automation_init.sh] WARNING: Skipping azure-nspkg as it is not installed.
    command	sh scripts/automation_generate.sh ../azure-sdk-for-python_tmp/generateInput.json ../azure-sdk-for-python_tmp/generateOutput.json
    cmderr	[automation_generate.sh] npm notice
    cmderr	[automation_generate.sh] npm notice New minor version of npm available! 9.6.7 -> 9.8.1
    cmderr	[automation_generate.sh] npm notice Changelog: <https://github.com/npm/cli/releases/tag/v9.8.1>
    cmderr	[automation_generate.sh] npm notice Run `npm install -g [email protected]` to update!
    cmderr	[automation_generate.sh] npm notice
  • ️✔️track2_azure-mgmt-appcontainers [View full logs]  [Preview SDK Changes]
    info	[Changelog] ### Features Added
    info	[Changelog]
    info	[Changelog]   - Added operation ContainerAppsAPIClientOperationsMixin.get_custom_domain_verification_id
    info	[Changelog]   - Added operation group ManagedEnvironmentUsagesOperations
    info	[Changelog]   - Added operation group UsagesOperations
    info	[Changelog]   - Model AuthConfig has a new parameter encryption_settings
    info	[Changelog]   - Model Ingress has a new parameter additional_port_mappings
    info	[Changelog]   - Model Login has a new parameter token_store
    info	[Changelog]   - Model ManagedEnvironment has a new parameter app_insights_configuration
    info	[Changelog]   - Model ManagedEnvironment has a new parameter open_telemetry_configuration
️⚠️ azure-sdk-for-java warning [Detail]
  • ⚠️Warning [Logs] Generate from 429352f7a75ab3af64a164d7bf01b898d166d6dd. 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
    cmderr	[init.sh] [notice] A new release of pip is available: 23.0.1 -> 23.2.1
    cmderr	[init.sh] [notice] To update, run: pip install --upgrade pip
    cmderr	[init.sh] [notice] A new release of pip is available: 23.0.1 -> 23.2.1
    cmderr	[init.sh] [notice] To update, run: pip install --upgrade pip
    command	./eng/mgmt/automation/generate.py ../azure-sdk-for-java_tmp/generateInput.json ../azure-sdk-for-java_tmp/generateOutput.json
  • ️✔️azure-resourcemanager-appcontainers [View full logs]  [Preview SDK Changes]
️️✔️ azure-sdk-for-go succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs] Generate from 429352f7a75ab3af64a164d7bf01b898d166d6dd. SDK Automation 14.0.0
    command	sh ./eng/scripts/automation_init.sh ../../../../../azure-sdk-for-go_tmp/initInput.json ../../../../../azure-sdk-for-go_tmp/initOutput.json
    command	generator automation-v2 ../../../../../azure-sdk-for-go_tmp/generateInput.json ../../../../../azure-sdk-for-go_tmp/generateOutput.json
  • ️✔️sdk/resourcemanager/appcontainers/armappcontainers [View full logs]  [Preview SDK Changes]
    info	[Changelog] ### Features Added
    info	[Changelog]
    info	[Changelog] - New function `*ClientFactory.NewManagedEnvironmentUsagesClient() *ManagedEnvironmentUsagesClient`
    info	[Changelog] - New function `*ClientFactory.NewUsagesClient() *UsagesClient`
    info	[Changelog] - New function `*ContainerAppsAPIClient.GetCustomDomainVerificationID(context.Context, *ContainerAppsAPIClientGetCustomDomainVerificationIDOptions) (ContainerAppsAPIClientGetCustomDomainVerificationIDResponse, error)`
    info	[Changelog] - New function `NewManagedEnvironmentUsagesClient(string, azcore.TokenCredential, *arm.ClientOptions) (*ManagedEnvironmentUsagesClient, error)`
    info	[Changelog] - New function `*ManagedEnvironmentUsagesClient.NewListPager(string, string, *ManagedEnvironmentUsagesClientListOptions) *runtime.Pager[ManagedEnvironmentUsagesClientListResponse]`
    info	[Changelog] - New function `NewUsagesClient(string, azcore.TokenCredential, *arm.ClientOptions) (*UsagesClient, error)`
    info	[Changelog] - New function `*UsagesClient.NewListPager(string, *UsagesClientListOptions) *runtime.Pager[UsagesClientListResponse]`
    info	[Changelog] - New struct `AppInsightsConfiguration`
    info	[Changelog] - New struct `BlobStorageTokenStore`
    info	[Changelog] - New struct `DataDogConfiguration`
    info	[Changelog] - New struct `DestinationsConfiguration`
    info	[Changelog] - New struct `EncryptionSettings`
    info	[Changelog] - New struct `IngressPortMapping`
    info	[Changelog] - New struct `ListUsagesResult`
    info	[Changelog] - New struct `LogsConfiguration`
    info	[Changelog] - New struct `MetricsConfiguration`
    info	[Changelog] - New struct `OpenTelemetryConfiguration`
    info	[Changelog] - New struct `TokenStore`
    info	[Changelog] - New struct `TracesConfiguration`
    info	[Changelog] - New struct `Usage`
    info	[Changelog] - New struct `UsageName`
    info	[Changelog] - New field `EncryptionSettings` in struct `AuthConfigProperties`
    info	[Changelog] - New field `AdditionalPortMappings` in struct `Ingress`
    info	[Changelog] - New field `TokenStore` in struct `Login`
    info	[Changelog] - New field `AppInsightsConfiguration`, `OpenTelemetryConfiguration` in struct `ManagedEnvironmentProperties`
    info	[Changelog]
    info	[Changelog] Total 0 breaking change(s), 39 additive change(s).
️️✔️ azure-sdk-for-js succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs] Generate from 429352f7a75ab3af64a164d7bf01b898d166d6dd. SDK Automation 14.0.0
    command	sh .scripts/automation_init.sh ../azure-sdk-for-js_tmp/initInput.json ../azure-sdk-for-js_tmp/initOutput.json
    warn	File azure-sdk-for-js_tmp/initOutput.json not found to read
    command	sh .scripts/automation_generate.sh ../azure-sdk-for-js_tmp/generateInput.json ../azure-sdk-for-js_tmp/generateOutput.json
  • ️✔️@azure/arm-appcontainers [View full logs]  [Preview SDK Changes]
    info	[Changelog] **Features**
    info	[Changelog]
    info	[Changelog]   - Added operation group ManagedEnvironmentUsages
    info	[Changelog]   - Added operation group Usages
    info	[Changelog]   - Added Interface AppInsightsConfiguration
    info	[Changelog]   - Added Interface BlobStorageTokenStore
    info	[Changelog]   - Added Interface DataDogConfiguration
    info	[Changelog]   - Added Interface DestinationsConfiguration
    info	[Changelog]   - Added Interface EncryptionSettings
    info	[Changelog]   - Added Interface GetCustomDomainVerificationIdOptionalParams
    info	[Changelog]   - Added Interface IngressPortMapping
    info	[Changelog]   - Added Interface ListUsagesResult
    info	[Changelog]   - Added Interface LogsConfiguration
    info	[Changelog]   - Added Interface ManagedEnvironmentUsagesListNextOptionalParams
    info	[Changelog]   - Added Interface ManagedEnvironmentUsagesListOptionalParams
    info	[Changelog]   - Added Interface MetricsConfiguration
    info	[Changelog]   - Added Interface OpenTelemetryConfiguration
    info	[Changelog]   - Added Interface TokenStore
    info	[Changelog]   - Added Interface TracesConfiguration
    info	[Changelog]   - Added Interface Usage
    info	[Changelog]   - Added Interface UsageName
    info	[Changelog]   - Added Interface UsagesListNextOptionalParams
    info	[Changelog]   - Added Interface UsagesListOptionalParams
    info	[Changelog]   - Added Type Alias GetCustomDomainVerificationIdResponse
    info	[Changelog]   - Added Type Alias ManagedEnvironmentUsagesListNextResponse
    info	[Changelog]   - Added Type Alias ManagedEnvironmentUsagesListResponse
    info	[Changelog]   - Added Type Alias UsagesListNextResponse
    info	[Changelog]   - Added Type Alias UsagesListResponse
    info	[Changelog]   - Interface AuthConfig has a new optional parameter encryptionSettings
    info	[Changelog]   - Interface Ingress has a new optional parameter additionalPortMappings
    info	[Changelog]   - Interface Login has a new optional parameter tokenStore
    info	[Changelog]   - Interface ManagedEnvironment has a new optional parameter appInsightsConfiguration
    info	[Changelog]   - Interface ManagedEnvironment has a new optional parameter openTelemetryConfiguration
️️✔️ azure-resource-manager-schemas succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs] Generate from 429352f7a75ab3af64a164d7bf01b898d166d6dd. Schema Automation 14.0.0
    command	.sdkauto/initScript.sh ../azure-resource-manager-schemas_tmp/initInput.json ../azure-resource-manager-schemas_tmp/initOutput.json
    warn	File azure-resource-manager-schemas_tmp/initOutput.json not found to read
    command	.sdkauto/generateScript.sh ../azure-resource-manager-schemas_tmp/generateInput.json ../azure-resource-manager-schemas_tmp/generateOutput.json
️❌ azure-powershell failed [Detail]
  • Pipeline Framework Failed [Logs] Generate from 429352f7a75ab3af64a164d7bf01b898d166d6dd. SDK Automation 14.0.0
    command	sh ./tools/SwaggerCI/init.sh ../azure-powershell_tmp/initInput.json ../azure-powershell_tmp/initOutput.json
    command	pwsh ./tools/SwaggerCI/psci.ps1 ../azure-powershell_tmp/generateInput.json ../azure-powershell_tmp/generateOutput.json
    SSL error: syscall failure: Broken pipe
    Error: SSL error: syscall failure: Broken pipe
  • ️✔️Az.app.DefaultTag [View full logs
    error	Fatal error: SSL error: syscall failure: Broken pipe
    error	The following packages are still pending:
    error		Az.app.DefaultTag
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Aug 17, 2023

Generated ApiView

Language Package Name ApiView Link
Go sdk/resourcemanager/appcontainers/armappcontainers https://apiview.dev/Assemblies/Review/be864ee7039d44519bab728bd39d46a6
Java azure-resourcemanager-appcontainers https://apiview.dev/Assemblies/Review/9105b011ed5f42cda071d608ef29000b
JavaScript @azure/arm-appcontainers https://apiview.dev/Assemblies/Review/037f681a38d34eeba8185349f6a514e3
Swagger Microsoft.App https://apiview.dev/Assemblies/Review/c535cc60e95d489ea163c4249fdcfd8c

@openapi-workflow-bot
Copy link

Please address or respond to feedback from the ARM API reviewer.
When you are ready to continue the ARM API review, please remove the ARMChangesRequested label.
This will notify the reviewer to have another look.
If the feedback provided needs further discussion, please use this Teams channel to post your questions - aka.ms/azsdk/support/specreview-channel.
Please include [ARM Query] in the title of your question to indicate that it is ARM-related.

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Aug 17, 2023

Automatic PR validation restarted. This comment will be populated with next steps to merge this PR once validation is completed. Please wait ⌛.

@Seris370 Seris370 removed the ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review label Aug 17, 2023
@openapi-workflow-bot openapi-workflow-bot bot added the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Aug 17, 2023
@michaelkira michaelkira requested a review from Juliehzl August 18, 2023 03:37
"description": "Configuration of Application Insights ",
"type": "object",
"properties": {
"connectionString": {
Copy link
Member

Choose a reason for hiding this comment

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

It seems to be a secret, would you please add x-ms-secret: true?

refs: https://github.com/Azure/autorest/blob/main/docs/extensions/readme.md#x-ms-secret

"description": "The data dog site",
"type": "string"
},
"key": {
Copy link
Member

Choose a reason for hiding this comment

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

It seems to be a secret, would you please add x-ms-secret: true?

refs: https://github.com/Azure/autorest/blob/main/docs/extensions/readme.md#x-ms-secret

@ms-henglu ms-henglu added the ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review label Aug 23, 2023
@openapi-workflow-bot openapi-workflow-bot bot removed the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Aug 23, 2023
@michaelkira michaelkira removed the ARMChangesRequested <valid label in PR review process>add this label when require changes after ARM review label Aug 25, 2023
@openapi-workflow-bot openapi-workflow-bot bot added the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Aug 25, 2023
@ms-henglu ms-henglu added the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Aug 25, 2023
@openapi-workflow-bot openapi-workflow-bot bot removed the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Aug 25, 2023
@kazrael2119
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@Alancere Alancere added the Approved-SdkBreakingChange-Go Approve the breaking change tracking for azure-sdk-for-go label Aug 28, 2023
@kazrael2119
Copy link
Contributor

remove "CI-BreakingChange-JavaScript" label because there is not any breaking

@kazrael2119
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@michaelkira
Copy link
Contributor Author

/pr RequestMerge

@rkmanda rkmanda merged commit 7d21bf4 into Azure:release-app-Microsoft.App-2023-08-01-preview Sep 12, 2023
zizw123 pushed a commit that referenced this pull request Nov 9, 2023
* Adds base for updating Microsoft.App from version preview/2023-05-02-preview to version 2023-08-01-preview

* Updates readme

* Updates API version in new specs and examples

* Add type to Certificate (#25380)

* Add type to Certificate

* lint

* Add gpus property to AvailableWorkloadProfiles. (#25328)

* Add gpus property to AvailableWorkloadProfiles.

* Fix.

* Pauld/re add build and builder (#25442)

* cherry picked the build and builder updates

* some more cleanup

* added arm id format, removed extra properties, and named the resource provisioning states

* Add OT and AI config (#25350)

* Add OT and AI config

* resolve comments

* Add service bind properties to dapr component (#25781)

* dapr component sb-bind definition add

* fix

* ran npx prettier

* npx prettier

---------

Co-authored-by: [email protected] <[email protected]>

* Adding patching scenario ARM API (#25446)

* cherry picked the build and builder updates

* some more cleanup

* Adding patching scenario and samples

* Added to readme

* Lint fix

* Linting

* Linting error fix

* Addressed comments and added extra properties

* Linting

* Fixing patch skip post action

* Linting

* Addressing comments

* Reverting patchDetails changes

* Fixing pipeline error

* Linting

* Removed invalid required tag

* Fixed examples

* Addressed comments

---------

Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>

* Update microsoft.app 2023-08-01-preview for new DaprSubscription API feature (#25631)

* ACA Dapr Subscription: WIP

Signed-off-by: Bernd Verst <[email protected]>

* Add Dapr Subscription examples

Signed-off-by: Bernd Verst <[email protected]>

* complete examples

Signed-off-by: Bernd Verst <[email protected]>

* Add DaprSubscription descriptions

Signed-off-by: Bernd Verst <[email protected]>

* include updated readme

Signed-off-by: Bernd Verst <[email protected]>

* fix casing

Signed-off-by: Bernd Verst <[email protected]>

* formatting

Signed-off-by: Bernd Verst <[email protected]>

* Use Types v5

Signed-off-by: Bernd Verst <[email protected]>

* Revert "Use Types v5"

v5 causes several mysterious SDK build errors in this branch on this PR. To avoid this error we will stick to v3 for now. We will upgrade to v5 in a future net new branch.

This reverts commit 6a43aac.

---------

Signed-off-by: Bernd Verst <[email protected]>

* Support log streaming on build resource (#25902)

* updated the build resource to support build upload and log streaming, with a token to be passed in via header

* add x-ms-secret and update buildendpoint -> endpoint

* retrieve log stream and upload endpoint tokens through post requests, as they are secrets

* remove secret flag from tokenendpoint property, as we expect users to retrieve it

* rename file

* updated format of how build object returns auth token

* fix example

* added expiration date to token response

* fix auth token to also include build resource information

* renamed getAuthToken -> listAuthToken and removed resource information from token response

* fix examples

* updated samples to be more accurate

* Update serviceBinds with clientType and customizedKeys (#26111)

* Update serviceBinds with clientType and customizedKeys

* force ci

* fix missing }

---------

Co-authored-by: Ahmed ElSayed <[email protected]>

* Adds new Dapr Component Resiliency Feature to existing version `Release app microsoft.app 2023 08 01 preview` (#26107)

* Add Dapr Component Resiliency Policies APIs to ContainerApps 2023-08-01-preview

Signed-off-by: Bernd Verst <[email protected]>

* Update deletion return code

* Fix variable name

* Remove invalid status code from example

* prettify

* Register new resource in readme

* Remove existing import cycle

* Use integers not integer strings

* Fix delete status codes

* More changes

* Update examples

* Revert "Remove existing import cycle" which causes unrelated failures

This reverts commit 8ce1a74.

---------

Signed-off-by: Bernd Verst <[email protected]>

* Add API spec for ContainerApp Resiliency (#25732)

Signed-off-by: Hal Spang <[email protected]>

* Add extendedlocation for job (#26196)

* Add extendedlocation for job

* Add sample for container apps on connectedEnvironment

* Fixes for examples

* Fix connected environment example

* Update old examples

* removing SourceToCloud patching from 08-01-preview (#26245)

* Add diagnostic routes to Container App Jobs (#26227)

* Add diagnostic routes to Container App Jobs

* Update

* Update

* Update

* Update

* Update

* Update

* Update operationIds

---------

Co-authored-by: Michimune Kohno <[email protected]>

* Update app resiliency rest api spec (#26357)

Signed-off-by: Yash Nisar <[email protected]>

* Fix ACA component resiliency examples (#26441)

* Fix arm review comments for container app (#26455)

* update

* update

* update

* Fix SDK build error for Microsoft.App (#26416)

* Fix SDK build error for Microsoft.App

* Update

* update

* update

* update

---------

Co-authored-by: Michimune Kohno <[email protected]>

* Fix arm review comments for container app (#26479)

* fix

* update

* update

* Fix detectorProperties request path (#26431)

* fix list exception

* change param to enum

* fix lint error

* revert not working changes

* fix enum

* rearrange

---------

Co-authored-by: Chenghui Yu <[email protected]>

* Enable dynamicJsonColumns for log analytics configuration (#26432)

* Enable dynamicJsonColumn for log analysis configuration

* Add missed s

* Fix code style

* Update container app usages value to float (#26448)

* update

* update

* update

* Fix App Resiliency retry example (#26560)

Signed-off-by: Yash Nisar <[email protected]>

* Fix lint warning for container app (#26523)

* fix warning

* fix warning

* upate

---------

Signed-off-by: Bernd Verst <[email protected]>
Signed-off-by: Hal Spang <[email protected]>
Signed-off-by: Yash Nisar <[email protected]>
Co-authored-by: zhenqxuMSFT <[email protected]>
Co-authored-by: yalixiang <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Michael Dai <[email protected]>
Co-authored-by: bgashirabake <[email protected]>
Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Harry Li <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Bernd Verst <[email protected]>
Co-authored-by: Ahmed ElSayed <[email protected]>
Co-authored-by: Ahmed ElSayed <[email protected]>
Co-authored-by: halspang <[email protected]>
Co-authored-by: LaylaLiu-gmail <[email protected]>
Co-authored-by: michimune <[email protected]>
Co-authored-by: Michimune Kohno <[email protected]>
Co-authored-by: Yash Nisar <[email protected]>
Co-authored-by: Seris370 <[email protected]>
Co-authored-by: Chenghui Yu <[email protected]>
sjanamma pushed a commit that referenced this pull request Nov 30, 2023
* Adds base for updating Microsoft.App from version preview/2023-05-02-preview to version 2023-08-01-preview

* Updates readme

* Updates API version in new specs and examples

* Add type to Certificate (#25380)

* Add type to Certificate

* lint

* Add gpus property to AvailableWorkloadProfiles. (#25328)

* Add gpus property to AvailableWorkloadProfiles.

* Fix.

* Pauld/re add build and builder (#25442)

* cherry picked the build and builder updates

* some more cleanup

* added arm id format, removed extra properties, and named the resource provisioning states

* Add OT and AI config (#25350)

* Add OT and AI config

* resolve comments

* Add service bind properties to dapr component (#25781)

* dapr component sb-bind definition add

* fix

* ran npx prettier

* npx prettier

---------

Co-authored-by: [email protected] <[email protected]>

* Adding patching scenario ARM API (#25446)

* cherry picked the build and builder updates

* some more cleanup

* Adding patching scenario and samples

* Added to readme

* Lint fix

* Linting

* Linting error fix

* Addressed comments and added extra properties

* Linting

* Fixing patch skip post action

* Linting

* Addressing comments

* Reverting patchDetails changes

* Fixing pipeline error

* Linting

* Removed invalid required tag

* Fixed examples

* Addressed comments

---------

Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>

* Update microsoft.app 2023-08-01-preview for new DaprSubscription API feature (#25631)

* ACA Dapr Subscription: WIP

Signed-off-by: Bernd Verst <[email protected]>

* Add Dapr Subscription examples

Signed-off-by: Bernd Verst <[email protected]>

* complete examples

Signed-off-by: Bernd Verst <[email protected]>

* Add DaprSubscription descriptions

Signed-off-by: Bernd Verst <[email protected]>

* include updated readme

Signed-off-by: Bernd Verst <[email protected]>

* fix casing

Signed-off-by: Bernd Verst <[email protected]>

* formatting

Signed-off-by: Bernd Verst <[email protected]>

* Use Types v5

Signed-off-by: Bernd Verst <[email protected]>

* Revert "Use Types v5"

v5 causes several mysterious SDK build errors in this branch on this PR. To avoid this error we will stick to v3 for now. We will upgrade to v5 in a future net new branch.

This reverts commit 6a43aac.

---------

Signed-off-by: Bernd Verst <[email protected]>

* Support log streaming on build resource (#25902)

* updated the build resource to support build upload and log streaming, with a token to be passed in via header

* add x-ms-secret and update buildendpoint -> endpoint

* retrieve log stream and upload endpoint tokens through post requests, as they are secrets

* remove secret flag from tokenendpoint property, as we expect users to retrieve it

* rename file

* updated format of how build object returns auth token

* fix example

* added expiration date to token response

* fix auth token to also include build resource information

* renamed getAuthToken -> listAuthToken and removed resource information from token response

* fix examples

* updated samples to be more accurate

* Update serviceBinds with clientType and customizedKeys (#26111)

* Update serviceBinds with clientType and customizedKeys

* force ci

* fix missing }

---------

Co-authored-by: Ahmed ElSayed <[email protected]>

* Adds new Dapr Component Resiliency Feature to existing version `Release app microsoft.app 2023 08 01 preview` (#26107)

* Add Dapr Component Resiliency Policies APIs to ContainerApps 2023-08-01-preview

Signed-off-by: Bernd Verst <[email protected]>

* Update deletion return code

* Fix variable name

* Remove invalid status code from example

* prettify

* Register new resource in readme

* Remove existing import cycle

* Use integers not integer strings

* Fix delete status codes

* More changes

* Update examples

* Revert "Remove existing import cycle" which causes unrelated failures

This reverts commit 8ce1a74.

---------

Signed-off-by: Bernd Verst <[email protected]>

* Add API spec for ContainerApp Resiliency (#25732)

Signed-off-by: Hal Spang <[email protected]>

* Add extendedlocation for job (#26196)

* Add extendedlocation for job

* Add sample for container apps on connectedEnvironment

* Fixes for examples

* Fix connected environment example

* Update old examples

* removing SourceToCloud patching from 08-01-preview (#26245)

* Add diagnostic routes to Container App Jobs (#26227)

* Add diagnostic routes to Container App Jobs

* Update

* Update

* Update

* Update

* Update

* Update

* Update operationIds

---------

Co-authored-by: Michimune Kohno <[email protected]>

* Update app resiliency rest api spec (#26357)

Signed-off-by: Yash Nisar <[email protected]>

* Fix ACA component resiliency examples (#26441)

* Fix arm review comments for container app (#26455)

* update

* update

* update

* Fix SDK build error for Microsoft.App (#26416)

* Fix SDK build error for Microsoft.App

* Update

* update

* update

* update

---------

Co-authored-by: Michimune Kohno <[email protected]>

* Fix arm review comments for container app (#26479)

* fix

* update

* update

* Fix detectorProperties request path (#26431)

* fix list exception

* change param to enum

* fix lint error

* revert not working changes

* fix enum

* rearrange

---------

Co-authored-by: Chenghui Yu <[email protected]>

* Enable dynamicJsonColumns for log analytics configuration (#26432)

* Enable dynamicJsonColumn for log analysis configuration

* Add missed s

* Fix code style

* Update container app usages value to float (#26448)

* update

* update

* update

* Fix App Resiliency retry example (#26560)

Signed-off-by: Yash Nisar <[email protected]>

* Fix lint warning for container app (#26523)

* fix warning

* fix warning

* upate

* update tests for jobs

* update tests for jobs

* fix name and resource values in examples

* fix name and resource values in examples

* fix name and resource values in examples

* fix name and resource values in examples

---------

Signed-off-by: Bernd Verst <[email protected]>
Signed-off-by: Hal Spang <[email protected]>
Signed-off-by: Yash Nisar <[email protected]>
Co-authored-by: Zunli Hu <[email protected]>
Co-authored-by: zhenqxuMSFT <[email protected]>
Co-authored-by: yalixiang <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Michael Dai <[email protected]>
Co-authored-by: bgashirabake <[email protected]>
Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Harry Li <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Bernd Verst <[email protected]>
Co-authored-by: Ahmed ElSayed <[email protected]>
Co-authored-by: Ahmed ElSayed <[email protected]>
Co-authored-by: halspang <[email protected]>
Co-authored-by: LaylaLiu-gmail <[email protected]>
Co-authored-by: michimune <[email protected]>
Co-authored-by: Michimune Kohno <[email protected]>
Co-authored-by: Yash Nisar <[email protected]>
Co-authored-by: Seris370 <[email protected]>
Co-authored-by: Chenghui Yu <[email protected]>
zman-ms pushed a commit that referenced this pull request Jan 10, 2024
* Adds base for updating Microsoft.App from version preview/2023-05-02-preview to version 2023-08-01-preview

* Updates readme

* Updates API version in new specs and examples

* Add type to Certificate (#25380)

* Add type to Certificate

* lint

* Add gpus property to AvailableWorkloadProfiles. (#25328)

* Add gpus property to AvailableWorkloadProfiles.

* Fix.

* Pauld/re add build and builder (#25442)

* cherry picked the build and builder updates

* some more cleanup

* added arm id format, removed extra properties, and named the resource provisioning states

* Add OT and AI config (#25350)

* Add OT and AI config

* resolve comments

* Add service bind properties to dapr component (#25781)

* dapr component sb-bind definition add

* fix

* ran npx prettier

* npx prettier

---------

Co-authored-by: [email protected] <[email protected]>

* Adding patching scenario ARM API (#25446)

* cherry picked the build and builder updates

* some more cleanup

* Adding patching scenario and samples

* Added to readme

* Lint fix

* Linting

* Linting error fix

* Addressed comments and added extra properties

* Linting

* Fixing patch skip post action

* Linting

* Addressing comments

* Reverting patchDetails changes

* Fixing pipeline error

* Linting

* Removed invalid required tag

* Fixed examples

* Addressed comments

---------

Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>

* Update microsoft.app 2023-08-01-preview for new DaprSubscription API feature (#25631)

* ACA Dapr Subscription: WIP

Signed-off-by: Bernd Verst <[email protected]>

* Add Dapr Subscription examples

Signed-off-by: Bernd Verst <[email protected]>

* complete examples

Signed-off-by: Bernd Verst <[email protected]>

* Add DaprSubscription descriptions

Signed-off-by: Bernd Verst <[email protected]>

* include updated readme

Signed-off-by: Bernd Verst <[email protected]>

* fix casing

Signed-off-by: Bernd Verst <[email protected]>

* formatting

Signed-off-by: Bernd Verst <[email protected]>

* Use Types v5

Signed-off-by: Bernd Verst <[email protected]>

* Revert "Use Types v5"

v5 causes several mysterious SDK build errors in this branch on this PR. To avoid this error we will stick to v3 for now. We will upgrade to v5 in a future net new branch.

This reverts commit 6a43aac.

---------

Signed-off-by: Bernd Verst <[email protected]>

* Support log streaming on build resource (#25902)

* updated the build resource to support build upload and log streaming, with a token to be passed in via header

* add x-ms-secret and update buildendpoint -> endpoint

* retrieve log stream and upload endpoint tokens through post requests, as they are secrets

* remove secret flag from tokenendpoint property, as we expect users to retrieve it

* rename file

* updated format of how build object returns auth token

* fix example

* added expiration date to token response

* fix auth token to also include build resource information

* renamed getAuthToken -> listAuthToken and removed resource information from token response

* fix examples

* updated samples to be more accurate

* Update serviceBinds with clientType and customizedKeys (#26111)

* Update serviceBinds with clientType and customizedKeys

* force ci

* fix missing }

---------

Co-authored-by: Ahmed ElSayed <[email protected]>

* Adds new Dapr Component Resiliency Feature to existing version `Release app microsoft.app 2023 08 01 preview` (#26107)

* Add Dapr Component Resiliency Policies APIs to ContainerApps 2023-08-01-preview

Signed-off-by: Bernd Verst <[email protected]>

* Update deletion return code

* Fix variable name

* Remove invalid status code from example

* prettify

* Register new resource in readme

* Remove existing import cycle

* Use integers not integer strings

* Fix delete status codes

* More changes

* Update examples

* Revert "Remove existing import cycle" which causes unrelated failures

This reverts commit 8ce1a74.

---------

Signed-off-by: Bernd Verst <[email protected]>

* Add API spec for ContainerApp Resiliency (#25732)

Signed-off-by: Hal Spang <[email protected]>

* Add extendedlocation for job (#26196)

* Add extendedlocation for job

* Add sample for container apps on connectedEnvironment

* Fixes for examples

* Fix connected environment example

* Update old examples

* removing SourceToCloud patching from 08-01-preview (#26245)

* Add diagnostic routes to Container App Jobs (#26227)

* Add diagnostic routes to Container App Jobs

* Update

* Update

* Update

* Update

* Update

* Update

* Update operationIds

---------

Co-authored-by: Michimune Kohno <[email protected]>

* Update app resiliency rest api spec (#26357)

Signed-off-by: Yash Nisar <[email protected]>

* Fix ACA component resiliency examples (#26441)

* Fix arm review comments for container app (#26455)

* update

* update

* update

* Fix SDK build error for Microsoft.App (#26416)

* Fix SDK build error for Microsoft.App

* Update

* update

* update

* update

---------

Co-authored-by: Michimune Kohno <[email protected]>

* Fix arm review comments for container app (#26479)

* fix

* update

* update

* Fix detectorProperties request path (#26431)

* fix list exception

* change param to enum

* fix lint error

* revert not working changes

* fix enum

* rearrange

---------

Co-authored-by: Chenghui Yu <[email protected]>

* Enable dynamicJsonColumns for log analytics configuration (#26432)

* Enable dynamicJsonColumn for log analysis configuration

* Add missed s

* Fix code style

* Update container app usages value to float (#26448)

* update

* update

* update

* Fix App Resiliency retry example (#26560)

Signed-off-by: Yash Nisar <[email protected]>

* Fix lint warning for container app (#26523)

* fix warning

* fix warning

* upate

---------

Signed-off-by: Bernd Verst <[email protected]>
Signed-off-by: Hal Spang <[email protected]>
Signed-off-by: Yash Nisar <[email protected]>
Co-authored-by: zhenqxuMSFT <[email protected]>
Co-authored-by: yalixiang <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Michael Dai <[email protected]>
Co-authored-by: bgashirabake <[email protected]>
Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Harry Li <[email protected]>
Co-authored-by: Paul Dorsch <[email protected]>
Co-authored-by: Bernd Verst <[email protected]>
Co-authored-by: Ahmed ElSayed <[email protected]>
Co-authored-by: Ahmed ElSayed <[email protected]>
Co-authored-by: halspang <[email protected]>
Co-authored-by: LaylaLiu-gmail <[email protected]>
Co-authored-by: michimune <[email protected]>
Co-authored-by: Michimune Kohno <[email protected]>
Co-authored-by: Yash Nisar <[email protected]>
Co-authored-by: Seris370 <[email protected]>
Co-authored-by: Chenghui Yu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved-SdkBreakingChange-Go Approve the breaking change tracking for azure-sdk-for-go Approved-SdkBreakingChange-Python ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review CI-BreakingChange-Go resource-manager
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants