Skip to content

Improve TraceLogging #5285

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Improve TraceLogging #5285

wants to merge 7 commits into from

Conversation

DrusTheAxe
Copy link
Member

Improve TraceLogging

  • Always include _GENERIC_PARTB_FIELDS_ENABLED in TraceLoggingWrite calls for rich log information
  • Always specify PDT Keyword per PII practices. Usually TraceLoggingKeyword(PDT_ProductAndServicePerformance) (was in these cases but any future devs reading this should consult with your privacy representative)

https://task.ms/56757819

@DrusTheAxe DrusTheAxe added area-Deployment Issues related to packaging, installation, runtime (e.g., SelfContained, Unpackaged) area-DynamicDependencies labels Mar 27, 2025
@DrusTheAxe DrusTheAxe added this to the 1.8 milestone Mar 27, 2025
@DrusTheAxe DrusTheAxe self-assigned this Mar 27, 2025
@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe DrusTheAxe enabled auto-merge (squash) March 28, 2025 00:32
@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe DrusTheAxe disabled auto-merge April 9, 2025 04:21
@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DrusTheAxe
Copy link
Member Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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

@AgneLukoseviciute AgneLukoseviciute AgneLukoseviciute approved these changes

@SaChinta-MSFT SaChinta-MSFT SaChinta-MSFT approved these changes

@cwruss cwruss Awaiting requested review from cwruss

@wcheng-msft wcheng-msft Awaiting requested review from wcheng-msft

@dabhattimsft dabhattimsft Awaiting requested review from dabhattimsft

Assignees

@DrusTheAxe DrusTheAxe

Labels
area-Deployment Issues related to packaging, installation, runtime (e.g., SelfContained, Unpackaged) area-DynamicDependencies
Projects
None yet
Milestone
1.8
Development

Successfully merging this pull request may close these issues.
3 participants
@DrusTheAxe @AgneLukoseviciute @SaChinta-MSFT