Skip to content

chore: add health check in docker#1452

Merged
Flouse merged 2 commits intoaxonweb3:mainfrom
Flouse:healthcheck-in-docker-dev
Sep 28, 2023
Merged

chore: add health check in docker#1452
Flouse merged 2 commits intoaxonweb3:mainfrom
Flouse:healthcheck-in-docker-dev

Conversation

@Flouse
Copy link
Copy Markdown
Contributor

@Flouse Flouse commented Sep 26, 2023

What this PR does / why we need it?

Without curl and jq, devtools/docker/health_check.sh was not working previously in the Axon docker container.

This PR fixed the Dockerfile and add healthcheck into the quick-start docker-compose file, to test it.

docker compose up -d --wait will wait for services to be running|healthy.

What is the impact of this PR?

No Breaking Change

PR relation:

Test

chain-axon-1
ghcr.io/flouse/axon:healthcheck-in-docker-dev
"/bin/sh -c /app/devtools/docker/docker-entrypoint.sh"
Up 31 seconds (healthy)

https://github.com/Flouse/axon/actions/runs/6314701964/job/17146622945#step:6:622

CI Settings

CI Usage

Tip: Check the CI you want to run below, and then comment /run-ci.

CI Switch

  • Web3 Compatible Tests
  • OCT 1-5 And 12-15
  • OCT 6-10
  • OCT 11
  • OCT 16-19
  • v3 Core Tests

CI Description

CI Name Description
Web3 Compatible Test Test the Web3 compatibility of Axon
v3 Core Test Run the compatibility tests provided by Uniswap V3
OCT 1-5 | 6-10 | 11 | 12-15 | 16-19 Run the compatibility tests provided by OpenZeppelin

@Flouse Flouse added the DevOps label Sep 26, 2023
@Flouse Flouse self-assigned this Sep 26, 2023
@Flouse Flouse requested a review from Simon-Tl September 26, 2023 16:07
@Flouse Flouse marked this pull request as ready for review September 26, 2023 16:10
@Flouse Flouse requested a review from a team as a code owner September 26, 2023 16:10
@Flouse Flouse requested a review from jjyr September 26, 2023 16:10
@Flouse Flouse requested review from a team, chaoticlonghair, jiangxianliang007 and jjyr and removed request for a team and jjyr September 27, 2023 06:50
@Flouse Flouse enabled auto-merge September 27, 2023 06:51
@KaoImin KaoImin changed the title chore: Add healthcheck in docker chore: add health check in docker Sep 27, 2023
@Flouse Flouse disabled auto-merge September 28, 2023 14:36
@Flouse Flouse merged commit 71f5bad into axonweb3:main Sep 28, 2023
@Flouse Flouse deleted the healthcheck-in-docker-dev branch September 28, 2023 14:37
@Flouse Flouse linked an issue Nov 5, 2023 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a health_check script into the Axon docker image

3 participants