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

https:// is considered invalid by hailctl config set batch/remote_tmpdir #13049

Closed
danking opened this issue May 15, 2023 · 0 comments · Fixed by #13051
Closed

https:// is considered invalid by hailctl config set batch/remote_tmpdir #13049

danking opened this issue May 15, 2023 · 0 comments · Fixed by #13051
Assignees
@daniel-goldstein

Comments

@danking
Copy link
Contributor

danking commented May 15, 2023

What happened?

In regards to the https vs hail-az, I get an error for https:
hailctl config set batch/remote_tmpdir https://kahlquisrefsa.blob.core.windows.net/test
Error: bad value 'https://kahlquisrefsa.blob.core.windows.net/test' for parameter 'batch/remote_tmpdir' should be valid cloud storage URI such as gs://my-bucket/batch-tmp/

Version

0.2.116

Relevant log output

No response
@daniel-goldstein daniel-goldstein self-assigned this May 15, 2023
@daniel-goldstein daniel-goldstein mentioned this issue May 15, 2023
Merged
danking pushed a commit that referenced this issue May 15, 2023
@daniel-goldstein
[hailctl] Fix validation for remote_tmpdir to allow azure https scheme (
0aaf389 
#13051)

Validation code was not updated when adding azure https support. Made
this check a bit more robust and tested locally.

Resolves #13049
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@daniel-goldstein daniel-goldstein

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[hailctl] Fix validation for remote_tmpdir to allow azure https scheme daniel-goldstein/hail
2 participants
@danking @daniel-goldstein