Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

metrics separation #5391

Merged
merged 10 commits into from
Apr 25, 2022
Merged

metrics separation #5391

merged 10 commits into from
Apr 25, 2022

Conversation

drahnr
Copy link
Contributor

@drahnr drahnr commented Apr 25, 2022

Separate the metrics definitions in some very old subsystem implementations, that haven't been touch in a while.

Mostly an extraction of #5314 to get that timely merged and avoid merge-pain.

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Apr 25, 2022
@drahnr drahnr self-assigned this Apr 25, 2022
@drahnr drahnr added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Apr 25, 2022
@drahnr drahnr requested review from slumber and eskimor April 25, 2022 16:23
@eskimor
Copy link
Member

eskimor commented Apr 25, 2022

Gladly you left collator-protocol untouched. (I am working on a PR that did the same thing for the collator protocol).

@drahnr
Copy link
Contributor Author

drahnr commented Apr 25, 2022

I touched it too much and could not extract the required changeset cleanly 😰 merge conflict upcoming..

@drahnr
Copy link
Contributor Author

drahnr commented Apr 25, 2022

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@drahnr drahnr added A8-mergeoncegreen and removed A0-please_review Pull request needs code review. labels Apr 25, 2022
@paritytech-processbot paritytech-processbot bot merged commit 121aa40 into master Apr 25, 2022
@paritytech-processbot paritytech-processbot bot deleted the bernhard-metrics-separation branch April 25, 2022 17:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants