Skip to content

Commit f13cc80

Browse files
idsulikndeloof
authored andcommitted
convert LabelFile into slice of strings
Signed-off-by: Suleiman Dibirov <[email protected]>
1 parent 46eba20 commit f13cc80

File tree

9 files changed

+17
-89
lines changed

9 files changed

+17
-89
lines changed

loader/full-struct_test.go

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -227,13 +227,9 @@ func services(workingDir, homeDir string) types.Services {
227227
"com.example.number": "42",
228228
"com.example.empty-label": "",
229229
},
230-
LabelFiles: []types.LabelFile{
231-
{
232-
Path: filepath.Join(workingDir, "example1.label"),
233-
},
234-
{
235-
Path: filepath.Join(workingDir, "example2.label"),
236-
},
230+
LabelFiles: []string{
231+
filepath.Join(workingDir, "example1.label"),
232+
filepath.Join(workingDir, "example2.label"),
237233
},
238234
Links: []string{
239235
"db",

loader/loader_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2331,8 +2331,8 @@ func TestLoadServiceWithLabelFile(t *testing.T) {
23312331
Services: types.Services{
23322332
"test": {
23332333
Name: "test",
2334-
LabelFiles: []types.LabelFile{
2335-
{Path: file.Name()},
2334+
LabelFiles: []string{
2335+
file.Name(),
23362336
},
23372337
},
23382338
},
@@ -2349,8 +2349,8 @@ func TestLoadServiceWithLabelFile_NotExists(t *testing.T) {
23492349
Services: types.Services{
23502350
"test": {
23512351
Name: "test",
2352-
LabelFiles: []types.LabelFile{
2353-
{Path: "test"},
2352+
LabelFiles: []string{
2353+
"test",
23542354
},
23552355
},
23562356
},

override/uncity.go

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ func init() {
5151
unique["services.*.env_file"] = envFileIndexer
5252
unique["services.*.expose"] = exposeIndexer
5353
unique["services.*.labels"] = keyValueIndexer
54-
unique["services.*.label_file"] = labelFileIndexer
5554
unique["services.*.links"] = keyValueIndexer
5655
unique["services.*.networks.*.aliases"] = keyValueIndexer
5756
unique["services.*.networks.*.link_local_ips"] = keyValueIndexer
@@ -228,16 +227,3 @@ func envFileIndexer(y any, p tree.Path) (string, error) {
228227
}
229228
return "", nil
230229
}
231-
232-
func labelFileIndexer(y any, p tree.Path) (string, error) {
233-
switch value := y.(type) {
234-
case string:
235-
return value, nil
236-
case map[string]any:
237-
if pathValue, ok := value["path"]; ok {
238-
return pathValue.(string), nil
239-
}
240-
return "", fmt.Errorf("label path attribute %s is missing", p)
241-
}
242-
return "", nil
243-
}

paths/resolve.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ func ResolveRelativePaths(project map[string]any, base string, remotes []RemoteR
3737
"services.*.build.context": r.absContextPath,
3838
"services.*.build.additional_contexts.*": r.absContextPath,
3939
"services.*.env_file.*.path": r.absPath,
40-
"services.*.label_file.*.path": r.absPath,
40+
"services.*.label_file.*": r.absPath,
4141
"services.*.extends.file": r.absExtendsPath,
4242
"services.*.develop.watch.*.path": r.absSymbolicLink,
4343
"services.*.volumes.*": r.absVolumeMount,

transform/canonical.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ func init() {
3030
transformers["services.*.build.additional_contexts"] = transformKeyValue
3131
transformers["services.*.depends_on"] = transformDependsOn
3232
transformers["services.*.env_file"] = transformEnvFile
33-
transformers["services.*.label_file"] = transformLabelFile
3433
transformers["services.*.extends"] = transformExtends
3534
transformers["services.*.networks"] = transformServiceNetworks
3635
transformers["services.*.volumes.*"] = transformVolumeMount

transform/labelfile.go

Lines changed: 0 additions & 55 deletions
This file was deleted.

types/derived.gen.go

Lines changed: 4 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

types/project.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -712,12 +712,12 @@ func loadEnvFile(envFile EnvFile, resolve dotenv.LookupFn) (Mapping, error) {
712712
return loadMappingFile(envFile.Path, envFile.Format, resolve)
713713
}
714714

715-
func loadLabelFile(labelFile LabelFile, resolve dotenv.LookupFn) (Mapping, error) {
716-
if _, err := os.Stat(labelFile.Path); os.IsNotExist(err) {
717-
return nil, fmt.Errorf("label file %s not found: %w", labelFile.Path, err)
715+
func loadLabelFile(labelFile string, resolve dotenv.LookupFn) (Mapping, error) {
716+
if _, err := os.Stat(labelFile); os.IsNotExist(err) {
717+
return nil, fmt.Errorf("label file %s not found: %w", labelFile, err)
718718
}
719719

720-
return loadMappingFile(labelFile.Path, labelFile.Format, resolve)
720+
return loadMappingFile(labelFile, "", resolve)
721721
}
722722

723723
func loadMappingFile(path string, format string, resolve dotenv.LookupFn) (Mapping, error) {

types/types.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ type ServiceConfig struct {
8989
Ipc string `yaml:"ipc,omitempty" json:"ipc,omitempty"`
9090
Isolation string `yaml:"isolation,omitempty" json:"isolation,omitempty"`
9191
Labels Labels `yaml:"labels,omitempty" json:"labels,omitempty"`
92-
LabelFiles []LabelFile `yaml:"label_file,omitempty" json:"label_file,omitempty"`
92+
LabelFiles []string `yaml:"label_file,omitempty" json:"label_file,omitempty"`
9393
CustomLabels Labels `yaml:"-" json:"-"`
9494
Links []string `yaml:"links,omitempty" json:"links,omitempty"`
9595
Logging *LoggingConfig `yaml:"logging,omitempty" json:"logging,omitempty"`

0 commit comments

Comments
 (0)