Skip to content

fix: quote mc-observability DB name in InfluxDB init to handle hyphen #178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MZC-CSC merged 11 commits into from
Jun 30, 2026
Merged

fix: quote mc-observability DB name in InfluxDB init to handle hyphen #178

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
f15edfa
Merge branch 'm-cmp:main' into main
MZC-CSC Jun 19, 2026
ce481b8
Merge pull request #48 from MZC-CSC/develop
MZC-CSC Jun 19, 2026
e18d7f5
Merge pull request #49 from MZC-CSC/develop
MZC-CSC Jun 22, 2026
c436452
fix(iam): FE iframe URL scheme 분기 및 data-manager nginx timeout 보강
dogfootman Jun 22, 2026
96f5bff
Merge branch 'm-cmp:main' into main
MZC-CSC Jun 23, 2026
6579e9c
Merge branch 'm-cmp:main' into main
MZC-CSC Jun 23, 2026
cd7016c
Merge branch 'm-cmp:main' into main
MZC-CSC Jun 26, 2026
e043141
Merge branch 'm-cmp:main' into main
MZC-CSC Jun 29, 2026
6a1221a
Merge branch 'm-cmp:main' into main
MZC-CSC Jun 30, 2026
bcbddc0
fix: quote mc-observability DB name in InfluxDB init to handle hyphen
yh-noh Jun 30, 2026
ad9bd5c
fix: quote mc-observability DB name in InfluxDB init to handle hyphen
MZC-CSC Jun 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions conf/docker/conf/mc-observability/influxdb/influxdb_init/init-database.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,8 @@

PREFIX="influx -host 127.0.0.1 -port $INFLUXDB_INIT_PORT -username ${INFLUXDB_ADMIN_USER} -password ${INFLUXDB_ADMIN_PASSWORD} -execute"

# Quote the DB name: it contains a hyphen, which the InfluxQL parser rejects
# when unquoted (the image's INFLUXDB_DB env triggers an unquoted CREATE and fails).
$PREFIX 'CREATE DATABASE "mc-observability"'
$PREFIX "CREATE DATABASE insight"
$PREFIX "CREATE DATABASE downsampling"
8 changes: 6 additions & 2 deletions conf/docker/docker-compose.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1368,7 +1368,9 @@ services:
- INFLUXDB_HTTP_AUTH_ENABLED=${MC_OBSERVABILITY_INFLUXDB_HTTP_AUTH_ENABLED:-true}
- INFLUXDB_ADMIN_USER=${MC_OBSERVABILITY_INFLUXDB_ADMIN_USER:-mc-agent}
- INFLUXDB_ADMIN_PASSWORD=${MC_OBSERVABILITY_INFLUXDB_ADMIN_PASSWORD:-mc-agent}
- INFLUXDB_DB=${MC_OBSERVABILITY_INFLUXDB_DB:-mc-observability}
# INFLUXDB_DB intentionally omitted: the image entrypoint issues an
# unquoted "CREATE DATABASE mc-observability" which fails on the hyphen.
# The DB is created (quoted) by influxdb_init/init-database.sh instead.
volumes:
- ./conf/mc-observability/influxdb/influxdb_init:/docker-entrypoint-initdb.d
- ./container-volume/mc-observability/influxdb/influxdb_data:/var/lib/influxdb
Expand Down Expand Up @@ -1408,7 +1410,9 @@ services:
- INFLUXDB_HTTP_AUTH_ENABLED=${MC_OBSERVABILITY_INFLUXDB_HTTP_AUTH_ENABLED:-true}
- INFLUXDB_ADMIN_USER=${MC_OBSERVABILITY_INFLUXDB_ADMIN_USER:-mc-agent}
- INFLUXDB_ADMIN_PASSWORD=${MC_OBSERVABILITY_INFLUXDB_ADMIN_PASSWORD:-mc-agent}
- INFLUXDB_DB=${MC_OBSERVABILITY_INFLUXDB_DB:-mc-observability}
# INFLUXDB_DB intentionally omitted: the image entrypoint issues an
# unquoted "CREATE DATABASE mc-observability" which fails on the hyphen.
# The DB is created (quoted) by influxdb_init/init-database.sh instead.
volumes:
- ./conf/mc-observability/influxdb/influxdb_init:/docker-entrypoint-initdb.d
- ./container-volume/mc-observability/influxdb/influxdb2_data:/var/lib/influxdb
Expand Down