Skip to content

Commit 592a4cb

Browse files
Merge pull request #15891 from jim-minter/tsb-loglevel
Automatic merge from submit-queue (batch tested with PRs 15861, 15865, 15873, 15848, 15891) oc cluster up: pass server loglevel to template service broker @gabemontero @deads2k fyi
2 parents 0ff757b + bf69cda commit 592a4cb

File tree

5 files changed

+13
-3
lines changed

5 files changed

+13
-3
lines changed

examples/templateservicebroker/templateservicebroker-template.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@ parameters:
1111
value: openshift-template-service-broker
1212
- name: KUBE_SYSTEM
1313
value: kube-system
14+
- name: LOGLEVEL
15+
value: "0"
1416
objects:
1517

1618
# to create the tsb server
@@ -41,6 +43,7 @@ objects:
4143
- "--tls-cert-file=/var/serving-cert/tls.crt"
4244
- "--tls-private-key-file=/var/serving-cert/tls.key"
4345
- "--template-namespace=${TSB_TEMPLATE_NAMESPACE}"
46+
- "--loglevel=${LOGLEVEL}"
4447
ports:
4548
- containerPort: 8443
4649
volumeMounts:

pkg/oc/bootstrap/bindata.go

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

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

+3-2
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ const (
2323
)
2424

2525
// InstallServiceCatalog checks whether the template service broker is installed and installs it if not already installed
26-
func (h *Helper) InstallTemplateServiceBroker(f *clientcmd.Factory, imageFormat string) error {
26+
func (h *Helper) InstallTemplateServiceBroker(f *clientcmd.Factory, imageFormat string, serverLogLevel int) error {
2727
osClient, kubeClient, err := f.Clients()
2828
if err != nil {
2929
return errors.NewError("cannot obtain API clients").WithCause(err).WithDetails(h.OriginLog())
@@ -45,7 +45,8 @@ func (h *Helper) InstallTemplateServiceBroker(f *clientcmd.Factory, imageFormat
4545
imageTemplate.Latest = false
4646

4747
if err = instantiateTemplate(osClient, clientcmd.ResourceMapper(f), tsbNamespace, tsbTemplateName, tsbNamespace, map[string]string{
48-
"IMAGE": imageTemplate.ExpandOrDie(""),
48+
"IMAGE": imageTemplate.ExpandOrDie(""),
49+
"LOGLEVEL": fmt.Sprint(serverLogLevel),
4950
}, true); err != nil {
5051
return errors.NewError("cannot instantiate logger accounts").WithCause(err)
5152
}

pkg/oc/bootstrap/docker/up.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -1066,7 +1066,7 @@ func (c *ClientStartConfig) InstallTemplateServiceBroker(out io.Writer) error {
10661066
}
10671067
// TODO we want to use this eventually, but until we have our own image for TSB, we have to hardcode this origin
10681068
//return c.OpenShiftHelper().InstallTemplateServiceBroker(f, c.imageFormat())
1069-
return c.OpenShiftHelper().InstallTemplateServiceBroker(f, c.Image)
1069+
return c.OpenShiftHelper().InstallTemplateServiceBroker(f, c.Image, c.ServerLogLevel)
10701070
}
10711071

10721072
// Login logs into the new server and sets up a default user and project

test/extended/testdata/bindata.go

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

0 commit comments

Comments
 (0)