Page MenuHomeDevCentral

Synchronize nodes.sls with NetBox pillar
Needs ReviewPublic

Authored by yousra on Sun, Feb 22, 20:03.
Tags
None
Referenced Files
F24669974: D3974.id10302.diff
Wed, Mar 4, 08:42
Unknown Object (File)
Wed, Mar 4, 06:17
Unknown Object (File)
Wed, Mar 4, 05:14
Unknown Object (File)
Wed, Mar 4, 05:06
Unknown Object (File)
Wed, Mar 4, 03:18
Unknown Object (File)
Tue, Mar 3, 21:57
Unknown Object (File)
Tue, Mar 3, 19:17
Unknown Object (File)
Tue, Mar 3, 18:14
Subscribers

Details

Summary

Used pillarize.py (D2790) to compare generated pillar data with nodes.sls and aligned both configurations.

Ref T2203

Test Plan

The new network configuration has been successfully deployed on all machines. All errors that occurred during the deployment were resolved, except for one documented in T2269.

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Skipped
Unit
No Test Coverage
Branch
pillar-nodes
Build Status
Buildable 6405
Build 6689: arc lint + arc unit

Event Timeline

yousra requested review of this revision.Sun, Feb 22, 20:03
yousra created this revision.

Fix priority for CARP in NetBox, and then in the file nodes.sls

Best practice:

  • MASTER router: advskew 0 (effective priority 255)
  • BACKUP router: advskew 100 (effective priority 155)

Use True and False (capitalized) to follow Salt's Python boolean conventions.

Add the flag ipv6_dhcp_duid for windriver

Hostnames should always be written in lowercase and the node name must match the hostname.

This comment was removed by yousra.

Adjusted the ordering of entries (and the content) in nodes.sls to maintain consistency with the generated pillar structure and ensure deterministic output.

Rename network interfaces on selected machines to standardize the naming convention across all hosts, using intranought and public.

Add canonical_public_ipv4 for Ysul

When I was trying to do salt 'router-001' state.apply roles/core/network, an error
occurs saying that canonical_public_ipv4 is missing, so the router-001 has it but not
the gre destination Ysul.

Add some changes : some flags where missing

dereckson requested changes to this revision.Wed, Mar 4, 07:37

Virtually OK, but one critical flag missing from pillarize for ongoing IPv6 connectivity

pillar/nodes/nodes.sls
357–358

That one really need hello_ipv6_ovh, we should understand why here pillarize.py doesn't add it.

This revision now requires changes to proceed.Wed, Mar 4, 07:37

The flag hello_ipv6_ovh was missing in web-001, because in netbox the ipv6 was in /56 not in /64.