8000 feat(integration/s3): tempURL, cache, doc, integration by yesnault · Pull Request #4221 · ovh/cds · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(integration/s3): tempURL, cache, doc, integration #4221

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

Merged
merged 2 commits into from
Apr 29, 2019

Conversation

yesnault
Copy link
Member
@yesnault yesnault commented Apr 24, 2019

Signed-off-by: Yvonnick Esnault yvonnick@esnau.lt

@yesnault yesnault force-pushed the ye-reynn-master-s3 branch 3 times, most recently from 4d4c6bd to f1c81f4 Compare April 27, 2019 22:17
@yesnault
Copy link
Member Author

new IT:

➜  cds git:(ye-reynn-master-s3) ✗ venom run --var cds.build.cdsctl=cdsctl tests/sc_workflow_storage_awss3_artifact.yml
SUCCESS tests/sc_workflow_storage_awss3_artifact.yml   22.21588221s

Total:6 Duration:22.218462937s
OK:6
KO:0
Skipped:0
TestSuite:1
TestCase:6
➜  cds git:(ye-reynn-master-s3) ✗ venom run --var cds.build.cdsctl=cdsctl tests/sc_workflow_storage_awss3_worker_artifact.yml 
SUCCESS tests/sc_workflow_storage_awss3_worker_artifact31.782532385s

Total:6 Duration:31.785587976s
OK:6
KO:0
Skipped:0
TestSuite:1
TestCase:6
➜  cds git:(ye-reynn-master-s3) ✗ venom run --var cds.build.cdsctl=cdsctl tests/sc_workflow_storage_awss3_worker_cache.yml 
SUCCESS tests/sc_workflow_storage_awss3_worker_cache.ym22.055815325s

Total:7 Duration:22.059108803s
OK:7
KO:0
Skipped:0
TestSuite:1
TestCase:7

Signed-off-by: Yvonnick Esnault <yvonnick@esnau.lt>
@yesnault yesnault force-pushed the ye-reynn-master-s3 branch from f1c81f4 to 69e977e Compare April 28, 2019 16:12
@yesnault yesnault marked this pull request as ready for review April 28, 2019 16:12
@yesnault yesnault merged this pull request into reynn-master-s3 Apr 29, 2019
- action [Artifact Downlaod]({{< relref "/docs/actions/artifact-download.md">}})
- [worker cache command]({{< relref "/docs/components/worker/cache">}})

Notice: per default, the storage is configured in CDS Configuration. This integration
Copy link
Contributor

Choose a reason for hiding this comment

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

by default instead per default no ?

With this integration, you can use a dedicated AWS S3 Bucket Storage on :

- action [Artifact Upload]({{< relref "/docs/actions/artifact-upload.md">}})
- action [Artifact Downlaod]({{< relref "/docs/actions/artifact-download.md">}})
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- action [Artifact Downlaod]({{< relref "/docs/actions/artifact-download.md">}})
- action [Artifact Download]({{< relref "/docs/actions/artifact-download.md">}})

if err != nil 8000 {
return "", "", sdk.WrapError(err, "failed to sign request")
}
log.Debug("AWS-S3-Store> StoreURL urlStr:%v key:%v hs:%v", urlStr, *key, hs)
Copy link
Contributor

Choose a reason for hiding this comment

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

key could not be nil ?

yesnault added a commit that referenced this pull request Apr 29, 2019
* feat(integration/s3): tempURL, cache, docs

Signed-off-by: Yvonnick Esnault <yvonnick@esnau.lt>
Signed-off-by: Yvonnick Esnault <yvonnick.esnault@corp.ovh.com>
yesnault added a commit that referenced this pull request Apr 29, 2019
* feat(integration/s3): tempURL, cache, docs

Signed-off-by: Yvonnick Esnault <yvonnick@esnau.lt>
Signed-off-by: Yvonnick Esnault <yvonnick.esnault@corp.ovh.com>
@yesnault yesnault deleted the ye-reynn-master-s3 branch April 30, 2019 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0