Skip to content

[DPE-7316] Stereo mode - split watcher charm#2

Closed
dragomirp wants to merge 10 commits into
mainfrom
16/dev
Closed

[DPE-7316] Stereo mode - split watcher charm#2
dragomirp wants to merge 10 commits into
mainfrom
16/dev

Conversation

@dragomirp
Copy link
Copy Markdown
Contributor

@dragomirp dragomirp commented May 5, 2026

Split of of canonical/postgresql-operator#1664.

  • Watcher side of the relation with the role config removed
  • Repo boilerplate

@dragomirp dragomirp added the enhancement New feature or request label May 5, 2026
@dragomirp dragomirp force-pushed the 16/dev branch 3 times, most recently from e811474 to bef034c Compare May 5, 2026 08:59
@dragomirp dragomirp changed the title 16/dev [DPE-7316] Stereo mode - split watcher charm May 5, 2026
on:
push:
branches:
- '*/edge'
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Assuming that the default branch will be 16/edge

Comment thread .github/zizmor.yaml
Copy link
Copy Markdown
Contributor Author

@dragomirp dragomirp May 5, 2026

Choose a reason for hiding this comment

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

Copy pasted from the Postgresql operator repo.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Overkill? We only have one config using this.

Comment thread src/config.py
class CharmConfig(BaseConfigModel):
"""Manager for the structured configuration."""

profile: Literal["testing", "production"]
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Pulling the data model charm lib.

Comment thread src/charm.py
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No checks set for refreshing.

@dragomirp dragomirp marked this pull request as ready for review May 5, 2026 22:14
@dragomirp dragomirp requested review from a team and taurus-forever and removed request for a team May 5, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Libraries: OK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants