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

containerapp Version 0.3.4 Release #4794

Merged
merged 35 commits into from
May 13, 2022

Conversation

StrawnSC
Copy link
Contributor

@StrawnSC StrawnSC commented May 9, 2022


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

Related command

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.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

@ghost ghost requested a review from yonzhan May 9, 2022 20:52
@ghost ghost added the Auto-Assign Auto assign by bot label May 9, 2022
@ghost ghost requested a review from wangzelin007 May 9, 2022 20:52
@ghost ghost assigned zhoxing-ms May 9, 2022
@ghost ghost added this to the May 2022 (2022-05-24) milestone May 9, 2022
@ghost ghost added the ContainerApp label May 9, 2022
@ghost ghost requested review from zhoxing-ms and jsntcy May 9, 2022 20:52
@yonzhan
Copy link
Collaborator

yonzhan commented May 9, 2022

containerapp

@StrawnSC StrawnSC changed the title containerapp May 11 Release containerapp Version 0.3.4 Release May 10, 2022
calvinsID and others added 11 commits May 12, 2022 09:34
* Change to use stable api version

* More helpful error message when managedEnvironmentId is msising

* Edit error message

* Fix error message
* Finished label and revision work.

* Finished dynamic weight update work.

* Fixed small bug.

* Added more items to revision table output.

* Use patch instead of create or update API for traffic set.

* Fixed small bug related to floats not rounding.

* Added help. Updated resourcenotfounderror. Fixed existing bug related to no name given for revision command and latest passed. Added revision name inference from --revision to revision label add.

* Fixed style.

Co-authored-by: Haroon Feisal <[email protected]>
@StrawnSC StrawnSC marked this pull request as ready for review May 12, 2022 21:17
Comment on lines 189 to 202
c.argument('traffic_weights', nargs='*', options_list=['--traffic-weight'], help="A list of revision weight(s) for the container app. Space-separated values in 'revision_name=weight' format. For latest revision, use 'latest=weight'")
c.argument('revision_weights', nargs='+', options_list=['--revision-weight'], help="A list of revision weight(s) for the container app. Space-separated values in 'revision_name=weight' format. For latest revision, use 'latest=weight'")
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we use the deprecate_info flag to the traffic_weights parameter to set deprecation information first to avoid directly causing breaking change to users?

Copy link
Contributor Author

@StrawnSC StrawnSC May 13, 2022

Choose a reason for hiding this comment

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

@zhoxing-ms essentially this renamed to --revision-weight to reduce confusion with the new parameter --label-weight. To get around the need for a deprecation, I can add --traffic-weight as an option for --revision-weight

Copy link
Contributor

Choose a reason for hiding this comment

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

I suggest using the deprecated_info for --traffic-weight if the --traffic-weight parameter is no longer recommended in the future, what do you think of it?

Copy link
Contributor Author

@StrawnSC StrawnSC May 13, 2022

Choose a reason for hiding this comment

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

Sounds good. I just added --traffic-weight as a deprecated argument option

@StrawnSC StrawnSC requested a review from zhoxing-ms May 13, 2022 06:05
@StrawnSC
Copy link
Contributor Author

@zhoxing-ms just made the two changes in response to your comments

@zhoxing-ms zhoxing-ms merged commit 6da2c72 into Azure:main May 13, 2022
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