Skip to content
This repository was archived by the owner on Feb 3, 2025. It is now read-only.

changing dnsmasq pre start script #2

Merged
mjudeikis merged 1 commit intojewzaam:release-4.7-azurefrom
Makdaam:idempotentresolv
Jul 1, 2021
Merged

changing dnsmasq pre start script #2
mjudeikis merged 1 commit intojewzaam:release-4.7-azurefrom
Makdaam:idempotentresolv

Conversation

@Makdaam
Copy link
Copy Markdown

@Makdaam Makdaam commented Jun 1, 2021

to an idempotent one and ready for both OVSSDN and OVN.
Adds a separate executable script.

@Makdaam
Copy link
Copy Markdown
Author

Makdaam commented Jun 1, 2021

  1. Cleaner than the 1-liner we had in the Systemd Unit.
  2. Works also on OVN clusters where br-ex is the main outbound interface instead of eth0.
  3. Can be executed multiple times without changing the resulting files.

Tested by replacing my local /vendor/ copy with this installer and creating a cluster with OVS, then starting a migration to OVN (https://docs.openshift.com/container-platform/4.7/networking/ovn_kubernetes_network_provider/migrate-from-openshift-sdn.html - requires rebooting masters manually), master and worker nodes come back in OVN mode.

@Makdaam Makdaam force-pushed the idempotentresolv branch from 4d6b43c to 1964ab6 Compare June 3, 2021 09:30
@mjudeikis mjudeikis merged commit 92bc15b into jewzaam:release-4.7-azure Jul 1, 2021
rolandmkunkel pushed a commit that referenced this pull request May 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants