Skip to content

Use example.invalid instead of example.org in kickstart_test tests - #98

Merged
jstodola merged 2 commits into
rhinstaller:develfrom
jstodola:fix/kickstart-test-example-domain
Sep 23, 2026
Merged

jstodola merged 2 commits into
rhinstaller:develfrom
jstodola:fix/kickstart-test-example-domain

Conversation

@jstodola

Copy link
Copy Markdown
Contributor

example.org is a real, publicly reachable domain, so a test that isn't fully mocked can make a genuine outbound HTTP request to it. On networks with an egress firewall that silently drops the connection, this hangs until the job times out instead of failing fast. example.invalid is the RFC 2606 domain guaranteed to never resolve.

jstodola and others added 2 commits September 23, 2026 17:17
example.org is a real, publicly reachable domain, so a test that isn't
fully mocked can make a genuine outbound HTTP request to it. On networks
with an egress firewall that silently drops the connection, this hangs
until the job times out instead of failing fast. example.invalid is the
RFC 2606 domain guaranteed to never resolve.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
These tests never mocked the boot.iso download, so they only passed
because example.org was a real, reachable host that requests.get()
didn't raise on. Switching to example.invalid (previous commit) made
the unmocked network call fail fast instead of hanging, exposing that
these tests aren't meant to perform real I/O.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@jstodola
jstodola force-pushed the fix/kickstart-test-example-domain branch from 034bd4d to 15067c9 Compare September 23, 2026 15:18
@jstodola
jstodola requested a review from jikortus September 23, 2026 15:21

@jikortus jikortus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you.

@jstodola
jstodola merged commit 4fdf054 into rhinstaller:devel Sep 23, 2026
3 of 4 checks passed
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.

2 participants