Skip to content

Commit aa3c157

Browse files
committed
Merge branch '3031-syntax-error-in-install-extensions-managed-script-on-distributed-logs-cluster' into 'main-1.17'
fix: syntax error in install-extensions script on SGDistributedLogs See merge request ongresinc/stackgres!1693
2 parents 04bc03b + 8df1ad1 commit aa3c157

3 files changed

Lines changed: 10 additions & 4 deletions

File tree

stackgres-k8s/src/operator/src/main/java/io/stackgres/operator/conciliation/factory/distributedlogs/DistributedLogsScript.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,12 @@ private StackGresScript getScript(StackGresDistributedLogsContext context) {
8888
cluster.getMetadata().getNamespace(),
8989
cluster.getMetadata().getName()) + "'")
9090
.collect(Collectors.joining(","));
91+
final String template1AndDatabaseList = Seq.of("'template1'")
92+
.append(context.getConnectedClusters().stream()
93+
.map(cluster -> "'" + FluentdUtil.databaseName(
94+
cluster.getMetadata().getNamespace(),
95+
cluster.getMetadata().getName()) + "'"))
96+
.collect(Collectors.joining(","));
9197
final String databaseAndRetenentionList = context.getConnectedClusters().stream()
9298
.map(cluster -> "['" + FluentdUtil.databaseName(
9399
cluster.getMetadata().getNamespace(),
@@ -122,7 +128,7 @@ private StackGresScript getScript(StackGresDistributedLogsContext context) {
122128
"/distributed-logs/install-extensions.sql"),
123129
StandardCharsets.UTF_8)
124130
.read()).get().formatted(
125-
Optional.of(databaseList).filter(Predicate.not(String::isEmpty)).orElse("null"),
131+
Optional.of(template1AndDatabaseList).filter(Predicate.not(String::isEmpty)).orElse("null"),
126132
timescaledbVersion,
127133
isPendingRestart))
128134
.endScript()
@@ -136,7 +142,7 @@ private StackGresScript getScript(StackGresDistributedLogsContext context) {
136142
"/distributed-logs/init.sql"),
137143
StandardCharsets.UTF_8)
138144
.read()).get().formatted(
139-
Optional.of(databaseList).filter(Predicate.not(String::isEmpty)).orElse("null")))
145+
Optional.of(template1AndDatabaseList).filter(Predicate.not(String::isEmpty)).orElse("null")))
140146
.endScript()
141147
.addNewScript()
142148
.withId(2)

stackgres-k8s/src/operator/src/main/resources/distributed-logs/init.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
DO $distributedlogsinit$
22
DECLARE database_name text;
33
BEGIN
4-
FOREACH database_name IN ARRAY ARRAY['template1',%1$s]
4+
FOREACH database_name IN ARRAY ARRAY[%1$s]
55
LOOP
66
IF EXISTS (SELECT * FROM pg_database WHERE datname = database_name) THEN
77
PERFORM dblink(

stackgres-k8s/src/operator/src/main/resources/distributed-logs/install-extensions.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
DO $upgradeextensions$
33
DECLARE database_name text;
44
BEGIN
5-
FOREACH database_name IN ARRAY ARRAY['template1',%1$s]
5+
FOREACH database_name IN ARRAY ARRAY[%1$s]
66
LOOP
77
IF EXISTS (SELECT * FROM pg_database WHERE datname = database_name) THEN
88
PERFORM dblink(

0 commit comments

Comments
 (0)