Skip to content

containerapp remove managed environment id#6041

Merged
zhoxing-ms merged 10 commits intoAzure:mainfrom
Greedygre:xinyu/fix_patch_and_remove_managed_environment_id
Mar 21, 2023
Merged

containerapp remove managed environment id#6041
zhoxing-ms merged 10 commits intoAzure:mainfrom
Greedygre:xinyu/fix_patch_and_remove_managed_environment_id

Conversation

@Greedygre
Copy link
Contributor


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

Related command

az containerapp create
az containerapp update
az containerapp hostname

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 pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

@ghost ghost requested a review from yonzhan March 20, 2023 07:12
@ghost ghost added the Auto-Assign Auto assign by bot label Mar 20, 2023
@ghost ghost requested review from wangzelin007 and yanzhudd March 20, 2023 07:12
@ghost ghost assigned zhoxing-ms Mar 20, 2023
@ghost ghost added this to the March 2023 (2023-04-04) milestone Mar 20, 2023
@ghost ghost added the ContainerApp label Mar 20, 2023
@ghost ghost requested review from jsntcy and zhoxing-ms March 20, 2023 07:12
@yonzhan
Copy link
Collaborator

yonzhan commented Mar 20, 2023

containerapp

@Greedygre Greedygre changed the title containerapp remove managed environment containerapp remove managed environment id Mar 21, 2023
@zhoxing-ms
Copy link
Contributor

May I ask if this PR change will cause a breaking change for customers?

@Greedygre
Copy link
Contributor Author

@

May I ask if this PR change will cause a breaking change for customers?

This change will not cause breaking change.
We have done compatibility to ensure that customers' behavior will not be changed.

@zhoxing-ms zhoxing-ms merged commit 9348437 into Azure:main Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot ContainerApp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants