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

Add ability to copy interval list files to gs directory [VS-191] #7467

Merged
merged 7 commits into from
Sep 15, 2021

Conversation

rsasch
Copy link

@rsasch rsasch commented Sep 14, 2021

@gatk-bot
Copy link

gatk-bot commented Sep 14, 2021

Travis reported job failures from build 36013
Failures in the following jobs:

Test Type JDK Job ID Logs
unit openjdk11 36013.13 logs
unit openjdk8 36013.3 logs

@gatk-bot
Copy link

gatk-bot commented Sep 14, 2021

Travis reported job failures from build 36015
Failures in the following jobs:

Test Type JDK Job ID Logs
unit openjdk11 36015.13 logs
unit openjdk8 36015.3 logs
unit openjdk11 36015.13 logs
unit openjdk8 36015.3 logs
unit openjdk11 36015.13 logs
unit openjdk8 36015.3 logs
unit openjdk11 36015.13 logs
unit openjdk11 36015.13 logs
unit openjdk8 36015.3 logs
unit openjdk11 36015.13 logs
unit openjdk8 36015.3 logs

Copy link
Contributor

@ahaessly ahaessly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

gsutil cp ~{service_account_json_path} local.service_account.json
gcloud auth activate-service-account --key-file=local.service_account.json
fi
gsutil -m cp *.interval_list $OUTPUT_GCS_DIR/
Copy link
Contributor

@RoriCremer RoriCremer Sep 14, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any organizational benefit to putting them into their own interval list specific directory? like the interval-files/ one gatk creates?

@rsasch rsasch merged commit a2dfaba into ah_var_store Sep 15, 2021
@rsasch rsasch deleted the rsa_add_interval_list_to_wf_outputs branch September 15, 2021 15:58
This was referenced Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants