Skip to content

Add '--suffix' parameter to 'netlab collect'#2571

Merged
ipspace merged 1 commit intodevfrom
collect-sfx
Aug 4, 2025
Merged

Add '--suffix' parameter to 'netlab collect'#2571
ipspace merged 1 commit intodevfrom
collect-sfx

Conversation

@ipspace
Copy link
Owner

@ipspace ipspace commented Aug 2, 2025

The 'netlab collect' command usually saves configuration files with the '.cfg' suffix (which can then be used in 'netlab up --reload' process). However, when building a multi-platform solution, it might be helpful to save the configuration files with a platform-specific suffix, for example 'eos.j2'. The '--suffix' parameter allows you to replace the standard 'cfg' suffix with something else

As a few 'fetch-config' task lists used hardcoded '.cfg' suffix, most of them were either fixed to use the 'cfg_suffix' variable or (even better) set 'ansible_net_config' fact instead of creating the output file directly.

The 'netlab collect' command usually saves configuration files with
the '.cfg' suffix (which can then be used in 'netlab up --reload'
process). However, when building a multi-platform solution, it might
be helpful to save the configuration files with a platform-specific
suffix, for example 'eos.j2'. The '--suffix' parameter allows you to
replace the standard 'cfg' suffix with something else

As a few 'fetch-config' task lists used hardcoded '.cfg' suffix,
most of them were either fixed to use the 'cfg_suffix' variable or
(even better) set 'ansible_net_config' fact instead of creating the
output file directly.
@ipspace
Copy link
Owner Author

ipspace commented Aug 2, 2025

I was sick-and-tired of renaming configuration files to platform-specific templates while writing the IS-IS labs, so I added another nerd knob to "netlab collect" and fixed a bit of technical debt while doing that. The FRR and SR Linux changes were tested; I hope I'm awake enough to do a find-and-replace on the CL tasklist ;)

Adding reviewers just in case someone wants to look at this or comment.

@ipspace ipspace merged commit f3053c1 into dev Aug 4, 2025
11 checks passed
@ipspace ipspace deleted the collect-sfx branch August 4, 2025 16:03
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