Skip to content

DP1410: add logic to handle networking behind a proxy, update readme#20

Merged
pwildber merged 1 commit into
mainfrom
DP-1410
Apr 13, 2026
Merged

DP1410: add logic to handle networking behind a proxy, update readme#20
pwildber merged 1 commit into
mainfrom
DP-1410

Conversation

@pwildber
Copy link
Copy Markdown
Contributor

🌐 Add Proxy Support to Deployment Scripts

This PR introduces optional proxy support across the on-prem deployment automation, enabling installations in environments where outbound traffic must go through an HTTP/HTTPS proxy.

✅ What’s Included

  1. Proxy Environment Variable Detection

The scripts now automatically detect standard proxy environment variables if they are set:

HTTP_PROXY, HTTPS_PROXY, NO_PROXY
Lowercase equivalents (http_proxy, etc.)
Optional: FTP_PROXY, ALL_PROXY

No changes are required for users who do not use a proxy.

  1. Podman Image Build Support

Proxy variables are now passed into temporary containers used during:

Podman image customization (build_podman_secret_image)

This ensures package installs and image preparation steps work correctly behind a proxy.

  1. Systemd Service Integration

Proxy variables are injected into generated systemd units:

Docker (iriusrisk-docker.service)
Podman (iriusrisk-podman.service)

This ensures all runtime containers inherit proxy configuration automatically.

  1. Documentation Updates
    Added a new “Proxy Configuration (Optional)” section to the README
    Includes step-by-step setup instructions and verification guidance
    🔁 Backward Compatibility
    Fully backward-compatible
    If no proxy variables are set, the deployment behaves exactly as before
    No breaking changes to existing workflows
    🧪 Testing Considerations

Tested scenarios:

✅ No proxy configured (baseline behavior unchanged)
✅ Proxy variables set → successful install and container runtime
✅ Podman image customization behind proxy
✅ Systemd services inherit proxy configuration correctly
📌 Summary

This change makes the deployment scripts compatible with restricted network environments while remaining completely transparent for standard installations.

@pwildber pwildber requested a review from jsanchezIR April 13, 2026 09:47
@pwildber pwildber merged commit a36f909 into main Apr 13, 2026
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