[DPE-6112] Pg16 tweaks (#893)

* Update dependency uv to v0.5.27 (#847)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* [DPE-6053] Fix ports (#846)

* Fix ports

Signed-off-by: Marcelo Henrique Neppel <[email protected]>

* Remove unused import

Signed-off-by: Marcelo Henrique Neppel <[email protected]>

* Remove unused charm library and fix unit tests

Signed-off-by: Marcelo Henrique Neppel <[email protected]>

* Fix identation

Signed-off-by: Marcelo Henrique Neppel <[email protected]>

* Bump libs

* Conditional set_ports logic

* Unit test

* Don't expect juju managed resources

---------

Signed-off-by: Marcelo Henrique Neppel <[email protected]>
Co-authored-by: Marcelo Henrique Neppel <[email protected]>

* Sync docs from Discourse (#801)

Co-authored-by: GitHub Actions <41898282+github-actions[bot]@users.noreply.github.com>

* Update dependency uv to v0.5.29 (#848)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* [MISC] Define charm utility properties (#842)

* Update data-platform-workflows to v29.1.0 (#849)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Use `charmcraft test` & concierge (#852)

* [MISC] Split topology script (#853)

* Update charmcraft.yaml build tools (#851)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update dependency uv to v0.5.31 (#858)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update canonical/data-platform-workflows action to v30 (#859)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Use _promote_charm.yaml (#860)

Use `charmcraft promote` and auto-generate release notes

* Update dependency cryptography to v44.0.1 [SECURITY] (#855)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Sync docs from Discourse (#854)

Co-authored-by: GitHub Actions <41898282+github-actions[bot]@users.noreply.github.com>

* Update data-platform-workflows to v30.0.2 (#863)

* Update canonical/has-signed-canonical-cla action to v2 (#865)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* [DPE-5827] Set all nodes to synchronous replicas (#784)

* Update patroni configuration

* Update test assertion

* Copy update_synchronous_node_count from VM

* Add unit test

* Set sync node count during upgrade

* Fix tls test

* Switchover primary

* Add different helper to get leader

* Add config boilerplate

* Use config value when setting sync node count

* Escape tuple

* Add policy values

* Add integration test

* Fix casting

* Fix test

* Update to spec

* Bump retry timout

* Switch to planned units

* Fix generator

* Update conf description

* Spread task

* Pass the charm

* [DPE-6484] Add scope to promote to primary (#850)

* Promote unit action

* Tweaks for failing REST calls

* VM parity

* [MISC] Enable RBAC in tests (#861)

* Enable RBAC

* Bump cosl

* Add trust

* Bump libs

* [MISC] Define charm constants (#862)

* Lock file maintenance Python dependencies (#835)

* Lock file maintenance Python dependencies

* Backoff boto3 1.36

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Dragomir Penev <[email protected]>

* Update canonical/data-platform-workflows action to v30.1.3 (#856)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update charmcraft.yaml build tools (#868)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Handle missing stanza output (#866)

* Update ghcr.io/canonical/charmed-postgresql:14.15-22.04_edge Docker digest to 71d49b6 (#870)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* [MISC] Disable storage test on arm (#872)

* Disable storage test on Arm

* Bump libs

* Correct import path

* Update charmcraft.yaml build tools (#871)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* [MISC] Add degraded status to primary message (#874)

* Add degraded status on to primary message

* Use const running tests

* [MISC] Sanitize PostgreSQL extra-user-roles arg (#876)

* [MISC] Fix PostgreSQL lib function signature (#879)

* [MISC] Skip backup tests without creds (#884)

* Bump libs

* Skip backups tests if no creds are set

* Check that the cloud config is set to a value

* Typo

* Update tests/integration/conftest.py

Co-authored-by: Carl Csaposs <[email protected]>

* Switch all checks to gets

* Typo

---------

Co-authored-by: Carl Csaposs <[email protected]>

* [DPE-6679] PostgreSQL Config Improvement (#867)

* config improvement

* config improvement

* config improvement

* config improvement

* config improvement

* config improvement

* config improvement

* config improvement

* config improvement

* config improvement

* Update dependency jinja2 to v3.1.6 [SECURITY] (#882)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update canonical/data-platform-workflows action to v30.2.0 (#887)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Reduce required approvals on Renovate pull requests by 1 (#880)

* Update dependency uv to v0.6.5 (#878)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update ghcr.io/canonical/charmed-postgresql Docker tag to v14.17 (#877)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update dependency uv to v0.6.7 (#890)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Update canonical/data-platform-workflows action to v31 (#891)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Try the new timescale

* Check PR on */edge

* Enable cache, disable pgaudit

* Use the right base

* Arm tests

---------

Signed-off-by: Marcelo Henrique Neppel <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Marcelo Henrique Neppel <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Sinclert Pérez <[email protected]>
Co-authored-by: Carl Csaposs <[email protected]>
Co-authored-by: Danylo Bereznenko <[email protected]>

Author: 7 people

.github/workflows/approve_renovate_pr.yaml

@@ -10,6 +10,6 @@ on:
 jobs:
   approve-pr:
     name: Approve Renovate pull request
-    uses: canonical/data-platform-workflows/.github/workflows/approve_renovate_pr.yaml@v30.2.0
+    uses: canonical/data-platform-workflows/.github/workflows/approve_renovate_pr.yaml@v31.0.0
     permissions:
       pull-requests: write  # Needed to approve PR

.github/workflows/check_pr.yaml

@@ -11,8 +11,9 @@ on:
       - edited
     branches:
       - main
+      - '*/edge'
 
 jobs:
   check-pr:
     name: Check pull request
-    uses: canonical/data-platform-workflows/.github/workflows/check_charm_pr.yaml@v30.2.0
+    uses: canonical/data-platform-workflows/.github/workflows/check_charm_pr.yaml@v31.0.0

.github/workflows/ci.yaml

@@ -27,7 +27,7 @@ on:
 jobs:
   lint:
     name: Lint
-    uses: canonical/data-platform-workflows/.github/workflows/lint.yaml@v30.2.0
+    uses: canonical/data-platform-workflows/.github/workflows/lint.yaml@v31.0.0
 
   unit-test:
     name: Unit test charm
@@ -49,9 +49,7 @@ jobs:
 
   build:
     name: Build charm
-    uses: canonical/data-platform-workflows/.github/workflows/[email protected]
-    with:
-      cache: false
+    uses: canonical/data-platform-workflows/.github/workflows/[email protected]
 
   integration-test:
     name: Integration test charm

.github/workflows/promote.yaml

@@ -25,7 +25,7 @@ on:
 jobs:
   promote:
     name: Promote charm
-    uses: canonical/data-platform-workflows/.github/workflows/_promote_charm.yaml@v30.2.0
+    uses: canonical/data-platform-workflows/.github/workflows/_promote_charm.yaml@v31.0.0
     with:
       track: '16'
       from-risk: ${{ inputs.from-risk }}

.github/workflows/release.yaml

@@ -44,7 +44,7 @@ jobs:
     name: Release charm
     needs:
       - ci-tests
-    uses: canonical/data-platform-workflows/.github/workflows/release_charm.yaml@v30.2.0
+    uses: canonical/data-platform-workflows/.github/workflows/release_charm.yaml@v31.0.0
     with:
       channel: ${{ github.ref_name }}
       artifact-prefix: ${{ needs.ci-tests.outputs.artifact-prefix }}

.github/workflows/sync_docs.yaml

@@ -10,7 +10,7 @@ on:
 jobs:
   sync-docs:
     name: Sync docs from Discourse
-    uses: canonical/data-platform-workflows/.github/workflows/sync_docs.yaml@v30.2.0
+    uses: canonical/data-platform-workflows/.github/workflows/sync_docs.yaml@v31.0.0
     with:
       reviewers: a-velasco
     permissions:

charmcraft.yaml

@@ -27,7 +27,7 @@ parts:
       PIP_BREAK_SYSTEM_PACKAGES=true python3 -m pip install --user --upgrade pip==25.0.1  # renovate: charmcraft-pip-latest
 
       # Use uv to install poetry so that a newer version of Python can be installed if needed by poetry
-      curl --proto '=https' --tlsv1.2 -LsSf https://github.com/astral-sh/uv/releases/download/0.6.5/uv-installer.sh | sh  # renovate: charmcraft-uv-latest
+      curl --proto '=https' --tlsv1.2 -LsSf https://github.com/astral-sh/uv/releases/download/0.6.7/uv-installer.sh | sh  # renovate: charmcraft-uv-latest
       # poetry 2.0.0 requires Python >=3.9
       if ! "$HOME/.local/bin/uv" python find '>=3.9'
       then

tests/integration/ha_tests/test_upgrade.py

@@ -12,7 +12,6 @@
 from pytest_operator.plugin import OpsTest
 from tenacity import Retrying, stop_after_attempt, wait_fixed
 
-from .. import markers
 from ..helpers import (
     APPLICATION_NAME,
     CHARM_BASE,
@@ -36,8 +35,6 @@
 TIMEOUT = 600
 
 
-# No arm edge
-@markers.amd64_only
 @pytest.mark.abort_on_fail
 async def test_deploy_latest(ops_test: OpsTest) -> None:
     """Simple test to ensure that the PostgreSQL and application charms get deployed."""
@@ -68,8 +65,6 @@ async def test_deploy_latest(ops_test: OpsTest) -> None:
     assert len(ops_test.model.applications[DATABASE_APP_NAME].units) == 3
 
 
-# No arm edge
-@markers.amd64_only
 @pytest.mark.abort_on_fail
 async def test_pre_upgrade_check(ops_test: OpsTest) -> None:
     """Test that the pre-upgrade-check action runs successfully."""
@@ -96,8 +91,6 @@ async def test_pre_upgrade_check(ops_test: OpsTest) -> None:
     assert stateful_set.spec.updateStrategy.rollingUpdate.partition == 2, "Partition not set to 2"
 
 
-# No arm edge
-@markers.amd64_only
 @pytest.mark.abort_on_fail
 async def test_upgrade_from_edge(ops_test: OpsTest, charm, continuous_writes) -> None:
     # Start an application that continuously writes data to the database.
@@ -160,8 +153,6 @@ async def test_upgrade_from_edge(ops_test: OpsTest, charm, continuous_writes) ->
     )
 
 
-# No arm edge
-@markers.amd64_only
 @pytest.mark.abort_on_fail
 async def test_fail_and_rollback(ops_test, charm, continuous_writes) -> None:
     # Start an application that continuously writes data to the database.

tests/integration/ha_tests/test_upgrade_from_stable.py

@@ -13,6 +13,7 @@
 from ..helpers import (
     APPLICATION_NAME,
     CHARM_BASE,
+    CHARM_BASE_NOBLE,
     DATABASE_APP_NAME,
     METADATA,
     count_switchovers,
@@ -34,15 +35,14 @@
 @pytest.mark.abort_on_fail
 async def test_deploy_stable(ops_test: OpsTest) -> None:
     """Simple test to ensure that the PostgreSQL and application charms get deployed."""
-    # TODO remove once we release to stable
-    pytest.skip("No 16/stable yet.")
     await asyncio.gather(
         ops_test.model.deploy(
             DATABASE_APP_NAME,
             num_units=3,
-            channel="16/stable",
+            # TODO move to stable once we release
+            channel="16/beta",
             trust=True,
-            base=CHARM_BASE,
+            base=CHARM_BASE_NOBLE,
         ),
         ops_test.model.deploy(
             APPLICATION_NAME,
@@ -62,8 +62,6 @@ async def test_deploy_stable(ops_test: OpsTest) -> None:
 @pytest.mark.abort_on_fail
 async def test_pre_upgrade_check(ops_test: OpsTest) -> None:
     """Test that the pre-upgrade-check action runs successfully."""
-    # TODO remove once we release to stable
-    pytest.skip("No 16/stable yet.")
     application = ops_test.model.applications[DATABASE_APP_NAME]
     if "pre-upgrade-check" not in await application.get_actions():
         logger.info("skipping the test because the charm from 14/stable doesn't support upgrade")
@@ -95,8 +93,6 @@ async def test_pre_upgrade_check(ops_test: OpsTest) -> None:
 @pytest.mark.abort_on_fail
 async def test_upgrade_from_stable(ops_test: OpsTest, charm):
     """Test updating from stable channel."""
-    # TODO remove once we release to stable
-    pytest.skip("No 16/stable yet.")
     # Start an application that continuously writes data to the database.
     logger.info("starting continuous writes to the database")
     await start_continuous_writes(ops_test, DATABASE_APP_NAME)

tests/integration/test_backups_aws.py

@@ -7,7 +7,6 @@
 from pytest_operator.plugin import OpsTest
 from tenacity import Retrying, stop_after_attempt, wait_exponential
 
-from . import architecture
 from .conftest import AWS
 from .helpers import (
     DATABASE_APP_NAME,
@@ -27,7 +26,7 @@
 FAILED_TO_INITIALIZE_STANZA_ERROR_MESSAGE = "failed to initialize stanza, check your S3 settings"
 S3_INTEGRATOR_APP_NAME = "s3-integrator"
 tls_certificates_app_name = "self-signed-certificates"
-tls_channel = "latest/edge" if architecture.architecture == "arm64" else "latest/stable"
+tls_channel = "latest/stable"
 tls_config = {"ca-common-name": "Test CA"}
 
 logger = logging.getLogger(__name__)