Skip to content

fix: include PHXDC-ai-pve1 in PVE_NODES for prune_iptables.sh#32

Merged
runleveldev merged 1 commit intomainfrom
issue-31
Sep 12, 2025
Merged

fix: include PHXDC-ai-pve1 in PVE_NODES for prune_iptables.sh#32
runleveldev merged 1 commit intomainfrom
issue-31

Conversation

@cmyers-mieweb
Copy link
Copy Markdown
Collaborator

Summary:
This PR updates the prune_iptables.sh script to include the PHXDC-ai-pve1 server in the PVE_NODES array. This ensures that containers running on PHXDC-ai-pve1 are considered when pruning port mappings, preventing accidental removal of valid port configurations from /etc/nginx/port_map.json.

Impact:

Prevents loss of port mappings for containers on PHXDC-ai-pve1.
Ensures correct connectivity and access for all relevant containers.
Closes: #31

Copy link
Copy Markdown

@FoxieHazmat FoxieHazmat left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Copy Markdown
Collaborator

@runleveldev runleveldev left a comment

Choose a reason for hiding this comment

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

Looks okay. May want to make this dynamic using pvecm in the future.

@runleveldev runleveldev merged commit cf55aaa into main Sep 12, 2025
3 checks passed
@runleveldev runleveldev deleted the issue-31 branch September 12, 2025 20:14
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.

Fix prune_iptables.sh to include pct list of PHXDC-ai-pve1 (HIGH PRIORITY)

3 participants