add support for GenericCloud image testing #144
No reviewers
Labels
No Label
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
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: testing/os-autoinst-distri-rocky-migrated#144
Loading…
Reference in New Issue
Block a user
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?
Description
This pull request will add support for GenericCloud image testing into the
templates.fif.json
file.How Has This Been Tested?
Requirements
You must pull or create the
cloudinit.iso
to run theGenericCloud
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 ofGenericCloud
images will need alternatecloudinit.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:
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".
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.As mentioned in the PR description it is expected that some tests will fail, more specifically this is the
autocloud
test for the usertest
. 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.