Skip to content

[DPE-10062] Switch to shared cluster code#1764

Draft
dragomirp wants to merge 6 commits into
16/edgefrom
dpe-10062-cluster
Draft

[DPE-10062] Switch to shared cluster code#1764
dragomirp wants to merge 6 commits into
16/edgefrom
dpe-10062-cluster

Conversation

@dragomirp

Copy link
Copy Markdown
Contributor

Issue

Solution

Checklist

  • I have added or updated any relevant documentation.
  • I have cleaned any remaining cloud resources from my accounts.

@dragomirp dragomirp added the enhancement New feature, UI change, or workload upgrade label Jun 11, 2026
@github-actions github-actions Bot added the Libraries: Out of sync The charm libs used are out-of-sync label Jun 11, 2026
@dragomirp dragomirp force-pushed the dpe-10062-cluster branch from 5670496 to 283b12e Compare June 11, 2026 15:48
@dragomirp dragomirp force-pushed the dpe-10062-cluster branch from 8c9d324 to 1fe33ca Compare June 11, 2026 19:51
marceloneppel added a commit that referenced this pull request Jun 12, 2026
Integration tests were importing constants via 'from constants import ...' (which eagerly imports
single_kernel_postgresql in its module body) and directly via 'from single_kernel_postgresql...
import'. The integration CI env runs 'poetry install --only integration --no-root' which does not
install the lib, causing ModuleNotFoundError at pytest collection time for every spread job.

Hardcode the needed constant values locally in each file, matching PR #1764's style, so no
integration file transitively imports the lib. One documented exception remains: test_charm.py
retains 'from single_kernel_postgresql.config.locales import LOCALES as SNAP_LOCALES' because it
asserts against a 510-entry locale list that cannot be practically hardcoded.

Signed-off-by: Marcelo Henrique Neppel <neppel2026@proton.me>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, UI change, or workload upgrade Libraries: Out of sync The charm libs used are out-of-sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant