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

webapp : Migrate vsts dependency to azure-devops #14821

Closed
arrownj opened this issue Aug 18, 2020 · 12 comments
Closed

webapp : Migrate vsts dependency to azure-devops #14821

arrownj opened this issue Aug 18, 2020 · 12 comments
Assignees
Labels
App Services az appservice feature-request Service Attention This issue is responsible by Azure service team.

Comments

@arrownj
Copy link
Contributor

arrownj commented Aug 18, 2020

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Currently appservice depends on vsts which is not maintained anymore. There will be no further support for this package if there are bugs for feature requests. The vsts package is migrated to azure-devops.

This is a request to migrate vsts to azure-devops in CLI side.

Describe alternatives you've considered

Additional context

@ghost ghost added the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Aug 18, 2020
@arrownj arrownj added App Services az appservice Service Attention This issue is responsible by Azure service team. labels Aug 18, 2020
@ghost ghost removed the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Aug 18, 2020
@ghost
Copy link

ghost commented Aug 18, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @antcp, @AzureAppServiceCLI.

@arrownj
Copy link
Contributor Author

arrownj commented Aug 18, 2020

Hi @panchagnula , could you please help check this ? Thanks

@ThejaChoudary ThejaChoudary changed the title Migrate vsts dependency to azure-devops AppService : Migrate vsts dependency to azure-devops Aug 18, 2020
@panchagnula
Copy link
Contributor

ACK @arrownj will take a look just to be sure is this the one you are referring to https://github.com/Azure/azure-cli/blob/dev/src/azure-cli/azure/cli/command_modules/appservice/vsts_cd_provider.py

Can you point me to the new vets package on devops so I can update the references & test on our end? Thanks!

@panchagnula panchagnula self-assigned this Aug 18, 2020
@panchagnula panchagnula added this to the S175 - For Ignite milestone Aug 18, 2020
@yungezz yungezz modified the milestones: S175 - For Ignite, S176 Sep 16, 2020
@yonzhan yonzhan removed this from the S176 milestone Oct 2, 2020
@btardif btardif added this to the S178 milestone Oct 21, 2020
@btardif
Copy link
Member

btardif commented Oct 21, 2020

@panchagnula is this still relevant?

@btardif btardif modified the milestones: S178, S179 Oct 28, 2020
@panchagnula panchagnula changed the title AppService : Migrate vsts dependency to azure-devops webapp : Migrate vsts dependency to azure-devops Oct 28, 2020
@panchagnula panchagnula modified the milestones: S179, S180 Dec 2, 2020
@Kotasudhakarreddy
Copy link
Contributor

Kotasudhakarreddy commented Dec 16, 2020

ACK @arrownj will take a look just to be sure is this the one you are referring to https://github.com/Azure/azure-cli/blob/dev/src/azure-cli/azure/cli/command_modules/appservice/vsts_cd_provider.py

Can you point me to the new vets package on devops so I can update the references & test on our end? Thanks!

@arrownj - could you please respond here.

@yonzhan yonzhan modified the milestones: S180, S181 Dec 26, 2020
@yonzhan yonzhan modified the milestones: S181, S182 Dec 26, 2020
@qwordy
Copy link
Member

qwordy commented Jan 20, 2021

Hi Kota,

I read the issue, read the code and searched information.
We advise to deprecate vsts-cd-manager and use azure-devops instead. vsts-cd-manager is not maintained now.

Please search
from .vsts_cd_provider import VstsContinuousDeliveryProvider
from vsts_cd_manager.continuous_delivery_manager import ContinuousDeliveryManager
in code.

https://pypi.org/project/vsts-cd-manager/
https://github.com/microsoft/vsts-cd-manager
https://pypi.org/project/azure-devops/
https://github.com/microsoft/azure-devops-python-api

@Kotasudhakarreddy
Copy link
Contributor

@qwordy - thanks. Will check and get back to you for any clarifications.

@Kotasudhakarreddy Kotasudhakarreddy modified the milestones: S182, S181 Jan 26, 2021
@yonzhan yonzhan removed this from the S182 milestone Feb 6, 2021
@Kotasudhakarreddy Kotasudhakarreddy added this to the S183 - For Ignite milestone Feb 17, 2021
@qwordy
Copy link
Member

qwordy commented Feb 22, 2021

Hi @Kotasudhakarreddy , is there any update?

@Kotasudhakarreddy
Copy link
Contributor

@qwordy I am on it. Targeted for S183.

@yonzhan yonzhan removed this from the S183 - For Ignite milestone Feb 26, 2021
@panchagnula panchagnula added this to the S184 milestone Mar 1, 2021
@panchagnula
Copy link
Contributor

@calvinsID assigning this to you to help identify the right functions with the new library for parity
this file needs to be updated
from vsts_cd_manager.continuous_delivery_manager import ContinuousDeliveryManager
https://github.com/Azure/azure-cli/blob/dev/src/azure-cli/azure/cli/command_modules/appservice/vsts_cd_provider.py

Old: https://github.com/microsoft/vsts-cd-manager
New: https://github.com/microsoft/azure-devops-python-api

@calvinsID
Copy link
Contributor

Created this item to track moving az functionapp devops-pipeline create command, which pulls in vsts package, to an extension to reduce weight: #18228

@panchagnula panchagnula modified the milestones: S188, S189 Jun 11, 2021
@yonzhan yonzhan removed this from the S189 milestone Jul 2, 2021
@seligj95
Copy link
Contributor

seligj95 commented Jul 9, 2021

this is complete

@seligj95 seligj95 closed this as completed Jul 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App Services az appservice feature-request Service Attention This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests