Bug 52552 – Make DockerScriptInit configurable

Bug 52552 - Make DockerScriptInit configurable


Summary:	Make DockerScriptInit configurable

Status:	RESOLVED DUPLICATE of bug 52839

Product:	UCS
Classification:	Unclassified
Component:	Docker
Version:	UCS 4.4
Hardware:	All All

Importance:	P3 enhancement (vote)
Target Milestone:	---
Assigned To:	App Center maintainers
QA Contact:	App Center maintainers

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2021-01-07 10:45 CET by Jan-Luca Kiok
Modified:	2021-03-24 10:53 CET (History)
CC List:	3 users (show)

See Also:
What kind of report is it?:	Feature Request
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:	Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):	Design
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Jan-Luca Kiok

2021-01-07 10:45:55 CET

Currently there is no way to edit the DockerScriptInit of an app.
For the Dashboard Database, which is a containerized Prometheus, the only way to change retention rules like saving period or database size is via flag on start as set by DockerScriptInit. (see https://github.com/prometheus/prometheus/issues/6188 for the reasoning)

While there might be a sufficient way to rewrite the Prometheus config, my guess is that with a more containerized UCS5 the need to change the ScriptInit will arise for other apps too.

Comment 1 Nico Gulden

2021-01-18 15:37:05 CET

How is this flag passed to the container? Via environment variable?

Can you please provide an example for a docker command that would configure the desired value?

Comment 2 Jan-Luca Kiok

2021-01-18 16:53:28 CET

I guess the word flag is ambiguous in this context. Speaking of it in terms of Golang flags, you have to supply it as a command option like:
"docker run prometheus --storage.tsdb.retention.time=30d --config.file=/etc/prometheus/prometheus.yml ..." which also would mean changing CMD in a Dockerfile or "command:" in docker-compose.
As stated in linked discussion, there is and will be no other way for now to configure this via file, env or similar.

Comment 3 Sönke Schwardt-Krummrich

2021-03-24 10:53:45 CET


*** This bug has been marked as a duplicate of bug 52839 ***