Skip to content

Conversation

@leonard520
Copy link
Contributor

@leonard520 leonard520 commented May 8, 2020

Add command "az spring-cloud app identity" to support Managed Identity feature


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.

@leonard520 leonard520 changed the title spring-cloud: add support to enable/disable MSI (#1523) [DO-NOT-MERGE]spring-cloud: add support to enable/disable MSI (#1523) May 8, 2020
@yungezz yungezz requested review from fengzhou-msft and qwordy May 8, 2020 23:42
@yungezz yungezz added Spring Cloud Spring Cloud related issues do-not-merge labels May 8, 2020
@yungezz
Copy link
Member

yungezz commented May 8, 2020

hi @leonard520 is there any change between this PR with previous PR? that one was reviewed and approved already

@leonard520
Copy link
Contributor Author

hi @leonard520 is there any change between this PR with previous PR? that one was reviewed and approved already

They are the same except I raise the version number to 0.2.4 in this one

@yungezz
Copy link
Member

yungezz commented May 9, 2020

thanks for clarification. Then just ping us when the PR is ready to merge, thanks. @leonard520

@leonard520
Copy link
Contributor Author

@yungezz @fengzhou-msft The backend is ready. Please help to merge the code. Thanks!

@leonard520 leonard520 changed the title [DO-NOT-MERGE]spring-cloud: add support to enable/disable MSI (#1523) spring-cloud: add support to enable/disable MSI (#1523) May 27, 2020
@yonzhan yonzhan added this to the S170 milestone May 27, 2020
@qwordy
Copy link
Member

qwordy commented May 27, 2020

Could you please add a brief description for this PR and link the previous same PR that is reviewed?

@leonard520
Copy link
Contributor Author

leonard520 commented May 27, 2020

#1523

@qwordy
Copy link
Member

qwordy commented May 27, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@leonard520
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 1671 in repo Azure/azure-cli-extensions

@qwordy qwordy merged commit f1734ee into Azure:master May 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Spring Cloud Spring Cloud related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants