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

HMS-3162: test: boot generated image and ensure ssh is working #34

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

mvo5
Copy link
Collaborator

@mvo5 mvo5 commented Dec 6, 2023

This PR adds the extra check that the generated image from podman run osbuild-deploy-container can actually boot in qemu and will open the ssh port.

@ochosi ochosi changed the title test: boot generated image and ensure ssh is working HMS-3162: test: boot generated image and ensure ssh is working Dec 6, 2023
@ondrejbudai
Copy link
Member

Looks good, happy to approved when #24 is merged and this is rebased. :)

@mvo5 mvo5 force-pushed the tmt-smoke-boot-vm branch from 0d8c943 to cceb14e Compare December 7, 2023 11:58
test/test_smoke.py Show resolved Hide resolved
@mvo5 mvo5 force-pushed the tmt-smoke-boot-vm branch 3 times, most recently from e8489e1 to d306cf5 Compare December 7, 2023 15:57
Copy link

@thrix thrix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo here

plans/all.fmf Outdated Show resolved Hide resolved
@mvo5 mvo5 force-pushed the tmt-smoke-boot-vm branch 2 times, most recently from 5868880 to 6557abb Compare December 7, 2023 16:28
@mvo5 mvo5 marked this pull request as ready for review December 7, 2023 16:39
@mvo5 mvo5 force-pushed the tmt-smoke-boot-vm branch from 6557abb to 3a330cf Compare December 7, 2023 17:49
@thrix
Copy link

thrix commented Dec 7, 2023

@mvo5 looks the action is not yet working well with RH ranch :(

It fails to report the correct URL seems.
That would be:

http://artifacts.osci.redhat.com/testing-farm/54e53734-4050-45df-abff-c79e365d354c/

So seems tests run, but they failed on:

E               FileNotFoundError: [Errno 2] No such file or directory: 'qemu-system-x86_64'

@thrix
Copy link

thrix commented Dec 7, 2023

@mvo5 never mind, you need to adjust tf_scope to private:

https://github.com/sclorg/testing-farm-as-github-action/blob/main/action.yml#L78

@thrix
Copy link

thrix commented Dec 7, 2023

My apologies for not realizing earlier, we do not use much this action 🤦

@mvo5 mvo5 force-pushed the tmt-smoke-boot-vm branch 2 times, most recently from 5d9594f to bd2d8c2 Compare December 7, 2023 19:04
@mvo5
Copy link
Collaborator Author

mvo5 commented Dec 7, 2023

My apologies for not realizing earlier, we do not use much this action 🤦

Thank you for all your help with this! It looks like it's working fine, I'm very excited.

@achilleas-k
Copy link
Member

Needs a rebase and conflict resolution.

Do some more testing of the generated image by booting it and
checking that ssh comes up. There will be a followup that will
actually login into the VM and ensure that also works.
@mvo5 mvo5 force-pushed the tmt-smoke-boot-vm branch from bd2d8c2 to f158d30 Compare December 8, 2023 07:51
@ondrejbudai ondrejbudai added this pull request to the merge queue Dec 8, 2023
Merged via the queue into osbuild:main with commit 5f13442 Dec 8, 2023
8 checks passed
@mvo5 mvo5 deleted the tmt-smoke-boot-vm branch December 8, 2023 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants