Re-enable healthcheck when we get the curl package in upstream #1

Closed
opened 2021-05-06 11:10:12 +00:00 by decentral1se · 6 comments
Owner
> https://github.com/Chocobozzz/PeerTube/issues/4045
Author
Owner
Dropped https://github.com/Chocobozzz/PeerTube/pull/4046
Author
Owner

Moving fast but also need to take a look at 0609cf8662.

Moving fast but also need to take a look at https://git.autonomic.zone/coop-cloud/peertube/commit/0609cf8662af4618acaf274113291f110eec7119.
Author
Owner

/api/v1/ping is available for peertube healthchecking!

`/api/v1/ping` is available for peertube healthchecking!
Author
Owner

It should be available now with the latest release 👍

It should be available now with the latest release 👍
Owner

We could also do this (right now!) using a native healthcheck, which is apparently preferable in general?

e.g. see coop-cloud/rocketchat, https://git.autonomic.zone/coop-cloud/rocketchat/src/branch/main/compose.yml#L735-L740

Although ideally that would be upstream too, it doesn't seem amazing to be maintaining our own code-in-compose-files 🤔🤔

We could also do this (right now!) using [a native healthcheck](https://ryaneschinger.com/blog/using-docker-native-health-checks/), which is apparently preferable in general? e.g. see coop-cloud/rocketchat, https://git.autonomic.zone/coop-cloud/rocketchat/src/branch/main/compose.yml#L735-L740 Although ideally that would be upstream too, it doesn't seem amazing to be maintaining our own code-in-compose-files 🤔🤔
Member

@3wordchant that's a very good idea, PR incoming :3

Also, I'll look into getting it upstream.

@3wordchant that's a very good idea, PR incoming :3 Also, I'll look into getting it upstream.
Sign in to join this conversation.
No Label
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/peertube#1
No description provided.