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

AWS - SNS publisher and lambda code #675

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukemun
Copy link
Contributor

@lukemun lukemun commented Jan 30, 2025

  • Script for upload 20 sns inventory update messages
    • contains a negative inv value which will throw an error
  • lambda function and corresponding libraries which are manually zipped and uploaded to AWS
  • Also updates .gitignore for python .venv

@lukemun lukemun requested a review from realkosty as a code owner January 30, 2025 15:02
Copy link

vercel bot commented Jan 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
empower ⬜️ Ignored (Inspect) Visit Preview Feb 3, 2025 5:21pm

Copy link

codecov bot commented Jan 30, 2025

Bundle Report

Bundle size has no change ✅

aws/inventory_update_sns.py Dismissed Show dismissed Hide dismissed
aws/inventory_update_sns.py Dismissed Show dismissed Hide dismissed
aws/lambda_demo/urllib3/util/ssl_.py Dismissed Show dismissed Hide dismissed
Copy link

codecov bot commented Jan 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 30.10%. Comparing base (9cea64b) to head (03e2515).
Report is 5 commits behind head on master.

✅ All tests successful. No failed tests found.

❌ Your project check has failed because the head coverage (30.10%) is below the target coverage (35.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #675      +/-   ##
==========================================
- Coverage   34.13%   30.10%   -4.03%     
==========================================
  Files          37       38       +1     
  Lines         920      940      +20     
  Branches      111      113       +2     
==========================================
- Hits          314      283      -31     
- Misses        589      640      +51     
  Partials       17       17              
Flag Coverage Δ
api 6.59% <ø> (ø)
frontend 47.06% <ø> (-7.69%) ⬇️
Components Coverage Δ
checkout_module 6.59% <ø> (ø)
product_component 43.30% <ø> (-13.11%) ⬇️

see 6 files with indirect coverage changes

Copy link
Contributor Author

@lukemun lukemun left a comment

Choose a reason for hiding this comment

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

Added comments to find relevant code easier


return messages

def usage_demo():
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Start of upload script

return latency

@serverless_function
def lambda_handler(event, context):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

lambda function (used for both SQS right now)

@lukemun lukemun requested a review from antonpirker January 30, 2025 15:05
@lukemun lukemun force-pushed the aws-distributed-tracing branch from 3b577a8 to 2aa0a8a Compare January 30, 2025 22:08
@lukemun lukemun force-pushed the aws-distributed-tracing branch from 2aa0a8a to 03e2515 Compare February 3, 2025 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant