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

Wrong environment variable name for migrate and seed sections of makefile #356

Closed
cadabrum opened this issue Feb 27, 2024 · 1 comment
Closed

Comments

@cadabrum
Copy link

The sections below use the SALES_DB_HOST environment variable

migrate:
	export SALES_DB_HOST=localhost; go run app/tooling/sales-admin/main.go migrate

seed: migrate
	export SALES_DB_HOST=localhost; go run app/tooling/sales-admin/main.go seed

The commands fail with an error:

export SALES_DB_HOST=localhost; go run app/tooling/sales-admin/main.go migrate
msg migrating database: migrate database: status check database: context deadline exceeded
exit status 1
make: *** [migrate] Error 1

Сorrect variable name SALES_DB_HOST_PORT?

@cadabrum cadabrum changed the title Wrong environment variable name for migrate and seed sections in makefile Wrong environment variable name for migrate and seed sections of makefile Feb 27, 2024
@ardan-bkennedy
Copy link
Contributor

Yea you are right. I will fix this now. Thanks!

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

No branches or pull requests

2 participants