ci: isolate flaky stop/start test from integration tests - #370
Merged
louiseschmidtgen merged 5 commits intoJun 11, 2026
Merged
Conversation
Split 'Run simple integration tests' into two steps: - The 4 k8s-dqlite-relevant tests run without continue-on-error so regressions fail the job. - test_microk8s_stop_start runs separately with continue-on-error until canonical/microk8s#5546 (add wait_for_installation() after start) lands on master. Use python -m pytest directly instead of tox -e simple so we can pass the -k filter without modifying the microk8s tox config. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
canonical/microk8s#5546 adds wait_for_installation() after microk8s.start, so coredns is guaranteed to be running before the assertion. The flakiness is fixed upstream; no need to tolerate failures here. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add cluster functions to microk8s_util: get_join_token, join_cluster, wait_until_ready, ready_nodes, get_local_node_status. These mirror the k8s-snap equivalents in test_util.util but use microk8s commands. get_join_token parses the output of 'microk8s add-node'; each call produces a unique token so the assert join_token != join_token_2 holds. - Remove skipif from test_three_node_load and test_three_node_read_load. Both tests now branch on USE_MICROK8S for join/wait/status operations. - Fix configure_argocd docstring (was '""" ""Configures...') and make kubectl command conditional on USE_MICROK8S. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
These tests only run against MicroK8s. Remove all USE_MICROK8S conditionals and the dead k8s-snap branches: - conftest.py: drop USE_MICROK8S env check, always use microk8s_util - test_multi_node.py: call microk8s_util directly, no branching - test_multi_node_read.py: same; configure_argocd uses microk8s kubectl - kube_burner.py: always use 'microk8s config', drop USE_MICROK8S - metrics.py: K8S_DQLITE_SERVICE is always microk8s.daemon-k8s-dqlite - tox.ini: drop USE_MICROK8S from passenv - microk8s-integration.yaml: drop 'export USE_MICROK8S=true' Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
BenchmarkResultsCurrent status |
louiseschmidtgen
merged commit Jun 11, 2026
38fd785
into
feat/microk8s-integration-tests
9 of 11 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closing — all changes merged directly into PR #369 (
feat/microk8s-integration-tests).