coreos / coreos/coreos-assembler

add CI coverage for `kola testiso --pxe-append-rootfs`

Open
#4,024 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

jira
Dominant language
Go
Stars
393
Forks
193
Avg merge
17h 6m
Merged PRs (30d)
5

Description

Currently we don't have any CI coverage for the PXE case:

Concatenate the initramfs and rootfs files together, and specify the combined file as the initrd. This method is slower and requires 4 GiB of RAM.

from the docs.

The kola testiso command supports --pxe-append-rootfs (underlying code), but we don't set that option when we run any of our default tests. We either need to create a new name for this type of test and add it to the default list OR we need to just blanket add it to a class of tests (like the PXE offline ones). I'm not sure which would be best.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the default kola test list and the existing PXE offline tests, then inspect how kola testiso options are passed. Decide whether this coverage belongs in a new test name or an existing PXE test class. Done means a default CI test exercises --pxe-append-rootfs and covers the combined initramfs/rootfs PXE path.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
ci-cd, testing
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.