chore: bring the lambda scope to 0.3.0 - #12
Closed
agustincelentano wants to merge 1 commit into
Closed
Conversation
Picks up the metrics fix: the catalog reaching the platform without the assume_role progress in front of it, each metric resolving its own statistic, and stream.iterator_age. Both references move together — the agent's clone and the scope catalog — so the specs and the scripts come from the same commit.
Contributor
Author
|
Cerrado: quedó obsoleto. Este PR movía el catálogo de |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moves the Lambda scope to
0.3.0, which carries the metrics fix(nullplatform/scopes-lambda#33).
What it picks up
The performance screen for Lambda scopes was showing container metrics — CPU,
memory, throughput — that a Lambda cannot answer. The cause was
assume_rolewriting its progress to stdout, which the telemetry workflows hand to the platform
as the response body: the catalog arrived behind two lines of text and could not be
parsed, so
GET /metricreturned 400 and the UI fell back to the container set.It also fixes the statistics. Every metric fell back to
Sumbecause the variablecarrying it was never set, so
Durationreported 1076 ms where the latency was2 ms, and
ConcurrentExecutionsreported 205 where the peak was 1.And it adds
stream.iterator_age, which is what tells whether the consumer of aDynamoDB stream is falling behind.
Both references move together
agent_repos_extra0.3.0nullplatform/locals.tf)0.3.0They have to stay in sync: the catalog provides the specs and the agent runs the
scripts, and a mismatch means the platform describes one version while the agent
executes another.
There is no release for this version yet, so
0.3.0is a branch cut frommainrather than from a tag.
Verified
Applied against the implementation. The agent's pod now clones
heads/0.3.0, andthe metric catalog reaches the platform intact — the workflow's stdout starts with
{instead of the assume_role line, and parses with its six metrics. Theperformance screen shows the Lambda graphs and no longer shows the container set.
Note on an unrelated failure
tofu planinnullplatform/fails onmodule.scope_definitions["containers"].data.external.action_specs["kill-instances"]with a jq parse error. It reproduces on
mainwithout these changes and belongs tothe containers scope, which tracks
scopes.git#beta— not to this bump. Worthlooking at separately.
🤖 Generated with Claude Code