search

Azure / draft-classic

A tool for developers to create cloud-native applications on Kubernetes.
https://draft.sh
MIT License
3.92k stars 395 forks source link

draft delete should delete docker related artifacts #382

Open michelleN opened 7 years ago

michelleN commented 7 years ago

Opening this issue to keep track of discussion in #379

As @fibonacci1729 brought up, at the moment, draft delete deletes a helm release but doesn't do anything with the Docker artifacts that get exist. There are implications of deleting this such as deleting layers that other images rely on. There are also different strategies for going about doing this in an intelligent manner as @bacongobbler mentioned. See above PR thread for more info.

bacongobbler commented 6 years ago

This might be nice under a feature flag or at least by gating with a yes/no prompt. The user should explicitly opt into this since docker images stored in the daemon and the registry are being used across other tools like what's implied in #541. :)