[SDNC] Fix typo in setenv script 14/113814/2
authorDan Timoney <dtimoney@att.com>
Mon, 12 Oct 2020 19:57:56 +0000 (15:57 -0400)
committerDan Timoney <dtimoney@att.com>
Tue, 13 Oct 2020 14:05:30 +0000 (10:05 -0400)
The setenv script used to set garbage collection options in Frankfurt
contains a typo.  Unfortunately, since the GC logging options are different
in Frankfurt vs Guilin, we cannot simply cherry pick the Guilin change that
fixed this - we must submit a separate change directly to Frankfurt.

This change changes the setenv script so that it will be identical to that
used in Guilin - the differences are now in sdnc values.yaml. There is
a new setting (gcLogOptions), which is set here to the proper values
to enable GC logging for Frankfurt.

Issue-ID: SDNC-1377
Signed-off-by: Dan Timoney <dtimoney@att.com>
Change-Id: Ibacdb598cc561d64d1e197d45fe485a4701e3dd8

kubernetes/sdnc/resources/config/conf/setenv
kubernetes/sdnc/values.yaml

index 7476e68..3f4ce22 100644 (file)
@@ -53,9 +53,8 @@
 if [ "x$JAVA_MAX_MEM" = "x" ]; then
     export JAVA_MAX_MEM="2048m"
 fi
-
-EXTRA_JAVA_OPTS: "-XX:+UseG1GC -XX:MaxGCPauseMillis={{.Values.config.odl.javaOptions.maxGCPauseMillis}} \
- -XX:ParallelGCThreads={{.Values.config.odl.javaOptions.parallelGCThreads}} -XX:+ParallelRefProcEnabled \
- -XX:+UseStringDeduplication -XX:+PrintGC -XX:+PrintGCDateStamps -XX:+PrintGCDetails \
- -XX:+PrintGCTimeStamps -XX:+UseGCLogFileRotation \
- -XX:NumberOfGCLogFiles={{.Values.config.odl.javaOptions.numberGGLogFiles}} -Xloggc:/var/log/onap/sdnc/gc-%t.log"
+EXTRA_JAVA_OPTS=${EXTRA_JAVA_OPTS:-"-XX:+UseG1GC \
+ -XX:MaxGCPauseMillis={{.Values.config.odl.javaOptions.maxGCPauseMillis}} \
+ -XX:ParallelGCThreads={{.Values.config.odl.javaOptions.parallelGCThreads}} \
+ -XX:+ParallelRefProcEnabled \
+ -XX:+UseStringDeduplication {{.Values.config.odl.javaOptions.gcLogOptions}}"}
index f16f3b1..2ca7ff4 100644 (file)
@@ -186,6 +186,7 @@ config:
       maxGCPauseMillis: 100
       parallelGCThreads : 3
       numberGGLogFiles: 10
+      gcLogOptions: "-XX:+PrintGC -XX:+PrintGCDateStamps -XX:+PrintGCDetails -XX:+PrintGCTimeStamps -XX:+UseGCLogFileRotation -XX:NumberOfGCLogFiles=10 -Xloggc:/var/log/onap/sdnc/gc-%t.log"
 
 # dependency / sub-chart configuration
 certInitializer: