Katenary/katenary
2
0
Fork 0
Code Issues 7 Pull Requests Actions Packages Projects Releases 15 Wiki Activity

fix(configmap): do not write env var in file CM

Browse Source 
File CM are configmap to store "static" data (file content), do not set
environment variables inside.
This commit is contained in:
Patrice Ferlet
2024-11-22 14:55:42 +01:00
parent 95f3abfa74
commit 1a1d2b5ee8
1 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
generator/configMap.go
View File

@@ -109,14 +109,14 @@ func NewConfigMap(service types.ServiceConfig, appName string, forFile bool) *Co
done[key] = true done[key] = true
} }
} }
} for key, env := range service.Environment {
for key, env := range service.Environment { _, isDropped := drop[key]
_, isDropped := drop[key] _, isDone := done[key]
_, isDone := done[key] if isDropped || isDone {
if isDropped || isDone { continue
continue }
cm.AddData(key, *env)
} }
cm.AddData(key, *env)
} }
return cm return cm
@@ -168,7 +168,6 @@ func (c *ConfigMap) AppendDir(path string) {
if err != nil { if err != nil {
log.Fatalf("Path %s does not exist\n", path) log.Fatalf("Path %s does not exist\n", path)
} }
log.Printf("Appending files from %s to configmap\n", path)
// recursively read all files in the path and add them to the configmap // recursively read all files in the path and add them to the configmap
if stat.IsDir() { if stat.IsDir() {
files, err := os.ReadDir(path) files, err := os.ReadDir(path)