Skip to content

Commit

Permalink
op-node,op-heartbeat: remove stale node heartbeat monitoring service (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
protolambda authored Aug 27, 2024
1 parent ced7889 commit 0da4ba1
Show file tree
Hide file tree
Showing 24 changed files with 18 additions and 868 deletions.
44 changes: 0 additions & 44 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1517,10 +1517,6 @@ workflows:
on_changes: op-e2e,packages/contracts-bedrock/src
uses_artifacts: true
requires: ["go-mod-download", "pnpm-monorepo"]
- go-test:
name: op-heartbeat-tests
module: op-heartbeat
requires: [ "go-mod-download" ]
- go-test:
name: op-batcher-tests
module: op-batcher
Expand Down Expand Up @@ -1607,7 +1603,6 @@ workflows:
- go-mod-download
- op-batcher-tests
- op-chain-ops-tests
- op-heartbeat-tests
- op-node-tests
- op-proposer-tests
- op-challenger-tests
Expand Down Expand Up @@ -1657,11 +1652,6 @@ workflows:
docker_name: op-conductor
docker_tags: <<pipeline.git.revision>>,<<pipeline.git.branch>>
# op-conductor is not part of the devnet, we don't save it.
- docker-build:
name: op-heartbeat-docker-build
docker_name: op-heartbeat
docker_tags: <<pipeline.git.revision>>,<<pipeline.git.branch>>
# op-heartbeat is not part of the devnet, we don't save it.
- docker-build:
name: da-server-docker-build
docker_name: da-server
Expand Down Expand Up @@ -1712,26 +1702,6 @@ workflows:
only: /^(da-server|ci-builder(-rust)?|ufm-[a-z0-9\-]*|op-[a-z0-9\-]*)\/v.*/
branches:
ignore: /.*/
- docker-build:
name: op-heartbeat-release
filters:
tags:
only: /^op-heartbeat\/v.*/
branches:
ignore: /.*/
docker_name: op-heartbeat
docker_tags: <<pipeline.git.revision>>
requires: ['hold']
platforms: "linux/amd64,linux/arm64"
publish: true
release: true
context:
- oplabs-gcr-release
- check-cross-platform:
name: op-heartbeat-cross-platform
op_component: op-heartbeat
requires:
- op-heartbeat-release
- docker-build:
name: op-node-docker-release
filters:
Expand Down Expand Up @@ -2107,20 +2077,6 @@ workflows:
op_component: op-conductor
requires:
- op-conductor-docker-publish
- docker-build:
name: op-heartbeat-docker-publish
docker_name: op-heartbeat
docker_tags: <<pipeline.git.revision>>,<<pipeline.git.branch>>
platforms: "linux/amd64,linux/arm64"
publish: true
context:
- oplabs-gcr
- slack
- check-cross-platform:
name: op-heartbeat-cross-platform
op_component: op-heartbeat
requires:
- op-heartbeat-docker-publish
- docker-build:
name: op-supervisor-docker-publish
docker_name: op-supervisor
Expand Down
1 change: 0 additions & 1 deletion .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
/op-challenger @ethereum-optimism/go-reviewers
/op-dispute-mon @ethereum-optimism/go-reviewers
/op-e2e @ethereum-optimism/go-reviewers
/op-heartbeat @ethereum-optimism/go-reviewers
/op-node @ethereum-optimism/go-reviewers
/op-node/rollup @protolambda @ajsutton
/op-alt-da @ethereum-optimism/go-reviewers
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/tag-service.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ on:
options:
- ci-builder
- ci-builder-rust
- op-heartbeat
- op-node
- op-batcher
- op-proposer
Expand Down
4 changes: 0 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@ The Optimism Immunefi program offers up to $2,000,042 for in-scope critical vuln
├── <a href="./op-chain-ops">op-chain-ops</a>: State surgery utilities
├── <a href="./op-challenger">op-challenger</a>: Dispute game challenge agent
├── <a href="./op-e2e">op-e2e</a>: End-to-End testing of all bedrock components in Go
├── <a href="./op-heartbeat">op-heartbeat</a>: Heartbeat monitor service
├── <a href="./op-node">op-node</a>: rollup consensus-layer client
├── <a href="./op-preimage">op-preimage</a>: Go bindings for Preimage Oracle
├── <a href="./op-program">op-program</a>: Fault proof program
Expand Down Expand Up @@ -118,11 +117,8 @@ The full set of components that have releases are:
- `op-batcher`
- `op-contracts`
- `op-challenger`
- `op-heartbeat`
- `op-node`
- `op-proposer`
- `op-ufm`
- `proxyd`

All other components and packages should be considered development components only and do not have releases.

Expand Down
17 changes: 0 additions & 17 deletions docker-bake.hcl
Original file line number Diff line number Diff line change
Expand Up @@ -53,10 +53,6 @@ variable "OP_DISPUTE_MON_VERSION" {
default = "${GIT_VERSION}"
}

variable "OP_HEARTBEAT_VERSION" {
default = "${GIT_VERSION}"
}

variable "OP_PROGRAM_VERSION" {
default = "${GIT_VERSION}"
}
Expand Down Expand Up @@ -152,19 +148,6 @@ target "op-conductor" {
tags = [for tag in split(",", IMAGE_TAGS) : "${REGISTRY}/${REPOSITORY}/op-conductor:${tag}"]
}

target "op-heartbeat" {
dockerfile = "ops/docker/op-stack-go/Dockerfile"
context = "."
args = {
GIT_COMMIT = "${GIT_COMMIT}"
GIT_DATE = "${GIT_DATE}"
OP_HEARTBEAT_VERSION = "${OP_HEARTBEAT_VERSION}"
}
target = "op-heartbeat-target"
platforms = split(",", PLATFORMS)
tags = [for tag in split(",", IMAGE_TAGS) : "${REGISTRY}/${REPOSITORY}/op-heartbeat:${tag}"]
}

target "da-server" {
dockerfile = "ops/docker/op-stack-go/Dockerfile"
context = "."
Expand Down
1 change: 0 additions & 1 deletion op-heartbeat/.gitignore

This file was deleted.

22 changes: 0 additions & 22 deletions op-heartbeat/Makefile

This file was deleted.

34 changes: 0 additions & 34 deletions op-heartbeat/cmd/main.go

This file was deleted.

48 changes: 0 additions & 48 deletions op-heartbeat/config.go

This file was deleted.

46 changes: 0 additions & 46 deletions op-heartbeat/flags/flags.go

This file was deleted.

Loading

0 comments on commit 0da4ba1

Please sign in to comment.