Skip to content
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 end-to-end tests #68

Merged
merged 2 commits into from Jun 21, 2023
Merged

Add end-to-end tests #68

merged 2 commits into from Jun 21, 2023

Conversation

sed-i
Copy link
Contributor

@sed-i sed-i commented Mar 26, 2023

Issue

Need end-to-end tests.

Solution

Add end-to-end tests (discourse post).

  • juju-info test
  • cos-agent test

Context

Grafana agent machine charm.

Testing Instructions

NTA.

Release Notes

Add end-to-end tests.

@sed-i
Copy link
Contributor Author

sed-i commented Apr 22, 2023

The tests pass locally (Juju 2.9.42, MicroK8s 1.25.9).

The tests fail in CI:

  • Random hooks fail with Juju 3.1.2, MicroK8s 1.26.3.
  • Cannot pin to Juju 2.9 because e2e will fail because of this juju bug.

Drop the "testing" bundle option

Finish the juju_info e2e test

Add e2e workflow to CI

Cleanup

Cancel in-progress CI

Fix

Still need to raise_on_error False

Retrigger CI (empty commit)

Dump logs on failure

Dump logs on failure

Use juju 2.9/stable for e2e (same as itests)

Use juju 3.1/stable for e2e

Strict microk8s for juju 3

Juju 3.1 everywhere

Fixes

Strict microk8s for juju 3

Add cos_agent test

Both tests in one

Pin pylibjuju to 3.1

Longer delay

Lint

Reduce scale to 1

Fix itest

Attempt Juju 2.9, MicroK8s 1.25

Attempt Juju 2.9, MicroK8s 1.25

Increase timeouts

Attempt Juju 3.0, MicroK8s 1.25

Revert. Let's keep retrtying with 2.9

This reverts commit bfc2900.

Try first lxd, then uk8s
@sed-i sed-i merged commit c8e23ef into main Jun 21, 2023
5 checks passed
@sed-i sed-i deleted the feature/e2e_test branch June 21, 2023 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants