[ENH] Add an option to dump the values of all the configuration variables

[frankcrawford]

If would be handy to add another to 08-ipv6-prefix to dump out all the values of the configuration variables.

This would help with debugging as you can quickly see what the configuration is, rather than assuming a setting, or possibly mistyping something and missing it.

This came up for me, because I installed the latest release with the pkg option, and forgot that NMCONF was different, so spent time tracking why it was not changed, where as with a dump of the actual settings, I probably would have quick realised I had the configuration files in the wrong place.

[sshambar]

Yeah, I debated the config move on packages (it was originally to allow incompatible versions of package and source versions to be installed simultaneously) but in the end I made the config backward compatible - but I really should have put a warning in the release notes (probably should still :). I also tried to find a "fool-proof" package upgrade scriptlet to copy the config over on initial upgrade, but figured it was too prone to "unintended consequences"...

But yes, some kind of config summary might be useful... which would be better?
a) have a "config" command option
b) show config in "status"
c) something else?

[frankcrawford]

I don't think the move is a bad idea for packages, and I did find there were a least 2 notes about it, but of course I didn't see it until after I was bitten by it, so, maybe a warning note would be handy.

As for the additional command, I think a separate "config" command would be best, as it isn't needed at the same time as using "status".

[sshambar]

Frank, I've added support for a new "config" command to both 08-ipv6-prefix and 09-ddns scripts. They will display all "non-default" values set, and show the "config-file" loaded to obtain the values (even if it's not found). They both have options to only display config for requested interfaces using{script} config <interface>..., and also an option to dump all documented variables (with their default values if not set) {script} config -a [ <interface>... ]

I've also added a 09-ddns help to match 08-ipv6-prefix help - I haven't decided if a status command makes sense for ddns yet :)

Let me know what you think, and if you think the output could be tweaked a bit!

[frankcrawford]

Thanks. I'll test it out this weekend.

[frankcrawford]

That looks good, both the ability to see the full list including defaults, or just the differences.

If there was any change, it would be to somehow mark the differences in the full list, but it isn't a critical issue.