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
test/e2e: update packaging tests #952
Conversation
320ce49
to
7fc6ac6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add some description to both commit messages why the change is needed. Now it is unclear why the change is done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with the changes and they make sense, but I think @jukkar is right that it would be nice to get something like these in the commit messages
- use 15.4 instead of 15.2 for OpenSUSE packaging tests
- drop EOL'd Ubuntu 21.04 from packaging tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
demo/lib/distro.bash does not have support for opensuse-15.4 yet, though. As a result, this packaging test will currently run on opensuse-15.3.
I tried out adding it:
opensuse-image-url() {
opensuse-15_4-image-url
}
opensuse-15_4-image-url() {
echo "https://download.opensuse.org/pub/opensuse/distribution/leap/15.4/appliances/openSUSE-Leap-15.4-JeOS.x86_64-15.4-OpenStack-Cloud-Current.qcow2"
}
However, Qemu seems to be stuck in boot-loop. This can be addressed in later commits, anyway.
7fc6ac6
to
99042f0
Compare
Added reasoning to the commit messages |
99042f0
to
f91d6b8
Compare
Ubuntu 21.04 is EOL'd a long time ago and we don't build packages for it so let's drop it from packaging tests as well.
Change openSUSE packaging tests from Leap 15.2 to 15.4 as that's the version that we currently build packages for.
f91d6b8
to
02fb8a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
No description provided.