PHANTOM
🇮🇳 IN
Skip to content

remove container_info analytics event#13848

Open
thrau wants to merge 2 commits intomainfrom
DAT-694-remove-container_info-payload
Open

remove container_info analytics event#13848
thrau wants to merge 2 commits intomainfrom
DAT-694-remove-container_info-payload

Conversation

@thrau
Copy link
Member

@thrau thrau commented Feb 25, 2026

Motivation

We've been reviewing telemetry events that are not useful to us, and this is one of them. Please find more details in linear DAT-694.

Changes

  • Remove container_info as analytics payload

Tests

Related

@thrau thrau requested a review from ackdav February 25, 2026 22:06
@thrau thrau added semver: patch Non-breaking changes which can be included in patch releases notes: skip Pull request does not have to be mentioned in the release notes labels Feb 25, 2026
@thrau thrau changed the title remove container_config analytics event remove container_info analytics event Feb 25, 2026
@thrau thrau added the docs: skip Pull request does not require documentation changes label Feb 25, 2026
@github-actions
Copy link

github-actions bot commented Feb 25, 2026

Test Results - Preflight, Unit

23 070 tests  ±0   21 179 ✅ ±0   6m 6s ⏱️ -21s
     1 suites ±0    1 891 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit c59f8d8. ± Comparison against base commit 9536c7a.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 25, 2026

S3 Image Test Results (AMD64 / ARM64)

    2 files      2 suites   8m 26s ⏱️
  573 tests   517 ✅  56 💤 0 ❌
1 146 runs  1 034 ✅ 112 💤 0 ❌

Results for commit c59f8d8.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 4s ⏱️ -1s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit c59f8d8. ± Comparison against base commit 9536c7a.

@github-actions
Copy link

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 39m 49s ⏱️ +2s
5 740 tests ±0  5 212 ✅ +5  527 💤 ±0  1 ❌ +1 
5 746 runs  ±0  5 212 ✅ +5  533 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit c59f8d8. ± Comparison against base commit 9536c7a.

@github-actions
Copy link

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   2h 7m 31s ⏱️ + 3m 12s
5 335 tests ±0  4 964 ✅ ±0  371 💤 ±0  0 ❌ ±0 
5 337 runs  ±0  4 964 ✅ ±0  373 💤 ±0  0 ❌ ±0 

Results for commit c59f8d8. ± Comparison against base commit 9536c7a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants