Skip to content

Commit

Permalink
branch got out of sync with develop
Browse files Browse the repository at this point in the history
more things
  • Loading branch information
snf2ye committed Dec 17, 2024
1 parent cc8d08c commit 200768f
Show file tree
Hide file tree
Showing 23 changed files with 954 additions and 844 deletions.
2 changes: 1 addition & 1 deletion .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ updates:
- "dependency"
- "gradle"
commit-message:
prefix: "[DCJ-400-gradle]"
prefix: "[DT-400-gradle]"
ignore:
# Google API dependencies use a version format that Dependabot wrongly interprets as semver.
- dependency-name: "com.google.apis:*"
Expand Down
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,13 +29,13 @@ See [started guide](docs/jade-getting-started.md) for information on running con
### Run TDR locally

To run TDR locally:
`./scripts/run local`
`./scripts/run start_local`

To run TDR in docker:
`./scripts/run docker`
`./scripts/run start_docker`

To run TDR locally and wait for debugger to attach on port 5005:
`./scripts/run local --debug-jvm`
`./scripts/run start_local --debug-jvm`

To have the code hot reload, enable automatic builds in intellij, go to:
`Preferences -> Build, Execution, Deployment -> Compiler`
Expand Down
1 change: 0 additions & 1 deletion scripts/compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ services:
volumes:
- /tmp/jade-dev-account.pem:/tmp/jade-dev-account.pem
- /tmp/jade-dev-account.json:/tmp/jade-dev-account.json
- /tmp/buffer-client-sa-account.json:/tmp/buffer-client-sa-account.json

database:
image: postgres:14
Expand Down
16 changes: 0 additions & 16 deletions scripts/render-configs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -112,15 +112,9 @@ GOOGLE_SA_CERT=/tmp/jade-dev-account.pem
# By default, RBS will use the tools project. GCP projects will automatically be deleted after 1 day.
# Other option: dev - this will allow for projects to persist for longer than 1 day
if [[ "${RBS_ENV}" == "tools" ]]; then
BUFFER_CLIENT_SERVICE_ACCOUNT_VAULT_PATH=secret/dsde/terra/kernel/integration/tools/buffer/client-sa
BUFFER_CLIENT_SERVICE_ACCOUNT_GSM_PROJECT=broad-dsde-qa
BUFFER_CLIENT_GSM_SECRET=buffer-client-sa-b64-integration
RBS_POOLID=datarepo_v1
RBS_INSTANCEURL=https://buffer.tools.integ.envs.broadinstitute.org
elif [[ "${RBS_ENV}" == "dev" ]]; then
BUFFER_CLIENT_SERVICE_ACCOUNT_VAULT_PATH=secret/dsde/terra/kernel/dev/dev/buffer/client-sa
BUFFER_CLIENT_SERVICE_ACCOUNT_GSM_PROJECT=broad-jade-dev
BUFFER_CLIENT_GSM_SECRET=buffer-client-sa-b64
RBS_POOLID=datarepo_v3
RBS_INSTANCEURL=https://buffer.dsde-dev.broadinstitute.org
else
Expand All @@ -131,16 +125,6 @@ fi
echo $RBS_POOLID > "/tmp/rbs-pool-id.txt"
echo $RBS_INSTANCEURL > "/tmp/rbs-instance-url.txt"

RBS_CLIENTCREDENTIALFILEPATH=/tmp/buffer-client-sa-account.json

if $USE_VAULT; then
vault read -field=key "$BUFFER_CLIENT_SERVICE_ACCOUNT_VAULT_PATH" \
| base64 -d > "$RBS_CLIENTCREDENTIALFILEPATH"
else
gcloud secrets versions access latest --project $BUFFER_CLIENT_SERVICE_ACCOUNT_GSM_PROJECT --secret $BUFFER_CLIENT_GSM_SECRET \
| jq -r '.key' | base64 -d > "$RBS_CLIENTCREDENTIALFILEPATH"
fi


VARIABLE_NAMES=(AZURE_SYNAPSE_WORKSPACENAME AZURE_CREDENTIALS_HOMETENANTID AZURE_CREDENTIALS_APPLICATIONID AZURE_CREDENTIALS_SECRET AZURE_SYNAPSE_SQLADMINUSER AZURE_SYNAPSE_SQLADMINPASSWORD AZURE_SYNAPSE_ENCRYPTIONKEY GOOGLE_APPLICATION_CREDENTIALS GOOGLE_SA_CERT RBS_POOLID RBS_INSTANCEURL)

Expand Down
2 changes: 1 addition & 1 deletion src/main/java/bio/terra/app/configuration/WebConfig.java
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,6 @@ public void configurePathMatch(PathMatchConfigurer configurer) {
public void addResourceHandlers(ResourceHandlerRegistry registry) {
registry
.addResourceHandler("/webjars/swagger-ui-dist/**")
.addResourceLocations("classpath:/META-INF/resources/webjars/swagger-ui-dist/4.3.0/");
.addResourceLocations("classpath:/META-INF/resources/webjars/swagger-ui-dist/5.18.2/");
}
}
7 changes: 6 additions & 1 deletion src/main/java/bio/terra/common/PdaoLoadStatistics.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
package bio.terra.common;

import com.fasterxml.jackson.annotation.JsonProperty;
import com.google.cloud.bigquery.JobStatistics;
import java.time.Instant;

Expand All @@ -9,7 +10,11 @@ public class PdaoLoadStatistics {
private final Instant startTime;
private final Instant endTime;

public PdaoLoadStatistics(long badRecords, long rowCount, Instant startTime, Instant endTime) {
public PdaoLoadStatistics(
@JsonProperty("badRecords") long badRecords,
@JsonProperty("rowCount") long rowCount,
@JsonProperty("startTime") Instant startTime,
@JsonProperty("endTime") Instant endTime) {
this.badRecords = badRecords;
this.rowCount = rowCount;
this.startTime = startTime;
Expand Down
Loading

0 comments on commit 200768f

Please sign in to comment.