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

Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) #28

Merged
merged 27 commits into from
Mar 21, 2022

Conversation

runefa
Copy link
Collaborator

@runefa runefa commented Mar 18, 2022


This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

@runefa runefa requested a review from calvinsID as a code owner March 18, 2022 18:40
Copy link
Owner

@calvinsID calvinsID left a comment

Choose a reason for hiding this comment

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

Awesome, thanks! Just a few small changes

@runefa runefa requested a review from calvinsID March 21, 2022 18:04
@runefa runefa merged commit c346bc7 into calvinsID:containerapp Mar 21, 2022
calvinsID pushed a commit that referenced this pull request Mar 22, 2022
… removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

Co-authored-by: Haroon Feisal <[email protected]>
runefa added a commit that referenced this pull request Mar 23, 2022
* Update workaround information in parameter helptext (#4321)

* update workaround information in helptext

* update setup.py to use 0.12.0 version

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

* [Release] Update index.json for extension [ cosmosdb-preview ] (#4323)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1296094

Last commit: https://github.com/Azure/azure-cli-extensions/commit/68fd2df68090503ccaa95a8135bfc651b0a5136b

* [Enterprise-Edge] : `az staticwebapp enterprise-edge enable` : register Microsoft.CDN provider automatically upon running enable command  (#4324)

* register Microsoft.CDN provider automatically

* bump version and fix style issues

* small fixes; use sdk method

* fix sdk call

* Update version (#4322)

* [Release] Update index.json for extension [ storage-preview ] (#4326)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1307240

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f6aa6d8194a7dec21f29b4c92dec5946c60699f1

* [Release] Update index.json for extension [ enterprise-edge ] (#4325)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1306508

Last commit: https://github.com/Azure/azure-cli-extensions/commit/7fc818250de852b5d6ad46ed537694f3ee1e8038

* Make cli zone redundant property be none (#4327)

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

* fix regex used to check node image version (#4319)

* Update Terminology end-to-end TLS to ingress-to-app TLS (#4320)

* Update Terminology end-to-end TLS to ingress-to-app TLS

* Update position

* Update for static analysis

* Add it for e2etls

* Remove useless line

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

* Fix certificate list output error (#4330)

* Fix certificate output error

* Update according to comments

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

* [k8s-configuration] Release v1.4.1 with fixed help text (#4328)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update CODEOWNERS

* Update azure-pipelines.yml

* Create pull.yml

* Update pull.yml

* Update pull.yml

* Update pipelines file

* Update k8s-configuration name

* Update test script params

* Update pipeline file

* Remove codeowners

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update custom pipelines files

* Add publish step to k8s-configuration

* Update pipeline to publish extension

* Update public extension pipeline

* Change condition variable

* Update pipeline naming

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

* Stage Public Version of k8s-extension 0.2.0 for official release (#15)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update public extension pipeline

* Change condition variable

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

Co-authored-by: [email protected] <Action - Fork Sync>

* Remove custom pipelines file

* Update extension description, remove private const

* Update pipeline file

* Disable check ref docs

* Disable refs docs

* Update to include better create warning logs and remove update context (#20)

* Update to include better create warning logs and remove update context

* Remove help text for update

* Fix spelling error

* Update message

* Fix k8s-extension conflict with private version

* Fix style errors

* Fix filename

* add customization for microsoft.azureml.kubernetes (#23)

* add customization for microsoft.azureml.kubernetes

* Update release history

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Add E2E Testing from Separate branch into internal code (#26)

* Add internal e2e testing

* Change to testing folder

* Inference CLI validation for Scoring FE (#24)

* cli validation starter

* added the call to the fe validation function

* nodeport validation not required

* test fix

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

* legal warning added (#27)

* Remove deprecated method logger.warn

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Add Azure Defender to E2E testing (#28)

* Add azure defender testing to e2e

* Remove the debug flag

* Add configuration testing

* Fix pipeline failures

* Make test script more intuitive

* Remove parameter from testing

* Add some debug

* Fix wrong location for k8s config whl

* Fix pip install upgrade issue

* Fix pip install upgrade issue

* Add Check for Provider Registration and Refactor (#19)

* Add check for provider registration and refactor

* Fix bug in checking registration

* Add license header to utils

* Update private key check and error messaging

* Update based on refactoring

* Fix failing tests

* Add provider registration check

* Create a test for uppercase url, address comments

* Add blank line to fix style check

* Testing increase to ubuntu-latest

* Update k8s-configuration Models to Track2 (#63)

* Update models to track2

* Increase k8s-configuration version number

* Update kind version

* Change error to warning because of DSA failure

* Upgrade helm operator chart version (#75)

* Pin helm version

* Bump version

* Migrate pipeline (#90)

* Disable updates on configuration tests (#89)

* Release k8s-configuration v1.2.0 for Flux v2 Public Preview (#86)

* Scaffold out the k8s-config package

* Base implementation of CLI commands

* Add create scenario and cleanup in consts

* Add help text to commands, params

* Add other clients to client factory

* Automatically installing the flux extension

* Move flux and extension into modules

* Updated the versioned sdks

* Push working command for testing

* Update to multi api versioned sdk

* Support other extension methods

* Fix nullity check

* Add source control provider

* Add scc commands

* Add defer logic for create with cache

* Use default extension with identity

* Fix identity creation

* Add kustomization caching

* Add formatters

* Add scc provider

* Add help text for k8s-config fluxv1

* Add help text for extension

* Allow force delete of extension and fluxconfiguration

* Add location to the extension model

* Update with latest from k8s-extension

* Add k8s-config testing

* Add license header

* Fix all style issues

* Update codeowners file

* Validate data before checking cluster compliance

* No kustomizations warning

* Fix identity issue in 2020-07-01

* Fix k8s regex

* Fix configuration name regex validation

* Fix name length validation

* Adding some validation warnings

* Add protected settings to request

* Exclude private test path

* Add suspend functionality

* Add correct values to build Kustomization

* Add no_wait

* Fix style issues

* Use base64 encoded httpsUser

* Fix formatting error and base64 encoding error

* Fix style issues

* Fix force

* Updated help text

* Style fixes

* Increase namespace maximum len

* Add managed cluster support to k8s-config

* Custom confirmation when prune is enabled

* Add flux commands to existing k8s-configuration

* Remove extension provider from CLI

* Fix style issues

* Override extension variables

* Strip newlines from known_hosts file

* Update help text and validators

* Strip newlines from known hosts

* Add provisioning state check for flux extension

* Pin helm version

* Remove validation from create command

* Add patch support with new SDK

* Add implementation for CRUD of source and kustomization

* Fix errors on patch

* Fix some bugs in patching properties

* Add fixes for patch in k8s-configuraiton

* Change duration formatting in table output

* Add validation and conversion for durations

* Bump verison and fix typo

* Fix bug with dependencies

* Fix linter and style issues

* Fix delete prune check

* Add flux testing

* Create separate jobs for scenarios

* Update error text

* Fix filepath suggestion from CLI team

* Fix unneeded file edit

* Add a Deployed Object List to the Flux CLI (#91)

* Enable a deployed object list on the CLI

* Show detail when extension install fails

* Bump version

* Update deployed object format (#93)

* Fix help text for consistency

* Enable Bucket Support in the CLI (#92)

* Update vendored_sdks

* Enable source kind generation factories and use kwargs to pass to provider

* Add better validation logic to source generator

* Move away from classes in the provider directory

* Fix style issues using black auto-formatter

* Fix linter failures

* Update identity with api version and rp in same function

* Bucket Testing for E2E Testing (#96)

* Bucket testing

* Add switching kind test to update CLI

* Bump version

* Fix help text and parameter naming for bucket (#100)

* Fix default help text for parameters (#101)

* Remove unneeded files

* Edit history with breaking change

Co-authored-by: [email protected] <Action - Fork Sync>
Co-authored-by: yuyue9284 <[email protected]>
Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Lia Kazakova <[email protected]>

* fix: 🐛 Validate deployment parameters with source type (#4329)

* fix: 🐛 Should enable persistent storage when updating App

* Validate deployment source parameters

* fix lint

* fix bugs

* fix test

* [IoT] IoT Extension Release v0.13.0 (#4333)

* Remove IoT Extension v0.10.14
* Add IoT Extension v0.13.0
* Update linter exclusions

* [Release] Update index.json for extension [ k8s-configuration ] (#4332)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1309054

Last commit: https://github.com/Azure/azure-cli-extensions/commit/649b29ab8f154006be6cc4fd2c53f242eec0a319

* update location (#4334)

* Update data plane client from swagger (#4318)

* Update generated client to latest Azure Quantum Python Data Plane

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

* Update Data-Plane client generate script to delete previous output folder contents (#4340)

* [Resource Mover] Update Resource Mover CLI (#4309)

- generate vendored SDK
- bump up version to 0.1.1

* [Release] Update index.json for extension [ resource-mover ] (#4343)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1317636

Last commit: https://github.com/Azure/azure-cli-extensions/commit/e9497c3fe0b619aa92548f21246f584fca9c3d4b

* [Databricks]az databricks workspace create: Add new parameters --publ… (#4331)

* [Databricks]az databricks workspace create: Add new parameters --public-network-access and --required-nsg-rules

update databricks sdk version and add new parameters

* Update

1.move databricks.models into load_arguments function
2.update extension version in

* [Logic] Upgrade SDK to fix deserialization errors (#4347)

* [Release] Update index.json for extension [ logic ] (#4349)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1321294

Last commit: https://github.com/Azure/azure-cli-extensions/commit/16595ffb29a5d5cc24593c3656d25fa47b71196c

* fetching tenantID from subscription object instead of graph client (to pickup tenant Id of sub where cc is getting created instead of user/spn's tenant) (#4336)

* Fetch the tenant ID from the subscription object instead of graphclient

* nit

* nit

* revert proxy tenantid change

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* Adding new module DataMigration (#4298)

* Adding initial settings to module

* Working on Linter, License and Cred scan errors

* Working on Static analysis error

* Hiding Register-ir commands for now

* Lint and Integration Tests solving

* Unhiding register-ir command and adding windows test to assessment command

* Making error messages proper

* Nit changes from comments

* MultiServer support for Assessment and nit changes

* Adding Proper Json format, lower case for access, seperate util for Windows check

* Correcting Lint and Static errors

* Removing to-* from command names

* Adding correct error formats

* Removing Update operation from DatabaseMigration

* Updating readme and help

* Nit: Readme corrections

* Added CLI Test Cases

* Removed IR AuthKeys and Unused Imports

* Adding Default IR Program files path for 32-bit error

* Nit and Static analysis fix

* Added File Extension Validation

* Removed unnecessary spaces and fixed static analysis Issues

* Deferred IR path and MSI Extension validations

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

* [Release] Update index.json for extension [ connectedk8s ] (#4350)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1323101

Last commit: https://github.com/Azure/azure-cli-extensions/commit/b8abdf1987ed799ed1d5a2cc846d8696c86cc5d7

* [AKS] az aks update/az aks nodepool update: Allow empty string as label value  (#4351)

* skip none check for labels

* use correct value

* change empty string check in aks nodepool update

* revert change

* fix value

* revert change

* [Release] Update index.json for extension [ datamigration ] (#4352)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1323628

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f412582fe051ce76f555fee010f3d8240118ed18

* Change link in datamigration setup file (#4353)

* add rdbms-connect to cloud shell (#4313)

* Update azurestackhci version to 0.2.3 (#4356)

* [Release] Update index.json for extension [ rdbms-connect ] (#4359)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1325769

Last commit: https://github.com/Azure/azure-cli-extensions/commit/83cde80e25e9273eece470369f18f5a3b5662249

* fix: fix builder validator (#4339)

* fix: fix builder validator

* fix

* [Release] Update index.json for extension [ databricks ] (#4346)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1321182

Last commit: https://github.com/Azure/azure-cli-extensions/commit/c908659851fa910522935d8e71c7d717b5e22824

* [Spring-Cloud] fix app deployment validator can retrieve app name from app and name (#4335)

* fix app deployment validator can retrieve app name from app and name

* fix lint

* Initial commit for 0.12.0 release (#4341)

* Initial commit for 0.12.0 release

* Update src/quantum/HISTORY.rst to use cost instead of price

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

* Update commands.py

* Update src/quantum/HISTORY.rst

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

* Restart CI/CD

Co-authored-by: Ricardo Espinoza <[email protected]>
Co-authored-by: Angela Burton <[email protected]>

* [Release] Update index.json for extension [ quantum ] (#4364)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1328638

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d28093638614c9af2cef3041d6422691c6bb1e99

* Baremetalinfrastructure extension stable release support (#4180)

* baremetalinfrastructure extension release support

* adding whl extension

* removing unnecessary files

* Alerts management new alert processing rule (#4337)

* Alerts management new alert processing rule

Co-authored-by: Liran Englender <[email protected]>
Co-authored-by: kai ru <[email protected]>

* chore(arcdata): arcdata version bump to 1.2.0 (#4365)

* chore(arcdata): version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* {AKS} Declare delete command and update recordings (#4367)

* add delete command

* fix location for gpu test

* update recordings

* [Image Copy] Fix the hyper_v_generation of image copy (#4372)

* Fix generation

* Fix CI

* Update ml extension to version 2.0.7 (#4370)

* Update ml extension to version 2.0.6

* fix linter issue

* Remove is_preview from snapshot related commands (#4376)

* {AKS} Use default serialization for maintenanceconfiguration commands (#4362)

* Use msrest deserilizer

* fix as per comments

* remove the import

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

* [Release] Update index.json for extension [ image-copy ] (#4373)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1332056

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f1bdd0318036276b6aa199103895b7e02d90356d

* Release 0.23.0 for azure dev ops (#4378)

* [AKS] Allow aks nodepool update --node-taints (#4363)

* allow edit taints and add test

* add missing parameter

* change var name

* .get_output_in_json()

* give value

* try fix'

* fix test

* address comments

* add yaml file to package data (#4382)

* [Release] Update index.json for extension [ alertsmanagement ] (#4366)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1329629

Last commit: https://github.com/Azure/azure-cli-extensions/commit/2e6b127b67489ae3ad36419a436e6529b4d38a73

* [CosmosDB] Mongo RBAC (#4389)

* Mongo RBAC

* Fixing Lint issues

* Fixing help Lint issue

* [Release] Update index.json for extension [ cosmosdb-preview ] (#4392)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1351532

Last commit: https://github.com/Azure/azure-cli-extensions/commit/9dac199b9a77beb191e870f44adb0d42b5757c63

* Fix test: change to use soft taint on system pool in test (#4393)

* change to soft taint on system pool

* outbound ip workaround

* {AKS} Remove hardcoded k8s version in test case `test_aks_create_and_update_ipv6_count` (#4394)

* remove hardcoded k8s version in test case

* update recording

* Update azure-pipelines.yml for Azure Pipelines (#4399)

* [Vm-repair] preview script bug fix (#4388)

* bug fix

* adding history

* changing history

* Apply suggestions from code review

* bug fix preview

* adding history

* version bump

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

* {AKS} Update version to 0.5.52 (#4377)

* update version to 0.5.52

* update history

update recording for test_aks_create_and_update_outbound_ips

show pip as workaround

update jmes check

fix

update recording

* Add clear for sso & cors  properties (#4371)

* AlertsManagement-CLI-GA-Fixes (#4400)

* 1. help file fix
2. removed redundant import from typing_extension

* Increased minor version and added it to the history file

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

* [Release] Update index.json for extension [ alertsmanagement ] (#4403)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1355227

Last commit: https://github.com/Azure/azure-cli-extensions/commit/fe44a1bb123a58b7e8248850bdc20555ca893406

* [Quantum] Workspace creation race condition fix (#4314)

* Add ARM templates

* Add ARM template that does role assignment

* Modify auth code and move template folder to operations

* Try AzureCliCredential

* Restore deleted Storage Account lines in template

* Fixed Deployment.properties problem

* Call _add_quantum_providers for validation

* Call _get_data_credentials

* Fix pylint infractions

* Eliminate Pylint message W1514

* Add test that does role assignment

* Use short storage account name in utils.py

* Use workspace name for deployment name

* Use workspace name for storage account deployment name.

* Use workspace name for deployment name and temporarily comment-out the provisioningState check in test

* Add workspace creation timeout

* Add ARM template file to wheel

* Delete commented-out lines and unused ARM files

* Fix static analysis violations

* Re-run CI/CD checks

* update url links (#4354)

Co-authored-by: cxznmhdcxz <>

* Change links for setup file till k8s-configuration module (#4348)

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

* [Public Preview] Azure Confidential Ledger Extension (#4276)

* generated code

* add recording

* update tests

* json deserialize

* regenerated sdk

* tests passed!

* run pre-pr checks

* update docstrings

* update codeowners

* ci fix attempts

* add to codeowners

* update service_name.json

* Update src/confidentialledger/azext_confidentialledger/tests/latest/example_steps.py

* Update src/confidentialledger/azext_confidentialledger/tests/latest/test_confidentialledger_scenario.py

* ci fixes again

* coverage file updated

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

* [Release] Update index.json for extension [ confidentialledger ] (#4410)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1358406

Last commit: https://github.com/Azure/azure-cli-extensions/commit/ddba03626f75e10984652bf74235bb3cd5c4c894

* [Stream Analytics] Bump up version to 0.1.2 (#4404)

* Refine help messages

* Add tests and recordings

* Fix style and linter checks

* update disk-pool-volume-create --mount-option to only accept one para… (#4396)

* update disk-pool-volume-create --mount-option to only accept one parameter

* updated mount option help and added unit tests

* mount type help text update

* {AKS} Clean up some deprecated code (#4412)

* remove validator for cluster_autoscaler_profile

* clean up unused **kwargs

* change to property & SimpleNamespace for nat_gateway_models

* fix lint issues

* remove another hardcoded k8s version in test

* update recording

* {account} Bump up `azure-mgmt-subscription` SDK to `3.0.0` (#4409)

* bump up subscription SDK

* update

* [Release] Update index.json for extension [ account ] (#4416)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1362133

Last commit: https://github.com/Azure/azure-cli-extensions/commit/8c9c7841dbc38d9c114c2b6a9de04d963577374f

* [Release] Update index.json for extension [ vmware ] (#4415)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1361421

Last commit: https://github.com/Azure/azure-cli-extensions/commit/8dbc3b19762c619c11613a5ef7748417e8cfe2ab

* [Release] Update index.json for extension [ aks-preview ] [ hack ] (#4408)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1357913

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d236554051bdd0f359ec1bd3ad4ece70cccd7f53

* [Release] Update index.json for extension [ stream-analytics ] (#4414)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1361403

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d9108c0305d48feb92a73cc63954380bc30b58a4

* {AKS} Vendor 2022-01-02-preview for aks-preview (#4417)

* vendor new sdk

* update default api

* fix check in test test_aks_nodepool_update_taints_msi

* update recordings'

* fix recordings

* fix check

* [Release] Update index.json for extension [ vm-repair ] (#4401)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1355202

Last commit: https://github.com/Azure/azure-cli-extensions/commit/2cd5fda928273f4cd5c5a0488d306a8d6afa5c48

* AKS CRG for Aks v20211101preview (#4292)

* add support for capacity reservation group

* fix an UT issue

* fix a static test issue

* add new test function for new get_crg_id

* remove type=str per comment.

* Adds test case for crg

* Adds test case for crg

* fixed a merging error

* fixed a merging error

* fixed a space error

* fix: add runtime version in build (#4420)

* fix: add runtime version in build

* fix npe

* fix

* Update src/spring-cloud/azext_spring_cloud/_buildservices_factory.py

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

* add test

* fix

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

* Add Arc Appliance extension 0.2.15 (#4421)

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

* AKS preview upgraded to 0.5.53 with CRG (#4422)

* add support for capacity reservation group

* fix an UT issue

* fix a static test issue

* add new test function for new get_crg_id

* remove type=str per comment.

* Adds test case for crg

* Adds test case for crg

* fixed a merging error

* fixed a merging error

* fixed a space error

* fixed a space error

* upgrade to 0.5.53

* [Spring-Cloud] Upgrade version (#4294)

* [Spring-Cloud] Upgrade version

* Update HISTORY.md

* Update HISTORY.md

* Update HISTORY.md

* Update HISTORY.md

* Update HISTORY.md

* Update HISTORY.md

* Update wording

* Update commands.py

* TOC - fixed Azure service names (#4425)

* fixed TOC service names

* fixed Azure service names

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

* [Release] Update index.json for extension [ aks-preview ] (#4423)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1368539

Last commit: https://github.com/Azure/azure-cli-extensions/commit/b01494990b0c5672b1513727f4adbf39f95ad338

* [Release] Update index.json for extension [ spring-cloud ] (#4426)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1370330

Last commit: https://github.com/Azure/azure-cli-extensions/commit/6593c6a9df7071b3052ac5cd516967eed27ea928

* Add support for host group id (#4419)

* some initial changes for host group

* add host-group-id support

* remove some unnecessary changes

* format _validators.py by adding extra line

* resolve comments

* add missing comma

* update test_decorator.py

* update version to 0.5.54

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

* [Release] Update index.json for extension [ aks-preview ] (#4430)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1373529

Last commit: https://github.com/Azure/azure-cli-extensions/commit/16bd206af6ca04510694a98a16e42e4fecef7ba4

* [AKS] Add network plugin none support (#4007)

* rebase

* improve help text clarity

* fixes

* closing params

* reset the count

* include location

* add test recording file

* add check network plugin in test

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

* remove csi deployment (#4431)

* {AKS} Update unit test coverage configuration (#4432)

* clean up ut

* update ut coverage config

* remove read_only property setting in ut

* [appservice-kube] Fix #4357, #4358 : support functions v4 and powershell functions (#4411)

* support functions v4; powershell functions

* add custom handler runtime and fix CI issue

* use specific error type

* set max CLI version

* fix CI issues

* fix style

* [Release] Update index.json for extension [ appservice-kube ] (#4434)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1374379

Last commit: https://github.com/Azure/azure-cli-extensions/commit/e305b4d01f56c38f94be004bea18d19bed534502

* [Release] Update index.json for extension [ aks-preview ] (#4433)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1374361

Last commit: https://github.com/Azure/azure-cli-extensions/commit/811a3192458606c794e76071c830872f4f5d003e

* Added version 2.1.1 for Azure ML CLI (#4418)

* CLI 2.0.3 public preview release

* Updated CLI release with linting fixes

* Updated .whl file

* Fixed conflict

* Fix change to 2.0.7 checksum

* Replaced 2.1.0 with 2.1.1

* AzureML CLI patch release 2.1.2 (#4442)

* Customcert and fix for non escaping values in helm (#4428)

* Use userValues.txt instead of --resuse-values in update command

* fix error msg

* added custom cert

* change setup.py

* Helmupdatefix (#4)

* Use userValues.txt instead of --resuse-values in update command

* fix error msg

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* fix output file

* Helmupdatefix (#5)

* Use userValues.txt instead of --resuse-values in update command

* fix error msg

* fix output file

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* place values file in .azure folder

* place values file in .azure folder

* Helmupdatefix (#6)

* Use userValues.txt instead of --resuse-values in update command

* fix error msg

* fix output file

* place values file in .azure folder

* place values file in .azure folder

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* place values file in .azure folder

* nit

* nit

* nit

* setp.py and history.rst updates

* modify the way of fetching userValues.txt path

* Waiting for LRO to complete before moving ahead with agent installation/deletion

* update history.rst

* fix

* fix location variable

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>
Co-authored-by: siriteja <[email protected]>

* [Release] Update index.json for extension [ connectedk8s ] (#4445)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1383624

Last commit: https://github.com/Azure/azure-cli-extensions/commit/6d0ff505113ca853fea82e5f0573a8edb96b63a6

* {appservice-kube} Ensure compatibility with upcoming CLI version (#4384)

* ensure compatibility with upcoming CLI version

* change min CLI version to 2.34.0

* Update src/appservice-kube/HISTORY.rst

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

* fix 'az functionapp create' compatibility

* Update HISTORY.rst

* remove unneeded methods

* fix style

* fix 'az webapp create'

* bump version no.

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

* add message of the day for aks agentpools (#4381)

* add message of the day for aks agentpools

* bump version

* add validation for linux only

* fix tests

* fix spacing

* try to fix ut

* fix: typo, remove cluster creation validation

* add missing comma

* ut

* use more specific validator error

* update recording

* add a line

* bump version, test fixes, encoding fixes

* fix lint

* fix missing coverage

* change to iter (#4449)

* Data migration - Adding SKU recommendation (#4436)

* Adding SKU Recommendation Command

* Adding multiserver support - currently fails

* Adding Support for DatabaseAllowList and DatabaseDenyList

* Correcting lint and style errors

* Adding help for perf-data-collection and sku-recommendation

* Adding changes as per suggestions

* Nit change: config_file_path

* Adding custom cmds to common group

* Updating version of extension

* [Release] Update index.json for extension [ datamigration ] (#4456)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1387498

Last commit: https://github.com/Azure/azure-cli-extensions/commit/7cf0cbe19dac82a16fbf054aa9f4bf10dd1989c5

* [Monitor] `az monitor app-insights web-test`: Support Application Insights Web Test (#4440)

* Update vendored SDK

* Add list/show/create/update/delete commands

* Add parameter actions

* Add help messages

* Fix style and linter checks

* Add tests and recording files

* Update version and changelog

* Restorable APIs for Tables and Gremlin. Added Latest backup info for Gremlin and Table and also added start and end time filter for collection listing. (#4385)

* Table/Gremlin changes (Generated code)

* Table/Gremlin code and test changes for new commands

* New change

* New changes

* Working

* New Vendored sdk

* Test Updates and Session Records.

* Merge changes for removing EnableGremlinV2 Support

* Fixing Linter

* Update ReadMe, History and Version.

* [Release] Update index.json for extension [ application-insights ] (#4458)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1389585

Last commit: https://github.com/Azure/azure-cli-extensions/commit/a1ac7443895dd794ae3a86dfabfab5c4ea04a5d3

* [Release] Update index.json for extension [ aks-preview ] (#4455)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1387121

Last commit: https://github.com/Azure/azure-cli-extensions/commit/e4edce2ed79082e0cc183148810da86a7c3a52ca

* [EdgeOrder] Initial Release (#4450)

* Add command groups

* Simplify long parameters

* Refine help messages and README.md

* Add tests and recording files

* Fix style and linter checks

* [Release] Update index.json for extension [ edgeorder ] (#4460)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1390385

Last commit: https://github.com/Azure/azure-cli-extensions/commit/a408888db3f9ab1c5ea63f096b0973d997e87a22

* DataShare CLI (#4454)

* adding auo-generated cli commands and basic-test file with recording, commit-1

* adding tests, commit-2

* adding tests, commit-3

* adding tests, commit-4

* Updating cli version to 2021-08-01 and modifying tests

* updating test files

* Fixing CI checks

* removing auto-generated test file

* addressing PR comments, rev-1

* reinstating experimental tag

* updating test recording file after updating the branch with main

* updating test recording file after merging azure-cli repo with main on local

* updating test file after installing newer version of storage

* [Release] Update index.json for extension [ cosmosdb-preview ] (#4459)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1389592

Last commit: https://github.com/Azure/azure-cli-extensions/commit/52bca4ddf4b52ff1bf36da92461b3981e463f63f

* [Release] Update index.json for extension [ datashare ] (#4461)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1390624

Last commit: https://github.com/Azure/azure-cli-extensions/commit/00f0762f671793f96aa5b1c8e773077121f2946b

* fix: 🐛 [Spring-Cloud] Failed to remove jvm option in Enterprise (#4444)

* fix: 🐛 [Spring-Cloud] Failed to remove jvm option in Enterprise

* fix test

* fix lint

* Add more test

* fix app update

* fix UT recording

* add log

* fix lint

* Add new WHL for SCVMM scenarios (#4465)

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

* [Spring-Cloud] Add a new parameter "--build-env" for app deploy (#4443)

* fix: fix the deploy of gradle project with multiple modules

* add a new parameter --build-env

* fix typos

* fix

* fix

* fix

* fix

* fix

* fix

* fix test

* fix test

* fix test

* fix test

* fix test

* fix

* Update src/spring-cloud/HISTORY.md

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

* update ClientRequestError to ArgumentUsageError

* fix test

* fix test

* fix test

* update validator

* add new line

* update history

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

* chore(arcdata): arcdata version bump to 1.2.1 (#4466)

* chore(arcdata): arcdata version bump to 1.2.1

* chore(arcdata): arcdata version bump to 1.2.1

* Add CLI version 0.1.48 (#4470)

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

* [Release] Update index.json for extension [ spring-cloud ] (#4467)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1393299

Last commit: https://github.com/Azure/azure-cli-extensions/commit/47fd457596e401a5f3c5524c176064cc46559696

* [Release] Update index.json for extension [ appservice-kube ] (#4481)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1386720

Last commit: https://github.com/Azure/azure-cli-extensions/commit/4fbcd8fcc049730f65d92a30da1eb8628559b76b

Co-authored-by: Azure CLI Team <[email protected]>

* chore(arcdata): arcdata version bump to 1.2.2 (#4473)

* chore(arcdata): arcdata version bump to 1.2.2

* chore(arcdata): arcdata version bump to 1.2.2

* Update version and history info for az quantum extension 0.13.0 (#4484)

* Added ArcAppliance 0.2.16 (#4487)

Co-authored-by: Sai Sankar Gochhayat <[email protected]>

* [Release] Update index.json for extension [ quantum ] (#4489)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1408510

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f659327e9aa96b842cbcfe94a58d5ca4609a8041

* {Quota}Generate quota with codegen (#4475)

* {Quota}Generate quota with codegen

* Update setup.py

* Update test_quota_crud.yaml

* Update test_quota_crud.yaml

* Update

* Update

* Update _help.py

* Update _help.py

* [Release] Update index.json for extension [ quota ] (#4496)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1412890

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d292d6d0295e5245cff22da0bdc4bbf38ed92dd7

* [Quantum] Show progress messages and dots before executing commands (#4480)

* Show the first progress indicator dot before starting ARM template deployment

* Show progress messages on 'execute' and 'job submit'

* Change f-string to simple string

* chore(arcdata): arcdata version bump to 1.2.3 (#4501)

* Revert "chore(arcdata): arcdata version bump to 1.2.3 (#4501)" (#4502)

This reverts commit c45ca8102492ece8d1f7594f25224612954a2986.

* chore(arcdata): arcdata version bump to 1.2.3 (#4503)

* [appservice-kube] Make compatible with new SDK (#4494)

* fix plan create and webapp create

* bump version, include bug fix from #4493, fix style, fix webapp update bug, add basic regression tests

* remove deprecated 'az webapp update' params

* Fix UUID issue for data disk (#4479)

* new version 0.4.2

* new version 0.4.2

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Update src/vm-repair/azext_vm_repair/custom.py

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

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

* Move cli to production subscription (#4505)

* {NetworkManager}Add new commands (#4413)

* {NetworkManager}Add new commands

Add new commands 'az network manager connection', 'az network manager connection management-group', 'az network manager scope connection', 'az network manager group static-members'

* Update _help.py

* Update _params.py

Bump up azure-mamt-network SDK to 2021_05_01_preview and Update  params and commands.

* Update

* Update

* Update

* Update

* Update

* Update

* update tests for new api version 2021-05-01-preview for network-manager extension

* Update

* Update

* Update custom.py

* Update

* Update

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

* CosmosDB Adding enable MaterializedViews parameter for create/update database … (#4499)

* Adding enable MaterializedViews parameter for create/update database account

* api-version=2021-11-15-preview

* CassandraEnableMaterializedViews

* Update src/cosmosdb-preview/HISTORY.rst

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

* Update src/cosmosdb-preview/README.md

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

* Update src/cosmosdb-preview/azext_cosmosdb_preview/_params.py

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

* Update linter_exclusions.yml

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

* [Release] Update index.json for extension [ network-manager ] (#4511)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1419833

Last commit: https://github.com/Azure/azure-cli-extensions/commit/03dc8173839f44a960b25e625c23dbc34932143b

* [Release] Update index.json for extension [ vm-repair ] (#4507)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1416685

Last commit: https://github.com/Azure/azure-cli-extensions/commit/b68739c74809633eb9d358034f7884a81a2158d6

* [Release] Update index.json for extension [ appservice-kube ] (#4504)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1415742

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f77b373146be6cb2f15b1cccec5311f161b23ce0

* {AKS} Fix several test cases (#4506)

* update

* update recording

* update subnet cidr for ingress test case

* update recordings

* fix coverage

* [Release] Update index.json for extension [ cosmosdb-preview ] (#4512)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1419899

Last commit: https://github.com/Azure/azure-cli-extensions/commit/387d57b8a1140d19738ace79864788814725d61b

* SSH Extension v.1.0.1 (#4474)

* Change how private ips are appended to the list (#4517)

* [Release] Update index.json for extension [ ssh ] (#4520)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1426796

Last commit: https://github.com/Azure/azure-cli-extensions/commit/55eee43c837115fe90760a1ebcb16edde6c8deb8

* ADAL to MSAL move for cli>=2.30.0 (#4468)

* ADAL to MSAL move for cli>=2.30.0

* nit

* nit

* nit

* add comment

* addressing review comments

* linting fixes

* linter fixes

* linter nit

* add license for new file

* update release date

* add token condition

* Update src/connectedk8s/azext_connectedk8s/_utils.py

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

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>

* [Release] Update index.json for extension [ connectedk8s ] (#4523)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1431525

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f13c4952c40bc1796a8e2c4b024d373140a6cfbf

* add aks update support for http proxy (#4469)

* add aks update support for http proxy

* adjust e2e tests

* fix decorator and tests

* add test

* fix: add back update bits

* fix recording

* fix decorator

* decorator should have trusted ca in ut

* new recording

* add real proxy vm

* remove extraneous option list

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

* fix tests

* fix ut

* fix proxy urls

* add credscan suppression

* bump setup py version

* add live_only because of custom feature required

* exclude test due to feature

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

* [Release] Update index.json for extension [ aks-preview ] (#4525)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1433736

Last commit: https://github.com/Azure/azure-cli-extensions/commit/456f486a75de4c6d19c8f861bbf8fe3f54ff8856

* [storage-preview] `az storage account create/update`: Support `--key-vault-federated-identity-client-id` and `--allowed-copy-scope` (#4498)

* `az storage account create/update`: Support `--key-vault-federated-identity-client-id` and `--allowed-copy-scope`

* rerun tests

* Update src/storage-preview/HISTORY.rst

* [Release] Update index.json for extension [ storage-preview ] (#4527)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1434569

Last commit: https://github.com/Azure/azure-cli-extensions/commit/1208f9ec30660177a7571a5482df3f7a40a41901

* [appservice-kube] `az webapp create`: Fix TypeError: 'NoneType' object is not subscriptable (#4518)

* fix plan create and webapp create

* bump version, include bug fix from #4493, fix style, fix webapp update bug, add basic regression tests

* remove deprecated 'az webapp update' params

* fix TypeError: 'NoneType' object is not subscriptable

* bump version

* better error message for nonextant customlocation

* fix name

* Fix functionapp devops-pipeline create (#4500)

* Fix functionapp devops-pipeline create

* Update version

* Bump up minimum cli core version to 2.23.0 (#4537)

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* [Release] Update index.json for extension [ connectedk8s ] (#4538)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1437831

Last commit: https://github.com/Azure/azure-cli-extensions/commit/a1959b123d4c11149adae2728ab5791949889d54

* [Release] Update index.json for extension [ functionapp ] (#4532)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1437244

Last commit: https://github.com/Azure/azure-cli-extensions/commit/182362788e6f0e70f67b4f8934ce6e170a207bd2

* [Release] Update index.json for extension [ appservice-kube ] (#4531)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1437153

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d2674f49819c6b1e953d55c5c650a5a0a8cfa559

* [CostManagement] `az costmanagement export`: Support `ActualCost` and `AmortizedCost` for export type (#4528)

* Fixing typo (#4541)

* [Release] Update index.json for extension [ costmanagement ] (#4544)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1439436

Last commit: https://github.com/Azure/azure-cli-extensions/commit/8080424bfe2b4cfc2c0bfee76ec918daeacc4d71

* Remove "packaging" module since 2.16.0 doesn't support it (#4545)

* Remove "packaging" module

* nit

* handle 2.30 case

* nit

* nit

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* [Release] Update index.json for extension [ connectedk8s ] (#4546)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1439941

Last commit: https://github.com/Azure/azure-cli-extensions/commit/411a1c893715364f6a4980c1b0de3e67618292cc

* [AuthV2] Nov 2022 Bug Fixes and Improvements (#4089)

* fixes

* static analysis done

* fix test recordings

* fix static analysis

* Update to throw AZCliErrors and add Unit Tests for new Error Cases

* Fix indentation issue

* Merge main and re-record tests

* change azclierror to argumentusageerror

* Fix indentation

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

* Add sdk for 2022-03-01-preview api generated here in this PR: (#4552)

https://github.com/Azure/azure-rest-api-specs/pull/18319

* {AKS} Vendor 2022-02-02-preview for aks-preview  (#4553)

* vendor 2022-02-02-preview

* update recordings

* manual update recording for test_aks_create_with_crg_id

* [Quantum] Output error information in table when Quantum job fails (#4463)

* Output status, error code, and error message (errorData) when job fails

* Add Target, Job ID, and Submission Time to table

* Let 'az quantum job output' show error info

* Add unit test cases and error handling for missing errorData items

* Delete commented-out code

* Add logging

* Add Job ID to error log, delete commented-out code

* Delete more commented-out code

* Use dict.get() function

* Alerts management contains operator support for resource type condition (#4554)

* added support for contains and doesNotContain operators for resource types filter

* 1. updated version to 0.2.1
2. updated history file accordingly

Co-authored-by: MIDDLEEAST\lienglender <[email protected]>

* Release 0.24.0 (#4516)

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

* [Quantum] Handle more storage types during workspace creation (#4524)

* Parameterize storage SKU and kind

* Get storage account parameters

* Add a break

* Add unit test

* Update test defaults

* Make minor changes to satisfy Pylint and Flake8

* Add storage account validation

* Add unit test cases

* Rearranged a logical expression to satisfy Flake8 style check

* Append 'GRS' (or 'grs') to all references to the new test storage account

* Re-run the CI/CD checks

* Change test defaults back to e2e-scenarios resource group

* Bulk fix to change az API links from using underscore to dash (#4551)

* fix_storage_azcopy_blob_sync_help (#4521)

* Add support for enabling Azure KeyVault KMS in aks-preview (#4547)

* support AzureKeyVaultKms

* fix

* [Release] Update index.json for extension [ aks-preview ] (#4558)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1450270

Last commit: https://github.com/Azure/azure-cli-extensions/commit/4a7de787a95b840679a5b24fb6e5c4c1e40c28a2

* update owners of aks-preview (#4557)

* Onboard command extension for Azure Managed Grafana service (#4495)

* POC: update with command list to implement

* consolidate control plane and dashboard commands

* half done: make resource group and location optional

* more commands and code style fixes

* add a validator to workspace https://bugs.python.org/issue9334

* catch up a few TODOs

* Consolidate data source CRUD commands with bug fixes and samples

* add help

* bug fixes on data source commands

* fix style errors

* fix bugs in data-source commands

* clean up command help

* upload whl file for tempoaray measures, before we publish it officially

* update readme and bug fixes

* update get_start doc

* Update get_start,md

* fix typos in get_start.md

* add vendor SDK

* new extension

* new extension built from azdev

* rename from ags to amg

* add tag support

* Update readme.rst

* undo non related changes

* rename readme to markdown

* address lint error

* address linter error

* more fix towards command lint error

* register the command module in a few common file

* add import

* support gallery import

* use deep copy

* address review feedback

* set the minimum cli core version

* fix a bug in 'az grafana user show'

* Remove the 'id' on creating dashboard to prevent 'Not Found' error

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Updated dapr enable/disable to current spec.

* Fixed oversight.

* Fixed dapr enable fixed create --dapr.

Co-authored-by: vivek-microsoft <[email protected]>
Co-authored-by: Vivek Sundararajan <[email protected]>
Co-authored-by: Azure CLI Bot <[email protected]>
Co-authored-by: Silas Strawn <[email protected]>
Co-authored-by: Yishi Wang <[email protected]>
Co-authored-by: Wenhao Zhang <[email protected]>
Co-authored-by: Wenhao Zhang <[email protected]>
Co-authored-by: FumingZhang <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>
Co-authored-by: yuyue9284 <[email protected]>
Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Lia Kazakova <[email protected]>
Co-authored-by: Yuwei Zhou <[email protected]>
Co-authored-by: Ryan K <[email protected]>
Co-authored-by: Ricardo Espinoza <[email protected]>
Co-authored-by: XField <[email protected]>
Co-authored-by: necusjz <[email protected]>
Co-authored-by: ZengTaoxu <[email protected]>
Co-authored-by: Yu Chen <[email protected]>
Co-authored-by: siriteja <[email protected]>
Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>
Co-authored-by: Ashutosh Suman <[email protected]>
Co-authored-by: Sachin Gangwar <[email protected]>
Co-authored-by: alice_wang <[email protected]>
Co-authored-by: Ni Yuanyuan <[email protected]>
Co-authored-by: Daeun Yim <[email protected]>
Co-authored-by: Ark…
calvinsID pushed a commit that referenced this pull request Mar 24, 2022
… removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

Co-authored-by: Haroon Feisal <[email protected]>
calvinsID added a commit that referenced this pull request Mar 29, 2022
* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Dapr moved from Template to Configuration

* Use aka.ms link for containerapps yaml

* Updated dapr enable/disable to current spec.

* Fixed oversight.

* Remove revisions-mode from containerapp update

* Fixed dapr enable property names. (#47)

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

* Fix exceptions with using --yaml in containerapp create/update

* Rename history msg

* Include fqdn in containerapp table output

* Added ingress messages.

* Revert history msg

* Add basic test case

* Remove managed-identity support for first release of CLI

* Need to investigate test flakiness

* Update _help.py

removing duplicate help

Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
Co-authored-by: Sisira Panchagnula <[email protected]>
calvinsID added a commit that referenced this pull request Apr 6, 2022
* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Fixed dapr in create.

* Revert "Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing."

This reverts commit 51bc543.

* Revert "Fixed dapr in create."

This reverts commit 37030ad.

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Reduced redundant code between revision copy and containerapp update.

* Fixed merge issues.

* Fixed merge conflicts, moved helper function

Co-authored-by: Calvin Chan <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
calvinsID pushed a commit that referenced this pull request Apr 6, 2022
…ure#4588)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update public extension pipeline

* Change condition variable

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

* Stage Public Version of k8s-extension 0.2.0 for official release (#15)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update public extension pipeline

* Change condition variable

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

Co-authored-by: [email protected] <Action - Fork Sync>

* Remove custom pipelines file

* Update extension description, remove private const

* Update pipeline file

* Disable refs docs

* Update to include better create warning logs and remove update context (#20)

* Update to include better create warning logs and remove update context

* Remove help text for update

* Fix spelling error

* Update message

* Fix k8s-extension conflict with private version

* Fix style errors

* Fix filename

* add customization for microsoft.azureml.kubernetes (#23)

* add customization for microsoft.azureml.kubernetes

* Update release history

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Add E2E Testing from Separate branch into internal code (#26)

* Add internal e2e testing

* Change to testing folder

* Inference CLI validation for Scoring FE (#24)

* cli validation starter

* added the call to the fe validation function

* nodeport validation not required

* test fix

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

* legal warning added (#27)

* Remove deprecated method logger.warn

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Add Azure Defender to E2E testing (#28)

* Add azure defender testing to e2e

* Remove the debug flag

* Add configuration testing

* Fix pipeline failures

* Make test script more intuitive

* Remove parameter from testing

* Fix wrong location for k8s config whl

* Fix pip upgrade issue

* Fix pip install upgrade issue

* Fix pip install issue

* delete resurce in testcase (#29)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Check Provider is Registered with Subscription Before Making Requests (#18)

* Add check for KubernetesConfiguration

* Disable pylint and rename

* Update provider registration link

* Update version

* Remove extra blank line

* Fix bug in import

* only validate scoring fe when inference is enabled (#31)

* only validate scoring fe when inference is enabled

* Fix versioning

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Provider registration case insensitive

* do not validate against scoring fe if inference is not enabled. (#33)

* do not validate against scoring fe if inference is not enabled.

* add inference enabled scenario

* refine

* increase sleeping time

* fix

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Add OSM as Public Preview Extension (#34)

* Add OSM as public preview extension

* Add osm testing

* Add release train to tests

* Fix failing osm test

* Upgrade pip in integration testing

* Remove ununsed import

* Fix release train check in update

* Parallelize E2E Testing (#36)

* Add OSM as public preview extension

* Add osm testing

* Update test logic to parallelize

* Fix test success checking

* Parallelize extension testing

* Better error checking logic

* Fix azureml deletion

* Fix private build (#40)

* change amlk8s to amlarc (#42)

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

* Servicebus client model changes (#44)

* Servicebus client model changes

* Fix testing script

* Update history file and pipeline

* Update min cli core version for track 2 updates

* Read SSL cert and key from files (#38)

* first sketch of the change

fixes

removed extra blank lines

changes regarding param renaming

added ssl tests

added more detail to the unit test

additional import

moved pem files out of public folder

fixed import

chenged import

changed import

unit tests fix

unit test fix

fixed unit tests

fixed unit test

unit test fix

changes int test cert and key

* test protected config

* fix test typo

* temporary changes reverted

* fixing tests

* fixed file paths

* removed accidentally added file

* changes according to review comments

* more changes according to review comments

* changes according to review comments

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

* Upgrade release version

* Liakaz/inference read ssl from file (#47)

* first sketch of the change

fixes

removed extra blank lines

changes regarding param renaming

added ssl tests

added more detail to the unit test

additional import

moved pem files out of public folder

fixed import

chenged import

changed import

unit tests fix

unit test fix

fixed unit tests

fixed unit test

unit test fix

changes int test cert and key

* test protected config

* fix test typo

* temporary changes reverted

* fixing tests

* fixed file paths

* removed accidentally added file

* changes according to review comments

* more changes according to review comments

* changes according to review comments

* fixed decode error

* renamed the experimental param

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

* Fix style issues (#51)

* Fixed scoring fe related extension param names (#49)

* fixed scoring fe related extension params

* bug fix and style fixes

* variable rename

* fixed the error type

* set cluster to prod by default

* Add distro validation for osm-arc (#50)

* Add distro validation for osm-arc

* fixed indentation

* Fix linting

* Resolve comments

* Add unit test

* fix lint

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

* Add distro validation for osm-arc (#50)

* Add distro validation for osm-arc

* fixed indentation

* Fix linting

* Resolve comments

* Add unit test

* fix lint

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

* Add distro validation for osm-arc (#53)

removed release-train logic

* Add Custom Delete Logic for Partners (#54)

* Add custom delete logic

* Fix failing unit tests

* Add warning message when deleting amlarc extension (#55)

* add warning message

* fix indentation

* Update release version

* Remove Pyhelm from OSM customization (#58)

* Fix OSM pyhelm bug

* Debug bootstrap error

* Update release message

* Remove pyhelm dependency

* Update tests to only check extensionconfig creation (#61)

* Update tests to only check extensionconfig creation

* Single set of CRUD for AzureML

* Debug logs for connectedk8s

* Increase open service mesh version number

* Update k8s-extension Models to Track2 (#64)

* Update k8s-extension models to track2

* Add debug for failed cleanup

* Increase version number

* Exit 0 on failed cleanup

* Fix identity in wrong place in model (#66)

* Readd osm-arc distro validation (#62)

* Add distro validation for osm-arc

removed release-train logic

* Readd osm_arc distro validation

* Fix style

* Rm space

* Edit test

* Fixed tests and error logic

* Remove dependency

* Add delete method

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

* Don't Send Identity Headers If In DF (#67)

* Don't send identity for clusters in dogfood

* Add location to model for identity

* Add identity validation to testing

* Use default extension with identity instead of Cassandra specific (#69)

* Remove the identity check for now

* Add -t for clusterType parameter (#71)

* Adding a flag for AKS to AMLARC migration and set up corresponding FE… (#65)

* Adding a flag for AKS to AMLARC migration and set up corresponding FE helm values

* Remove one extra line

* Adding Scoring FE IS_AKS_MIGRATION check logic for helm values

Co-authored-by: Harry Yang <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* remove version requirement and auto upgrade minor version check (#72)

* Custom User Confirmation for Partners (#70)

* Custom user confirmation

* Check for disable confirm prompty for confirmation

* Add yes to delete command

* Code cleanup and style fixes (#73)

* Enabled identity by default (#74)

* Increase version

* Fix df check and add unit test (#77)

* Bump extension version

* Pin helm version

* Extensions GA changes into Public Branch (#79)

* Add openservicemesh back

* OpenServiceMesh import

* Update osm with new extension model

* Add back private file

* Add Azure ML to list of private extensions (#16)

* Update k8s-custom-pipelines.yml

* Add Microsoft.PolicyInsights extension (#17)

* Add Policy extension

* Update comment

* Update args

* Fix linting errors

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

* Add HISTORY_private file for private preview

* Change versioning scheme

* Update the code for supporting both extensions at once

* Fix style issue

* Remove old consts file

* change the resource tag from managed_by:amlk8s to created_by:amlk8s-e… (#22)

* change the resource tag from managed_by:amlk8s to created_by:amlk8s-extension

* remove the lock when creating resources

* fix lint

* update version and HISTORY_private.rst

* change error message

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

* Update the beta version with upstream

* Update the private history file

* Add upgrade pip to pipeline

* Move pip install within virtualenv

* Merge in k8s-extension/public (0.3.1) (#32)

* delete resurce in testcase (#29)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Check Provider is Registered with Subscription Before Making Requests (#18)

* Add check for KubernetesConfiguration

* Disable pylint and rename

* Update provider registration link

* Update version

* Remove extra blank line

* Fix bug in import

* only validate scoring fe when inference is enabled (#31)

* only validate scoring fe when inference is enabled

* Fix versioning

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Update private release

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

* Release Version 0.4.0-b1 (#37)

* Merge k8s-extension/public into k8s-extension/private

* Update the version

* Fix testing concurrency

* K8s extension/private 0.4.0b2 (#41)

* Fix private build (#40)

* Update version

* Upgrade to v0.5.2

* Fix policy bug

* Increase private version

* Update consts_private.py

* Increase private version

* Increase version with public

* Add flux to private version

* Update models for 2021-05-01-preview

* Add async models to version

* Add no wait to delete and create

* support managed cluster

* Bump version

* Pin helm version

* Add cmd to delete call

* Add force deletion

* add dapr extension (#78)

Signed-off-by: Ji An Liu <[email protected]>

* Fix failing integration tests

* Adding the GA changes for private branch

* Fix confirm prompt

* Fix update E2E tests

Co-authored-by: jonathan-innis <[email protected]>
Co-authored-by: [email protected] <Action - Fork Sync>
Co-authored-by: nreisch <[email protected]>
Co-authored-by: yuyue9284 <[email protected]>
Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: anagg929 <[email protected]>
Co-authored-by: Ji'an Liu <[email protected]>
Co-authored-by: nanthi <nanthi@NANTHI01>

* Fix configuration settings in update

* Only provide confirmation when specifying settings

* Fix style issues

* Cassandra tests with update (#81)

* Add Microsoft.PolicyInsights extension for public preview (#83)

* Add Azure Policy

* Remove custom configuration and update tests

* Yuyu3/fix upgrade public (#85)

* populate configuration protected settings for azureml

bump version && add log

fetch connection string only if configuration protected settings are set

update ssl key

* bump the version

* reverse changes on version and HISTORY.rst

* inferenceLoadBalancerHA

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

* Remove Parallel Powershell Jobs (#82)

* Unparallelize tests

* Moved location of pipeline file

* Remove the parallel invoke expression calls

* Add templates to testing

* Remove policy update test from extension E2E (#88)

* feIsNodePort, feIsInternalLoadBalancer (#87)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Fix history file

* Add one more prompt for amlarc extension update (#94)

* Add one more prompt for amlarc extension update

* fix pylint issue

* fix pylint issue

* fix pylint issue

* fix pylint issue

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

* Update Identity Creation for Appliance to Latest Version (#95)

* Update appliance API to latest version for identity

* Create a utils file with get parent_api_version

* Fix style errors

* Bump version

* Remove additional entry from history

* Do not create identity with appliances (#97)

* Bump version

* support sslSecret (#99)

* support sslSecret

* fix

* fix error message

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

* Bump version to 1.0.4

* Increase OSM verison to valid version (#104)

* remove inference private review warning message (#102)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Update unit testing from osm to dapr (#106)

* osm-ext: enable system identities (#105)

* remove inference private review warning message (#109)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Build Extension in Template (#110)

* Build extension using a template

* Update build template in run test

* Fix private build issues (#112)

* Upgrade vendored_sdks to 2022-03-01 (#113)

* Upgrade vendored_sdks to 2022-03-01

* Update recordings file

* Bump version and changelog

* Remove unneeded files

Co-authored-by: [email protected] <Action - Fork Sync>
Co-authored-by: yuyue9284 <[email protected]>
Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Lia Kazakova <[email protected]>
Co-authored-by: Niranjan Shankar <[email protected]>
Co-authored-by: jingyizhu99 <[email protected]>
Co-authored-by: Harry Yang <[email protected]>
Co-authored-by: Harry Yang <[email protected]>
Co-authored-by: Thomas Stringer <[email protected]>
Co-authored-by: NarayanThiru <[email protected]>
Co-authored-by: nreisch <[email protected]>
Co-authored-by: anagg929 <[email protected]>
Co-authored-by: Ji'an Liu <[email protected]>
Co-authored-by: nanthi <nanthi@NANTHI01>
Co-authored-by: youhuatuyh <[email protected]>
Co-authored-by: Youhua Tu <[email protected]>
Co-authored-by: Youhua Tu <[email protected]>
Co-authored-by: Jun <[email protected]>
Co-authored-by: Jun Min <[email protected]>
Co-authored-by: Sanya Kochhar <[email protected]>
calvinsID pushed a commit that referenced this pull request Apr 6, 2022
…3-01 for GA (Azure#4587)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update CODEOWNERS

* Update azure-pipelines.yml

* Create pull.yml

* Update pull.yml

* Update pull.yml

* Update pipelines file

* Update k8s-configuration name

* Update test script params

* Update pipeline file

* Remove codeowners

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update custom pipelines files

* Add publish step to k8s-configuration

* Update pipeline to publish extension

* Update public extension pipeline

* Change condition variable

* Update pipeline naming

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

* Stage Public Version of k8s-extension 0.2.0 for official release (#15)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update public extension pipeline

* Change condition variable

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

Co-authored-by: [email protected] <Action - Fork Sync>

* Remove custom pipelines file

* Update extension description, remove private const

* Update pipeline file

* Disable check ref docs

* Disable refs docs

* Update to include better create warning logs and remove update context (#20)

* Update to include better create warning logs and remove update context

* Remove help text for update

* Fix spelling error

* Update message

* Fix k8s-extension conflict with private version

* Fix style errors

* Fix filename

* add customization for microsoft.azureml.kubernetes (#23)

* add customization for microsoft.azureml.kubernetes

* Update release history

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Add E2E Testing from Separate branch into internal code (#26)

* Add internal e2e testing

* Change to testing folder

* Inference CLI validation for Scoring FE (#24)

* cli validation starter

* added the call to the fe validation function

* nodeport validation not required

* test fix

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

* legal warning added (#27)

* Remove deprecated method logger.warn

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Add Azure Defender to E2E testing (#28)

* Add azure defender testing to e2e

* Remove the debug flag

* Add configuration testing

* Fix pipeline failures

* Make test script more intuitive

* Remove parameter from testing

* Add some debug

* Fix wrong location for k8s config whl

* Fix pip install upgrade issue

* Fix pip install upgrade issue

* Add Check for Provider Registration and Refactor (#19)

* Add check for provider registration and refactor

* Fix bug in checking registration

* Add license header to utils

* Update private key check and error messaging

* Update based on refactoring

* Fix failing tests

* Add provider registration check

* Create a test for uppercase url, address comments

* Add blank line to fix style check

* Testing increase to ubuntu-latest

* Update k8s-configuration Models to Track2 (#63)

* Update models to track2

* Increase k8s-configuration version number

* Update kind version

* Change error to warning because of DSA failure

* Upgrade helm operator chart version (#75)

* Pin helm version

* Bump version

* Migrate pipeline (#90)

* Disable updates on configuration tests (#89)

* Release k8s-configuration v1.2.0 for Flux v2 Public Preview (#86)

* Scaffold out the k8s-config package

* Base implementation of CLI commands

* Add create scenario and cleanup in consts

* Add help text to commands, params

* Add other clients to client factory

* Automatically installing the flux extension

* Move flux and extension into modules

* Updated the versioned sdks

* Push working command for testing

* Update to multi api versioned sdk

* Support other extension methods

* Fix nullity check

* Add source control provider

* Add scc commands

* Add defer logic for create with cache

* Use default extension with identity

* Fix identity creation

* Add kustomization caching

* Add formatters

* Add scc provider

* Add help text for k8s-config fluxv1

* Add help text for extension

* Allow force delete of extension and fluxconfiguration

* Add location to the extension model

* Update with latest from k8s-extension

* Add k8s-config testing

* Add license header

* Fix all style issues

* Update codeowners file

* Validate data before checking cluster compliance

* No kustomizations warning

* Fix identity issue in 2020-07-01

* Fix k8s regex

* Fix configuration name regex validation

* Fix name length validation

* Adding some validation warnings

* Add protected settings to request

* Exclude private test path

* Add suspend functionality

* Add correct values to build Kustomization

* Add no_wait

* Fix style issues

* Use base64 encoded httpsUser

* Fix formatting error and base64 encoding error

* Fix style issues

* Fix force

* Updated help text

* Style fixes

* Increase namespace maximum len

* Add managed cluster support to k8s-config

* Custom confirmation when prune is enabled

* Add flux commands to existing k8s-configuration

* Remove extension provider from CLI

* Fix style issues

* Override extension variables

* Strip newlines from known_hosts file

* Update help text and validators

* Strip newlines from known hosts

* Add provisioning state check for flux extension

* Pin helm version

* Remove validation from create command

* Add patch support with new SDK

* Add implementation for CRUD of source and kustomization

* Fix errors on patch

* Fix some bugs in patching properties

* Add fixes for patch in k8s-configuraiton

* Change duration formatting in table output

* Add validation and conversion for durations

* Bump verison and fix typo

* Fix bug with dependencies

* Fix linter and style issues

* Fix delete prune check

* Add flux testing

* Create separate jobs for scenarios

* Update error text

* Fix filepath suggestion from CLI team

* Fix unneeded file edit

* Add a Deployed Object List to the Flux CLI (#91)

* Enable a deployed object list on the CLI

* Show detail when extension install fails

* Bump version

* Update deployed object format (#93)

* Fix help text for consistency

* Enable Bucket Support in the CLI (#92)

* Update vendored_sdks

* Enable source kind generation factories and use kwargs to pass to provider

* Add better validation logic to source generator

* Move away from classes in the provider directory

* Fix style issues using black auto-formatter

* Fix linter failures

* Update identity with api version and rp in same function

* Bucket Testing for E2E Testing (#96)

* Bucket testing

* Add switching kind test to update CLI

* Bump version

* Fix help text and parameter naming for bucket (#100)

* Fix default help text for parameters (#101)

* Edit history with breaking change

* Prepare GA FluxConfiguration 2022-03-01 (#103)

* Add vendored sdks for new api-version

* Update the new depends on definition

* Update table formatting

* BucketDefinition to BucketPatchDefinition

* Remove http url warning

* Update vendored_sdks

* Update action file to add depenencies

* Remove preview from command groups

* Update changelog

* Make dependencies none when not specified

* Remove unneeded files

Co-authored-by: [email protected] <Action - Fork Sync>
Co-authored-by: yuyue9284 <[email protected]>
Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Lia Kazakova <[email protected]>
runefa added a commit that referenced this pull request Apr 7, 2022
* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Fixed dapr in create.

* Revert "Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing."

This reverts commit 51bc543.

* Revert "Fixed dapr in create."

This reverts commit 37030ad.

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Dapr moved from Template to Configuration

* Use aka.ms link for containerapps yaml

* Updated dapr enable/disable to current spec.

* Fixed oversight.

* Remove revisions-mode from containerapp update

* Fixed dapr enable property names. (#47)

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

* Fix exceptions with using --yaml in containerapp create/update

* Rename history msg

* Include fqdn in containerapp table output

* Added ingress messages.

* Revert history msg

* Reduced redundant code between revision copy and containerapp update.

* Fixed merge issues.

* Fixed merge conflicts, moved helper function

Co-authored-by: Calvin Chan <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
calvinsID added a commit that referenced this pull request Apr 12, 2022
…Azure#4660)

* Marchp1s and add back Identity (#57)

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Fixed dapr in create.

* Revert "Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing."

This reverts commit 51bc543.

* Revert "Fixed dapr in create."

This reverts commit 37030ad.

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Dapr moved from Template to Configuration

* Use aka.ms link for containerapps yaml

* Updated dapr enable/disable to current spec.

* Fixed oversight.

* Remove revisions-mode from containerapp update

* Fixed dapr enable property names. (#47)

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

* Fix exceptions with using --yaml in containerapp create/update

* Rename history msg

* Include fqdn in containerapp table output

* Added ingress messages.

* Revert history msg

* Reduced redundant code between revision copy and containerapp update.

* Fixed merge issues.

* Fixed merge conflicts, moved helper function

Co-authored-by: Calvin Chan <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>

* Fix help for linter

* various fixes, helptext (#59)

* Fixes (#60)

* Updated managed identity + help. (#61)

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

* Added user-assigned and system-assigned to containerapp create. (#62)

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

* Bump version to 0.1.1 (#63)

* Added more specific MSI help text. (#64)

* Added more specific MSI help text.

* Updated help text.

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

* Bump to 0.3.0 (#65)

Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
runefa added a commit that referenced this pull request Apr 18, 2022
* [storage-preview] `az storage account create/update`: Support `--enable-sftp` and `--enable-local-user` (#4140)

* Bump `azure-mgmt-storage` to 2021-08-01

* linter

* update SDK

* Update SDK

* Copy `az storage account create/update` from CLI main repo

* `az storage account create/update`: Support `--enable-sftp` and `--enable-local-user`

* update sdk

* fix test

* re-record test

* [Spring-Cloud] add Build Service related command for enterprise tier (#4295)

* [Spring-Cloud] Add Build service for enterprise tier

* fix

* fix

* Update src/spring-cloud/azext_spring_cloud/_params.py

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

* Update src/spring-cloud/azext_spring_cloud/_params.py

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

* fix

* fix

* fix

* fix

* fix

* Update src/spring-cloud/azext_spring_cloud/_buildservices.py

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

* Update src/spring-cloud/azext_spring_cloud/_buildservices.py

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

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* Update src/spring-cloud/azext_spring_cloud/_params.py

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

* Update src/spring-cloud/azext_spring_cloud/_params.py

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

* fix

* add delete test

* fix

* Update spring_cloud_instance.py

* Update commands.py

* fix

* fix

* Update _params.py

* fix

* fix test

* fix test

* fix test

Co-authored-by: Yuwei Zhou <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>

* Rename Ldap Parameter Names. (#4315)

* managed cassandra commands for cli extension

* remove node-status, add backups

* fix style and add backup command

* AuthenticationMethod param

* add sku, availability and idenitty parameters

* bugfix

* add backup key uri

* python vendor sdk changes

* bugfix

* recorded scenario tests

* bugfix on cassandra_stop_start

* update invoke-command arguments

* bugfix

* conditionally set ldap properties

* ldap property bugfix

* linter fix

* linter fix

* remove public commands from preview

* linter fix in help text

* linter fix

* use backup show and backup list

* preview python sdk

* bugfix

* removing whitespace

* Update History.rst

* update setup.py

* rename parameters

* parameter rename and help text

* update version history

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

* [Storage] `az storage account local-user`: add new command group local-user with create, update, delete, list, show, list-key, and regenerate-password (#4125)

* Bump `azure-mgmt-storage` to 2021-08-01

* linter

* update SDK

* start create_local_user

* continue create_local_user base64 error

* local user commands done, test pending

* test

* add help

* add help, fix credscan

* lint

* credscan

* credscan

* add back files deleted by accident

* Update _configuration.py

* refactor

* version

* lint

* add delete test

* fix PR comments

* lint, credscan

* date before release

* show command

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

* [storage-preview] `az storage account create/update`: Support `--sam-account-name` and `--account-type` (#4316)

* Bump `azure-mgmt-storage` to 2021-08-01

* linter

* update SDK

* Update SDK

* Copy `az storage account create/update` from CLI main repo

* `az storage account create/update`: Support `--enable-sftp` and `--enable-local-user`

* update sdk

* fix test

* re-record test

* `az storage account create/update`: Support `--sam-account-name` and `--account-type`

* add test

* Update workaround information in parameter helptext (#4321)

* update workaround information in helptext

* update setup.py to use 0.12.0 version

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

* [Release] Update index.json for extension [ cosmosdb-preview ] (#4323)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1296094

Last commit: https://github.com/Azure/azure-cli-extensions/commit/68fd2df68090503ccaa95a8135bfc651b0a5136b

* [Enterprise-Edge] : `az staticwebapp enterprise-edge enable` : register Microsoft.CDN provider automatically upon running enable command  (#4324)

* register Microsoft.CDN provider automatically

* bump version and fix style issues

* small fixes; use sdk method

* fix sdk call

* Update version (#4322)

* [Release] Update index.json for extension [ storage-preview ] (#4326)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1307240

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f6aa6d8194a7dec21f29b4c92dec5946c60699f1

* [Release] Update index.json for extension [ enterprise-edge ] (#4325)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1306508

Last commit: https://github.com/Azure/azure-cli-extensions/commit/7fc818250de852b5d6ad46ed537694f3ee1e8038

* Make cli zone redundant property be none (#4327)

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

* fix regex used to check node image version (#4319)

* Update Terminology end-to-end TLS to ingress-to-app TLS (#4320)

* Update Terminology end-to-end TLS to ingress-to-app TLS

* Update position

* Update for static analysis

* Add it for e2etls

* Remove useless line

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

* Fix certificate list output error (#4330)

* Fix certificate output error

* Update according to comments

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

* [k8s-configuration] Release v1.4.1 with fixed help text (#4328)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update CODEOWNERS

* Update azure-pipelines.yml

* Create pull.yml

* Update pull.yml

* Update pull.yml

* Update pipelines file

* Update k8s-configuration name

* Update test script params

* Update pipeline file

* Remove codeowners

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update custom pipelines files

* Add publish step to k8s-configuration

* Update pipeline to publish extension

* Update public extension pipeline

* Change condition variable

* Update pipeline naming

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

* Stage Public Version of k8s-extension 0.2.0 for official release (#15)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update public extension pipeline

* Change condition variable

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

Co-authored-by: [email protected] <Action - Fork Sync>

* Remove custom pipelines file

* Update extension description, remove private const

* Update pipeline file

* Disable check ref docs

* Disable refs docs

* Update to include better create warning logs and remove update context (#20)

* Update to include better create warning logs and remove update context

* Remove help text for update

* Fix spelling error

* Update message

* Fix k8s-extension conflict with private version

* Fix style errors

* Fix filename

* add customization for microsoft.azureml.kubernetes (#23)

* add customization for microsoft.azureml.kubernetes

* Update release history

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Add E2E Testing from Separate branch into internal code (#26)

* Add internal e2e testing

* Change to testing folder

* Inference CLI validation for Scoring FE (#24)

* cli validation starter

* added the call to the fe validation function

* nodeport validation not required

* test fix

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

* legal warning added (#27)

* Remove deprecated method logger.warn

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Add Azure Defender to E2E testing (#28)

* Add azure defender testing to e2e

* Remove the debug flag

* Add configuration testing

* Fix pipeline failures

* Make test script more intuitive

* Remove parameter from testing

* Add some debug

* Fix wrong location for k8s config whl

* Fix pip install upgrade issue

* Fix pip install upgrade issue

* Add Check for Provider Registration and Refactor (#19)

* Add check for provider registration and refactor

* Fix bug in checking registration

* Add license header to utils

* Update private key check and error messaging

* Update based on refactoring

* Fix failing tests

* Add provider registration check

* Create a test for uppercase url, address comments

* Add blank line to fix style check

* Testing increase to ubuntu-latest

* Update k8s-configuration Models to Track2 (#63)

* Update models to track2

* Increase k8s-configuration version number

* Update kind version

* Change error to warning because of DSA failure

* Upgrade helm operator chart version (#75)

* Pin helm version

* Bump version

* Migrate pipeline (#90)

* Disable updates on configuration tests (#89)

* Release k8s-configuration v1.2.0 for Flux v2 Public Preview (#86)

* Scaffold out the k8s-config package

* Base implementation of CLI commands

* Add create scenario and cleanup in consts

* Add help text to commands, params

* Add other clients to client factory

* Automatically installing the flux extension

* Move flux and extension into modules

* Updated the versioned sdks

* Push working command for testing

* Update to multi api versioned sdk

* Support other extension methods

* Fix nullity check

* Add source control provider

* Add scc commands

* Add defer logic for create with cache

* Use default extension with identity

* Fix identity creation

* Add kustomization caching

* Add formatters

* Add scc provider

* Add help text for k8s-config fluxv1

* Add help text for extension

* Allow force delete of extension and fluxconfiguration

* Add location to the extension model

* Update with latest from k8s-extension

* Add k8s-config testing

* Add license header

* Fix all style issues

* Update codeowners file

* Validate data before checking cluster compliance

* No kustomizations warning

* Fix identity issue in 2020-07-01

* Fix k8s regex

* Fix configuration name regex validation

* Fix name length validation

* Adding some validation warnings

* Add protected settings to request

* Exclude private test path

* Add suspend functionality

* Add correct values to build Kustomization

* Add no_wait

* Fix style issues

* Use base64 encoded httpsUser

* Fix formatting error and base64 encoding error

* Fix style issues

* Fix force

* Updated help text

* Style fixes

* Increase namespace maximum len

* Add managed cluster support to k8s-config

* Custom confirmation when prune is enabled

* Add flux commands to existing k8s-configuration

* Remove extension provider from CLI

* Fix style issues

* Override extension variables

* Strip newlines from known_hosts file

* Update help text and validators

* Strip newlines from known hosts

* Add provisioning state check for flux extension

* Pin helm version

* Remove validation from create command

* Add patch support with new SDK

* Add implementation for CRUD of source and kustomization

* Fix errors on patch

* Fix some bugs in patching properties

* Add fixes for patch in k8s-configuraiton

* Change duration formatting in table output

* Add validation and conversion for durations

* Bump verison and fix typo

* Fix bug with dependencies

* Fix linter and style issues

* Fix delete prune check

* Add flux testing

* Create separate jobs for scenarios

* Update error text

* Fix filepath suggestion from CLI team

* Fix unneeded file edit

* Add a Deployed Object List to the Flux CLI (#91)

* Enable a deployed object list on the CLI

* Show detail when extension install fails

* Bump version

* Update deployed object format (#93)

* Fix help text for consistency

* Enable Bucket Support in the CLI (#92)

* Update vendored_sdks

* Enable source kind generation factories and use kwargs to pass to provider

* Add better validation logic to source generator

* Move away from classes in the provider directory

* Fix style issues using black auto-formatter

* Fix linter failures

* Update identity with api version and rp in same function

* Bucket Testing for E2E Testing (#96)

* Bucket testing

* Add switching kind test to update CLI

* Bump version

* Fix help text and parameter naming for bucket (#100)

* Fix default help text for parameters (#101)

* Remove unneeded files

* Edit history with breaking change

Co-authored-by: [email protected] <Action - Fork Sync>
Co-authored-by: yuyue9284 <[email protected]>
Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Lia Kazakova <[email protected]>

* fix: 🐛 Validate deployment parameters with source type (#4329)

* fix: 🐛 Should enable persistent storage when updating App

* Validate deployment source parameters

* fix lint

* fix bugs

* fix test

* [IoT] IoT Extension Release v0.13.0 (#4333)

* Remove IoT Extension v0.10.14
* Add IoT Extension v0.13.0
* Update linter exclusions

* [Release] Update index.json for extension [ k8s-configuration ] (#4332)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1309054

Last commit: https://github.com/Azure/azure-cli-extensions/commit/649b29ab8f154006be6cc4fd2c53f242eec0a319

* update location (#4334)

* Update data plane client from swagger (#4318)

* Update generated client to latest Azure Quantum Python Data Plane

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

* Update Data-Plane client generate script to delete previous output folder contents (#4340)

* [Resource Mover] Update Resource Mover CLI (#4309)

- generate vendored SDK
- bump up version to 0.1.1

* [Release] Update index.json for extension [ resource-mover ] (#4343)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1317636

Last commit: https://github.com/Azure/azure-cli-extensions/commit/e9497c3fe0b619aa92548f21246f584fca9c3d4b

* [Databricks]az databricks workspace create: Add new parameters --publ… (#4331)

* [Databricks]az databricks workspace create: Add new parameters --public-network-access and --required-nsg-rules

update databricks sdk version and add new parameters

* Update

1.move databricks.models into load_arguments function
2.update extension version in

* [Logic] Upgrade SDK to fix deserialization errors (#4347)

* [Release] Update index.json for extension [ logic ] (#4349)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1321294

Last commit: https://github.com/Azure/azure-cli-extensions/commit/16595ffb29a5d5cc24593c3656d25fa47b71196c

* fetching tenantID from subscription object instead of graph client (to pickup tenant Id of sub where cc is getting created instead of user/spn's tenant) (#4336)

* Fetch the tenant ID from the subscription object instead of graphclient

* nit

* nit

* revert proxy tenantid change

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* Adding new module DataMigration (#4298)

* Adding initial settings to module

* Working on Linter, License and Cred scan errors

* Working on Static analysis error

* Hiding Register-ir commands for now

* Lint and Integration Tests solving

* Unhiding register-ir command and adding windows test to assessment command

* Making error messages proper

* Nit changes from comments

* MultiServer support for Assessment and nit changes

* Adding Proper Json format, lower case for access, seperate util for Windows check

* Correcting Lint and Static errors

* Removing to-* from command names

* Adding correct error formats

* Removing Update operation from DatabaseMigration

* Updating readme and help

* Nit: Readme corrections

* Added CLI Test Cases

* Removed IR AuthKeys and Unused Imports

* Adding Default IR Program files path for 32-bit error

* Nit and Static analysis fix

* Added File Extension Validation

* Removed unnecessary spaces and fixed static analysis Issues

* Deferred IR path and MSI Extension validations

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

* [Release] Update index.json for extension [ connectedk8s ] (#4350)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1323101

Last commit: https://github.com/Azure/azure-cli-extensions/commit/b8abdf1987ed799ed1d5a2cc846d8696c86cc5d7

* [AKS] az aks update/az aks nodepool update: Allow empty string as label value  (#4351)

* skip none check for labels

* use correct value

* change empty string check in aks nodepool update

* revert change

* fix value

* revert change

* [Release] Update index.json for extension [ datamigration ] (#4352)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1323628

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f412582fe051ce76f555fee010f3d8240118ed18

* Change link in datamigration setup file (#4353)

* add rdbms-connect to cloud shell (#4313)

* Update azurestackhci version to 0.2.3 (#4356)

* [Release] Update index.json for extension [ rdbms-connect ] (#4359)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1325769

Last commit: https://github.com/Azure/azure-cli-extensions/commit/83cde80e25e9273eece470369f18f5a3b5662249

* fix: fix builder validator (#4339)

* fix: fix builder validator

* fix

* [Release] Update index.json for extension [ databricks ] (#4346)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1321182

Last commit: https://github.com/Azure/azure-cli-extensions/commit/c908659851fa910522935d8e71c7d717b5e22824

* [Spring-Cloud] fix app deployment validator can retrieve app name from app and name (#4335)

* fix app deployment validator can retrieve app name from app and name

* fix lint

* Initial commit for 0.12.0 release (#4341)

* Initial commit for 0.12.0 release

* Update src/quantum/HISTORY.rst to use cost instead of price

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

* Update commands.py

* Update src/quantum/HISTORY.rst

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

* Restart CI/CD

Co-authored-by: Ricardo Espinoza <[email protected]>
Co-authored-by: Angela Burton <[email protected]>

* [Release] Update index.json for extension [ quantum ] (#4364)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1328638

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d28093638614c9af2cef3041d6422691c6bb1e99

* Baremetalinfrastructure extension stable release support (#4180)

* baremetalinfrastructure extension release support

* adding whl extension

* removing unnecessary files

* Alerts management new alert processing rule (#4337)

* Alerts management new alert processing rule

Co-authored-by: Liran Englender <[email protected]>
Co-authored-by: kai ru <[email protected]>

* chore(arcdata): arcdata version bump to 1.2.0 (#4365)

* chore(arcdata): version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* chore(arcdata): arcdata version bump to 1.2.0

* {AKS} Declare delete command and update recordings (#4367)

* add delete command

* fix location for gpu test

* update recordings

* [Image Copy] Fix the hyper_v_generation of image copy (#4372)

* Fix generation

* Fix CI

* Update ml extension to version 2.0.7 (#4370)

* Update ml extension to version 2.0.6

* fix linter issue

* Remove is_preview from snapshot related commands (#4376)

* {AKS} Use default serialization for maintenanceconfiguration commands (#4362)

* Use msrest deserilizer

* fix as per comments

* remove the import

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

* [Release] Update index.json for extension [ image-copy ] (#4373)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1332056

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f1bdd0318036276b6aa199103895b7e02d90356d

* Release 0.23.0 for azure dev ops (#4378)

* [AKS] Allow aks nodepool update --node-taints (#4363)

* allow edit taints and add test

* add missing parameter

* change var name

* .get_output_in_json()

* give value

* try fix'

* fix test

* address comments

* add yaml file to package data (#4382)

* [Release] Update index.json for extension [ alertsmanagement ] (#4366)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1329629

Last commit: https://github.com/Azure/azure-cli-extensions/commit/2e6b127b67489ae3ad36419a436e6529b4d38a73

* [CosmosDB] Mongo RBAC (#4389)

* Mongo RBAC

* Fixing Lint issues

* Fixing help Lint issue

* [Release] Update index.json for extension [ cosmosdb-preview ] (#4392)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1351532

Last commit: https://github.com/Azure/azure-cli-extensions/commit/9dac199b9a77beb191e870f44adb0d42b5757c63

* Fix test: change to use soft taint on system pool in test (#4393)

* change to soft taint on system pool

* outbound ip workaround

* {AKS} Remove hardcoded k8s version in test case `test_aks_create_and_update_ipv6_count` (#4394)

* remove hardcoded k8s version in test case

* update recording

* Update azure-pipelines.yml for Azure Pipelines (#4399)

* [Vm-repair] preview script bug fix (#4388)

* bug fix

* adding history

* changing history

* Apply suggestions from code review

* bug fix preview

* adding history

* version bump

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

* {AKS} Update version to 0.5.52 (#4377)

* update version to 0.5.52

* update history

update recording for test_aks_create_and_update_outbound_ips

show pip as workaround

update jmes check

fix

update recording

* Add clear for sso & cors  properties (#4371)

* AlertsManagement-CLI-GA-Fixes (#4400)

* 1. help file fix
2. removed redundant import from typing_extension

* Increased minor version and added it to the history file

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

* [Release] Update index.json for extension [ alertsmanagement ] (#4403)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1355227

Last commit: https://github.com/Azure/azure-cli-extensions/commit/fe44a1bb123a58b7e8248850bdc20555ca893406

* [Quantum] Workspace creation race condition fix (#4314)

* Add ARM templates

* Add ARM template that does role assignment

* Modify auth code and move template folder to operations

* Try AzureCliCredential

* Restore deleted Storage Account lines in template

* Fixed Deployment.properties problem

* Call _add_quantum_providers for validation

* Call _get_data_credentials

* Fix pylint infractions

* Eliminate Pylint message W1514

* Add test that does role assignment

* Use short storage account name in utils.py

* Use workspace name for deployment name

* Use workspace name for storage account deployment name.

* Use workspace name for deployment name and temporarily comment-out the provisioningState check in test

* Add workspace creation timeout

* Add ARM template file to wheel

* Delete commented-out lines and unused ARM files

* Fix static analysis violations

* Re-run CI/CD checks

* update url links (#4354)

Co-authored-by: cxznmhdcxz <>

* Change links for setup file till k8s-configuration module (#4348)

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

* [Public Preview] Azure Confidential Ledger Extension (#4276)

* generated code

* add recording

* update tests

* json deserialize

* regenerated sdk

* tests passed!

* run pre-pr checks

* update docstrings

* update codeowners

* ci fix attempts

* add to codeowners

* update service_name.json

* Update src/confidentialledger/azext_confidentialledger/tests/latest/example_steps.py

* Update src/confidentialledger/azext_confidentialledger/tests/latest/test_confidentialledger_scenario.py

* ci fixes again

* coverage file updated

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

* [Release] Update index.json for extension [ confidentialledger ] (#4410)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1358406

Last commit: https://github.com/Azure/azure-cli-extensions/commit/ddba03626f75e10984652bf74235bb3cd5c4c894

* [Stream Analytics] Bump up version to 0.1.2 (#4404)

* Refine help messages

* Add tests and recordings

* Fix style and linter checks

* update disk-pool-volume-create --mount-option to only accept one para… (#4396)

* update disk-pool-volume-create --mount-option to only accept one parameter

* updated mount option help and added unit tests

* mount type help text update

* {AKS} Clean up some deprecated code (#4412)

* remove validator for cluster_autoscaler_profile

* clean up unused **kwargs

* change to property & SimpleNamespace for nat_gateway_models

* fix lint issues

* remove another hardcoded k8s version in test

* update recording

* {account} Bump up `azure-mgmt-subscription` SDK to `3.0.0` (#4409)

* bump up subscription SDK

* update

* [Release] Update index.json for extension [ account ] (#4416)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1362133

Last commit: https://github.com/Azure/azure-cli-extensions/commit/8c9c7841dbc38d9c114c2b6a9de04d963577374f

* [Release] Update index.json for extension [ vmware ] (#4415)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1361421

Last commit: https://github.com/Azure/azure-cli-extensions/commit/8dbc3b19762c619c11613a5ef7748417e8cfe2ab

* [Release] Update index.json for extension [ aks-preview ] [ hack ] (#4408)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1357913

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d236554051bdd0f359ec1bd3ad4ece70cccd7f53

* [Release] Update index.json for extension [ stream-analytics ] (#4414)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1361403

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d9108c0305d48feb92a73cc63954380bc30b58a4

* {AKS} Vendor 2022-01-02-preview for aks-preview (#4417)

* vendor new sdk

* update default api

* fix check in test test_aks_nodepool_update_taints_msi

* update recordings'

* fix recordings

* fix check

* [Release] Update index.json for extension [ vm-repair ] (#4401)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1355202

Last commit: https://github.com/Azure/azure-cli-extensions/commit/2cd5fda928273f4cd5c5a0488d306a8d6afa5c48

* AKS CRG for Aks v20211101preview (#4292)

* add support for capacity reservation group

* fix an UT issue

* fix a static test issue

* add new test function for new get_crg_id

* remove type=str per comment.

* Adds test case for crg

* Adds test case for crg

* fixed a merging error

* fixed a merging error

* fixed a space error

* fix: add runtime version in build (#4420)

* fix: add runtime version in build

* fix npe

* fix

* Update src/spring-cloud/azext_spring_cloud/_buildservices_factory.py

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

* add test

* fix

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

* Add Arc Appliance extension 0.2.15 (#4421)

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

* AKS preview upgraded to 0.5.53 with CRG (#4422)

* add support for capacity reservation group

* fix an UT issue

* fix a static test issue

* add new test function for new get_crg_id

* remove type=str per comment.

* Adds test case for crg

* Adds test case for crg

* fixed a merging error

* fixed a merging error

* fixed a space error

* fixed a space error

* upgrade to 0.5.53

* [Spring-Cloud] Upgrade version (#4294)

* [Spring-Cloud] Upgrade version

* Update HISTORY.md

* Update HISTORY.md

* Update HISTORY.md

* Update HISTORY.md

* Update HISTORY.md

* Update HISTORY.md

* Update wording

* Update commands.py

* TOC - fixed Azure service names (#4425)

* fixed TOC service names

* fixed Azure service names

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

* [Release] Update index.json for extension [ aks-preview ] (#4423)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1368539

Last commit: https://github.com/Azure/azure-cli-extensions/commit/b01494990b0c5672b1513727f4adbf39f95ad338

* [Release] Update index.json for extension [ spring-cloud ] (#4426)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1370330

Last commit: https://github.com/Azure/azure-cli-extensions/commit/6593c6a9df7071b3052ac5cd516967eed27ea928

* Add support for host group id (#4419)

* some initial changes for host group

* add host-group-id support

* remove some unnecessary changes

* format _validators.py by adding extra line

* resolve comments

* add missing comma

* update test_decorator.py

* update version to 0.5.54

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

* [Release] Update index.json for extension [ aks-preview ] (#4430)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1373529

Last commit: https://github.com/Azure/azure-cli-extensions/commit/16bd206af6ca04510694a98a16e42e4fecef7ba4

* [AKS] Add network plugin none support (#4007)

* rebase

* improve help text clarity

* fixes

* closing params

* reset the count

* include location

* add test recording file

* add check network plugin in test

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

* remove csi deployment (#4431)

* {AKS} Update unit test coverage configuration (#4432)

* clean up ut

* update ut coverage config

* remove read_only property setting in ut

* [appservice-kube] Fix #4357, #4358 : support functions v4 and powershell functions (#4411)

* support functions v4; powershell functions

* add custom handler runtime and fix CI issue

* use specific error type

* set max CLI version

* fix CI issues

* fix style

* [Release] Update index.json for extension [ appservice-kube ] (#4434)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1374379

Last commit: https://github.com/Azure/azure-cli-extensions/commit/e305b4d01f56c38f94be004bea18d19bed534502

* [Release] Update index.json for extension [ aks-preview ] (#4433)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1374361

Last commit: https://github.com/Azure/azure-cli-extensions/commit/811a3192458606c794e76071c830872f4f5d003e

* Added version 2.1.1 for Azure ML CLI (#4418)

* CLI 2.0.3 public preview release

* Updated CLI release with linting fixes

* Updated .whl file

* Fixed conflict

* Fix change to 2.0.7 checksum

* Replaced 2.1.0 with 2.1.1

* AzureML CLI patch release 2.1.2 (#4442)

* Customcert and fix for non escaping values in helm (#4428)

* Use userValues.txt instead of --resuse-values in update command

* fix error msg

* added custom cert

* change setup.py

* Helmupdatefix (#4)

* Use userValues.txt instead of --resuse-values in update command

* fix error msg

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* fix output file

* Helmupdatefix (#5)

* Use userValues.txt instead of --resuse-values in update command

* fix error msg

* fix output file

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* place values file in .azure folder

* place values file in .azure folder

* Helmupdatefix (#6)

* Use userValues.txt instead of --resuse-values in update command

* fix error msg

* fix output file

* place values file in .azure folder

* place values file in .azure folder

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* place values file in .azure folder

* nit

* nit

* nit

* setp.py and history.rst updates

* modify the way of fetching userValues.txt path

* Waiting for LRO to complete before moving ahead with agent installation/deletion

* update history.rst

* fix

* fix location variable

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>
Co-authored-by: siriteja <[email protected]>

* [Release] Update index.json for extension [ connectedk8s ] (#4445)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1383624

Last commit: https://github.com/Azure/azure-cli-extensions/commit/6d0ff505113ca853fea82e5f0573a8edb96b63a6

* {appservice-kube} Ensure compatibility with upcoming CLI version (#4384)

* ensure compatibility with upcoming CLI version

* change min CLI version to 2.34.0

* Update src/appservice-kube/HISTORY.rst

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

* fix 'az functionapp create' compatibility

* Update HISTORY.rst

* remove unneeded methods

* fix style

* fix 'az webapp create'

* bump version no.

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

* add message of the day for aks agentpools (#4381)

* add message of the day for aks agentpools

* bump version

* add validation for linux only

* fix tests

* fix spacing

* try to fix ut

* fix: typo, remove cluster creation validation

* add missing comma

* ut

* use more specific validator error

* update recording

* add a line

* bump version, test fixes, encoding fixes

* fix lint

* fix missing coverage

* change to iter (#4449)

* Data migration - Adding SKU recommendation (#4436)

* Adding SKU Recommendation Command

* Adding multiserver support - currently fails

* Adding Support for DatabaseAllowList and DatabaseDenyList

* Correcting lint and style errors

* Adding help for perf-data-collection and sku-recommendation

* Adding changes as per suggestions

* Nit change: config_file_path

* Adding custom cmds to common group

* Updating version of extension

* [Release] Update index.json for extension [ datamigration ] (#4456)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1387498

Last commit: https://github.com/Azure/azure-cli-extensions/commit/7cf0cbe19dac82a16fbf054aa9f4bf10dd1989c5

* [Monitor] `az monitor app-insights web-test`: Support Application Insights Web Test (#4440)

* Update vendored SDK

* Add list/show/create/update/delete commands

* Add parameter actions

* Add help messages

* Fix style and linter checks

* Add tests and recording files

* Update version and changelog

* Restorable APIs for Tables and Gremlin. Added Latest backup info for Gremlin and Table and also added start and end time filter for collection listing. (#4385)

* Table/Gremlin changes (Generated code)

* Table/Gremlin code and test changes for new commands

* New change

* New changes

* Working

* New Vendored sdk

* Test Updates and Session Records.

* Merge changes for removing EnableGremlinV2 Support

* Fixing Linter

* Update ReadMe, History and Version.

* [Release] Update index.json for extension [ application-insights ] (#4458)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1389585

Last commit: https://github.com/Azure/azure-cli-extensions/commit/a1ac7443895dd794ae3a86dfabfab5c4ea04a5d3

* [Release] Update index.json for extension [ aks-preview ] (#4455)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1387121

Last commit: https://github.com/Azure/azure-cli-extensions/commit/e4edce2ed79082e0cc183148810da86a7c3a52ca

* [EdgeOrder] Initial Release (#4450)

* Add command groups

* Simplify long parameters

* Refine help messages and README.md

* Add tests and recording files

* Fix style and linter checks

* [Release] Update index.json for extension [ edgeorder ] (#4460)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1390385

Last commit: https://github.com/Azure/azure-cli-extensions/commit/a408888db3f9ab1c5ea63f096b0973d997e87a22

* DataShare CLI (#4454)

* adding auo-generated cli commands and basic-test file with recording, commit-1

* adding tests, commit-2

* adding tests, commit-3

* adding tests, commit-4

* Updating cli version to 2021-08-01 and modifying tests

* updating test files

* Fixing CI checks

* removing auto-generated test file

* addressing PR comments, rev-1

* reinstating experimental tag

* updating test recording file after updating the branch with main

* updating test recording file after merging azure-cli repo with main on local

* updating test file after installing newer version of storage

* [Release] Update index.json for extension [ cosmosdb-preview ] (#4459)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1389592

Last commit: https://github.com/Azure/azure-cli-extensions/commit/52bca4ddf4b52ff1bf36da92461b3981e463f63f

* [Release] Update index.json for extension [ datashare ] (#4461)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1390624

Last commit: https://github.com/Azure/azure-cli-extensions/commit/00f0762f671793f96aa5b1c8e773077121f2946b

* fix: 🐛 [Spring-Cloud] Failed to remove jvm option in Enterprise (#4444)

* fix: 🐛 [Spring-Cloud] Failed to remove jvm option in Enterprise

* fix test

* fix lint

* Add more test

* fix app update

* fix UT recording

* add log

* fix lint

* Add new WHL for SCVMM scenarios (#4465)

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

* [Spring-Cloud] Add a new parameter "--build-env" for app deploy (#4443)

* fix: fix the deploy of gradle project with multiple modules

* add a new parameter --build-env

* fix typos

* fix

* fix

* fix

* fix

* fix

* fix

* fix test

* fix test

* fix test

* fix test

* fix test

* fix

* Update src/spring-cloud/HISTORY.md

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

* update ClientRequestError to ArgumentUsageError

* fix test

* fix test

* fix test

* update validator

* add new line

* update history

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

* chore(arcdata): arcdata version bump to 1.2.1 (#4466)

* chore(arcdata): arcdata version bump to 1.2.1

* chore(arcdata): arcdata version bump to 1.2.1

* Add CLI version 0.1.48 (#4470)

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

* [Release] Update index.json for extension [ spring-cloud ] (#4467)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1393299

Last commit: https://github.com/Azure/azure-cli-extensions/commit/47fd457596e401a5f3c5524c176064cc46559696

* [Release] Update index.json for extension [ appservice-kube ] (#4481)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1386720

Last commit: https://github.com/Azure/azure-cli-extensions/commit/4fbcd8fcc049730f65d92a30da1eb8628559b76b

Co-authored-by: Azure CLI Team <[email protected]>

* chore(arcdata): arcdata version bump to 1.2.2 (#4473)

* chore(arcdata): arcdata version bump to 1.2.2

* chore(arcdata): arcdata version bump to 1.2.2

* Update version and history info for az quantum extension 0.13.0 (#4484)

* Added ArcAppliance 0.2.16 (#4487)

Co-authored-by: Sai Sankar Gochhayat <[email protected]>

* [Release] Update index.json for extension [ quantum ] (#4489)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1408510

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f659327e9aa96b842cbcfe94a58d5ca4609a8041

* {Quota}Generate quota with codegen (#4475)

* {Quota}Generate quota with codegen

* Update setup.py

* Update test_quota_crud.yaml

* Update test_quota_crud.yaml

* Update

* Update

* Update _help.py

* Update _help.py

* [Release] Update index.json for extension [ quota ] (#4496)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1412890

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d292d6d0295e5245cff22da0bdc4bbf38ed92dd7

* [Quantum] Show progress messages and dots before executing commands (#4480)

* Show the first progress indicator dot before starting ARM template deployment

* Show progress messages on 'execute' and 'job submit'

* Change f-string to simple string

* chore(arcdata): arcdata version bump to 1.2.3 (#4501)

* Revert "chore(arcdata): arcdata version bump to 1.2.3 (#4501)" (#4502)

This reverts commit c45ca8102492ece8d1f7594f25224612954a2986.

* chore(arcdata): arcdata version bump to 1.2.3 (#4503)

* [appservice-kube] Make compatible with new SDK (#4494)

* fix plan create and webapp create

* bump version, include bug fix from #4493, fix style, fix webapp update bug, add basic regression tests

* remove deprecated 'az webapp update' params

* Fix UUID issue for data disk (#4479)

* new version 0.4.2

* new version 0.4.2

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Fixed an UUID issue by getting the data disk attached to the VM after VM creation

* Update src/vm-repair/azext_vm_repair/custom.py

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

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

* Move cli to production subscription (#4505)

* {NetworkManager}Add new commands (#4413)

* {NetworkManager}Add new commands

Add new commands 'az network manager connection', 'az network manager connection management-group', 'az network manager scope connection', 'az network manager group static-members'

* Update _help.py

* Update _params.py

Bump up azure-mamt-network SDK to 2021_05_01_preview and Update  params and commands.

* Update

* Update

* Update

* Update

* Update

* Update

* update tests for new api version 2021-05-01-preview for network-manager extension

* Update

* Update

* Update custom.py

* Update

* Update

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

* CosmosDB Adding enable MaterializedViews parameter for create/update database … (#4499)

* Adding enable MaterializedViews parameter for create/update database account

* api-version=2021-11-15-preview

* CassandraEnableMaterializedViews

* Update src/cosmosdb-preview/HISTORY.rst

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

* Update src/cosmosdb-preview/README.md

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

* Update src/cosmosdb-preview/azext_cosmosdb_preview/_params.py

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

* Update linter_exclusions.yml

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

* [Release] Update index.json for extension [ network-manager ] (#4511)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1419833

Last commit: https://github.com/Azure/azure-cli-extensions/commit/03dc8173839f44a960b25e625c23dbc34932143b

* [Release] Update index.json for extension [ vm-repair ] (#4507)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1416685

Last commit: https://github.com/Azure/azure-cli-extensions/commit/b68739c74809633eb9d358034f7884a81a2158d6

* [Release] Update index.json for extension [ appservice-kube ] (#4504)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1415742

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f77b373146be6cb2f15b1cccec5311f161b23ce0

* {AKS} Fix several test cases (#4506)

* update

* update recording

* update subnet cidr for ingress test case

* update recordings

* fix coverage

* [Release] Update index.json for extension [ cosmosdb-preview ] (#4512)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1419899

Last commit: https://github.com/Azure/azure-cli-extensions/commit/387d57b8a1140d19738ace79864788814725d61b

* SSH Extension v.1.0.1 (#4474)

* Change how private ips are appended to the list (#4517)

* [Release] Update index.json for extension [ ssh ] (#4520)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1426796

Last commit: https://github.com/Azure/azure-cli-extensions/commit/55eee43c837115fe90760a1ebcb16edde6c8deb8

* ADAL to MSAL move for cli>=2.30.0 (#4468)

* ADAL to MSAL move for cli>=2.30.0

* nit

* nit

* nit

* add comment

* addressing review comments

* linting fixes

* linter fixes

* linter nit

* add license for new file

* update release date

* add token condition

* Update src/connectedk8s/azext_connectedk8s/_utils.py

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

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>

* [Release] Update index.json for extension [ connectedk8s ] (#4523)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1431525

Last commit: https://github.com/Azure/azure-cli-extensions/commit/f13c4952c40bc1796a8e2c4b024d373140a6cfbf

* add aks update support for http proxy (#4469)

* add aks update support for http proxy

* adjust e2e tests

* fix decorator and tests

* add test

* fix: add back update bits

* fix recording

* fix decorator

* decorator should have trusted ca in ut

* new recording

* add real proxy vm

* remove extraneous option list

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

* fix tests

* fix ut

* fix proxy urls

* add credscan suppression

* bump setup py version

* add live_only because of custom feature required

* exclude test due to feature

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

* [Release] Update index.json for extension [ aks-preview ] (#4525)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1433736

Last commit: https://github.com/Azure/azure-cli-extensions/commit/456f486a75de4c6d19c8f861bbf8fe3f54ff8856

* [storage-preview] `az storage account create/update`: Support `--key-vault-federated-identity-client-id` and `--allowed-copy-scope` (#4498)

* `az storage account create/update`: Support `--key-vault-federated-identity-client-id` and `--allowed-copy-scope`

* rerun tests

* Update src/storage-preview/HISTORY.rst

* [Release] Update index.json for extension [ storage-preview ] (#4527)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1434569

Last commit: https://github.com/Azure/azure-cli-extensions/commit/1208f9ec30660177a7571a5482df3f7a40a41901

* [appservice-kube] `az webapp create`: Fix TypeError: 'NoneType' object is not subscriptable (#4518)

* fix plan create and webapp create

* bump version, include bug fix from #4493, fix style, fix webapp update bug, add basic regression tests

* remove deprecated 'az webapp update' params

* fix TypeError: 'NoneType' object is not subscriptable

* bump version

* better error message for nonextant customlocation

* fix name

* Fix functionapp devops-pipeline create (#4500)

* Fix functionapp devops-pipeline create

* Update version

* Bump up minimum cli core version to 2.23.0 (#4537)

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* [Release] Update index.json for extension [ connectedk8s ] (#4538)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1437831

Last commit: https://github.com/Azure/azure-cli-extensions/commit/a1959b123d4c11149adae2728ab5791949889d54

* [Release] Update index.json for extension [ functionapp ] (#4532)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1437244

Last commit: https://github.com/Azure/azure-cli-extensions/commit/182362788e6f0e70f67b4f8934ce6e170a207bd2

* [Release] Update index.json for extension [ appservice-kube ] (#4531)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1437153

Last commit: https://github.com/Azure/azure-cli-extensions/commit/d2674f49819c6b1e953d55c5c650a5a0a8cfa559

* [CostManagement] `az costmanagement export`: Support `ActualCost` and `AmortizedCost` for export type (#4528)

* Fixing typo (#4541)

* [Release] Update index.json for extension [ costmanagement ] (#4544)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1439436

Last commit: https://github.com/Azure/azure-cli-extensions/commit/8080424bfe2b4cfc2c0bfee76ec918daeacc4d71

* Remove "packaging" module since 2.16.0 doesn't support it (#4545)

* Remove "packaging" module

* nit

* handle 2.30 case

* nit

* nit

Co-authored-by: Siri Teja Reddy Kasireddy <[email protected]>

* [Release] Update index.json for extension [ connectedk8s ] (#4546)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1439941

Last commit: https://github.com/Azure/azure-cli-extensions/commit/411a1c893715364f6a4980c1b0de3e67618292cc

* [AuthV2] Nov 2022 Bug Fixes and Improvements (#4089)

* fixes

* static analysis done

* fix test recordings

* fix static analysis

* Update to throw AZCliErrors and add Unit Tests for new Error Cases

* Fix indentation issue

* Merge main and re-record tests

* change azclierror to argumentusageerror

* Fix indentation

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

* Add sdk for 2022-03-01-preview api generated here in this PR: (#4552)

https://github.com/Azure/azure-rest-api-specs/pull/18319

* {AKS} Vendor 2022-02-02-preview for aks-preview  (#4553)

* vendor 2022-02-02-preview

* update recordings

* manual update recording for test_aks_create_with_crg_id

* [Quantum] Output error information in table when Quantum job fails (#4463)

* Output status, error code, and error message (errorData) when job fails

* Add Target, Job ID, and Submission Time to table

* Let 'az quantum job output' show error info

* Add unit test cases and error handling for missing errorData items

* Delete commented-out code

* Add logging

* Add Job ID to error log, delete commented-out code

* Delete more commented-out code

* Use dict.get() function

* Alerts management contains operator support for resource type condition (#4554)

* added support for contains and doesNotContain operators for resource types filter

* 1. updated version to 0.2.1
2. updated history file accordingly

Co-authored-by: MIDDLEEAST\lienglender <[email protected]>

* Release 0.24.0 (#4516)

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

* [Quantum] Handle more storage types during workspace creation (#4524)

* Parameterize storage SKU and kind

* Get storage account parameters

* Add a break

* Add unit test

* Update test defaults

* Make minor changes to satisfy Pylint and Flake8

* Add storage account validation

* Add unit test cases

* Rearranged a logical expression to satisfy Flake8 style check

* Append 'GRS' (or 'grs') to all references to the new test storage account

* Re-run the CI/CD checks

* Change test defaults back to e2e-scenarios resource group

* Bulk fix to change az API links from using underscore to dash (#4551)

* fix_storage_azcopy_blob_sync_help (#4521)

* Add support for enabling Azure KeyVault KMS in aks-preview (#4547)

* support AzureKeyVaultKms

* fix

* [Release] Update index.json for extension [ aks-preview ] (#4558)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1450270

Last commit: https://github.com/Azure/azure-cli-extensions/commit/4a7de787a95b840679a5b24fb6e5c4c1e40c28a2

* update owners of aks-preview (#4557)

* Onboard command extension for Azure Managed Grafana service (#4495)

* POC: update with command list to implement

* consolidate control plane and dashboard commands

* half done: make resource group and location optional

* more commands and code style fixes

* add a validator to workspace https://bugs.python.org/issue9334

* catch up a few TODOs

* Consolidate data source CRUD commands with bug fixes and samples

* add help

* bug fixes on data source commands

* fix style errors

* fix bugs in data-source commands

* clean up command help

* upload whl file for tempoaray measures, before we publish it officially

* update readme and bug fixes

* update get_start doc

* Update get_start,md

* fix typos in get_start.md

* add vendor SDK

* new extension

* new extension built from azdev

* rename from ags to amg

* add tag support

* Update readme.rst

* undo non related changes

* rename readme to markdown

* address lint error

* address linter error

* more fix towards command lint error

* register the command module in a few common file

* add import

* support gallery import

* use deep copy

* address review feedback

* set the minimum cli core version

* fix a bug in 'az grafana user show'

* Remove the 'id' on creating dashboard to prevent 'Not Found' error

* fix validation for kms options (#4560)

* [Release] Update index.json for extension [ alertsmanagement ] (#4556)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1447524

Last commit: https://github.com/Azure/azure-cli-extensions/commit/1e8404b25d94f6f63b7ed7aee229ab9f4273cbe9

* [Release] Update index.json for extension [ amg ] (#4559)

Triggered by Azure CLI Extensions Release Pipeline - ADO_BUILD_ID=1451213

Last commit: https://github.com/Azure/azure-cli-extensions/commit/0ad46009734e4f72c0584f0a00e9bca17dffe7bd

* {CI} Pin `Jinja2` to 3.0.3 (#4572)

* ContainerApp extension (#4441)

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating wa…
runefa added a commit that referenced this pull request Apr 22, 2022
…mmands, log streaming commands (#72)

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Dapr moved from Template to Configuration

* Use aka.ms link for containerapps yaml

* Updated dapr enable/disable to current spec.

* Fixed oversight.

* Remove revisions-mode from containerapp update

* Fixed dapr enable property names. (#47)

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

* Fix exceptions with using --yaml in containerapp create/update

* Rename history msg

* Include fqdn in containerapp table output

* Added ingress messages.

* Revert history msg

* Add basic test case

* Remove managed-identity support for first release of CLI

* Need to investigate test flakiness

* Update _help.py

removing duplicate help

* Added prototype of container up.

* Fixed deploy from acr registry image infer credentials issue.

* Tried to add source.

* Added acr build.

* Finished acr build functionality.

* Added acr create functionality and pull registry from existing containerapp if it exists.

* Fixed bugs.

* Check if rg exists and create one with name if it doesn't.

* initial containerapp ssh implementation

* fix interactive commands (vim); handle ctrl + c instead of exiting

* fix style and linter issues

* Added disable verbose. Moved utils into utils.py.

* fix for ssh for windows clients

* fix for unix

* Disable verbose now uses sdk poller so it gives running animation.

* Added helps for params. Added error handling for non acr registry passed with source. Ignore param disable_warnings for create and no_wait for up.

* Updated disable_warnings ignore. Removed disable_warnings from update_containerapp.

* add terminal resizing

* reorganize code; implement terminal resizing, add startup command param, etc

* organize code, remove token from warning output

* add validations, add replica commands

* use the correct API for fetching default container; remove is_preview

* Renamed silent to quiet.

* Fixed style issues.

* add log streaming, bump version number and add to HISTORY.rst

* add basic ssh test

* Added workspace name and fqdn to dry_run_str. Added indicators of if the resources are new or existing to dry_run_str.

* fix ssh test for windows

* Check RP for location when not provided. Open Dockerfile and use EXPOSE for CA port.

* fix windows arrow keys after exit

* fix typo, add logstream test, remove token from logstream output

* Removed print statement.

* Updated dockerfile expose automatic ingress feature.

* Removed dry run str, added dry run obj instead.

* use bearer auth; fix --command bug

* add handling for smooth transition to new URL path

* Fixed merge conflict.

* fix merge conflicts

* Create env if name passed and it doesn't exist.

* Added missing import from merge.

* Added prototype for new environment workflow.

* Finished environment logic.

* Minor updates before demo.

* add 'az containerapp github up' (wip)

* various fixes for demo

* rearrange github up code

* merge haroonf/containerappup

* start up refactor

* add --repo to up and refactor up

* reorganize code more; fix various bugs

* fix linter issues, fix lingering exec/tail improvements

* update history

* update output

* bug fixes for --repo

* fix --source bug

* fix --source

* minor bug fixes

* Added API change.

* Finished API change, added helloworld image auto ingress, checked provisioning state beforehand.

* fixes for sisira's comments

* fix minor typo

* bug fix where commands fail if providing registry creds

* Fixed style issues.

* Updated help and version text.

Co-authored-by: Calvin Chan <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
Co-authored-by: Sisira Panchagnula <[email protected]>
runefa added a commit that referenced this pull request Apr 22, 2022
* Marchp1s and add back Identity (#57)

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Fixed dapr in create.

* Revert "Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing."

This reverts commit 51bc543.

* Revert "Fixed dapr in create."

This reverts commit 37030ad.

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Dapr moved from Template to Configuration

* Use aka.ms link for containerapps yaml

* Updated dapr enable/disable to current spec.

* Fixed oversight.

* Remove revisions-mode from containerapp update

* Fixed dapr enable property names. (#47)

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

* Fix exceptions with using --yaml in containerapp create/update

* Rename history msg

* Include fqdn in containerapp table output

* Added ingress messages.

* Revert history msg

* Reduced redundant code between revision copy and containerapp update.

* Fixed merge issues.

* Fixed merge conflicts, moved helper function

Co-authored-by: Calvin Chan <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>

* Fix help for linter

* various fixes, helptext (#59)

* Fixes (#60)

* Updated managed identity + help. (#61)

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

* Added user-assigned and system-assigned to containerapp create. (#62)

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

* Bump version to 0.1.1 (#63)

* Added more specific MSI help text. (#64)

* Added more specific MSI help text.

* Updated help text.

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

* Bump to 0.3.0 (#65)

* Container App Test suite (#67)

* Add tests for containerapp create

* All tests under the same function to share environment - need to figure how to get multiple functions to share environment

* Basic tests

* use new GH actions API

* remove live only recordings

* update CODEOWNERS

* fix API version naming

* Managed Identity Tests (#69)

* Added managed identity tests.

* Fixed msi tests.

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

* resolve review comments

* Managed Identity Fixes (#71)

* Added managed identity tests.

* Fixed msi tests.

* Added live_only to managed identity tests.

* Changed region to eastus2 from canary.

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

* Update src/containerapp/azext_containerapp/_params.py

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

Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
Co-authored-by: Silas Strawn <[email protected]>
Co-authored-by: Sisira Panchagnula <[email protected]>
StrawnSC added a commit that referenced this pull request May 19, 2022
* Marchp1s and add back Identity (#57)

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Fixed dapr in create.

* Revert "Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing."

This reverts commit 51bc543.

* Revert "Fixed dapr in create."

This reverts commit 37030ad.

* Ingress enable --transport default. Secret list returns empty array. Secret update prints message saying user needs to restart their apps. Added show-values flag to secret list. Fixed yaml datetime field issues, replaced x00 values that also came up during testing.

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Dapr moved from Template to Configuration

* Use aka.ms link for containerapps yaml

* Updated dapr enable/disable to current spec.

* Fixed oversight.

* Remove revisions-mode from containerapp update

* Fixed dapr enable property names. (#47)

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

* Fix exceptions with using --yaml in containerapp create/update

* Rename history msg

* Include fqdn in containerapp table output

* Added ingress messages.

* Revert history msg

* Reduced redundant code between revision copy and containerapp update.

* Fixed merge issues.

* Fixed merge conflicts, moved helper function

Co-authored-by: Calvin Chan <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>

* Fix help for linter

* various fixes, helptext (#59)

* Fixes (#60)

* Updated managed identity + help. (#61)

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

* Added user-assigned and system-assigned to containerapp create. (#62)

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

* Bump version to 0.1.1 (#63)

* Added more specific MSI help text. (#64)

* Added more specific MSI help text.

* Updated help text.

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

* Bump to 0.3.0 (#65)

* Container App Test suite (#67)

* Add tests for containerapp create

* All tests under the same function to share environment - need to figure how to get multiple functions to share environment

* Basic tests

* use new GH actions API

* remove live only recordings

* update CODEOWNERS

* fix API version naming

* Managed Identity Tests (#69)

* Added managed identity tests.

* Fixed msi tests.

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

* resolve review comments

* Managed Identity Fixes (#71)

* Added managed identity tests.

* Fixed msi tests.

* Added live_only to managed identity tests.

* Changed region to eastus2 from canary.

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

* Update src/containerapp/azext_containerapp/_params.py

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

* 4/26 release: Up with --repo/--browse, exec (ssh) command, replica commands, log streaming commands  (#72)

* Skeleton code

* az containerapp env show

* List kube/managed environments

* Create kube environment, wait doesn't work yet

* Update containerapp stubs (check if it is supported now)

* Containerapp env delete, polling not working yet

* Added polling for create and delete

* Use Microsoft.App RP for show, list, delete command

* Create containerapp env using Microsoft.App RP

* Add optional containerapp env create arguments

* Remove old kube environment code, naming fixes

* Containerapp create almost done

* Done containerapp create, except for --yaml. Need to test

* Containerapp show, list

* Fix helptext

* Containerapp delete

* Containerapp update. Needs secrets api to be implemented, and testing

* Add scale command

* Various validations, small fixes

* listSecrets API for updates, autogen log analytics for env

* Use space delimiter for secrets and env variables

* Verify sub is registered to Microsoft.ContainerRegistration if creating vnet enabled env, remove logs-type parameter

* Containerapp create --yaml

* Fix updating registry to do create or update

* Fix containerapp update command. Add image-name parameter to support multi container updates. Fix updating registries, containers and secrets

* started update with --yaml. Need to do create or update for when an attribute is a list of items

* use space delimiter for startup_command and args, instead of comma delimiter

* Traffic weights

* List and show revisions

* az containerapp revision restart, activate, deactivate

* Add ability for users to clear args/command in az containerapp update

* Various fixes, traffic weights fixes

* Verify subnet subscription is registered to Microsoft.ContainerServices

* GitHub Actions Update (#17)

* Added models. Finished transferring Calvin's previous work.

* Updated wrong models.

* Updated models in custom.py, added githubactionclient.

* Updated envelope to be correct.

* Small bug fixes.

* Updated error handling. Fixed bugs. Initial working state.

* Added better error handling.

* Added error messages for tokens with inappropriate access rights.

* Added back get_acr_cred.

* Fixed problems from merge conflict.

* Updated names of imports from ._models.py to fix pylance erros.

* Removed random imports.

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

* Remove --location since location must be same as managed env

* Add options for flag names: --env-vars and --registry-srever

* Empty string to clear env_vars

* Default revisions_mode to single

* Infer acr credentials if it is acr and credentials are not provided

* fix help msg

* if image is hosted on acr, and no registry server is supplied, infer the registry server

* Added subgroups (Ingress, Registry, Secret) and updated revisions (#18)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Modified update_containerapp_yaml to support updating from non-current revision.

Authored-by: Haroon Feisal <[email protected]>

* More p0 fixes (#20)

* Remove --registry-login-server, only allow --registry-server

* Rename --environment-variables to --env-vars

* If no image is supplied, use default quickstart image

* Update help text (#21)

* Update help text

* Update punctuation

* master -> main

* New 1.0.1 version

* Added identity commands + --assign-identity flag to containerapp create (#8)

* Added identity show and assign.

* Finisheed identity remove.

* Added helps, updated identity remove to work with identity names instead of requiring identity resource ids.

* Moved helper function to utils.

* Require --identities flag when removing identities.

* Added message for assign identity with no specified identity.

* Added --assign-identity flag to containerapp create.

* Moved assign-identity flag to containerapp create.

* Fixed small logic error on remove identities when passing duplicate identities. Added warnings for certain edge cases.

* Updated param definition for identity assign --identity default.

* Added identity examples in help.

* Made sure secrets were not removed when assigning identities. Added tolerance for [system] passed with capital letters.

* Fixed error from merge.

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

* Dapr Commands (#23)

* Added ingress subgroup.

* Added help for ingress.

* Fixed ingress traffic help.

* Added registry commands.

* Updated registry remove util to clear secrets if none remaining. Added warning when updating existing registry. Added registry help.

* Changed registry delete to remove.

* Added error message if user tries to remove non assigned registry.

* Changed registry add back to registry set.

* Added secret subgroup commands.

* Removed yaml support from secret set.

* Changed secret add to secret set. Updated consistency between secret set and secret delete. Added secret help. Require at least one secret passed with --secrets for secret commands.

* Changed param name for secret delete from --secrets to --secret-names. Updated help.

* Changed registry remove to registry delete.

* Fixed bug in registry delete.

* Added revision mode set and revision copy.

* Added dapr enable and dapr disable. Need to test more.

* Added list, show, set dapr component. Added dapr enable, disable.

* Added delete dapr delete.

* Added helps and param text.

* Changed dapr delete to dapr remove to match with dapr set.

* Commented out managed identity for whl file.

* Uncommented.

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

* Rename --image-name to --container-name

* Remove allowInsecure since it was messing with the api parsing

* Fix for env var being empty string

* Rename to --dapr-instrumentation-key, only infer ACR credentials if --registry-server is provided

* Remove az containerapp scale

* Fix delete containerapp errors

* Remove ingress, dapr flags from az containerapp update/revision copy

* Fix revision list -o table

* Help text fix

* Bump extension to 0.1.2

* Update managed identities and Dapr help text (#25)

* Update managed identities and Dapr help text

* Update Dapr flags

* Add secretref note

* Env var options + various bug fixes (#26)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

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

* Fixed style issues, various bug fixes (#27)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

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

* Update src/containerapp/azext_containerapp/tests/latest/test_containerapp_scenario.py

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

* Specific Error Types + Bugfixes (Help, remove app-subnet-resource-id, removed env-var alias, added help text for --name) (#28)

* Moved dapr arguments to env as a subgroup.

* Added env variable options.

* Changed revision mode set to revision set-mode.

* Added env var options to revision copy.

* Fixed revision copy bug related to env secret refs.

* Changed registry and secret delete to remove. Added registry param helps. Removed replica from table output and added trafficWeight.

* Updating warning text.

* Updated warning text once more.

* Made name optional for revision copy if from-revision flag is passed.

* Fixed whitespace style issues.

* Styled clients and utils to pass pylint.

* Finished client.py pylint fixes.

* Fixed pylint issues.

* Fixed flake8 commands and custom.

* Fixed flake issues in src.

* Added license header to _sdk_models.

* Added confirmation for containerapp delete.

* Update helps for identity, revision. Removed env-var alias for set-env-vars. Added name param help.

* Removed app-subnet-resource-id.

* Updated infrastructure subnet param help.

* Check if containerapp resource exists before attempting to delete.

* Added check before deleting managed env.

* Changed error types to be more specific.

* Removed check before deletion. Removed comments.

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

* Reset to 0.1.0 version, remove unneeded options-list

* Update min cli core version

* Fixed style issues. (#30)

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

* Fix linter issues

* Use custom-show-command

* Removed --ids from revision, secret, registry list.

* Add linter exclusions

* Fix polling on delete containerapp

* Fix error handling

* Add Container App Service

* Fix flake linter

* Fix help text

* Mark extension as preview

* Add python 3.9 and 3.10 as supported

* Remove registries and secrets from az containerapp update, in favor of registry and secret subgroup

* Fix YAML not working

* Move import to inside deserialize function

* Dapr moved from Template to Configuration

* Use aka.ms link for containerapps yaml

* Updated dapr enable/disable to current spec.

* Fixed oversight.

* Remove revisions-mode from containerapp update

* Fixed dapr enable property names. (#47)

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

* Fix exceptions with using --yaml in containerapp create/update

* Rename history msg

* Include fqdn in containerapp table output

* Added ingress messages.

* Revert history msg

* Add basic test case

* Remove managed-identity support for first release of CLI

* Need to investigate test flakiness

* Update _help.py

removing duplicate help

* Added prototype of container up.

* Fixed deploy from acr registry image infer credentials issue.

* Tried to add source.

* Added acr build.

* Finished acr build functionality.

* Added acr create functionality and pull registry from existing containerapp if it exists.

* Fixed bugs.

* Check if rg exists and create one with name if it doesn't.

* initial containerapp ssh implementation

* fix interactive commands (vim); handle ctrl + c instead of exiting

* fix style and linter issues

* Added disable verbose. Moved utils into utils.py.

* fix for ssh for windows clients

* fix for unix

* Disable verbose now uses sdk poller so it gives running animation.

* Added helps for params. Added error handling for non acr registry passed with source. Ignore param disable_warnings for create and no_wait for up.

* Updated disable_warnings ignore. Removed disable_warnings from update_containerapp.

* add terminal resizing

* reorganize code; implement terminal resizing, add startup command param, etc

* organize code, remove token from warning output

* add validations, add replica commands

* use the correct API for fetching default container; remove is_preview

* Renamed silent to quiet.

* Fixed style issues.

* add log streaming, bump version number and add to HISTORY.rst

* add basic ssh test

* Added workspace name and fqdn to dry_run_str. Added indicators of if the resources are new or existing to dry_run_str.

* fix ssh test for windows

* Check RP for location when not provided. Open Dockerfile and use EXPOSE for CA port.

* fix windows arrow keys after exit

* fix typo, add logstream test, remove token from logstream output

* Removed print statement.

* Updated dockerfile expose automatic ingress feature.

* Removed dry run str, added dry run obj instead.

* use bearer auth; fix --command bug

* add handling for smooth transition to new URL path

* Fixed merge conflict.

* fix merge conflicts

* Create env if name passed and it doesn't exist.

* Added missing import from merge.

* Added prototype for new environment workflow.

* Finished environment logic.

* Minor updates before demo.

* add 'az containerapp github up' (wip)

* various fixes for demo

* rearrange github up code

* merge haroonf/containerappup

* start up refactor

* add --repo to up and refactor up

* reorganize code more; fix various bugs

* fix linter issues, fix lingering exec/tail improvements

* update history

* update output

* bug fixes for --repo

* fix --source bug

* fix --source

* minor bug fixes

* Added API change.

* Finished API change, added helloworld image auto ingress, checked provisioning state beforehand.

* fixes for sisira's comments

* fix minor typo

* bug fix where commands fail if providing registry creds

* Fixed style issues.

* Updated help and version text.

Co-authored-by: Calvin Chan <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
Co-authored-by: Sisira Panchagnula <[email protected]>

* Fixed small issue with test.

* Removed flake exclusions and removed type=str from params.

* Fixed repo bug when searching for dockerfile, increased timeout on github action complete, fixed datetime import issue from style fix.

* Added env var changes.

* Assume port if ingress is provided with image and port is not.

* Fixed small helloworld error.

* Fixed logger typo.

* Search for acr before creating one.

* Fixed bug where only --environment is passed. Changed hash on acr name to make it more unique. Tiny change in find_existing_acr.

* error out if dockerfile not found (--repo)

* Fixed bug with --image. Changed logger warning output. Disabled warnings on the registry update code for containerapp up. Added HELLOWORLD constant.

* Disabled no_wait. Added better error handling for up API calls. Updated ingress infer warning text. Fixed typo. Moved create_if_needed to environment.

* fix ACR length cap; enforce name/secret limits; trigger GH action if needed (update with GH actions already extant); fail command if GH action ends in failure

* force exact match for ACR retrieval (prevents secrets issues for --repo)

* fix hashing and add GH validations

* don't retrieve a registry if one provided; take RG from env if possible

* Fixed --registry-server with --image bug. (#78)

* Fixed --registry-server with --image bug.

* Fixed style issues.

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

* use SP creds if provided

* fix github actions (less polling)

* Added prototype for env check.

* Honor location and environment passed to create new containerapp (even if a CA exists on subscription with the same name) (#79)

* Create new Containerapp if user passes env name even if a CA exists with the same name.

* Create a new app if location doesn't match any other app.

* Fixed small bug, added better error handling for multiple environments with the same name on subscription.

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

* print created SP name/id; prevent using ACR names longer than 20 chars with --repo; add basic --image test

* fix style; add license header

* Finished core logic.

* add max core cli version 2.36.0

* make ACR name more unique (must be globally unique)

* Finished logic.

* sort workflows by date before selecting one

* log workflow

* Added error message with eligible locations if users pass uneligible location.

* Added function to check if env already exists so we don't try to update the location value of an existing environment.

* Added error handling for location northcentralusstage. Added list of eligible locations to unallowed location error message.

* Small fixes, implemented check_env_name_on_rg.

* location bug fix

* fix style

* bump version number

* Updates to tests (#82)

* Updates to update tests

* Update api version for create

* Remove recordings (#83)

* prevent using --only-show-errors, --output, -o in up

* Added FileShare commands. (#84)

* Added FileShare commands.

* Updated params to match spec. Added param help.

* Added help. Removed --ids support.

* Removed automatic import statements.

* Added back type in help.

* Added tests.

* Updated param names to better reflect AzureFile dependency.

* Added validation to ensure share name and account name are longer than 2 characters. Seems to be an API issue.

* Added warning message if user is updating existing storage.

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

* Fixed bug. (#86)

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

* register log analytics resource provider if not registered when creating an env with up

* fix style

* Fixed linter issue.

* fix linter issues

* update history file

* Moved constant to constants.py.

* add timeout to container app ping for ssh/logstream

* Various tests (Ingress, Traffic, Dapr, Env) (#87)

* Added env tests.

* Added ingress tests.

* Added ingress traffic tests.

* Fixed small issue.

* Added remove test for dapr components.

* Removed live_only from containerapp commands. Added recordings.

* Removed live_only from env commands.

* Removed sleep 60 since we use poll for delete anyways.

* Wrote better dapr-component tests. Fixed old tests that required live only.

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

* Revert "Various tests (Ingress, Traffic, Dapr, Env) (#87)" (#88)

This reverts commit 40ca5b9.

* Reverted fileshare. (#90)

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

* Tests (dapr-components, env, ingress, traffic) (#89)

* Added env tests.

* Added ingress tests.

* Added ingress traffic tests.

* Fixed small issue.

* Added remove test for dapr components.

* Removed live_only from containerapp commands. Added recordings.

* Removed live_only from env commands.

* Removed sleep 60 since we use poll for delete anyways.

* Wrote better dapr-component tests. Fixed old tests that required live only.

* Added suppression for log analytics dummy secrets.

* Rerecorded tests.

* rerecord failing tests

* Updated credscan.

Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Silas Strawn <[email protected]>

* adding certs cmd & test

* adding hostname cmds & test

* changes based on comments

* changes based on comments

Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Calvin Chan <[email protected]>
Co-authored-by: Haroon Feisal <[email protected]>
Co-authored-by: Anthony Chu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
Co-authored-by: Silas Strawn <[email protected]>
Co-authored-by: Sisira Panchagnula <[email protected]>
StrawnSC pushed a commit that referenced this pull request Aug 1, 2022
* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update public extension pipeline

* Change condition variable

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

* Stage Public Version of k8s-extension 0.2.0 for official release (#15)

* Create pull.yml

* Update pull.yml

* Update azure-pipelines.yml

* Initial commit of k8s-extension

* Update pipelines file

* Update CODEOWNERS

* Update private preview pipelines

* Remove open service mesh from public release

* Update pipeline files

* Update public extension pipeline

* Change condition variable

* Add version to public preview/private preview

* Update pipelines

* Add different testing based on private branch

* Add annotations to extension model

* Update k8s-custom-pipelines.yml

* Update SDKs with Updated Swagger Spec for 2020-07-01-preview (#13)

* Update sdks with updated swagger spec

* Update version and history rst

* Reorder release history timeline

* Fix ExtensionInstanceForCreate for import

* remove py2 bdist support

* Add custom table formatting

* Remove unnecessary files

* Fix style issues

* Fix branch based on comments

* Update identity piece manually

* Don't handle defaults at the CLI level

* Remove defaults from CLI client

* Check null target namespace with namespace scope

* Update style

* Add cassandra operator and location to model

Co-authored-by: [email protected] <Action - Fork Sync>

* Remove custom pipelines file

* Update extension description, remove private const

* Update pipeline file

* Disable refs docs

* Update to include better create warning logs and remove update context (#20)

* Update to include better create warning logs and remove update context

* Remove help text for update

* Fix spelling error

* Update message

* Fix k8s-extension conflict with private version

* Fix style errors

* Fix filename

* add customization for microsoft.azureml.kubernetes (#23)

* add customization for microsoft.azureml.kubernetes

* Update release history

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Add E2E Testing from Separate branch into internal code (#26)

* Add internal e2e testing

* Change to testing folder

* Inference CLI validation for Scoring FE (#24)

* cli validation starter

* added the call to the fe validation function

* nodeport validation not required

* test fix

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

* legal warning added (#27)

* Remove deprecated method logger.warn

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Update k8s-custom-pipelines.yml for Azure Pipelines

* Add Azure Defender to E2E testing (#28)

* Add azure defender testing to e2e

* Remove the debug flag

* Add configuration testing

* Fix pipeline failures

* Make test script more intuitive

* Remove parameter from testing

* Fix wrong location for k8s config whl

* Fix pip upgrade issue

* Fix pip install upgrade issue

* Fix pip install issue

* delete resurce in testcase (#29)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Check Provider is Registered with Subscription Before Making Requests (#18)

* Add check for KubernetesConfiguration

* Disable pylint and rename

* Update provider registration link

* Update version

* Remove extra blank line

* Fix bug in import

* only validate scoring fe when inference is enabled (#31)

* only validate scoring fe when inference is enabled

* Fix versioning

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Provider registration case insensitive

* do not validate against scoring fe if inference is not enabled. (#33)

* do not validate against scoring fe if inference is not enabled.

* add inference enabled scenario

* refine

* increase sleeping time

* fix

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Add OSM as Public Preview Extension (#34)

* Add OSM as public preview extension

* Add osm testing

* Add release train to tests

* Fix failing osm test

* Upgrade pip in integration testing

* Remove ununsed import

* Fix release train check in update

* Parallelize E2E Testing (#36)

* Add OSM as public preview extension

* Add osm testing

* Update test logic to parallelize

* Fix test success checking

* Parallelize extension testing

* Better error checking logic

* Fix azureml deletion

* Fix private build (#40)

* change amlk8s to amlarc (#42)

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

* Servicebus client model changes (#44)

* Servicebus client model changes

* Fix testing script

* Update history file and pipeline

* Update min cli core version for track 2 updates

* Read SSL cert and key from files (#38)

* first sketch of the change

fixes

removed extra blank lines

changes regarding param renaming

added ssl tests

added more detail to the unit test

additional import

moved pem files out of public folder

fixed import

chenged import

changed import

unit tests fix

unit test fix

fixed unit tests

fixed unit test

unit test fix

changes int test cert and key

* test protected config

* fix test typo

* temporary changes reverted

* fixing tests

* fixed file paths

* removed accidentally added file

* changes according to review comments

* more changes according to review comments

* changes according to review comments

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

* Upgrade release version

* Liakaz/inference read ssl from file (#47)

* first sketch of the change

fixes

removed extra blank lines

changes regarding param renaming

added ssl tests

added more detail to the unit test

additional import

moved pem files out of public folder

fixed import

chenged import

changed import

unit tests fix

unit test fix

fixed unit tests

fixed unit test

unit test fix

changes int test cert and key

* test protected config

* fix test typo

* temporary changes reverted

* fixing tests

* fixed file paths

* removed accidentally added file

* changes according to review comments

* more changes according to review comments

* changes according to review comments

* fixed decode error

* renamed the experimental param

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

* Fix style issues (#51)

* Fixed scoring fe related extension param names (#49)

* fixed scoring fe related extension params

* bug fix and style fixes

* variable rename

* fixed the error type

* set cluster to prod by default

* Add distro validation for osm-arc (#50)

* Add distro validation for osm-arc

* fixed indentation

* Fix linting

* Resolve comments

* Add unit test

* fix lint

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

* Add distro validation for osm-arc (#50)

* Add distro validation for osm-arc

* fixed indentation

* Fix linting

* Resolve comments

* Add unit test

* fix lint

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

* Add distro validation for osm-arc (#53)

removed release-train logic

* Add Custom Delete Logic for Partners (#54)

* Add custom delete logic

* Fix failing unit tests

* Add warning message when deleting amlarc extension (#55)

* add warning message

* fix indentation

* Update release version

* Remove Pyhelm from OSM customization (#58)

* Fix OSM pyhelm bug

* Debug bootstrap error

* Update release message

* Remove pyhelm dependency

* Update tests to only check extensionconfig creation (#61)

* Update tests to only check extensionconfig creation

* Single set of CRUD for AzureML

* Debug logs for connectedk8s

* Increase open service mesh version number

* Update k8s-extension Models to Track2 (#64)

* Update k8s-extension models to track2

* Add debug for failed cleanup

* Increase version number

* Exit 0 on failed cleanup

* Fix identity in wrong place in model (#66)

* Readd osm-arc distro validation (#62)

* Add distro validation for osm-arc

removed release-train logic

* Readd osm_arc distro validation

* Fix style

* Rm space

* Edit test

* Fixed tests and error logic

* Remove dependency

* Add delete method

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

* Don't Send Identity Headers If In DF (#67)

* Don't send identity for clusters in dogfood

* Add location to model for identity

* Add identity validation to testing

* Use default extension with identity instead of Cassandra specific (#69)

* Remove the identity check for now

* Add -t for clusterType parameter (#71)

* Adding a flag for AKS to AMLARC migration and set up corresponding FE… (#65)

* Adding a flag for AKS to AMLARC migration and set up corresponding FE helm values

* Remove one extra line

* Adding Scoring FE IS_AKS_MIGRATION check logic for helm values

Co-authored-by: Harry Yang <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* remove version requirement and auto upgrade minor version check (#72)

* Custom User Confirmation for Partners (#70)

* Custom user confirmation

* Check for disable confirm prompty for confirmation

* Add yes to delete command

* Code cleanup and style fixes (#73)

* Enabled identity by default (#74)

* Increase version

* Fix df check and add unit test (#77)

* Bump extension version

* Pin helm version

* Extensions GA changes into Public Branch (#79)

* Add openservicemesh back

* OpenServiceMesh import

* Update osm with new extension model

* Add back private file

* Add Azure ML to list of private extensions (#16)

* Update k8s-custom-pipelines.yml

* Add Microsoft.PolicyInsights extension (#17)

* Add Policy extension

* Update comment

* Update args

* Fix linting errors

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

* Add HISTORY_private file for private preview

* Change versioning scheme

* Update the code for supporting both extensions at once

* Fix style issue

* Remove old consts file

* change the resource tag from managed_by:amlk8s to created_by:amlk8s-e… (#22)

* change the resource tag from managed_by:amlk8s to created_by:amlk8s-extension

* remove the lock when creating resources

* fix lint

* update version and HISTORY_private.rst

* change error message

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

* Update the beta version with upstream

* Update the private history file

* Add upgrade pip to pipeline

* Move pip install within virtualenv

* Merge in k8s-extension/public (0.3.1) (#32)

* delete resurce in testcase (#29)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Check Provider is Registered with Subscription Before Making Requests (#18)

* Add check for KubernetesConfiguration

* Disable pylint and rename

* Update provider registration link

* Update version

* Remove extra blank line

* Fix bug in import

* only validate scoring fe when inference is enabled (#31)

* only validate scoring fe when inference is enabled

* Fix versioning

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: jonathan-innis <[email protected]>

* Update private release

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

* Release Version 0.4.0-b1 (#37)

* Merge k8s-extension/public into k8s-extension/private

* Update the version

* Fix testing concurrency

* K8s extension/private 0.4.0b2 (#41)

* Fix private build (#40)

* Update version

* Upgrade to v0.5.2

* Fix policy bug

* Increase private version

* Update consts_private.py

* Increase private version

* Increase version with public

* Add flux to private version

* Update models for 2021-05-01-preview

* Add async models to version

* Add no wait to delete and create

* support managed cluster

* Bump version

* Pin helm version

* Add cmd to delete call

* Add force deletion

* add dapr extension (#78)

Signed-off-by: Ji An Liu <[email protected]>

* Fix failing integration tests

* Adding the GA changes for private branch

* Fix confirm prompt

* Fix update E2E tests

Co-authored-by: jonathan-innis <[email protected]>
Co-authored-by: [email protected] <Action - Fork Sync>
Co-authored-by: nreisch <[email protected]>
Co-authored-by: yuyue9284 <[email protected]>
Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: anagg929 <[email protected]>
Co-authored-by: Ji'an Liu <[email protected]>
Co-authored-by: nanthi <nanthi@NANTHI01>

* Fix configuration settings in update

* Only provide confirmation when specifying settings

* Fix style issues

* Cassandra tests with update (#81)

* Add Microsoft.PolicyInsights extension for public preview (#83)

* Add Azure Policy

* Remove custom configuration and update tests

* Yuyu3/fix upgrade public (#85)

* populate configuration protected settings for azureml

bump version && add log

fetch connection string only if configuration protected settings are set

update ssl key

* bump the version

* reverse changes on version and HISTORY.rst

* inferenceLoadBalancerHA

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

* Remove Parallel Powershell Jobs (#82)

* Unparallelize tests

* Moved location of pipeline file

* Remove the parallel invoke expression calls

* Add templates to testing

* Remove policy update test from extension E2E (#88)

* feIsNodePort, feIsInternalLoadBalancer (#87)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Fix history file

* Add one more prompt for amlarc extension update (#94)

* Add one more prompt for amlarc extension update

* fix pylint issue

* fix pylint issue

* fix pylint issue

* fix pylint issue

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

* Update Identity Creation for Appliance to Latest Version (#95)

* Update appliance API to latest version for identity

* Create a utils file with get parent_api_version

* Fix style errors

* Bump version

* Remove additional entry from history

* Do not create identity with appliances (#97)

* Bump version

* support sslSecret (#99)

* support sslSecret

* fix

* fix error message

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

* Bump version to 1.0.4

* Increase OSM verison to valid version (#104)

* remove inference private review warning message (#102)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Update unit testing from osm to dapr (#106)

* osm-ext: enable system identities (#105)

* remove inference private review warning message (#109)

Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Build Extension in Template (#110)

* Build extension using a template

* Update build template in run test

* Fix private build issues (#112)

* Upgrade vendored_sdks to 2022-03-01 (#113)

* Upgrade vendored_sdks to 2022-03-01

* Update recordings file

* Bump version and changelog

* Update AzureMLKubernetes install parameters on inferenceRouterServiceType and internalLoadBalancerProvider (#114)

* Add one more prompt for amlarc extension update

* fix pylint issue

* fix pylint issue

* fix pylint issue

* fix pylint issue

* merge from AzureArcForKubernetes/azure-cli-extensions

* Update AzureMLKubernetes install parameters on inferenceRouterServiceType and internalLoadBalancerProvider

* update error message

* update error message

* fix pipeline failures

* fix test failure

Co-authored-by: Youhua Tu <[email protected]>
Co-authored-by: Youhua Tu <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* Change extension validation logic osm-arc (#116)

* added default identity support for microsoft.azurebackup.backupagent (#115)

* Change extension validation logic osm-arc

Signed-off-by: nshankar13 <[email protected]>

Co-authored-by: sangitaray2021 <[email protected]>
Co-authored-by: Jonathan Innis <[email protected]>

* [k8s-extension] Add Managed Identity Auth support for ContainerInsights Extension (#118)

* container insights aad auth support

* container insights aad auth support

* container insights aad auth support

* handle useAADAuth setting

* handle useAADAuth setting

* delete dcr-a if its exists incase of MSI auth

* fix formatting

* fix formatting

* fix formatting

* fix formatting

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

* fix install issue on aks (#119)

* fix install issue on aks

* Removing checking distribution

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

* [k8s-extension] Bring back containerInsights solution addition in msi mode (#120)

* bring back containerinsights solution in dcr mode

* bring back containerinsights solution in dcr mode

* Bump release version

* Remove backup agent from mapping

* Fix history file

* clusterip (#121)

* clusterip

* fix pr test

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

* No default values for the patch scenario with extension (#122)

* Bump version to 1.2.1

* Rename inferenceLoadBalancerHA (#123)

* Rename inferenceLoadBalancerHA

Rename inferenceLoadBalancerHA to inferenceRouterHA and unify related logic

* Update AzureMLKubernetes.py

* Remove unused comments to trigger pipeline

* Update AzureMLKubernetes.py

* Yuyu3/remove relay (#124)

* remove relay

* disable service bus by default, disable relay if target cluster is managed clusters.

* move nginx ingress logic

* remove service bus in tests

* update HISTORY

* If no default values provided for update, assign an empty dict.

* set original_extension_config_settings to empty dict if it is None.

* bump version to 1.2.2

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

* Update test documentation for partners (#132)

* remove warning message (#129)

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

* Use cluster scale to control clusterPurpose and inferenceRouterHA (#130)

* Use cluster scale to control clusterPurpose and inferenceRouterHA

* Update AzureMLKubernetes.py

To solve transient gate error

* Update AzureMLKubernetes.py

* Update AzureMLKubernetes.py

* Update AzureMLKubernetes.py

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

* Fix warning message returned on PATCH (#133)

* Do not invoke 'create_or_update' if resource is already exists. (#135)

* init

* update HISTORY

* do not populate clusterPurpose if infrenceRouterHA is not set.

* use correct exception type.

* use correct exception type.

* fix lint

* use correct exception type.

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

* [k8s-extension]  ContainerInsights Extension Managed Identity Auth Onboarding updates  (#137)

* bring back containerinsights solution in dcr mode

* bring back containerinsights solution in dcr mode

* update to stream group and DCR API version

* Only test k8s-extension

* [k8s-extension] ContainerInsights Extension Managed Identity Auth Onboarding related bug fixes (#141)

* fix bugs

* fix indent issues

* remove get workspace key for msi auth

* fix pr feedback

* Fix osm-arc installation non-connected clusters

Signed-off-by: nshankar13 <[email protected]>

* k8s-extension azuredefender namespace to mdc (#152)

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

* change codeowners for k8s extension & configuration (#150)

* change codeowners for k8s extension & configuration

* change code owners

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

* bump k8s-extension version to 1.2.5

* delete testing files

Co-authored-by: Jonathan Innis <[email protected]>
Co-authored-by: [email protected] <Action - Fork Sync>
Co-authored-by: yuyue9284 <[email protected]>
Co-authored-by: Yue Yu <[email protected]>
Co-authored-by: Lia Kazakova <[email protected]>
Co-authored-by: Niranjan Shankar <[email protected]>
Co-authored-by: jingyizhu99 <[email protected]>
Co-authored-by: Harry Yang <[email protected]>
Co-authored-by: Harry Yang <[email protected]>
Co-authored-by: Thomas Stringer <[email protected]>
Co-authored-by: NarayanThiru <[email protected]>
Co-authored-by: nreisch <[email protected]>
Co-authored-by: anagg929 <[email protected]>
Co-authored-by: Ji'an Liu <[email protected]>
Co-authored-by: nanthi <nanthi@NANTHI01>
Co-authored-by: youhuatuyh <[email protected]>
Co-authored-by: Youhua Tu <[email protected]>
Co-authored-by: Youhua Tu <[email protected]>
Co-authored-by: Jun <[email protected]>
Co-authored-by: Jun Min <[email protected]>
Co-authored-by: Sanya Kochhar <[email protected]>
Co-authored-by: sangitaray2021 <[email protected]>
Co-authored-by: Ganga Mahesh Siddem <[email protected]>
Co-authored-by: WangDian <[email protected]>
Co-authored-by: Bavneet Singh <[email protected]>
Co-authored-by: Niv Ben Shabat <[email protected]>
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.

2 participants