Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: reescreve o nomad_bootstrap.sh em pyhton #18

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Rehzende
Copy link
Contributor

@Rehzende Rehzende commented Dec 6, 2021

Reescreve o script de bootstrap do nomad em pyhton

  • Garanta que seu topic/feature/bugfix branch tenha uma branch nomeada e não a sua branch main esteja no PR
  • Dê um titulo que expresse o objetivo do PR
  • Associe seu PR a uma Issue criada no repositósito. Caso seja uma correção de linguagem ou pequenas correções, não é necessário
  • Descreva o objetivo do PR
  • Inclua links relevantes para a sua modificação/sugestão/correção
  • Descreva um passo-a-passo para testar o seu PR

Issue

close #12

Objetivo

Referências

Como testar

Dentro da pasta templates:
Crie dois arquivos, server.hcl.tpl e client.hcl.tpl

e cole o código abaixo.

client.hcl.tpl

{% include "nomad.hcl.j2" %}

client {
  enabled = true

  server_join {
    retry_join = [<RETRY_JOIN>]
  }
}

server.hcl.tpl

{% include "nomad.hcl.j2" %}

server {
  enabled          = true
  bootstrap_expect = <BOOTSTRAP_EXPECT>

  server_join {
    retry_join = [<RETRY_JOIN>]
  }
}

Rode os comando abaixo:

python3 nomad_bootstrap.py server  -b 3 -r "provider=gce project_name=meu-projeto tag_value=nomad-server" --nomad_path .

e 

python3 nomad_bootstrap.py client -r "provider=gce project_name=meu-projeto tag_value=nomad-server" --nomad_path ./

Os arquivos terão alguma propiedades substituidas.

Rehzende and others added 2 commits November 24, 2021 22:00
reecreve o script de bootstrap do nomad em pyhton

Co-authored-by: Thiago Finardi <tfinardi@gmail.com>
Co-authored-by: Felipe Nobrega <lipenodias@gmail.com>
Co-authored-by: Marcelo Mansur mansur.ufmg@gmail.com
Co-authored-by: Luiz Aoqui <lgfa29@gmail.com>
Co-authored-by: Danilo Rocha<snifbr@gmail.com>
Copy link
Contributor

@lgfa29 lgfa29 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ficou bem legal o script! Acho que vai ajudar bastante na hora de usar e adicionar variáveis novas 🙂

Deixei alguns comentários mais de formatação e organização de código. Tenho também mais umas outras ideias que a gente pode ir adicionando aos poucos depois também.

O CI tá quebrando porque precisa fazer algumas modificações a mais:

E também está faltando uma entrada no CHANGELOG 😉

templates/nomad_bootstrap.py Outdated Show resolved Hide resolved
templates/nomad_bootstrap.py Outdated Show resolved Hide resolved
templates/nomad_bootstrap.py Outdated Show resolved Hide resolved
@Rehzende Rehzende requested a review from lgfa29 March 9, 2022 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Precisamos tornar mais simples os inputs do script de inicialização
3 participants