Skip to content
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.

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

WX-1122 ApplicationInsights for Cromwell #260

Merged
merged 13 commits into from
Jun 22, 2023
Merged
Changes from 1 commit
Commits
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
Prev Previous commit
Next Next commit
Logback config
jgainerdewar committed Jun 12, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit 07d259749932ebd12a625e2e0425400721d69e65
40 changes: 39 additions & 1 deletion coa-helm/templates/cromwell.yaml
Original file line number Diff line number Diff line change
@@ -13,7 +13,12 @@ spec:
- name: APPLICATIONINSIGHTS_CONNECTION_STRING
value: {{ .Values.config.applicationInsightsConnectionString | quote }}
args:
- java -Djava.io.tmpdir=/cromwell-tmp/ -Dconfig.file=/configuration/cromwell/cromwell-application.conf -jar /app/cromwell.jar server
- java
-Djava.io.tmpdir=/cromwell-tmp/
-Dconfig.file=/configuration/cromwell/cromwell-application.conf
-Dlogback.configurationFile=/configuration/cromwell/logback.xml
-jar /app/cromwell.jar
server
command:
- /bin/sh
- -c
@@ -67,6 +72,39 @@ status:
{{ end }}
{{ define "coa-helm.cromwell-config" }}
data:
logback.xml: |-
<configuration>
<!-- Enhanced thread and date reporting. -->
<conversionRule
conversionWord="et"
converterClass="cromwell.core.logging.EnhancedThreadConverter"
/>
<conversionRule
conversionWord="ed"
converterClass="cromwell.core.logging.EnhancedDateConverter"
/>

<!-- We are sending logs to ApplicationInsights, no need to print them to console as well. -->
<appender name="NOP_APPENDER" class="ch.qos.logback.core.helpers.NOPAppender">
<encoder>
<pattern>%ed{yyyy-MM-dd HH:mm:ss,SSS} %et %-5level - %msg%n</pattern>
</encoder>
</appender>

<root level="INFO">
<appender-ref ref="NOP_APPENDER" />
</root>

<logger name="liquibase" level="WARN"/>
<logger name="com.zaxxer.hikari" level="ERROR"/>
<logger name="HikariPool" level="ERROR"/>
<logger name="com.google.cloud.hadoop.gcsio.GoogleCloudStorageReadChannel" level="ERROR"/>
<logger name="org.semanticweb.owlapi.utilities.Injector" level="ERROR"/>

<!-- Guilty of logging ERROR under non-erroneous conditions -->
<logger name="com.azure.storage.blob.nio" level="OFF"/>
</configuration>

cromwell-application.conf: |-
include required(classpath("application"))