Skip to content

[SVLS-703] Document short timeouts effecting the Extension process#180

Merged
DylanLovesCoffee merged 2 commits into
mainfrom
dylan/update-docs
Nov 14, 2023
Merged

[SVLS-703] Document short timeouts effecting the Extension process#180
DylanLovesCoffee merged 2 commits into
mainfrom
dylan/update-docs

Conversation

@DylanLovesCoffee

Copy link
Copy Markdown
Contributor

Documenting where very short timeouts may cause subsequent invocations to not run handler code due to a failure in the Extension's INIT phase. Adds a reference to the documentation update for INIT_REPORT logs announced here: https://aws.amazon.com/about-aws/whats-new/2023/11/aws-lambda-troubleshoot-errors-timeouts-init-restore-phases/

Should also display here: https://docs.datadoghq.com/serverless/libraries_integrations/extension/#overhead

@DylanLovesCoffee
DylanLovesCoffee requested a review from a team as a code owner November 14, 2023 16:26

@maycmlee maycmlee left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Left a couple of suggestions!

Comment thread README.md Outdated
Co-authored-by: May Lee <[email protected]>
@DylanLovesCoffee
DylanLovesCoffee merged commit eebb47c into main Nov 14, 2023
@DylanLovesCoffee
DylanLovesCoffee deleted the dylan/update-docs branch November 14, 2023 19:09
duncanista added a commit that referenced this pull request Jun 17, 2026
## Overview

Consolidates the 6 open Dependabot/ADMS vulnerability PRs into a single
PR.

| Source PR | Manifest | Notes |
|---|---|---|
| #1258 | `integration-tests/lambda/otlp-java/pom.xml` |
`io.opentelemetry:opentelemetry-api` 1.58.0 → 1.63.0 |
| #1259 | `local_tests/golang/go.{mod,sum}` | `aws-sdk-go`,
`golang.org/x/net`, `golang.org/x/sys` minor bumps |
| #1260 | `integration-tests/lambda/otlp-node/package*.json` |
Superseded by #1263 (lock already has `@grpc/[email protected]`,
`@protobufjs/utf8` override carried over) |
| #1261 | `integration-tests/package*.json` | `uuid` 9 → 14 (ported
manually onto #1262 to avoid lock conflict) |
| #1262 | `integration-tests/package*.json` | `axios` 1.15 → 1.18, plus
overrides for `@opentelemetry/core`, `fast-xml-parser`,
`fast-xml-builder`, `fast-uri`, `brace-expansion` |
| #1263 | `integration-tests/lambda/otlp-node/package*.json` | Major
OTel SDK bump (`@opentelemetry/sdk-node` 0.54 → 0.219, `protobufjs`
7.5.4 → 7.6.4) |

Closes the following Dependabot alerts: #191, #192 (`@grpc/grpc-js`);
#183#190 + older axios alerts; protobufjs alerts via the major OTel SDK
bump; uuid #180; OpenTelemetry Java #176.

## Testing

GitHub Actions checks (Build All, Test Suite, Clippy, Audit, CodeQL)
passed on each individual source PR. `dd-gitlab/*` checks failed
identically on all 6 — appears to be an unrelated infra/permissions
issue with the bot-authored branches. Re-running CI on this branch
(created from a maintainer account) should clear them.

The vendored deps live entirely under `integration-tests/` and
`local_tests/`, so they have no impact on the shipped extension binary.

---------

Co-authored-by: gh-worker-campaigns-3e9aa4[bot] <244854796+gh-worker-campaigns-3e9aa4[bot]@users.noreply.github.com>
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.

3 participants