Skip to content
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.

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

[release/8.0-rc1] Unify with logging category matching #90642

Merged
merged 3 commits into from
Aug 16, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 15, 2023

Backport of #90559 to release/8.0-rc1

There's a new feature to configure metrics in RC1. This PR addresses feedback about making the matching rules consistent with Logging.

Customer Impact

Consistent matching rules will make the config file formats easier to understand and document.

Testing

Unit tests

Risk

Low, the feature is new in rc1.

@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Aug 15, 2023
@Tratcher Tratcher requested a review from tarekgh August 15, 2023 22:13
@Tratcher Tratcher self-assigned this Aug 15, 2023
@tarekgh tarekgh added Servicing-consider Issue for next servicing release review area-System.Diagnostics.Metric and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Aug 15, 2023
@tarekgh tarekgh added this to the 8.0.0 milestone Aug 15, 2023
@tarekgh
Copy link
Member

tarekgh commented Aug 16, 2023

@Tratcher can you rebase the changes with release/8.0-rc1

@Tratcher Tratcher force-pushed the backport/pr-90559-to-release/8.0-rc1 branch from 4c2aa78 to 2554d8e Compare August 16, 2023 15:24
@Tratcher
Copy link
Member

Rebased

@Tratcher Tratcher added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Aug 16, 2023
@carlossanlop
Copy link
Member

carlossanlop commented Aug 16, 2023

@Tratcher I can bypass the "Update branch" restriction. Otherwise the CI will restart. Ping me when the CI finishes for this new re-run and I can merge it for you (no need to press on "Update branch" again).

@Tratcher
Copy link
Member

@carlossanlop ready when you are.

@carlossanlop carlossanlop merged commit b6a0e1c into release/8.0-rc1 Aug 16, 2023
103 of 107 checks passed
@carlossanlop carlossanlop deleted the backport/pr-90559-to-release/8.0-rc1 branch August 16, 2023 17:58
@ghost ghost locked as resolved and limited conversation to collaborators Sep 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants