Skip to content
This repository has been archived by the owner on Aug 13, 2019. It is now read-only.

Document S3 inventory lambda #217

Merged
merged 2 commits into from
Aug 21, 2017
Merged

Conversation

leplatrem
Copy link
Collaborator

No description provided.

@leplatrem leplatrem requested a review from Natim August 18, 2017 13:59
jobs/README.rst Outdated
Load latest S3 inventory
========================

A command downloads the latest S3 manifests from Mozilla archives, and create records on a remote Kinto server.
Copy link
Contributor

Choose a reason for hiding this comment

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

A command to download...

jobs/README.rst Outdated
* ``NB_RETRY_REQUEST`` (default: ``3``)
* ``TIMEOUT_SECONDS`` (default: ``300``)

The following entry point can be used as an Amazon Lambda function:
Copy link
Contributor

Choose a reason for hiding this comment

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

Because the name of the lambda is so different, I didn't believe this was the same as the above. How about changing to something like To use this script as an Amazon Lambda function, use the entry point ``buildhub.lambda_s3_inventory.lambda_handler`` ?

@leplatrem leplatrem merged commit 9aa52b1 into master Aug 21, 2017
@leplatrem leplatrem deleted the document-s3-inventory-lambda branch August 21, 2017 08:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants