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

docker: Add docker-compose.yml and modify Dockerfile #109

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

Conversation

angad-k
Copy link
Member

@angad-k angad-k commented Jan 25, 2022

Modifies Dockerfile to actually run pinger instead of just running the version command. Adds a docker-compose.yml for easy startup.

@angad-k angad-k requested a review from h3llix January 25, 2022 18:02
docker-compose.yml Outdated Show resolved Hide resolved
docker-compose.yml Show resolved Hide resolved
Copy link
Contributor

@murex971 murex971 left a comment

Choose a reason for hiding this comment

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

make corresponding changes in docs folder

deploy/pinger.apache.conf Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
docs/src/contributing/build-process.md Outdated Show resolved Hide resolved
deploy/pinger.apache.conf Outdated Show resolved Hide resolved
agent.sample.yml Show resolved Hide resolved
Comment on lines +13 to +14
password: password
db_name: pinger
Copy link
Contributor

Choose a reason for hiding this comment

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

use env variables here as we discussed with viper

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.

3 participants