Enhance telemetry tracking and data privacy in migration process#3087
Draft
sevoku wants to merge 3 commits into
Draft
Enhance telemetry tracking and data privacy in migration process#3087sevoku wants to merge 3 commits into
sevoku wants to merge 3 commits into
Conversation
Updated telemetry properties to track the current step in the schema conversion process without including domain labels. This change enhances clarity in telemetry data by focusing on the step type rather than the domain context.
Improves telemetry tracking for AI model errors by adding detailed cause type, cause code, and cause message to the telemetry context. This ensures better insights into error occurrences during migration processes.
Updated the migration telemetry context to mask additional sensitive identifiers including subscriptionId, accountName, and resourceGroup. Improved logging to ensure only predefined OII identifiers are emitted, enhancing data privacy during telemetry reporting.
Contributor
🎉 Build Summary🔗 Source
📦 Package Information
🧪 Test Results
✅ Build StatusAll checks completed successfully! |
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
1 participant
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.
This pull request enhances telemetry and error reporting in the migration panel, with a focus on improved masking of sensitive information, more structured error context, and clearer telemetry for schema conversion steps. It also refines which environment details are logged during provisioning.
Telemetry and error reporting improvements:
tenantId, but alsosubscriptionId,accountName, andresourceGroupfrom error messages, while continuing to emit only allowed identifiers in telemetry. (migrationTelemetry.ts)aiErrorCauseTypeand a boundedaiErrorCauseCodeto telemetry, and ensures raw error messages are only included in user-consented issue reports, not telemetry. (migrationTelemetry.ts)Schema conversion telemetry refinements:
phase3SchemaConversion.ts)lastSteptelemetry property to only the sub-step name (e.g.,containerDesign), removing domain-specific prefixes for clarity and consistency. (phase3SchemaConversion.ts) [1] [2] [3] [4] [5] [6] [7] [8]Provisioning telemetry adjustments:
accountName,subscriptionId) are emitted in telemetry, and removes emission ofresourceGroupas a separate property. (phase4Provisioning.ts) [1] [2]sampleDataInsertedproperty from telemetry, streamlining reported metrics. (phase4Provisioning.ts)