Add `test_smoke` test #469

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.

Sign up for GitHub

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

Jump to bottom

Closed

BON4 wants to merge 332 commits into canonical:main from BON4:test-charmed-postgresql-k8s-smoke-test

Contributor

BON4 commented May 4, 2024

Issue #468

This pull request is based on the same PR but for VM postgresql-operator

All this cases should be covered:

Test Application. VM resources is removed after application removal. Install app, remove app (removal test), check no garbage left.
Test Unit. Re-Installation. Remove unit with --force --no-wait, storage should be detached.
Test Unit. Garbage ignorance. Charm should deploy in dirty environment with garbage storage.
Test Application. Application should deploy in dirty environment with garbage storage from another application.

marceloneppel and others added 30 commits

March 11, 2023 00:15


          Mark unstable tests (canonical#108)

8beaf9d

* Mark unstable tests

* Reorganize TLS test to use unstable mark

* Mark test_redeploy_charm_same_model as unstable


          Fix duplicate K8S service ports names (canonical#106)

8624cf1

* Fix duplicate K8S service ports names

* Fix expected resources list


          Fix action password key (canonical#109)

ab2871f


          Fix AWS bucket name and region (canonical#111)

558010a


          [MISC] Add database field in the provider-side databag (canonical#112)

7cf7f57

* Update postgresql_provider.py

* fix linting

---------

Co-authored-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>


          Don't consider failed backups when restoring (canonical#113)

f207cf2


          Add IP address to DNS SANs list (canonical#117)

70fbe6e


          Add resource creation right before check about k8s primary service (c…

5dd72b7

…anonical#116)


          Enable backup on replicas and hide replica from PgBouncer while doing…

4c68dba

… backup (canonical#114)

* Initial code for backing up on replicas

* Add initial TLS settings on primary

* Adjust pgBackRest configuration to use standbys

* Implement working TLS server

* Fix TLS errors

* Fix stanza retrieval, restore test and unit test

* Enable Mattermost test

* Fix backup settings check

* Revert S3 settings

* Add TLS check

* Reorganize TLS server logic

* Fix multiple primaries situation

* Remove log calls

* Turn off connectivity only on replicas

* Use planned units when checking whether there are replicas

* Revert last change

* Improve TLS flag documentation

* Improve logic that controls backups on standbys


          Add Tutorial and Landing page and update README and CONTRIBUTING guid…

5dea18a

…es (canonical#115)

* Add Tutorial and Landing page and update README and CONTRIBUTING guides


          Merge mermaid diagrams into Discource documentation source (canonical…

36eba8a

…#119)


          Configure Renovate (canonical#122)

3a28787

* Add renovate.json

* Team config

* Regex tweaks

* Pin exact versions

* Scaledown includes

---------

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


          Backup diagrams (canonical#123)

efec44d

* Add On S3 Credentials Changed Hook diagram

* Add On Restore Hook diagram

* Add On List Backups Hook diagram

* Add On Create Backup Hook diagram

* Add note about restore status

* Add note about TLS settings change


          Documentation Sync with Discourse (canonical#120)

68e806c

---------

Co-authored-by: Alex Lutay <1928266+taurus-forever@users.noreply.github.com>


          Update Python dependencies (major) (canonical#125)

f5044e4

* Update Python dependencies

* Add cryptography build dependency

* Holding rfc3986

---------

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


          Update canonical/charming-actions action to v2.2.5 (canonical#127)

edd9dcd

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


          Update Python dependencies (canonical#126)

d10ceb8

* Update Python dependencies

* Holding httpcore

* Enforcing httpx versions instead of ignoring pinned deps

---------

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


          Migrating to Rock OCI image (canonical#118)

7a3fc9a

* Initial switch

* Use stable tls-certificates-operator

* Temporarily manually installing ca-certs

* Revert "Temporarily manually installing ca-certs"

This reverts commit 2fd47db.

* Read version from the Rock image

* Switch to GHCR Rock


          Setting workload version to snap in rock version (canonical#128)

444062a


          Update Python dependencies (canonical#132)

02da146

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


          Sync GitHub issues to Jira (canonical#131)

f6d2350

Ported from canonical/mysql-operator#108


          Enable Sync Mode (canonical#134)

ae91d62

https://warthogs.atlassian.net/browse/DPE-1456

---------

Co-authored-by: Dragomir Penev <dragomir.penev@canonical.com>


          Update Python dependencies (canonical#138)

6063c40

* Update Python dependencies

* Update httpx dep checks

* Update lock and requirements

---------

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


          Use release_charm.yaml and release to 14/edge (canonical#130)

da1e44d


          Update track from dpe to 14 in URLs (canonical#140)

beb07d1


          Update Python dependencies (canonical#139)

0b5cc82

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


          Update README to follow the common style for all SQL charms (canonica…

85a8bdd

…l#142)

As discussed, let's keep README as small as possible and
use Discourse for detailed manuals.


          Kill DB process (PostgreSQL), full cluster restart and full cluster c…

b17cfe0

…rash tests (canonical#136)

* Enable sync mode

* Remove custom conf

* Add new expected k8s endpoint to helper function

* Fix TLS test

* Add pebble health check

* Fix freeze db test

* Enable restart db test

* Remove commented code

* Fix unit tests

* Add missing tests

* Improve TLS management

* Add health check update

* Remove health check

* Add health check

* Prevent multiple primaries

* Remove unused code

* Check max number written

* Check writes on all instances

* Remove master_start_timeout setting

* Improve logs retrieval

* Add CA chain to trusted certificates

* Revert "Remove master_start_timeout setting"

This reverts commit 84d810d.

* Use the right PG process in the HA tests

* Fix order in test

* Remove unused call to SIGCONT

* Enable restart DB test

* Add initial full cluster crash test

* Add remaining cluster restart and crash tests

* Kill right process

* Create helper function for duplicate code

* Add telemetry step

* Increase replan timeout

* Stabilize tests

* Change tester application connection

* Add timeout to the database operations

* Remove telemetry step

* Add prints

* Add back the last written value file

* Pin packaging version

* Fix PostgreSQL DB process full cluster restart/crash

* Fix helper for freeze DB process test

* Add sleep

* Reduce extended pebble restart delay

* Remove prints

* Remove sleep

* Fix poetry lock

* Separate signal logic

* Improve loop wait setting change

* Change calls order

* Improve description about timeout

* Rename boolean functions according the best practices

---------

Co-authored-by: Dragomir Penev <dragomir.penev@canonical.com>


          Update dependency attrs to v23 (canonical#145)

d4eeb59

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


          Test network cut (canonical#135)

ce08115

* Enable sync mode

* Remove custom conf

* Add new expected k8s endpoint to helper function

* Fix TLS test

* Add pebble health check

* Fix freeze db test

* Enable restart db test

* Remove commented code

* Fix unit tests

* Add missing tests

* Add helpers

* Improve TLS management

* Add health check update

* Remove health check

* Add health check

* Prevent multiple primaries

* Remove unused code

* Add network cut test

* Add initial test structure

* Update test steps

* Fix test to make it work

* Remove unused code

* Uncomment code

* Remove unused code

* Add checks for member isolation

* Improve description about timeout

* Simplify shell scripts

* Fix chaos mesh scripts

---------

Co-authored-by: Dragomir Penev <dragomir.penev@canonical.com>

marceloneppel and others added 21 commits

April 4, 2024 08:33


          Show error on upgrade action (canonical#432)

30e04fc

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>


          [DPE-3570] Add primary status message after cluster bootstrap (canoni…

04ec619

…cal#435)

* Add primary status message after cluster bootstrap

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Add and update unit tests

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Remove _set_primary_status_message method

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

---------

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>


          [DPE-3885] Enhance the error message if charm was deployed without `-…

f71299b

…-trust` flag (canonical#440)

* improve error message for trust flag

* fix message text

* fix docstring

* fix explanation

* add integration test for new trust behavior

* test different app name for deploy

* fix bug

* enable rbac from inside test

* fix typo

* use sudo with microk8s enable

* remove check true

* fix linting

* test subprocess call

* remove fast-forward

* test connectivity and enable rbac

* fix integration test + update endpoints

* revert poetry lock change

* reposition endpoint update in event

* reposition cleanup resource call


          Update data-platform-workflows to v13.1.1 (canonical#438)

73ddf22

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


          Lock file maintenance Python dependencies (canonical#436)

bc0e732

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


          Smoke testing. Garbage storage ignorance. Recourses confilcts test.

3fa6f0b


          Added check_tls_replication for checking replicas encrypted connect…

f965fc6

…ion (canonical#444)


          [DPE-4068] Convert all tests from unittest to pytest + enable/disable…

debf9a6

… secrets in every test (canonical#452)

* convert all tests to pytest

* remove secret remove hook

* Revert "remove secret remove hook"


          [MISC] Update Rock and renovate tweaks (canonical#439)

fecf204

* Update Rock

* Renovate tweaks

* Slower ffwd

* Check only for blocked status

* Update agents

* Port can_connect from VM

* Check update_config result

* Still handle psycopg2 exceptions

* Wrong check

* Check all units

* Restart connection check

* Lock file maintenance Python dependencies

* Don't release retart hook until PG is back up

* Increase restart need atempts

* New rock

* Check PG is up

* Revert tweaks

---------

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


          unify unit test suites (canonical#462)

e71ea77


          Rerender and reenable pgbackrest config and service (canonical#461)

80e9ff0


          Lock file maintenance Python dependencies (canonical#443)

9c73611

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


          [MISC] Update data-platform-libs: data_interfaces to 34 and upgrade t…

09f7824

…o 16 (canonical#454)

* update libs

* fix unit tests

* add peer check

* adapt src code and unit tests

* separate tests in 2 versions

* review suggestions

* add new review suggestions


          [DPE-2671] integration test (canonical#464)

5dee451

* Rerender and reenable pgbackrest config and service

* Charm lib bump

* Delete pod test

* Add logs

* Don't log config file


          [DPE-2897] Cross-region async replication (canonical#447)

bb1c73e

* Add async replication implementation

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Backup standby pgdata folder

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Improve comments and logs

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Remove unused constant

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Remove warning log call and add optional type hint

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Revert poetry.lock

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Revert poetry.lock

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

---------

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>


          Smoke testing. Garbage storage ignorance. Recourses confilcts test.

5af235d


          Smoke testing. Garbage storage ignorance. Recourses confilcts test.

0f79ddf


          Update dependency libjuju 2 to v2.9.49.0 (canonical#467)

20782ed

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


          Fix secret label (canonical#472)

19417e6

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>


          Smoke testing. Garbage storage ignorance. Recourses confilcts test.

1b32e41


          Merge branch 'canonical:main' into test-charmed-postgresql-k8s-smoke-…

8e5cc24

…test

BON4 marked this pull request as ready for review

May 15, 2024 21:34

dragomirp requested review from dragomirp, lucasgameiroborges, marceloneppel and taurus-forever

May 20, 2024 14:30


          Smoke testing. Garbage storage ignorance. Recourses confilcts test.

f7d7ee2

BON4 closed this

BON4 force-pushed the test-charmed-postgresql-k8s-smoke-test branch from 9146fd1 to f7d7ee2 Compare

May 20, 2024 17:08

BON4 deleted the test-charmed-postgresql-k8s-smoke-test branch

May 20, 2024 17:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet