search

dice-cyfronet / atmosphere

Atmosphere cloud platform
MIT License
7 stars 3 forks source link

Move destroy VM on compute site from VM into VmDestroyWorker #69

Open mkasztelnik opened 9 years ago

mkasztelnik commented 9 years ago

Right now all logic responsible for destroying VM on compute site is located inside Atmosphere::VirtualMachine model class. I propose to move this logic from Atmosphere::VirtualMachine into Atmosphere::Cloud::VmDestroyWorker. As a conclusion ORM VM model manipulation will not have any side effects.

@tbartynski, @nowakowski, @bwilk - what do you think?

nowakowski commented 9 years ago

Yes, this seems sensible.

tbartynski commented 9 years ago

OK but this is a minor issue. Let's do that if there is nothing more important to do.