search

pkusys / TGS

Artifacts for our NSDI'23 paper TGS
Apache License 2.0
69 stars 20 forks source link

[SUCK] TGS remove all docker containers without ANY notice #9

Open Naplesoul opened 1 month ago

Naplesoul commented 1 month ago

worker.py line 243 wrote: subprocess.call('docker rm $(docker ps -aq)', shell=True) I totally cannot understand why TGS has to remove all the containers on my machine. And also, if you really have to do that, you should notify the user before you doing that, but I did not see any notice from the shell or in your documents. It is unacceptable to me because important data in my containers are also removed. I think you guys did a terrible job of making your system open source.

dracit7 commented 1 month ago

Terrifying.

Just imagine someone, for reproducing the results in the paper, running this catastrophic script on a multi-tenant server...