Fix foremanctl install for IPv6 deployments#22125
Merged
shubhamsg199 merged 7 commits intoJul 16, 2026
Merged
Conversation
Contributor
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds the deploy_network_type field to the upstream-pr-install foremanctl job template execution so that IPv6 environments configure COPR via HTTPS_PROXY correctly during upstream PR installs. Sequence diagram for upstream-pr-install JT execution with deploy_network_typesequenceDiagram
participant Host
participant ForemanctlJobExecution
Host->>ForemanctlJobExecution: install_satellite_foremanctl(job_template upstream-pr-install, deploy_network_type)
ForemanctlJobExecution->>ForemanctlJobExecution: execute()
alt [deploy_network_type is ipv6]
Note over ForemanctlJobExecution: COPR repo uses HTTPS_PROXY on host
else [deploy_network_type is ipv4]
Note over ForemanctlJobExecution: COPR repo enabled without IPv6 proxy handling
end
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Consider guarding
deploy_network_type=settings.server.network_typein casesettings.serverornetwork_typeis unset or missing, to avoid attribute errors during JT execution. - If
deploy_network_typeis optional for this job template, you might want to pass it only when non-None to avoid overriding any default behavior on the JT side.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Consider guarding `deploy_network_type=settings.server.network_type` in case `settings.server` or `network_type` is unset or missing, to avoid attribute errors during JT execution.
- If `deploy_network_type` is optional for this job template, you might want to pass it only when non-None to avoid overriding any default behavior on the JT side.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Alleny244
approved these changes
Jul 13, 2026
Contributor
|
Do we have any successful executions we can use to verify that the fix really works? |
Gauravtalreja1
force-pushed
the
use-ipv6-upstream-pr-install
branch
from
July 13, 2026 13:46
64eef1f to
92454d5
Compare
Member
Author
|
Collaborator
|
PRT Result |
Gauravtalreja1
force-pushed
the
use-ipv6-upstream-pr-install
branch
from
July 13, 2026 21:04
92454d5 to
249aaaf
Compare
Member
Author
|
Collaborator
|
PRT Result |
evgeni
reviewed
Jul 14, 2026
Member
This fails to deploy Candlepin, but it deployed PostgreSQL and Valkey just fine before that, so the issue is not the Proxy (anymore) |
evgeni
approved these changes
Jul 14, 2026
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
…manctl runs Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
On IPv6-only hosts, candlepin healthcheck routes through the HTTP proxy and gets 503, causing systemd to timeout and kill the service. Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Gauravtalreja1
force-pushed
the
use-ipv6-upstream-pr-install
branch
2 times, most recently
from
July 14, 2026 17:08
206ebd2 to
d94968c
Compare
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Gauravtalreja1
force-pushed
the
use-ipv6-upstream-pr-install
branch
from
July 14, 2026 17:17
d94968c to
20a80f5
Compare
Member
Author
|
Collaborator
|
PRT Result |
evgeni
reviewed
Jul 15, 2026
Member
Author
|
evgeni
reviewed
Jul 15, 2026
Collaborator
|
PRT Result |
2 tasks
Disabling automatic proxy forwarding via http_proxy=false in containers section. Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Gauravtalreja1
force-pushed
the
use-ipv6-upstream-pr-install
branch
from
July 15, 2026 14:48
ad2a554 to
bee0801
Compare
Member
Author
|
Collaborator
|
PRT Result |
2 tasks
evgeni
approved these changes
Jul 15, 2026
jameerpathan111
approved these changes
Jul 16, 2026
shubhamsg199
approved these changes
Jul 16, 2026
Contributor
|
2 failures will be fixed with theforeman/obsah#127 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem Statement
upstream-pr-installJT execution fails for IPv6 because we enable the COPR repository on the host usingHTTPS_PROXYwheneverdeploy_network_typeis set. That configuration is currently missing, causing the test setup to fail on IPv6 deployments.Solution
deploy_network_typeto theupstream-pr-installJT execution so the COPR repository is enabled viaHTTPS_PROXYand the IPv6 test setup completes successfully.Related Issues
Summary by Sourcery
Enhancements: