Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

improve system information sent in session and container_info#13680

Merged
thrau merged 1 commit intomainfrom
improve-system-information
Feb 4, 2026
Merged

improve system information sent in session and container_info#13680
thrau merged 1 commit intomainfrom
improve-system-information

Conversation

@thrau
Copy link
Copy Markdown
Member

@thrau thrau commented Feb 2, 2026

Motivation

Adding a bit more system information to telemetry.

  • In add logging for basic container information #7855 we introduced a container_info event that we haven't really been using all that much because it doesn't container super useful information. I extended it to have more information about the container.
  • The session payload mostly said "Linux" which wasn't all that helpful. This method that we already use in pro gives us finer-grained system information. instead of Linux it will now say something like Linux,5.19.0-32-generic,x86_64

Why both, and the seemingly duplicated data? I didn't want to send uname.version as part of the session payload. Most information we need is in the system string we generate with this new method, and having that in the session payload will be very useful. container_information has it's own utility and having access to the version field there may be helpful for some cases.

Changes

  • Add more system information to session data

Tests

Related

@thrau thrau added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Feb 2, 2026
@thrau thrau force-pushed the improve-system-information branch from 42074a0 to 511ecb3 Compare February 3, 2026 14:10
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026

S3 Image Test Results (AMD64 / ARM64)

    2 files  ±0      2 suites  ±0   7m 54s ⏱️ -12s
  559 tests ±0    507 ✅ ±0   52 💤 ±0  0 ❌ ±0 
1 118 runs  ±0  1 014 ✅ ±0  104 💤 ±0  0 ❌ ±0 

Results for commit 759f1a6. ± Comparison against base commit bb27e3e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026

Test Results - Preflight, Unit

23 114 tests  ±0   21 255 ✅ ±0   6m 12s ⏱️ -9s
     1 suites ±0    1 859 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 759f1a6. ± Comparison against base commit bb27e3e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026

Test Results (amd64) - Acceptance

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

Results for commit 759f1a6. ± Comparison against base commit bb27e3e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 38m 37s ⏱️
5 610 tests 5 049 ✅ 561 💤 0 ❌
5 616 runs  5 049 ✅ 567 💤 0 ❌

Results for commit 759f1a6.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 3, 2026

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 58m 23s ⏱️ - 1m 0s
5 189 tests ±0  4 798 ✅ ±0  391 💤 ±0  0 ❌ ±0 
5 191 runs  ±0  4 798 ✅ ±0  393 💤 ±0  0 ❌ ±0 

Results for commit 759f1a6. ± Comparison against base commit bb27e3e.

♻️ This comment has been updated with latest results.

@thrau thrau force-pushed the improve-system-information branch from 511ecb3 to 759f1a6 Compare February 4, 2026 13:22
Copy link
Copy Markdown
Member

@vittoriopolverino vittoriopolverino left a comment

Choose a reason for hiding this comment

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

🚀

@thrau thrau merged commit d1407d6 into main Feb 4, 2026
49 checks passed
@thrau thrau deleted the improve-system-information branch February 4, 2026 21:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

3 participants