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

feat(#99): add couch2pg exporter compose file to run stand alone without watchdog #102

Merged
merged 5 commits into from
May 10, 2024

Conversation

mrjones-plip
Copy link
Contributor

@mrjones-plip mrjones-plip commented May 3, 2024

This PR adds YAML files to allow watchdog to monitor a remote postgres instances for couch2pg backlog but not need any postgress credentials or have an SSH tunnel to the server. see medic/cht-docs#1378 for steps to test

per #99

@mrjones-plip mrjones-plip marked this pull request as ready for review May 3, 2024 23:08
@mrjones-plip mrjones-plip changed the title feat(#99): Add compose file for couch2pg exporter running stand alone without watchdog feat(#99): add couch2pg exporter compose file to run stand alone without watchdog May 4, 2024
Copy link
Collaborator

@jkuester jkuester left a comment

Choose a reason for hiding this comment

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

My only concern around this approach is with the extra complexity of remembering which compose files to include when running Docker commands. 😓 However, I think this is the right approach and we should look into simplifying docker commands in a separate task: #103

exporters/postgres/scrape-custom.yml Outdated Show resolved Hide resolved
exporters/postgres/scrape-only.yml Outdated Show resolved Hide resolved
.gitignore Show resolved Hide resolved
@mrjones-plip mrjones-plip merged commit 7817175 into main May 10, 2024
3 checks passed
@mrjones-plip mrjones-plip deleted the stand-alone-couch2pg-backlog-99 branch May 10, 2024 19:23
@medic-ci
Copy link
Collaborator

🎉 This PR is included in version 1.13.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants