Skip to content

Commit ab79a0c

Browse files
committed
ensure builder labels override with proper priority
1 parent 4b59312 commit ab79a0c

File tree

4 files changed

+38
-1
lines changed

4 files changed

+38
-1
lines changed

test/extended/builds/labels.go

+6-1
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,11 @@ func ExpectOpenShiftLabels(labels map[string]string) error {
113113
return fmt.Errorf("Built image doesn't contain proper Docker image labels. Missing %q label", label)
114114
}
115115
}
116-
116+
if labels["io.k8s.display-name"] != "overridden" {
117+
return fmt.Errorf("Existing label was not overridden with user specified value: %s=%s", labels["io.k8s.display-name"], labels["overridden"])
118+
}
119+
if labels["io.openshift.builder-version"] != "overridden2" {
120+
return fmt.Errorf("System generated label was not overridden with user specified value: %s=%s", labels["io.openshift.builder-version"], labels["overridden2"])
121+
}
117122
return nil
118123
}

test/extended/testdata/bindata.go

+16
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

test/extended/testdata/test-docker-build.json

+8
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,14 @@
3434
{
3535
"name": "user-specified-label",
3636
"value": "arbitrary-value"
37+
},
38+
{
39+
"name": "io.k8s.display-name",
40+
"value": "overridden"
41+
},
42+
{
43+
"name": "io.openshift.builder-version",
44+
"value": "overridden2"
3745
}
3846
]
3947
}

test/extended/testdata/test-s2i-build.json

+8
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,14 @@
4040
{
4141
"name": "user-specified-label",
4242
"value": "arbitrary-value"
43+
},
44+
{
45+
"name": "io.k8s.display-name",
46+
"value": "overridden"
47+
},
48+
{
49+
"name": "io.openshift.builder-version",
50+
"value": "overridden2"
4351
}
4452
]
4553
}

0 commit comments

Comments
 (0)