Adding the pause, hibernate and resume features between the AWS and F…#759
Adding the pause, hibernate and resume features between the AWS and F…#759raissonsouto wants to merge 3 commits into
Conversation
jadsonluan
left a comment
There was a problem hiding this comment.
Oi, @raissonsouto !
Primeiramente, bom trabalho. O código está muito bem organizado e testado. 🚀
Tem um pequeno problema que impede de testar os métodos que você implementou na cloud: os métodos isPaused e isHibernated não estão implementados, dessa forma uma instância nunca sai de PAUSING/HIBERNATING porque esses métodos sempre retornam false.
Você precisa implementar eles e adicionar os estados na classe AwsV2StateMapper (dá uma olhada no da google cloud), dai para isso seria interessante ver como se chamam os estados de quando uma VM está pausada/hibernada.
Quando você corrigir isso, solicita novamente a revisão da PR. Qualquer dúvida sobre me manda uma mensagem no discord ou no slack.
|
Certo! Vou fazer essas alterações. |
No description provided.