Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

AWS Setup: uploaded yaml and code files#45

Open
kabathla wants to merge 2 commits intowavefrontHQ:masterfrom
kabathla:kbathla/INT-1037
Open

AWS Setup: uploaded yaml and code files#45
kabathla wants to merge 2 commits intowavefrontHQ:masterfrom
kabathla:kbathla/INT-1037

Conversation

@kabathla
Copy link
Copy Markdown
Contributor

@kabathla kabathla commented Aug 9, 2021

As part of https://wavefront.atlassian.net/browse/INT-1037, I have uploaded yaml files which are used to create the cloudformation template and the lambda code (to create the AWS Setup) written in Python is also uploaded.

@kabathla kabathla changed the title uploaded yaml and code files AWS Setup: uploaded yaml and code files Aug 9, 2021
@kabathla
Copy link
Copy Markdown
Contributor Author

kabathla commented Aug 9, 2021

@yogeshprasad, can you please review.

FYI
@srinivas-kandula

Copy link
Copy Markdown
Contributor

@yogeshprasad yogeshprasad left a comment

Choose a reason for hiding this comment

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

Please scan the Python file using pylint, flake8 and pydocstring.

@kabathla
Copy link
Copy Markdown
Contributor Author

kabathla commented Aug 17, 2021

for flake8, there are no scanning issues:
Screenshot 2021-08-17 at 1 19 26 PM

for pylint, we have the following scan outcomes:
LambdaCode.py:1:0: C0103: Module name "LambdaCode" doesn't conform to snake_case naming style (invalid-name)-- This does not matter as code would be in AWS Lambda
LambdaCode.py:3:0: E0401: Unable to import 'boto3' (import-error): This library is being used in Lambda code
LambdaCode.py:4:0: E0401: Unable to import 'botocore.vendored' (import-error) : This library is being used in Lambda code to import requests package
LambdaCode.py:7:0: R0914: Too many local variables (30/15) (too-many-locals): the variables are required for storing different parameters which are being passed by CLI
LambdaCode.py:7:19: W0613: Unused argument 'context' (unused-argument): this is part of lambda code function syntax

Screenshot 2021-08-17 at 1 20 38 PM

for pydocstring, there is a scan comment to add the description of the function which I have added already.

Screenshot 2021-08-17 at 1 24 31 PM

Copy link
Copy Markdown
Contributor

@yogeshprasad yogeshprasad left a comment

Choose a reason for hiding this comment

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

Minor comment otherwise LGTM

role_arn, prefix, bucket_name)
if bucket_name != "":
api_url = hostname+"/api/v2/cloudintegration"
requests.post(api_url,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we please confirm the status here

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