Skip to content

Commit d5f454d

Browse files
Merge pull request #18191 from spadgett/cluster-up-remove-old-asset-config
Automatic merge from submit-queue (batch tested with PRs 18191, 18264, 18235, 18251, 18271). cluster up: remove obsolete asset config properties Follow on to #18114 Requires openshift/origin-web-console-server#20 /assign @deads2k @jwforres @sdodson FYI /hold
2 parents ae43279 + 296729c commit d5f454d

File tree

4 files changed

+12
-83
lines changed

4 files changed

+12
-83
lines changed

install/origin-web-console/console-config.yaml

Lines changed: 4 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,10 @@ clusterInfo:
66
logoutPublicURL: ""
77
masterPublicURL: https://127.0.0.1:8443
88
metricsPublicURL: ""
9-
# TODO: The new extensions properties cannot be set until
10-
# origin-web-console-server has been updated with the API changes since
11-
# `extensions` in the old asset config was an array.
12-
#extensions:
13-
# scriptURLs: []
14-
# stylesheetURLs: []
15-
# properties: null
9+
extensions:
10+
scriptURLs: []
11+
stylesheetURLs: []
12+
properties: null
1613
features:
1714
inactivityTimeoutMinutes: 0
1815
clusterResourceOverridesEnabled: false
@@ -25,19 +22,3 @@ servingInfo:
2522
maxRequestsInFlight: 0
2623
namedCertificates: null
2724
requestTimeoutSeconds: 0
28-
29-
# START deprecated properties
30-
# These properties have been renamed and will be removed from the install
31-
# in a future pull. Keep both the old and new properties for now so that
32-
# the install is not broken while the origin-web-console image is updated.
33-
extensionDevelopment: false
34-
extensionProperties: null
35-
extensionScripts: null
36-
extensionStylesheets: null
37-
extensions: null
38-
loggingPublicURL: ""
39-
logoutURL: ""
40-
masterPublicURL: https://127.0.0.1:8443
41-
metricsPublicURL: ""
42-
publicURL: https://127.0.0.1:8443/console/
43-
# END deprecated properties

pkg/oc/bootstrap/bindata.go

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

pkg/oc/bootstrap/docker/openshift/webconsole.go

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -73,20 +73,6 @@ func (h *Helper) InstallWebConsole(f *clientcmd.Factory, imageFormat string, ser
7373
clusterInfo["metricsPublicURL"] = metricsURL
7474
}
7575

76-
// START deprecated properties
77-
// These properties have been renamed and will be removed from cluster up
78-
// in a future pull. Keep both the old and new properties for now so that
79-
// the cluster up is not broken while the origin-web-console image is updated.
80-
consoleConfig["publicURL"] = publicURL
81-
consoleConfig["masterPublicURL"] = masterURL
82-
if len(loggingURL) > 0 {
83-
consoleConfig["loggingPublicURL"] = loggingURL
84-
}
85-
if len(metricsURL) > 0 {
86-
consoleConfig["metricsPublicURL"] = metricsURL
87-
}
88-
// END deprecated properties
89-
9076
// serialize it back out as a string to use as a template parameter
9177
updatedConfig, err := yaml.Marshal(consoleConfig)
9278
if err != nil {

test/extended/testdata/bindata.go

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

0 commit comments

Comments
 (0)