tests: add microk8s spread test
Metadata
Current evaluation
Merged after single reviewer approval and passing CI. Adds a spread test to verify microk8s builds and installs correctly, deferring instrumentation to a future update. Changes affect 2 files (+35/-0 lines).
Suggested action: —
No scores available.
Issue body
Test that microk8s builds fine and install. Instrumenting the
installation comes later.
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after single reviewer approval and passing CI. Adds a spread test to verify microk8s builds and installs correctly, deferring instrumentation to a future update. Changes affect 2 files (+35/-0 lines). | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a pull request adding a spread test to verify microk8s builds and installs correctly. Installation instrumentation is deferred to a future update. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to add a spread test verifying microk8s builds and installs correctly. Initial test coverage implemented, with installation instrumentation planned for a subsequent update. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1522 tests(spread): don't use microk8s for dependencies tests | charmcraft | merged | Merged to remove microk8s from spread dependency tests, deleting seven lines. Approved by two reviewers and dependent on PR #1521. | |
| #2203 tests: add spread suite for kbuild plugin | snapcraft | merged | Merged following approval and passing CI. Resolves LP: #1783835 by migrating Kbuild plugin integration tests to a Spread suite across six files, increasing overall coverage by 0.02%. | |
| #13 test(spread): add more spread tests | snapcraft-rocks | merged | Successfully merged spread tests covering architectures, chisel slices, craftctl, package-repositories, and patchelf. Approved by two reviewers across 15 files (+206/-5 lines). | |
| #4433 tests(spread): add remote-build spread test | snapcraft | merged | Merged changes adding spread tests for new and legacy remote-build, resolving issue #4398. Approved by two reviewers with passing CI checks. | |
| #2983 tests: add LXD spread test | snapcraft | merged | Merged after reviewer approval and passing CI checks. Added LXD spread test to verify LXD builds and installs. Modified two files with 29 added lines. | |
| #127 Fix spread tests | imagecraft | merged | Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines). | |
| #1287 tests: minor cleanups on the spread tests | snapcraft | merged | Merged minor spread test cleanups into master. Modified four files with 16 additions and 6 deletions. Approved by one reviewer with zero unresolved comments, maintaining 95.27% coverage upon integration. | |
| #2183 tests: add spread suite for cmake plugin | snapcraft | merged | Merged into master, resolving LP #1783830 by migrating CMake plugin integration tests to a Spread suite. Approved, passed CI, and maintained 91.23% code coverage across 17 modified files. | |
| #2188 tests: support running spread suite in autopkgtests | snapcraft | merged | Merged. Adds an autopkgtest suite to run the spread test suite during releases. CI failures were resolved by fixing missing build dependencies and temporarily disabling armhf execution. Approved by one reviewer and merged. | |
| #4105 tests: check for submodules before running spread | snapcraft | merged | Merged. Adds a pre-run check for the snapd-testing-tools submodule in spread tests to provide a clear error if missing. Approved by two reviewers, passed CI, and introduced no coverage changes. |