search

Rapha-Borges / oke-free

Uma maneira fácil de garantir seu próprio cluster Kubernetes gratuitamente e para sempre
GNU General Public License v3.0
891 stars 381 forks source link

fix: comandos criacao cluster #7

Closed muritadb closed 10 months ago

muritadb commented 10 months ago

Alterei a lista de comandos para a criação do Cluster, observei que na live você utilizou um comando que não estava na lista originalmente. Adicionei o comando e alterei a ordem 🙂.

albertofp commented 10 months ago

Não previsa do plan, fazendo apply vc já tem a relação de toda a infra que vai ser criada e pede pra confirmar antes de criar qualquer coisa.

Mas sim se quiser ter isso num arquivo blz, só é opcional mesmo

Rapha-Borges commented 10 months ago

Agradeço pela sua contribuição.

Em termos de boas práticas, acredito que seja mais apropriado utilizar o terraform plan. No entanto, optei pelo terraform apply para simplificar o processo de implantação para aqueles que não estão familiarizados com o funcionamento do Terraform.

Vou manter o PR aberto, aceitando sugestões sobre qual seria a abordagem mais adequada.

badtuxx commented 10 months ago

Acredito que pode ser modificado, pois o apply não está usando -auto-approve, portanto o apply vai executar o plan no inicio da execução. Porém, caso queira ter certeza do que será feito, aconselho a ter o plan separado e passar o -auto-approve no apply.

Rapha-Borges commented 10 months ago

Seguindo a ideia do @badtuxx fiz esse exemplo

image

muritadb commented 10 months ago

@Rapha-Borges iria sugerir um comando opcional para esse passo mesmo, até para quem (EU incluso) não entende muito dos comandos.

badtuxx commented 10 months ago

@Rapha-Borges Nao nao, na verdade é o plan e depois o apply com auto approve. Rodar o apply com auto approve sem o plan antes é arriscado demais.

Rapha-Borges commented 10 months ago

@badtuxx Ah sim, erro meu!! Mas ainda não está no repositório, somente um exemplo.

@muritadb Acha que seguindo o exemplo que mandei (corrigindo os comandos, conforme o Jeferson pontuou) ficou claro? Se sim, pode alterar no seu PR para fazermos o merge.

muritadb commented 10 months ago

@Rapha-Borges @badtuxx Fui alterar as mudanças e na verdade criei uma nova PR, foi mal galera

muritadb commented 10 months ago

Link para a PR: https://github.com/Rapha-Borges/oke-free/pull/16