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: update base image version and update default configuration files #61

Merged
merged 1 commit into from
Jan 12, 2024

Conversation

njuguna-n
Copy link
Contributor

Updates related to this issue. The next step is to push the updated image to Docker Hub

@@ -2,7 +2,7 @@
input {
couchdb_changes {
always_reconnect => true
db => "couchdb"
db => "medic"
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't the purpose of COUCHDB_DBS to define a list of databases you want to sync? Can we use that environment variable here? However, it may only work when it contains one database e.g "medic" and not a list e.g "medic medic_sentinel"

Copy link
Member

@andrablaj andrablaj Jan 12, 2024

Choose a reason for hiding this comment

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

@njuguna-n can you please answer @lorerod's question above? I know you two had some conversations, and capturing the outcome for the public could benefit those who want to know more.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change is meant to update the default database that logstash listens to. @lorerod your last sentence is true as well, to be able to target multiple databases you need to do that on separate .conf files for example medic_sentinel should have a medic_sentinel.conf file.

@njuguna-n njuguna-n merged commit 674582d into main Jan 12, 2024
4 checks passed
@njuguna-n njuguna-n deleted the 38-update-logstash-image branch January 12, 2024 12:30
medic-ci pushed a commit that referenced this pull request Sep 10, 2024
# 1.0.0 (2024-09-10)

### Bug Fixes

* Change env variables according to cht pipeline updates ([#71](#71)) ([c89aadf](c89aadf))
* Fix numbering ([#50](#50)) ([5c93300](5c93300))

### Features

* **#107:** Adds multi-db watcher support ([#113](#113)) ([279d8f2](279d8f2)), closes [#107](#107) [#107](#107)
* **#112:** drop support for multiple copies of every document ([#115](#115)) ([b46f288](b46f288)), closes [#112](#112) [#118](#118)
* **#129:** add back automatic pipeline updates ([#130](#130)) ([fc73fd7](fc73fd7)), closes [#129](#129) [#129](#129) [#129](#129)
* **#1:** first release ([ff0fedd](ff0fedd)), closes [#1](#1)
* **#25:** custom databases ([#33](#33)) ([cd10db0](cd10db0)), closes [#25](#25)
* **#78:** full refresh on changed objects, only incremental runs continously ([0869ee9](0869ee9)), closes [#78](https://github.com/medic/cht-sync/issues/78)
* add versioning and releases ([a528aba](a528aba))
* bind sequence token path to host for persistence ([#88](#88)) ([e1c3953](e1c3953))
* remove superset container and update Readme ([#64](#64)) ([8acbc93](8acbc93))
* update logstash base image version and update default configuration files ([#61](#61)) ([674582d](674582d))
* update postgres version to 16 ([8bf1e84](8bf1e84))
@medic-ci
Copy link

🎉 This PR is included in version 1.0.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.

4 participants