-
Notifications
You must be signed in to change notification settings - Fork 20
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.
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
Add test_functional
test
#460
base: main
Are you sure you want to change the base?
Changes from all commits
75a88d5
50a7044
dcf2e2a
f9f7c4b
9df30a0
d632b43
6deb1d2
3f139ea
e7e07d7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,289 @@ | ||
#!/usr/bin/env python3 | ||
# Copyright 2023 Canonical Ltd. | ||
# See LICENSE file for licensing details. | ||
|
||
import logging | ||
import uuid | ||
from typing import Dict, Tuple | ||
|
||
import boto3 | ||
import pytest | ||
from pytest_operator.plugin import OpsTest | ||
from tenacity import Retrying, stop_after_attempt, stop_after_delay, wait_exponential, wait_fixed | ||
|
||
from ..helpers import ( | ||
APPLICATION_NAME, | ||
CHARM_SERIES, | ||
DATABASE_APP_NAME, | ||
construct_endpoint, | ||
get_machine_from_unit, | ||
) | ||
from .helpers import ( | ||
check_db, | ||
check_graceful_shutdown, | ||
check_success_recovery, | ||
create_db, | ||
is_postgresql_ready, | ||
lxc_restart_service, | ||
) | ||
|
||
TEST_DATABASE_NAME = "test_database" | ||
DUP_APPLICATION_NAME = "postgres-test-dup" | ||
S3_INTEGRATOR_APP_NAME = "s3-integrator" | ||
|
||
logger = logging.getLogger(__name__) | ||
|
||
AWS = "AWS" | ||
|
||
|
||
@pytest.fixture(scope="module") | ||
async def cloud_configs(ops_test: OpsTest, github_secrets) -> None: | ||
# Define some configurations and credentials. | ||
configs = { | ||
AWS: { | ||
"endpoint": "https://s3.amazonaws.com", | ||
"bucket": "data-charms-testing", | ||
"path": f"/postgresql-vm/{uuid.uuid1()}", | ||
"region": "us-east-1", | ||
}, | ||
} | ||
credentials = { | ||
AWS: { | ||
"access-key": github_secrets["AWS_ACCESS_KEY"], | ||
"secret-key": github_secrets["AWS_SECRET_KEY"], | ||
}, | ||
} | ||
yield configs, credentials | ||
# Delete the previously created objects. | ||
logger.info("deleting the previously created backups") | ||
for cloud, config in configs.items(): | ||
session = boto3.session.Session( | ||
aws_access_key_id=credentials[cloud]["access-key"], | ||
aws_secret_access_key=credentials[cloud]["secret-key"], | ||
region_name=config["region"], | ||
) | ||
s3 = session.resource( | ||
"s3", endpoint_url=construct_endpoint(config["endpoint"], config["region"]) | ||
) | ||
bucket = s3.Bucket(config["bucket"]) | ||
for bucket_object in bucket.objects.filter(Prefix=config["path"].lstrip("/")): | ||
bucket_object.delete() | ||
|
||
|
||
@pytest.mark.group(1) | ||
@pytest.mark.abort_on_fail | ||
async def test_instance_graceful_restart(ops_test: OpsTest, charm: str) -> None: | ||
"""Test graceful restart of a service.""" | ||
async with ops_test.fast_forward(): | ||
# Deploy the charm. | ||
logger.info("deploying charm") | ||
await ops_test.model.deploy( | ||
charm, | ||
application_name=APPLICATION_NAME, | ||
num_units=1, | ||
series=CHARM_SERIES, | ||
config={"profile": "testing"}, | ||
) | ||
|
||
logger.info("waiting for idle") | ||
await ops_test.model.wait_for_idle(apps=[APPLICATION_NAME], status="active", timeout=1500) | ||
assert ops_test.model.applications[APPLICATION_NAME].units[0].workload_status == "active" | ||
|
||
primary_name = ops_test.model.applications[APPLICATION_NAME].units[0].name | ||
|
||
logger.info("waiting for postgresql") | ||
for attempt in Retrying(stop=stop_after_delay(15 * 3), wait=wait_fixed(3), reraise=True): | ||
with attempt: | ||
assert await is_postgresql_ready(ops_test, primary_name) | ||
|
||
# Get unit hostname and IP. | ||
logger.info("wait for hostname") | ||
primary_hostname = await get_machine_from_unit(ops_test, primary_name) | ||
|
||
logger.info("restarting service") | ||
await lxc_restart_service(primary_hostname) | ||
|
||
logger.info("waiting for idle") | ||
await ops_test.model.wait_for_idle( | ||
apps=[APPLICATION_NAME], status="active", timeout=1500, raise_on_error=False | ||
) | ||
assert ops_test.model.applications[APPLICATION_NAME].units[0].workload_status == "active" | ||
|
||
logger.info("check graceful shutdown") | ||
assert await check_graceful_shutdown(ops_test, primary_name) | ||
|
||
logger.info("check success recovery") | ||
assert await check_success_recovery(ops_test, primary_name) | ||
|
||
logger.info("remove application") | ||
for attempt in Retrying(stop=stop_after_delay(15 * 3), wait=wait_fixed(3), reraise=True): | ||
with attempt: | ||
await ops_test.model.remove_application(APPLICATION_NAME, block_until_done=True) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Testing on Juju 3.5.1, I often faced the following error: unit-postgresql-test-app-0: 14:50:09 ERROR juju.worker.uniter.operation hook "pgdata-storage-detaching" (via hook dispatching script: dispatch) failed: exit status 1
unit-postgresql-test-app-0: 14:55:11 ERROR unit.postgresql-test-app/0.juju-log Uncaught exception while in charm code:
Traceback (most recent call last):
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/./src/charm.py", line 1645, in <module>
main(PostgresqlOperatorCharm)
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/venv/ops/main.py", line 544, in main
manager.run()
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/venv/ops/main.py", line 520, in run
self._emit()
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/venv/ops/main.py", line 509, in _emit
_emit_charm_event(self.charm, self.dispatcher.event_name)
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/venv/ops/main.py", line 143, in _emit_charm_event
event_to_emit.emit(*args, **kwargs)
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/venv/ops/framework.py", line 352, in emit
framework._emit(event)
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/venv/ops/framework.py", line 851, in _emit
self._reemit(event_path)
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/venv/ops/framework.py", line 941, in _reemit
custom_handler(event)
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/./src/charm.py", line 432, in _on_pgdata_storage_detaching
primary = self._patroni.get_primary(unit_name_pattern=True)
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/./src/charm.py", line 705, in _patroni
self._peer_members_ips,
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/./src/charm.py", line 735, in _peer_members_ips
addresses = self.members_ips
File "/var/lib/juju/agents/unit-postgresql-test-app-0/charm/./src/charm.py", line 756, in members_ips
return set(json.loads(self._peers.data[self.app].get("members_ips", "[]")))
AttributeError: 'NoneType' object has no attribute 'data'
The test got stuck at this point. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have been testing it for almost a week, and can't reproduce this error for juju 3.5.1 nor for 3.4 and 3.3. Can you elaborate and help me with this one? What version of lxd are you using? Charmcraft, Snapcraft? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sure.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could you test on Juju 3.5.1? Like it was in original issue by you? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sure. I tested again with Juju 3.5.1. Those are the version I used:
I also copied this branch to another PR to test it. The error is the following: https://github.com/canonical/postgresql-operator/actions/runs/9881101326/job/27384074248#step:28:2995 |
||
|
||
|
||
@pytest.mark.group(1) | ||
@pytest.mark.abort_on_fail | ||
async def test_instance_forceful_restart(ops_test: OpsTest, charm: str) -> None: | ||
"""Test forceful restart of a service.""" | ||
async with ops_test.fast_forward(): | ||
# Deploy the charm. | ||
logger.info("deploying charm") | ||
await ops_test.model.deploy( | ||
charm, | ||
application_name=APPLICATION_NAME, | ||
num_units=1, | ||
series=CHARM_SERIES, | ||
config={"profile": "testing"}, | ||
) | ||
|
||
logger.info("waiting for idle") | ||
await ops_test.model.wait_for_idle(apps=[APPLICATION_NAME], status="active", timeout=1500) | ||
assert ops_test.model.applications[APPLICATION_NAME].units[0].workload_status == "active" | ||
|
||
primary_name = ops_test.model.applications[APPLICATION_NAME].units[0].name | ||
|
||
logger.info("waiting for postgresql") | ||
for attempt in Retrying(stop=stop_after_delay(15 * 3), wait=wait_fixed(3), reraise=True): | ||
with attempt: | ||
assert await is_postgresql_ready(ops_test, primary_name) | ||
|
||
# Get unit hostname and IP. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have the same question from above regarding retrieving the IP. |
||
logger.info("wait for hostname") | ||
primary_hostname = await get_machine_from_unit(ops_test, primary_name) | ||
|
||
logger.info("restarting service with force") | ||
await lxc_restart_service(primary_hostname, force=True) | ||
|
||
logger.info("waiting for idle") | ||
await ops_test.model.wait_for_idle( | ||
apps=[APPLICATION_NAME], status="active", timeout=1500, raise_on_error=False | ||
) | ||
assert ops_test.model.applications[APPLICATION_NAME].units[0].workload_status == "active" | ||
|
||
logger.info("check forceful shutdown") | ||
assert not await check_graceful_shutdown(ops_test, primary_name) | ||
|
||
logger.info("check success recovery") | ||
assert await check_success_recovery(ops_test, primary_name) | ||
|
||
logger.info("remove application") | ||
for attempt in Retrying(stop=stop_after_delay(15 * 3), wait=wait_fixed(3), reraise=True): | ||
with attempt: | ||
await ops_test.model.remove_application(APPLICATION_NAME, block_until_done=True) | ||
|
||
|
||
@pytest.mark.group(1) | ||
@pytest.mark.abort_on_fail | ||
async def test_instance_backup_with_restart( | ||
ops_test: OpsTest, cloud_configs: Tuple[Dict, Dict], charm | ||
) -> None: | ||
"""Test instance backup after recovery.""" | ||
async with ops_test.fast_forward(): | ||
logger.info("deploying s3") | ||
await ops_test.model.deploy(S3_INTEGRATOR_APP_NAME) | ||
|
||
for cloud, config in cloud_configs[0].items(): | ||
# Deploy and relate PostgreSQL to S3 integrator (one database app for each cloud for now | ||
# as archive_mode is disabled after restoring the backup) | ||
logger.info("deploying charm") | ||
await ops_test.model.deploy( | ||
charm, | ||
application_name=DATABASE_APP_NAME, | ||
num_units=1, | ||
series=CHARM_SERIES, | ||
config={"profile": "testing"}, | ||
) | ||
|
||
logger.info("relate s3") | ||
await ops_test.model.relate(DATABASE_APP_NAME, S3_INTEGRATOR_APP_NAME) | ||
|
||
# Configure and set access and secret keys. | ||
logger.info(f"configuring S3 integrator for {cloud}") | ||
await ops_test.model.applications[S3_INTEGRATOR_APP_NAME].set_config(config) | ||
action = await ops_test.model.units.get(f"{S3_INTEGRATOR_APP_NAME}/0").run_action( | ||
"sync-s3-credentials", | ||
**cloud_configs[1][cloud], | ||
) | ||
await action.wait() | ||
await ops_test.model.wait_for_idle( | ||
apps=[DATABASE_APP_NAME, S3_INTEGRATOR_APP_NAME], status="active", timeout=1500 | ||
) | ||
|
||
primary_unit = ops_test.model.applications[DATABASE_APP_NAME].units[0] | ||
primary_name = primary_unit.name | ||
|
||
# Write some data. | ||
logger.info("write data before backup") | ||
await create_db(ops_test, DATABASE_APP_NAME, TEST_DATABASE_NAME) | ||
|
||
# Run the "create backup" action. | ||
logger.info("creating a backup") | ||
action = await ops_test.model.units.get(primary_name).run_action("create-backup") | ||
await action.wait() | ||
backup_status = action.results.get("backup-status") | ||
assert backup_status, "backup hasn't succeeded" | ||
await ops_test.model.wait_for_idle( | ||
apps=[DATABASE_APP_NAME, S3_INTEGRATOR_APP_NAME], status="active", timeout=1000 | ||
) | ||
|
||
# Run the "list backups" action. | ||
logger.info("listing the available backups") | ||
action = await ops_test.model.units.get(primary_name).run_action("list-backups") | ||
await action.wait() | ||
backups = action.results.get("backups") | ||
assert backups, "backups not outputted" | ||
await ops_test.model.wait_for_idle(status="active", timeout=1500) | ||
|
||
# Write some data. | ||
logger.info("write data after backup") | ||
await create_db(ops_test, DATABASE_APP_NAME, TEST_DATABASE_NAME + "_dup") | ||
|
||
logger.info("wait for hostname") | ||
primary_hostname = await get_machine_from_unit(ops_test, primary_name) | ||
|
||
logger.info("restarting service with force") | ||
await lxc_restart_service(primary_hostname, force=True) | ||
|
||
logger.info("waiting for idle") | ||
await ops_test.model.wait_for_idle( | ||
apps=[DATABASE_APP_NAME], status="active", timeout=1500, raise_on_error=False | ||
) | ||
assert ( | ||
ops_test.model.applications[DATABASE_APP_NAME].units[0].workload_status == "active" | ||
) | ||
|
||
logger.info("check forceful shutdown") | ||
assert not await check_graceful_shutdown(ops_test, primary_name) | ||
|
||
logger.info("check success recovery") | ||
assert await check_success_recovery(ops_test, primary_name) | ||
|
||
# Run the "restore backup" action. | ||
for attempt in Retrying( | ||
stop=stop_after_attempt(10), wait=wait_exponential(multiplier=1, min=2, max=30) | ||
): | ||
with attempt: | ||
logger.info("restoring the backup") | ||
most_recent_backup = backups.split("\n")[-1] | ||
backup_id = most_recent_backup.split()[0] | ||
action = await primary_unit.run_action("restore", **{"backup-id": backup_id}) | ||
await action.wait() | ||
restore_status = action.results.get("restore-status") | ||
assert restore_status, "restore hasn't succeeded" | ||
|
||
# Wait for the restore to complete. | ||
logger.info("wait for restore") | ||
await ops_test.model.wait_for_idle(status="active", timeout=1500) | ||
|
||
logger.info("checking data consistency") | ||
assert await check_db(ops_test, DATABASE_APP_NAME, TEST_DATABASE_NAME) | ||
assert not await check_db(ops_test, DATABASE_APP_NAME, TEST_DATABASE_NAME + "_dup") | ||
|
||
logger.info("remove application") | ||
for attempt in Retrying( | ||
stop=stop_after_delay(15 * 3), wait=wait_fixed(3), reraise=True | ||
): | ||
with attempt: | ||
await ops_test.model.remove_application( | ||
DATABASE_APP_NAME, block_until_done=True | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it really retrieving the IP or only the hostname?