Skip to content

Comments

Onboarded CE to Telemetry#1207

Merged
AhmedBM merged 4 commits intodevfrom
ahbenmes/onboard_ce_telemetry
Nov 20, 2025
Merged

Onboarded CE to Telemetry#1207
AhmedBM merged 4 commits intodevfrom
ahbenmes/onboard_ce_telemetry

Conversation

@AhmedBM
Copy link
Contributor

@AhmedBM AhmedBM commented Nov 14, 2025

Description

  • Added OSConfigTelemetryStatusTrace calls to all OsConfigLogError statements to capture telemetry

Checklist

  • I have read the contribution guidelines.
  • I added unit-tests to validate my changes. All unit tests are passing.
  • I have merged the latest dev branch prior to this PR submission.
  • I ran pre-commit on my changes prior to this PR submission.
  • I submitted this PR against the dev branch.

@AhmedBM AhmedBM requested review from a team as code owners November 14, 2025 20:49
@github-actions
Copy link

github-actions bot commented Nov 14, 2025

Test Results

 63 files   63 suites   57m 20s ⏱️
  8 tests   8 ✅  0 💤 0 ❌
252 runs  230 ✅ 22 💤 0 ❌

Results for commit 33bc3c2.

♻️ This comment has been updated with latest results.

MariusNi
MariusNi previously approved these changes Nov 14, 2025
Copy link
Contributor

@MariusNi MariusNi left a comment

Choose a reason for hiding this comment

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

:shipit:

kkanas
kkanas previously approved these changes Nov 18, 2025
Copy link
Contributor

@kkanas kkanas left a comment

Choose a reason for hiding this comment

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

.

@kkanas kkanas dismissed their stale review November 18, 2025 17:53

New Ui confused me.

@AhmedBM AhmedBM force-pushed the ahbenmes/onboard_ce_telemetry branch from bfa9331 to 33bc3c2 Compare November 20, 2025 18:40
@MariusNi MariusNi dismissed their stale review November 20, 2025 19:27

revoking review

Copy link
Contributor

@MariusNi MariusNi left a comment

Choose a reason for hiding this comment

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

:shipit:

@AhmedBM AhmedBM merged commit 139847e into dev Nov 20, 2025
118 of 120 checks passed
@AhmedBM AhmedBM deleted the ahbenmes/onboard_ce_telemetry branch November 20, 2025 19:28
MariusNi pushed a commit that referenced this pull request Jan 13, 2026
MariusNi pushed a commit that referenced this pull request Jan 14, 2026
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.

4 participants