Azure / mec-app-solution-accelerator

Application Solution Accelerator for Multi-access Edge Compute (MEC)
MIT License
37 stars 10 forks source link

[P1] Uninstall with Script should not throw errors #152

Closed CESARDELATORRE closed 1 month ago

CESARDELATORRE commented 1 month ago

Not critical, not urgent, but Uninstall with Script should not throw errors.

It should be handled consistently with no errors.

image
virubiotor commented 1 month ago

Fixed in ef2a4ed

CESARDELATORRE commented 1 month ago

OK, closing then.

CESARDELATORRE commented 1 month ago

Re-opening it. IT still throws errors in the latest version I just tested:

image

But this is NOT urgent, please, leave it for a time when we have finished all the priorities.

virubiotor commented 1 month ago

It was a race condition between the delete namespace and the other resources, I have change it so that we only delete the namespace and recursively all its resources. Fixed in 57d354d

CESARDELATORRE commented 1 month ago

This is fixed now. :)

image