Skip to content

Update Dependencies #244

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

Merged
merged 1 commit into from
Jan 11, 2022
Merged

Update Dependencies #244

merged 1 commit into from
Jan 11, 2022

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Nov 5, 2021

Autogenerated PR to update all deps to latest versions

@github-actions github-actions bot requested a review from a team as a code owner November 5, 2021 10:01
@codecov-commenter
Copy link

codecov-commenter commented Nov 6, 2021

Codecov Report

Merging #244 (6a5c499) into main (f83ae4a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #244   +/-   ##
=======================================
  Coverage   83.56%   83.56%           
=======================================
  Files          35       35           
  Lines        1339     1339           
  Branches      284      284           
=======================================
  Hits         1119     1119           
  Misses        196      196           
  Partials       24       24           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f83ae4a...6a5c499. Read the comment docs.

@DarcyRaynerDD DarcyRaynerDD force-pushed the update-dependencies branch 8 times, most recently from 2977f06 to 9f2a351 Compare November 15, 2021 10:06
@DarcyRaynerDD DarcyRaynerDD force-pushed the update-dependencies branch 5 times, most recently from da19882 to fe4872c Compare November 20, 2021 10:01
@DarcyRaynerDD DarcyRaynerDD force-pushed the update-dependencies branch 7 times, most recently from 2b96984 to 517bbbb Compare November 29, 2021 10:01
@DarcyRaynerDD DarcyRaynerDD force-pushed the update-dependencies branch 7 times, most recently from b91a631 to 97c2820 Compare December 6, 2021 10:01
@DarcyRaynerDD DarcyRaynerDD force-pushed the update-dependencies branch 8 times, most recently from 3ea768b to b942172 Compare December 23, 2021 10:01
@DarcyRaynerDD DarcyRaynerDD force-pushed the update-dependencies branch 6 times, most recently from 6a9790b to 6272a96 Compare December 31, 2021 10:01
@DarcyRaynerDD DarcyRaynerDD force-pushed the update-dependencies branch 7 times, most recently from b052691 to 8bb6417 Compare January 7, 2022 10:01
@DarcyRaynerDD DarcyRaynerDD force-pushed the update-dependencies branch 2 times, most recently from 6a5c499 to 56d1eec Compare January 10, 2022 10:01
@zARODz11z
Copy link
Contributor

zARODz11z commented Jan 10, 2022

I checked out update-dependencies, ran the local unit tests and integration tests, deployed the Node14 and Node12 layers to sandbox (sa-east-1) under Datadog-Node14-x-AndrewTest and Datadog-Node12-x-AndrewTest

Next I utilized the lambda-layer-sample-node from the cloud-integrations-lambdas repo (https://github.com/DataDog/cloud-integrations-lambdas/tree/master/lambda-layer-sample-node). I used that project to deploy a function in Nodejs12.x and Nodejs14.x instrumented with the dd-lambda-js layer I deployed to sandbox earlier(Datadog-NodeXX-X-AndrewTest). From there I invoked the function and viewed the results in Datadog.
https://app.datadoghq.com/functions/lambda-layer-sample-node-andrew-test-dev-hello12xAndrew%3Asa-east-1%3A601427279990%3Aaws?graphType=flamegraph&source=sls-plugin&spanID=5103684422861508559&spanViewType=metadata&tab=Overview&traceID=6919906129688037618&start=1641243478752&end=1641848278752&paused=false

Link to custom metric sent by layer https://app.datadoghq.com/metric/explorer?from_ts=1641819350988&to_ts=1641822950988&live=false&tile_size=m&exp_metric=hello.js10x.dog_2&exp_agg=avg&exp_row_type=metric

Everything i've read about the dd-lambda-js layer seems to be working. (I learned a lot about the layer by tinkering)

@zARODz11z
Copy link
Contributor

fyi, I do not have gov cloud access yet (it is pending) so someone else may have to publish the layer.

@zARODz11z zARODz11z merged commit 0031257 into main Jan 11, 2022
@zARODz11z zARODz11z deleted the update-dependencies branch January 11, 2022 13:30
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.

4 participants