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

How to clean up after running the example? #11

Open
yoDon opened this issue Aug 27, 2022 · 0 comments · May be fixed by #12
Open

How to clean up after running the example? #11

yoDon opened this issue Aug 27, 2022 · 0 comments · May be fixed by #12

Comments

@yoDon
Copy link

yoDon commented Aug 27, 2022

This issue is for a:

- [X] documentation issue or request

Expected/desired behavior

The last line of the README.md states

"Note: Don't forget to delete your resources, or at least, scale them way back. Thanks for trying this sample."

For those of us who are new to Azure Container Apps and who've been burned in the past from a cost perspective when expensive resources ended up still running after trying a sample like this, encountering that last line with zero guidance on how to either delete the resources or "scale them way back" does not make it feel very reassuring to try running this sample.

I think it's enough to do an az group delete --resource-group orleans-cluster-on-container-apps (or whatever you named your resource group to when editing deploy.yml to force the deployment) but I'm unclear on how to tackle the scaling down variant (is that just turning the WorkerServiceClient instances down to 0 or is there something else that should be scaled down?).

EDIT-UPDATE:

The Azure AD Application created by the az ad sp create-for-rbac ... command does not get deleted when you delete the resource group. Deleting the Azure AD Application should be possible using the cli via az ad sp delete ... but I was unable to successfully delete the created Azure AD app via the ci and needed to do it manually via the Azure portal UI (search for OrleansACASample in the portal to find it).

yoDon added a commit to yoDon/Orleans-Cluster-on-Azure-Container-Apps that referenced this issue Aug 28, 2022
@yoDon yoDon linked a pull request Aug 28, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant