Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RFE] kola should support to start previous build to do external tests #3714

Open
HuijingHei opened this issue Feb 2, 2024 · 4 comments
Open
Assignees
Labels
jira for syncing to jira

Comments

@HuijingHei
Copy link
Member

Feature Request

Get suggestion by @jlebon , see coreos/rpm-ostree#4806 (comment):

Kola also has support for downloading the previous image, but as part of the run-upgrade suite; I think we should really fold that back into kola run proper and support external tests.

Desired Feature

To test rpm-ostree upgrade on old build, should download the previous build and start from it.

Add --from-previous option to download previous build and start, instead of the new built image. For example:
kola run --from-previous ext.rpm-ostree.test

Example Usage

To test rpm-ostree upgrade on previous build, can run with kola run --from-previous ext.rpm-ostree.test, in the test should upgrade rpm-ostree first, reboot and do tests.

Other Information

@HuijingHei
Copy link
Member Author

@jlebon not sure my understanding is correct, could you help to review when you get a chance? Thanks!

@dustymabe
Copy link
Member

I feel like there's a lot of overlap here with cosa buildfetch --build=xyz --stream=testing-devel && cosa kola run --build=xyz.

@jlebon
Copy link
Member

jlebon commented Feb 2, 2024

Add --from-previous option to download previous build and start, instead of the new built image.

The way I was thinking about this was that it'd be another field in the test metadata like from: ..., and then we could support different syntaxes like from: previous (last build on that same stream), from: STREAM (latest build on STREAM), or from: STREAM@VERSION.

The key thing here is that the test is just like any other in the list and is run as part of the same kola run invocation (in practice, we'll probably want upgrade tests that use anything other than from: previous to be gated behind a requiredTag).

I feel like there's a lot of overlap here with cosa buildfetch --build=xyz --stream=testing-devel && cosa kola run --build=xyz.

Yeah, definitely. That's basically what we do today. This is an RFE to fold it into kola and make the UX even nicer (e.g. that'd let us simplify the kola() step a lot and ideally make it easier to run the extended upgrade test locally). Probably not high-priority but also not too hard since we already have the main pieces there.

@HuijingHei
Copy link
Member Author

Sorry for the misunderstood, does this mean: add from: ... to kola.json like following?

#!/bin/bash
set -xeuo pipefail
## kola:
##   from: previous / STREAM / STREAM@VERSION
##   tags: needs-internet
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jira for syncing to jira
Projects
None yet
Development

No branches or pull requests

3 participants