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

fix: correção de bug no Makefile - remoção da palavra 'COMPANIES' das variaveis POSTGRES #78

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

thisiscleverson
Copy link
Contributor

Correção de bug na variável de ambiente no comando start-database.

alterei os nomes das variáveis de ambientes no comando start-database do Makefile de:

-e POSTGRES_COMPANIES_PASSWORD=$(POSTGRES_COMPANIES_PASSWORD) \
-e POSTGRES_COMPANIES_USER=$(POSTGRES_COMPANIES_USER) \
-e POSTGRES_COMPANIES_DB=$(POSTGRES_COMPANIES_DB) \

Para:

-e POSTGRES_PASSWORD=$(POSTGRES_COMPANIES_PASSWORD) \
-e POSTGRES_USER=$(POSTGRES_COMPANIES_USER) \
-e POSTGRES_DB=$(POSTGRES_COMPANIES_DB) \

Por que da alterações?

Ao executar o comando run, o script dava um erro do tipo:

wait-for-it: timeout occurred after waiting 90 seconds for localhost:5432
make: *** [Makefile:175: wait-database] Error 124

vi que no pr (#76 ) as variáveis de ambientes foram alteradas. E as variáveis que cria a imagem doo banco de dados também foram alteradas com a palavra COMPANIES, isso impedia que o banco de dados fosse iniciado.

@thisiscleverson thisiscleverson changed the title fix: remoção da palavra 'COMPANIES' das variaveis POSTGRES fix: correção de bug no Makefile - remoção da palavra 'COMPANIES' das variaveis POSTGRES Dec 6, 2024
@ogecece
Copy link
Member

ogecece commented Dec 11, 2024

@thisiscleverson você não passa frio, pois está coberto de razão! Reproduzi aqui o problema e a solução. Valeu!

@ogecece ogecece merged commit 292bcfb into okfn-brasil:main Dec 11, 2024
1 check passed
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.

2 participants