~canonical-kernel/+git/kteam-tools:nwager/cleanup-chroot

Last commit made on 2024-01-08
Get this branch:
git clone -b nwager/cleanup-chroot https://git.launchpad.net/~canonical-kernel/+git/kteam-tools
Members of Canonical Kernel can upload to this branch. Log in for directions.

Branch merges

Branch information

Name:
nwager/cleanup-chroot
Repository:
lp:~canonical-kernel/+git/kteam-tools

Recent commits

dd1974c... by Juerg Haefliger

cranky: chroot: make it backwards compatible

Make cranky chroot backwards compatible:
  1) If a new session is not found, try to find an old one and operate
     on that.
  2) Parse the old sources|configs commandline option (but ignore it).

Emit deprecation warnings for the above and add a deprecation notice to
the script usage.

https://warthogs.atlassian.net/browse/KERNTT-724
Signed-off-by: Juerg Haefliger <email address hidden>

fec4e9b... by Noah Wager

cranky: move builder-extra.asc to gpg.d dir

https://warthogs.atlassian.net/browse/KERNTT-724
Signed-off-by: Noah Wager <email address hidden>
[juergh: Update cranky/cmds.d/chroot accordingly.]
Signed-off-by: Juerg Haefliger <email address hidden>

a96f40b... by Noah Wager

cranky: chroot: add which handling

Users can specify chroot session names (appended to the normal chroot
prefixes) using the --which option, like the old <sources|configs>. If
left out, it will create a default session.

The kernel series can be specified with: only the series, the handle,
or a path. If the series is omitted, it tries to infer the series from
the current directory.

Some help messages have been updated accordingly.

https://warthogs.atlassian.net/browse/KERNTT-724
Signed-off-by: Noah Wager <email address hidden>
[juergh: Limited the commit to adding --which commandline argument,
 trimmed the commit message.]
Signed-off-by: Juerg Haefliger <email address hidden>

fa4925c... by Noah Wager

cranky: chroot: accept either a series or a full handle

Add the option to pass a series by itself, which will default to
SERIES:linux, or pass a full handle as usual.

Also made some small formatting changes, like indenting case statements,
making the fallthrough case more readable, and other small changes.

https://warthogs.atlassian.net/browse/KERNTT-724
Signed-off-by: Noah Wager <email address hidden>
[juergh: Don't use distro-info (might not be up-to-date).]
Signed-off-by: Juerg Haefliger <email address hidden>

8cbaa86... by Noah Wager

cranky: chroot: add option to run as different user

`cranky chroot run <handle> -u <user> -- <cmd>` will run the commands in
the chroot as the specified user. This is helpful if e.g. you want to
run as root to add packages.

https://warthogs.atlassian.net/browse/KERNTT-724
Signed-off-by: Noah Wager <email address hidden>
[juergh: Fix argument order, use $USER, add --user long option.]
Signed-off-by: Juerg Haefliger <email address hidden>

3b4be3c... by Noah Wager

cranky: chroot: remove unused sources/configs session separation

From Juerg Haefliger: cranky chroot splits `sources` and `configs` session
types in case the sessions have differences, but this is not the case.
Remove the separation in favor of a single session per series.

https://warthogs.atlassian.net/browse/KERNTT-724
Signed-off-by: Noah Wager <email address hidden>

bd8fecc... by Andrea Righi

chroot-setup: add Rust build-depends for noble (6.7)

Signed-off-by: Andrea Righi <email address hidden>
Acked-by: Roxana Nicolescu <email address hidden>
Signed-off-by: Andrea Righi <email address hidden>

014cca4... by Andrea Righi

chroot-setup: add Rust build-depends for noble (6.7)

Signed-off-by: Andrea Righi <email address hidden>

8c960d0... by Roxana Nicolescu

adt-matrix: Include main linux package version in triggers for retry

For dkms modules that do not have any specific tests, the matrix will run
`dkms-autopkgtest` (part of dkms source) which consists of building the
module only. The test will run only if the `trigger` parameter contains
the linux main package version because it expects the following version
format:
linux-<derivative>/<kernel_version>-<abi_version>.<upload_version>

For retry, the `trigger` param consists of only meta version which has the
following format:
linux-<derivative>/<kernel_version>.<abi_version>.<upload_version>
which contains a dot instead of a '-' and prevents the test from
running and it marks it as 'Passed'.

Fix this by including the main version for retries too.
Note: When the test is triggered for the first time, the main version is
included, that's why the first run is always correct. The problem appears
only for retries. This broken logic prevents dkms builds failures from
being properly detected if the test is rerun (which usually happens because
people try the test 1-2 times before confirming if there's an issue)

Signed-off-by: Roxana Nicolescu <email address hidden>
Acked-by: Andy Whitcroft <email address hidden>
Signed-off-by: Roxana Nicolescu <email address hidden>

1cb1b49... by Roxana Nicolescu

adt-matrix: Include main linux package version in triggers for retry

For dkms modules that do not have any specific tests, the matrix will run
`dkms-autopkgtest` (part of dkms source) which consists of building the
module only. The test will run only if the `trigger` parameter contains
the linux main package version because it expects the following version
format:
linux-<derivative>/<kernel_version>-<abi_version>.<upload_version>

For retry, the `trigger` param consists of only meta version which has the
following format:
linux-<derivative>/<kernel_version>.<abi_version>.<upload_version>
which contains a dot instead of a '-' and prevents the test from
running and it marks it as 'Passed'.

Fix this by including the main version for retries too.
Note: When the test is triggered for the first time, the main version is
included, that's why the first run is always correct. The problem appears
only for retries. This broken logic prevents dkms builds failures from
being properly detected if the test is rerun (which usually happens because
people try the test 1-2 times before confirming if there's an issue)

Signed-off-by: Roxana Nicolescu <email address hidden>