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

cleanup: port replica-healthcheck to chain-mon #6438

Merged
merged 1 commit into from
Jul 28, 2023
Merged

Conversation

tynes
Copy link
Contributor

@tynes tynes commented Jul 25, 2023

Description

This ports the replica-healthcheck service to the chain-mon package. The idea is that all of the simple typescript services that monitor the chain should live in the same package. This was the final package that was not following this standard. This way it will be easier to run the various monitoring services since they will all be the same docker image, given different entrypoints. It will also be easier to isolate and update the deps.

@tynes tynes requested review from a team as code owners July 25, 2023 05:23
@tynes tynes requested review from twoshark and smartcontracts July 25, 2023 05:23
@changeset-bot
Copy link

changeset-bot bot commented Jul 25, 2023

⚠️ No Changeset found

Latest commit: 3d0178e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify
Copy link

netlify bot commented Jul 25, 2023

Deploy Preview for opstack-docs canceled.

Name Link
🔨 Latest commit 5f96f67
🔍 Latest deploy log https://app.netlify.com/sites/opstack-docs/deploys/64bf5c3de28015000804a52c

@tynes tynes requested review from maurelian and removed request for smartcontracts July 25, 2023 05:23
This ports the `replica-healthcheck` service to the `chain-mon`
package. The idea is that all of the simple typescript services
that monitor the chain should live in the same package. This was
the final package that was not following this standard. This way
it will be easier to run the various monitoring services since
they will all be the same docker image, given different entrypoints.
It will also be easier to isolate and update the deps.
@tynes tynes force-pushed the cleanup/replica-mon branch from 5f96f67 to 3d0178e Compare July 26, 2023 21:25
@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #6438 (3d0178e) into develop (90111ed) will decrease coverage by 0.01%.
Report is 2 commits behind head on develop.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #6438      +/-   ##
===========================================
- Coverage    40.73%   40.72%   -0.01%     
===========================================
  Files          465      465              
  Lines        33472    33472              
  Branches      1949     1949              
===========================================
- Hits         13636    13633       -3     
- Misses       18359    18364       +5     
+ Partials      1477     1475       -2     
Flag Coverage Δ
bedrock-go-tests 43.91% <ø> (-0.01%) ⬇️
cannon-go-tests 62.40% <ø> (ø)
chain-mon-tests 26.95% <ø> (ø)
common-ts-tests 26.82% <ø> (ø)
contracts-bedrock-tests 44.54% <ø> (-0.08%) ⬇️
contracts-ts-tests 6.88% <ø> (ø)
core-utils-tests 44.24% <ø> (ø)
sdk-next-tests 42.60% <ø> (ø)
sdk-tests 42.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 3 files with indirect coverage changes

@tynes tynes merged commit 63d5a75 into develop Jul 28, 2023
@tynes tynes deleted the cleanup/replica-mon branch July 28, 2023 00:07
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.

2 participants