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.

Sign up for GitHub

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

Jump to bottom

DRS Locator container image has changed the name of the env vars to access the DRS location service #391

Closed
BMurri opened this issue Sep 8, 2023 · 1 comment · Fixed by #390
Closed

DRS Locator container image has changed the name of the env vars to access the DRS location service #391

BMurri opened this issue Sep 8, 2023 · 1 comment · Fixed by #390
Assignees
@BMurri
Labels
bug Something isn't working
Milestone
next

Comments

@BMurri
Copy link
Collaborator

BMurri commented Sep 8, 2023

Describe the bug
On Dec 15, 2022 broadinstitute/cromwell-drs-localizer:develop changed the names of the environment variables it uses to genericize the names of the locator service it is pointed to. TES is programmed to populate one of those variables.

Additional context
broadinstitute/cromwell-drs-localizer:develop now needs to use the environment variable DRS_RESOLVER_URL instead of the former variable MARTHA_URL to locate the DRS resolver service.
@BMurri BMurri added the bug Something isn't working label Sep 8, 2023
@BMurri BMurri self-assigned this Sep 8, 2023
@BMurri BMurri added this to the next milestone Sep 8, 2023
@BMurri BMurri mentioned this issue Sep 8, 2023
Merged
@BMurri
Copy link
Collaborator Author

BMurri commented Sep 8, 2023

This is actually assigned to @jgainerdewar but it needs to be touched in order to make that assignment.

@BMurri BMurri added enhancement New feature or request up for grabs Available for community contributions. Please ask in the issue if you'd like to implement it bug Something isn't working and removed bug Something isn't working enhancement New feature or request up for grabs Available for community contributions. Please ask in the issue if you'd like to implement it labels Sep 8, 2023
@BMurri BMurri mentioned this issue Sep 8, 2023
Open
@BMurri BMurri mentioned this issue Nov 29, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BMurri BMurri

Labels
bug Something isn't working
Projects
None yet
Milestone
next
Development

Successfully merging a pull request may close this issue.

Fix DRS resolver env var name jgainerdewar/ga4gh-tes
1 participant
@BMurri