Merge pull request #527 from srl-labs/fix-ceos-startupcfg

fix variable shadowing in ceos startup cfg templating

docs/rn/0.15.md

@@ -43,6 +43,9 @@ With that in mind we launched the containerlab's own [discord server](https://di
 * Fixed directories creation for `ignite` runtimes
 * Fixed release notes links in `containerlab version` command
 
+### 0.15.2
+* Fixed startup-config provisioning for ceos nodes [#526](https://github.com/srl-labs/containerlab/issues/526)
+
 [^1]: as opposed to a VM-way by running Cumulus with [vrnetlab](../manual/vrnetlab.md)
 [^2]: both with `ignite` runtime or `docker` runtime
 [^3]: starting with SR Linux 21.3, the license file is optional

nodes/ceos/ceos.go

@@ -123,9 +123,9 @@ func createCEOSFiles(node *types.NodeConfig) error {
 		if err != nil {
 			return err
 		}
-		cfgTemplate = string(c)
+		tpl := string(c)
 
-		err = node.GenerateConfig(node.ResStartupConfig, cfgTemplate)
+		err = node.GenerateConfig(node.ResStartupConfig, tpl)
 		if err != nil {
 			return err
 		}
@@ -146,6 +146,7 @@ func ceosPostDeploy(ctx context.Context, r runtime.ContainerRuntime, nodeCfg *ty
 	if nodeCfg.StartupConfig != "" {
 		return nil
 	}
+
 	// regenerate ceos config since it is now known which IP address docker assigned to this container
 	err := nodeCfg.GenerateConfig(nodeCfg.ResStartupConfig, cfgTemplate)
 	if err != nil {