Skip to content

Add configuration flexibility for network and MTU settings - #246

Open
chenwng wants to merge 2 commits into
shiftstack:mainfrom
chenwng:fix-network-issues
Open

Add configuration flexibility for network and MTU settings#246
chenwng wants to merge 2 commits into
shiftstack:mainfrom
chenwng:fix-network-issues

Conversation

@chenwng

@chenwng chenwng commented Aug 26, 2026

Copy link
Copy Markdown

This PR fixes following issues I found during deployment.

  • os-net-config fails when the max_mtu of dummy interfaces is 0 . Add enable_dummy_mtu to skip setting mtu for dummy interfaces.
  • If external_fip_pool_start and external_fip_pool_end are not defined, network external will not be created. Running make prepare_stack_testconfig will fail when setting the gateway for the private router. Add external_network to allow using hostonly.
  • If rhsm_enabled is not true, NtpServer in playbooks/templates/standalone_parameters.yaml.j2 will not be set, which results chrony sync failure during deployment in internal network. Move NtpServer out of rhsm_enabled check to allow setting internal ntp server.

* Make dummy interface MTU optional via enable_dummy_mtu flag to handle
cases where max mtu = 0 causes os-net-config failures.
* Make external network name configurable to allow using hostonly network
as router external gateway.
* Move NTP server configuration outside RHSM block for independent control.
@chenwng
chenwng force-pushed the fix-network-issues branch from cb6350a to c643b94 Compare August 26, 2026 12:26
@mandre

mandre commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

I believe I've already fixed this issue last time I deployed with dev-install. I'm not in front of the computer right now so can't confirm, have a look at the open PRs in this repo.

@chenwng

chenwng commented Aug 27, 2026

Copy link
Copy Markdown
Author

I found #245 tries to fix the mtu issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants