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 CLI options --output-public and --output-context to set X-WPS-Output-Context header easily #569

Merged
merged 4 commits into from
Oct 5, 2023

Conversation

fmigneault
Copy link
Collaborator

@fmigneault fmigneault commented Oct 4, 2023

Changes

  • Add CLI execute options --output-public/-oP and --output-context/-oC OUTPUT_CONTEXT that add the
    specified X-WPS-Output-Context header to request the relevant output storage location of Job results.

References

@fmigneault fmigneault self-assigned this Oct 4, 2023
@github-actions github-actions bot added ci/doc Issue related to documentation of the package ci/tests Tests of the package and features feature/cli Issues or features related to CLI operations. labels Oct 4, 2023
@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #569 (2c6d05f) into master (6642ec9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #569   +/-   ##
=======================================
  Coverage   84.55%   84.55%           
=======================================
  Files          79       79           
  Lines       17788    17793    +5     
  Branches     2708     2709    +1     
=======================================
+ Hits        15040    15045    +5     
  Misses       2020     2020           
  Partials      728      728           
Files Coverage Δ
weaver/cli.py 85.90% <100.00%> (+0.06%) ⬆️

@fmigneault fmigneault merged commit 3680c5d into master Oct 5, 2023
47 checks passed
@fmigneault fmigneault deleted the cli-public-output-context branch October 5, 2023 22:47
fmigneault added a commit to bird-house/birdhouse-deploy that referenced this pull request Oct 10, 2023
## Overview

Allow users to submit a Weaver job requesting to store outputs to the public location instead of their user-workspace.

## Changes

**Non-breaking changes**
- Users-submitted jobs are not enforced to their user-workspace location if specifically requesting `public` location.

**Breaking changes**
- n/a

## Related Issue / Discussion

- relates to crim-ca/weaver#569
- resolves [DAC-190](https://crim-ca.atlassian.net/browse/DAC-190)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/doc Issue related to documentation of the package ci/tests Tests of the package and features feature/cli Issues or features related to CLI operations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant