Skip to content

Smoke test K3S in MinimalVM - #25409

Merged
ricardobranco777 merged 1 commit into
os-autoinst:masterfrom
volodymyrkatkalov:minimalvm_k3s
Apr 28, 2026
Merged

ricardobranco777 merged 1 commit into
os-autoinst:masterfrom
volodymyrkatkalov:minimalvm_k3s

Conversation

@volodymyrkatkalov

@volodymyrkatkalov volodymyrkatkalov commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

Introduce a new smoke test for k3s in Minimal-VM.
Re-use existing functions ifrom lib/containers/k8s.pm as much as possible.

@github-actions

Copy link
Copy Markdown

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

Comment thread tests/containers/k3s_smoketest.pm Outdated
Comment thread lib/containers/k8s.pm Outdated
Comment thread tests/containers/k3s_smoketest.pm Outdated
Comment thread tests/containers/k3s_smoketest.pm Outdated
Comment thread tests/containers/k3s_smoketest.pm Outdated
@volodymyrkatkalov
volodymyrkatkalov force-pushed the minimalvm_k3s branch 4 times, most recently from 125e0ff to 6641817 Compare April 23, 2026 16:05
@volodymyrkatkalov
volodymyrkatkalov marked this pull request as ready for review April 24, 2026 07:22
mloviska
mloviska previously approved these changes Apr 24, 2026
Comment thread lib/containers/k8s.pm Outdated
Comment thread tests/containers/k3s_smoketest.pm Outdated
Comment thread tests/containers/k3s_smoketest.pm Outdated
Comment thread tests/containers/k3s_smoketest.pm Outdated
Comment thread tests/containers/k3s_smoketest.pm Outdated
@mergify
mergify Bot dismissed linguini-dev’s stale review April 24, 2026 13:28

Pull request has been modified.

@volodymyrkatkalov
volodymyrkatkalov force-pushed the minimalvm_k3s branch 3 times, most recently from 4dc0604 to 769b34e Compare April 24, 2026 17:57
@volodymyrkatkalov
volodymyrkatkalov force-pushed the minimalvm_k3s branch 4 times, most recently from 2d35170 to 57312f5 Compare April 24, 2026 20:14
@volodymyrkatkalov
volodymyrkatkalov marked this pull request as draft April 24, 2026 20:40
@volodymyrkatkalov
volodymyrkatkalov marked this pull request as ready for review April 27, 2026 08:50
Comment thread tests/containers/k3s_smoketest.pm Outdated
Introduce a new smoke test for k3s in Minimal-VM.
Re-use existing functions ifrom lib/containers/k8s.pm
as much as possible.

- Related ticket: https://progress.opensuse.org/issues/199595
@ricardobranco777
ricardobranco777 merged commit e536d93 into os-autoinst:master Apr 28, 2026
15 checks passed
Comment thread lib/main_containers.pm
}

if ($runtime eq 'k3s') {
if ('CONTAINER_K3S_SMOKETEST') {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I realize now that this variable is not needed at all. Because we already have a smoketest below: containers/run_container_in_k3s that we'd likely also want to run.

volodymyrkatkalov added a commit to volodymyrkatkalov/os-autoinst-distri-opensuse that referenced this pull request Apr 29, 2026
volodymyrkatkalov added a commit to volodymyrkatkalov/os-autoinst-distri-opensuse that referenced this pull request Apr 29, 2026
fsimorda pushed a commit to fsimorda/os-autoinst-distri-opensuse that referenced this pull request May 4, 2026
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.

6 participants