dev 1.1.0 // Change log output for merge from external keystore

[derek-shnosh]

Goals

• Avoid outputting the configuration to the log file in plain text.
• Still have a 'chunky' log message for the successful merge to an external keystore and template association.

Concerns

• Will the identity and templatename variables pass through from def get_template?

[derek-shnosh]

I think I've discovered why the full config is being dumped to the log, maybe you can confirm @PackeTsar.

We just had two of twelve switches only get a partial config. I'm unsure if this was because the switch didn't parse it all or because ZTP didn't send it all, since my current log message is simply log("cfact.request: Returning the merged config to TFTPy.")

If so, I'd like to instead suggest that the merged config files are written to disk somewhere; e.g. /etc'ztp/merged_cfgs.

A write erase and reload resulted in the switch getting the full configuration. For now, I have put the default log message back to assist with troubleshooting if this happens again.

Update: Another switch got a partial config. Checked the log file and saw the full configuration as returned to TFPTy. Like before, this was resolved by a write erase and reload.