testing/os-autoinst-distri-rocky-migrated
10
0
Fork 0
Code Issues 12 Pull requests 1 Projects Releases Packages Wiki Activity

add support for GenericCloud image testing #144

Merged
tcooper merged 1 commit from add_cloud_image_support into develop 2023-02-20 21:46:38 +00:00
Conversation 2 Commits 1 Files changed 1 +86 -3
tcooper commented 2023-01-26 18:13:58 +00:00 (Migrated from github.com)
Copy link

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
Copy link

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)
Copy link

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)
Copy link

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.
Reviewers
No reviewers
lumarel
StackKorora
akatch
AlanMarshall
Labels
Clear labels   
priority: critical

This issue or pull request is of utmost priority

  
priority: high

This issue or pull request is of high priority

  
priority: low

This issue or pull request is of low priority

  
priority: medium

This issue or pull request is of medium priority

  
status: abandoned

This issue or pull request has been abandoned or is stale

  
status: duplicate

This issue or pull request already exists

  
status: invalid

This doesn't seem right

  
status: peer review needed

Requires at least one review in order to be merged

  
status: wontfix

This will not be worked on

  
tag: content

Additions to (or modifications of) content in the repository

  
tag: good first issue

Good for newcomers

  
tag: help wanted

Extra attention is needed

  
test suite

This issue or pull request is for test suite changes

  
type: bug

Something isn't working

  
type: documentation

Improvements or additions to documentation

  
type: enhancement

New feature or request

  
type: maintenance

To improve or continue maintenance for something

  
type: question

Further information is requested

No labels
priority: critical
priority: high
priority: low
priority: medium
status: abandoned
status: duplicate
status: invalid
status: peer review needed
status: wontfix
tag: content
tag: good first issue
tag: help wanted
test suite
type: bug
type: documentation
type: enhancement
type: maintenance
type: question
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
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.
Delete branch "add_cloud_image_support"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?