janeczku / docker-machine-vultr

:m: Docker Machine driver for Vultr Cloud
MIT License
192 stars 25 forks source link

Add snapshot support #20

Closed twang2218 closed 7 years ago

twang2218 commented 7 years ago

Fixes #15

Signed-off-by: Tao Wang twang2218@gmail.com

twang2218 commented 7 years ago

ping @janeczku

janeczku commented 7 years ago

@twang2218 I think we should automatically set OSID 164 when the vultr-snapshot-id is specified?

twang2218 commented 7 years ago

Yes, the OSID should be 164 in case vultr-snapshot-id is specified. I update the commit to assign the OSID to 164 if it's 159 (defaultOS), so user still be able to assign another OSID, in case if there is a future changes in the API, and the driver code haven't been updated yet.

janeczku commented 7 years ago

Thanks so much @twang2218!