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.

Sign up for GitHub

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

Jump to bottom

Add test cover for "az containerapp service XX delete" commands #6302

Closed
yash-nisar opened this issue May 18, 2023 · 2 comments · Fixed by #6327
Closed

Add test cover for "az containerapp service XX delete" commands #6302

yash-nisar opened this issue May 18, 2023 · 2 comments · Fixed by #6327
Assignees
@yash-nisar
Labels
Auto-Assign Auto assign by bot bug This issue requires a change to an existing behavior in the product in order to be resolved. ContainerApp CXP Attention This issue is handled by CXP team.

Comments

@yash-nisar
Copy link
Contributor

  • If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

Related command

az containerapp service redis delete
az containerapp service postgres delete

Extension name (the extension in question)

containerapp

Description of issue (in as much detail as possible)

Add test cover for the following commands:
az containerapp service redis delete
az containerapp service postgres delete
@ghost ghost added Auto-Assign Auto assign by bot ContainerApp labels May 18, 2023
@yonzhan
Copy link
Collaborator

yonzhan commented May 18, 2023

Thank you for opening this issue, we will look into it.

@ghost ghost added the CXP Attention This issue is handled by CXP team. label May 18, 2023
@yash-nisar yash-nisar mentioned this issue May 18, 2023
Merged
3 tasks
@yash-nisar
Copy link
Contributor Author

@yonzhan Can you assign it to me ?

yash-nisar added a commit to yash-nisar/azure-cli-extensions that referenced this issue May 25, 2023
@yash-nisar
Include test coverage for "containerapp service redis/postgres delete"
d12caad 
Closes Azure#6302
@yash-nisar yash-nisar mentioned this issue May 25, 2023
Merged
3 tasks
zhoxing-ms pushed a commit that referenced this issue May 26, 2023
@yash-nisar
Include test coverage for "containerapp service redis/postgres delete" (
ec36c77 
#6327)

Closes #6302
@navba-MSFT navba-MSFT added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yash-nisar yash-nisar

Labels
Auto-Assign Auto assign by bot bug This issue requires a change to an existing behavior in the product in order to be resolved. ContainerApp CXP Attention This issue is handled by CXP team.
Projects
None yet
Milestone
No milestone
3 participants
@yash-nisar @yonzhan @navba-MSFT