Skip to content

[MISC] Check that the watcher is connected#18

Merged
dragomirp merged 5 commits into
16/edgefrom
start-checker
May 22, 2026
Merged

[MISC] Check that the watcher is connected#18
dragomirp merged 5 commits into
16/edgefrom
start-checker

Conversation

@dragomirp
Copy link
Copy Markdown
Contributor

@dragomirp dragomirp commented May 19, 2026 •
edited
Loading

  • Check RAFT connectivity when starting the observer
  • Revert watcher default port to 2222

@dragomirp dragomirp added the bug Something isn't working label May 19, 2026
dragomirp
dragomirp commented May 22, 2026
View reviewed changes
Comment thread src/raft_controller.py
dragomirp
dragomirp commented May 22, 2026
View reviewed changes
Comment thread templates/watcher.service.j2
dragomirp
dragomirp commented May 22, 2026
View reviewed changes
Comment thread src/raft_controller.py
dragomirp
dragomirp commented May 22, 2026
View reviewed changes
Comment thread src/raft_controller.py
@dragomirp dragomirp marked this pull request as ready for review May 22, 2026 11:15
@dragomirp dragomirp requested a review from a team as a code owner May 22, 2026 11:15
@dragomirp dragomirp requested review from carlcsaposs-canonical, juju-charm-bot, marceloneppel and taurus-forever and removed request for a team May 22, 2026 11:15
taurus-forever
taurus-forever previously approved these changes May 22, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@taurus-forever taurus-forever left a comment

Choose a reason for hiding this comment

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

Thank you!

dragomirp
dragomirp commented May 22, 2026
View reviewed changes
Comment thread src/relations/watcher_requirer.py
marceloneppel
marceloneppel previously approved these changes May 22, 2026
View reviewed changes
@dragomirp dragomirp dismissed stale reviews from marceloneppel and taurus-forever via 9247128 May 22, 2026 14:26
dragomirp
dragomirp commented May 22, 2026
View reviewed changes
Comment thread src/raft_controller.py
dragomirp
dragomirp commented May 22, 2026
View reviewed changes
Comment thread src/raft_controller.py
dragomirp
dragomirp commented May 22, 2026
View reviewed changes
Comment thread src/raft_controller.py
@dragomirp dragomirp merged commit 63d00ab into 16/edge May 22, 2026
41 of 45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@taurus-forever taurus-forever taurus-forever approved these changes

@marceloneppel marceloneppel marceloneppel approved these changes

@carlcsaposs-canonical carlcsaposs-canonical Awaiting requested review from carlcsaposs-canonical carlcsaposs-canonical is a code owner automatically assigned from canonical/data-postgresql

@juju-charm-bot juju-charm-bot Awaiting requested review from juju-charm-bot juju-charm-bot is a code owner automatically assigned from canonical/data-postgresql

Assignees

No one assigned

Labels

bug Something isn't working Libraries: OK

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dragomirp @taurus-forever @marceloneppel