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.

Sign up for GitHub

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

Jump to bottom

Provide lambda function ARN instead of role ARN to ingestion ECR #677

Merged
merged 2 commits into from
May 3, 2024
Merged

Provide lambda function ARN instead of role ARN to ingestion ECR #677

merged 2 commits into from
May 3, 2024

Conversation

A-Ashiq
Copy link
Contributor

@A-Ashiq A-Ashiq commented May 3, 2024

No description provided.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 3, 2024 •
edited
Loading

unit test coverage report

Title Lines Statements Branches Functions
lambda-producer-handler Coverage: 100%
 100% (41/41) 25% (2/8) 100% (8/8)
legacy-dashboard-redirect-viewer-request Coverage: 100%
 100% (16/16) 100% (4/4) 100% (3/3)
public-api-cloud-front-viewer-request Coverage: 100%
 100% (18/18) 100% (12/12) 100% (3/3)
Title Tests Skipped Failures Errors Time
lambda-producer-handler 16 0 💤 0 ❌ 0 🔥 0.89s ⏱️
legacy-dashboard-redirect-viewer-request 10 0 💤 0 ❌ 0 🔥 0.523s ⏱️
public-api-cloud-front-viewer-request 11 0 💤 0 ❌ 0 🔥 0.528s ⏱️

@A-Ashiq A-Ashiq merged commit c0cf5e9 into main May 3, 2024
7 of 8 checks passed
@A-Ashiq A-Ashiq deleted the bug/ingestion-lambda-image-permissions/CDD-1690 branch May 3, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phill-stanley phill-stanley phill-stanley approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@A-Ashiq @phill-stanley