Open yarcat opened 3 months ago
Hi @yarcat thanks for your work on this.
I will test for S3 when you've confirmed it working for GCP - This seems a really useful addition and I'd quite like this API feature finished as soon as possible.
@robballantyne I've updated the PR with the test information performed. And I've uploaded a credentials initialization fix. Please verify S3.
Previously only S3 buckets export destination was supported. This PR introduces the following changes:
s3_url
andgcp_url
to the result object. Note: previousurl
is also supported for backward compatibility.Testing
Environment
Image configuration:
Existing extra environment variables
New environment variables provided to the image
Workflow request/response
IMPORTANT: Note an empty
"gcp": {}
which requests GCP upload using default settings. You can pass a project name to ensure it isn't empty and isn't ignored if default env variables are not set.Request
Response:
Validation request/response
Request
Response
Error test
Providing unavailable
"gcp": { "bucket_name": "abc" }