add support for GenericCloud image testing #144

Merged
tcooper merged 1 commits from add_cloud_image_support into develop 2023-02-20 21:46:38 +00:00
tcooper commented 2023-01-26 18:13:58 +00:00 (Migrated from github.com)

Description

This pull request will add support for GenericCloud image testing into the templates.fif.json file.

How Has This Been Tested?

openqa-cli api -X POST isos ARCH=x86_64 DISTRI=rocky FLAVOR=GenericCloud_Base-qcow2-qcow2 VERSION=9.1 BUILD=9.1-GenericCloud_Base-$(date +%Y%m%d).0 HDD_2=Rocky-9-GenericCloud-Base.latest.x86_64.qcow2  HDD_2_URL=https://dl.rockylinux.org/pub/rocky/9.1/images/x86_64/Rocky-9-GenericCloud-Base.latest.x86_64.qcow2

openqa-cli api -X POST isos ARCH=x86_64 DISTRI=rocky FLAVOR=GenericCloud_LVM-qcow2-qcow2 VERSION=9.1 BUILD=9.1-GenericCloud_LVM-$(date +%Y%m%d).0 HDD_2=Rocky-9-GenericCloud-LVM.latest.x86_64.qcow2  HDD_2_URL=https://dl.rockylinux.org/pub/rocky/9.1/images/x86_64/Rocky-9-GenericCloud-LVM.latest.x86_64.qcow2

openqa-cli api -X POST isos ARCH=x86_64 DISTRI=rocky FLAVOR=GenericCloud_Base-qcow2-qcow2 VERSION=8.7 BUILD=8-GenericCloud_Base-$(date +%Y%m%d).0 HDD_2=Rocky-8-GenericCloud-Base.latest.x86_64.qcow2  HDD_2_URL=https://dl.rockylinux.org/pub/rocky/8/images/x86_64/Rocky-8-GenericCloud-Base.latest.x86_64.qcow2

openqa-cli api -X POST isos ARCH=x86_64 DISTRI=rocky FLAVOR=GenericCloud_LVM-qcow2-qcow2 VERSION=8.7 BUILD=8-GenericCloud_LVM-$(date +%Y%m%d).0 HDD_2=Rocky-8-GenericCloud-LVM.latest.x86_64.qcow2  HDD_2_URL=https://dl.rockylinux.org/pub/rocky/8/images/x86_64/Rocky-8-GenericCloud-LVM.latest.x86_64.qcow2

Requirements

You must pull or create the cloudinit.iso to run the GenericCloud tests in openQA. Currently @tcooper has a fork of Cloud-Init-ISO to produce the minimal content to satisfy this requirement. It's possible more advanced tests of GenericCloud images will need alternate cloudinit.iso content and those can/should be generated as required.

NOTE: Some tests do not currently pass in each OS but adding support for running these tests will allow test/debug of the actual tests.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
# Description This pull request will add support for GenericCloud image testing into the `templates.fif.json` file. # How Has This Been Tested? ```bash openqa-cli api -X POST isos ARCH=x86_64 DISTRI=rocky FLAVOR=GenericCloud_Base-qcow2-qcow2 VERSION=9.1 BUILD=9.1-GenericCloud_Base-$(date +%Y%m%d).0 HDD_2=Rocky-9-GenericCloud-Base.latest.x86_64.qcow2 HDD_2_URL=https://dl.rockylinux.org/pub/rocky/9.1/images/x86_64/Rocky-9-GenericCloud-Base.latest.x86_64.qcow2 openqa-cli api -X POST isos ARCH=x86_64 DISTRI=rocky FLAVOR=GenericCloud_LVM-qcow2-qcow2 VERSION=9.1 BUILD=9.1-GenericCloud_LVM-$(date +%Y%m%d).0 HDD_2=Rocky-9-GenericCloud-LVM.latest.x86_64.qcow2 HDD_2_URL=https://dl.rockylinux.org/pub/rocky/9.1/images/x86_64/Rocky-9-GenericCloud-LVM.latest.x86_64.qcow2 openqa-cli api -X POST isos ARCH=x86_64 DISTRI=rocky FLAVOR=GenericCloud_Base-qcow2-qcow2 VERSION=8.7 BUILD=8-GenericCloud_Base-$(date +%Y%m%d).0 HDD_2=Rocky-8-GenericCloud-Base.latest.x86_64.qcow2 HDD_2_URL=https://dl.rockylinux.org/pub/rocky/8/images/x86_64/Rocky-8-GenericCloud-Base.latest.x86_64.qcow2 openqa-cli api -X POST isos ARCH=x86_64 DISTRI=rocky FLAVOR=GenericCloud_LVM-qcow2-qcow2 VERSION=8.7 BUILD=8-GenericCloud_LVM-$(date +%Y%m%d).0 HDD_2=Rocky-8-GenericCloud-LVM.latest.x86_64.qcow2 HDD_2_URL=https://dl.rockylinux.org/pub/rocky/8/images/x86_64/Rocky-8-GenericCloud-LVM.latest.x86_64.qcow2 ``` # Requirements You must pull or create the `cloudinit.iso` to run the `GenericCloud` tests in openQA. Currently @tcooper has a [fork of Cloud-Init-ISO](https://github.com/tcooper/Cloud-Init-ISO/tree/develop) to produce the minimal content to satisfy this requirement. It's possible more advanced tests of `GenericCloud` images will need alternate `cloudinit.iso` content and those can/should be generated as required. NOTE: Some tests do not currently pass in each OS but adding support for running these tests will allow test/debug of the actual tests. # Checklist: - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] Any dependent changes have been merged and published in downstream modules
StackKorora (Migrated from github.com) reviewed 2023-01-26 18:13:58 +00:00
lumarel (Migrated from github.com) reviewed 2023-01-26 18:13:58 +00:00
AlanMarshall (Migrated from github.com) reviewed 2023-02-19 13:56:13 +00:00
AlanMarshall (Migrated from github.com) left a comment

Testing as specified above I found the 14 tests that use only root login all pass but the 2 that in addition try to login as user consistently fail "Login incorrect".

Testing as specified above I found the 14 tests that use only root login all pass but the 2 that in addition try to login as user consistently fail "Login incorrect".
akatch commented 2023-02-19 20:42:36 +00:00 (Migrated from github.com)

I can confirm Alan's findings - the cloud_autocloud tests (LVM and Base, both 8 and 9) fail at the postinstall login step with "password incorrect" for the test user. Everything else looks good.

I can confirm Alan's findings - the `cloud_autocloud` tests (LVM and Base, both 8 and 9) fail at the postinstall login step with "password incorrect" for the test user. Everything else looks good.
tcooper commented 2023-02-20 19:32:06 +00:00 (Migrated from github.com)

As mentioned in the PR description it is expected that some tests will fail, more specifically this is the autocloud test for the user test. A solution has been proposed in the discussion in MM (there are at least two options). In order to allow merge of this PR as it exists and was requested could @AlanMarshall and @akatch please approve the existing PR.

As mentioned in the PR description it is expected that some tests will fail, more specifically this is the `autocloud` test for the user `test`. A solution has been proposed in the discussion in MM (there are at least two options). In order to allow merge of this PR as it exists and was requested could @AlanMarshall and @akatch please approve the existing PR.
akatch (Migrated from github.com) approved these changes 2023-02-20 19:49:32 +00:00
AlanMarshall (Migrated from github.com) approved these changes 2023-02-20 21:34:02 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: testing/os-autoinst-distri-rocky-migrated#144
No description provided.