mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-25 09:06:02 +03:00
[GH-ISSUE #897] Question: Do we really need healthcheck? #557
Labels
No labels
expected: maybe someday
expected: next release
expected: release after next
expected: unlikely unless contributed
good first ticket
help wanted
pull-request
scope: all users
scope: windows users
size: easy
size: hard
size: medium
size: medium
status: backlog
status: blocked
status: done
status: idea-phase
status: needs followup
status: wip
status: wontfix
touches: API/CLI/Spec
touches: configuration
touches: data/schema/architecture
touches: dependencies/packaging
touches: docs
touches: js
touches: views/replayers/html/css
why: correctness
why: functionality
why: performance
why: security
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ArchiveBox#557
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @upsuper on GitHub (Nov 27, 2021).
Original GitHub issue: https://github.com/ArchiveBox/ArchiveBox/issues/897
Today I noticed that my disk write rate increased over the past months, and when I investigated, I found Docker did a large amount of disk write, which was seemingly from regularly writing
config.v2.jsonin the directory of ArchiveBox container. When I inspected that file, I found that there are logs for healthcheck, and the container has been in unhealthy state for a long time because I changed the service port of the container indocker-compose.yml. I tried bringing it back to healthy state by changing the port back, but the problem was still there, and the disk write rate I could see at that point was ~15MB per 30s. Eventually I disabled healthcheck indocker-compose.ymland this problem stops.There are two observations:
So I wonder whether it's really meaningful to have the healthcheck at all. It doesn't seem to be useful that it can inform users if something goes wrong, and it seems to be wasting resources. Should we drop it? Or disable it by default in the provided
docker-compose.yml?@pirate commented on GitHub (Nov 30, 2021):
huh I don't see any healthcheck enabled in https://github.com/ArchiveBox/ArchiveBox/blob/dev/docker-compose.yml, I explicitly don't add those and leave them to the user to configure if they need it.
@upsuper commented on GitHub (Nov 30, 2021):
My guess would be that if the
Dockerfilecontains healthcheck, it's enabled by default, unless it's overridden indocker-compose.yml. If the intention is that it's left to user, maybe the compose file should have it disabled with a comment?@mAAdhaTTah commented on GitHub (Nov 30, 2021):
Healthcheck is configured here.
@pirate commented on GitHub (Dec 3, 2021):
ah right, that's really old. commented it out:
github.com/ArchiveBox/ArchiveBox@b40a1e19d1archivebox initloses metadata #1863archivebox initloses metadata #3374