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

New workflow to deploy the lambda function #261

Merged
merged 4 commits into from
Jun 13, 2023

Conversation

kov
Copy link

@kov kov commented Feb 16, 2022

Build the lambda function, package it as expected by AWS lambda, then update it on AWS.

For now this is only triggered manually, so that the update can be performed only when it makes sense to the developers. In the future we can make a release being tagged trigger this workflow.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 16, 2022
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
.github/workflows/lambda.yaml Show resolved Hide resolved
.github/workflows/lambda.yaml Outdated Show resolved Hide resolved
@obbardc
Copy link
Contributor

obbardc commented Feb 16, 2022

we should probably explain what secrets need to be added too

Gustavo Noronha Silva added 2 commits March 9, 2022 09:11
Signed-off-by: Gustavo Noronha Silva <gustavo.noronha@collabora.com>
Triggered when new releases are tagged or manually.

Signed-off-by: Gustavo Noronha Silva <gustavo.noronha@collabora.com>
@kov kov force-pushed the lambda-upload branch from edac8e2 to 8e98000 Compare March 9, 2022 12:12
@kov
Copy link
Author

kov commented Mar 9, 2022

we should probably explain what secrets need to be added too

explain where? I can add it to the doc on deploying the lambda function I guess, after it has been validated

.github/workflows/lambda.yaml Outdated Show resolved Hide resolved
Signed-off-by: Santosh Mahto <santosh.mahto@collabora.com>
Signed-off-by: Santosh Mahto <santosh.mahto@collabora.com>
Copy link
Contributor

@davide125 davide125 left a comment

Choose a reason for hiding this comment

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

I think this is good to go, we can't really test it until it's merged as this is an on demand workdlow.

@davide125 davide125 merged commit 4e0210e into facebookexperimental:main Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants