Repository navigation
Fix OTLP collector wiring, workflow permissions, Node 20 and tool caching (#1785-#1788) - #1809
Merged
hman38705 merged 4 commits intoOct 19, 2026
Conversation
|
@jhaydeeee-web Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
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.
Closes #1785
Closes #1786
Closes #1787
Closes #1788
Summary
9e86168: infra: add OTLP endpoint variable and collector egress rules (#1785)
Adds
infrastructure/terraform/otel_collector.tfwithotlp_endpoint,otlp_collector_cidr_blocksandotlp_collector_sg_idvariables, plus conditional ECS task egress rules for TCP 4317 and 4318. The ECS module now injectsOTLP_ENDPOINTinto the API container when an endpoint is set, anddocs/DISTRIBUTED_TRACING.mddocuments the setup.8c200bb: ci: add least-privilege top-level permissions to six workflows (#1786)
Adds a top-level
permissions: contents: readblock to accessibility, dependency-scan, docker, license-check, openapi-validation and protect-changelog. In dependency-scan, the Trivy job getssecurity-events: writeand the report job getsissuesandpull-requests: writeso SARIF upload and PR comments keep working.240f8f6: ci: bump Node 18 to 20 in workflows and frontend engines (#1787)
Moves every
node-version: 18in test, openapi-validation, dependency-scan and license-check, and the accessibilityNODE_VERSION, to Node 20. Raisesfrontend/package.jsonengines to>=20.0.0, in line with the TTS Dockerfile.e26dd97: ci: cache prebuilt cargo tool binaries instead of recompiling (#1788)
Adds
actions/cachesteps for~/.cargo/binbinaries of soroban-cli, cargo-llvm-cov, cargo-audit, cargo-nextest and cargo-deny in test, contract-deployment, dependency-scan and license-check. Each install now runs only when the binary is missing (command -v <bin> || cargo install ...), so cache hits skip compilation.