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.

Sign up for GitHub

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

Jump to bottom

fix(cmd-api-server): healtcheck broken due to missing wget binary #2894

Closed
petermetz opened this issue Nov 16, 2023 · 0 comments · Fixed by #2895 or #3324
Closed

fix(cmd-api-server): healtcheck broken due to missing wget binary #2894

petermetz opened this issue Nov 16, 2023 · 0 comments · Fixed by #2895 or #3324
Assignees
@petermetz
Labels
bug Something isn't working Developer_Experience P1 Priority 1: Highest
Milestone
v2.0.0

Comments

@petermetz
Copy link
Contributor

Description

The latest version of the API server's container image fails healthcheck like this (despite being healthy otherwise):

 {
      "Start": "2023-11-15T21:49:56.884066207-08:00",
      "End": "2023-11-15T21:49:56.922505045-08:00",
      "ExitCode": 127,
      "Output": "/healthcheck.sh: 3: wget: not found\n"
  },

Acceptance Criteria

  1. Healthcheck is passing when the API server is working fine
  2. Published an image pre-emptively to GHCR as proof of fix
@petermetz petermetz added bug Something isn't working Developer_Experience P1 Priority 1: Highest labels Nov 16, 2023
@petermetz petermetz added this to the v2.0.0 milestone Nov 16, 2023
@petermetz petermetz self-assigned this Nov 16, 2023
petermetz added a commit to petermetz/cacti that referenced this issue Nov 16, 2023
@petermetz
fix(cmd-api-server): healthcheck broken due to missing wget binary
a69c98a 
1. wget now gets installed explicitly
2. upgraded to ubuntu 22.04 LTS for the base image
3. upgraded nodejs to v20 LTS and npm to v10
3. Published the container image corresponding to this commit tagged as
`ghcr.io/hyperledger/cactus-cmd-api-server:2023-11-16-issue2894`

[skip ci]

Fixes hyperledger-cacti#2894

Signed-off-by: Peter Somogyvari <[email protected]>
@petermetz petermetz mentioned this issue Nov 16, 2023
Merged
5 tasks
petermetz added a commit to petermetz/cacti that referenced this issue Nov 17, 2023
@petermetz
fix(cmd-api-server): healthcheck broken due to missing wget binary
792f3e8 
1. wget now gets installed explicitly
2. upgraded to ubuntu 22.04 LTS for the base image
3. upgraded nodejs to v20 LTS and npm to v10
3. Published the container image corresponding to this commit tagged as
`ghcr.io/hyperledger/cactus-cmd-api-server:2023-11-16-issue2894`

[skip ci]

Fixes hyperledger-cacti#2894

Signed-off-by: Peter Somogyvari <[email protected]>
petermetz added a commit that referenced this issue Nov 17, 2023
@petermetz
fix(cmd-api-server): healthcheck broken due to missing wget binary
8f1ca3f 
1. wget now gets installed explicitly
2. upgraded to ubuntu 22.04 LTS for the base image
3. upgraded nodejs to v20 LTS and npm to v10
3. Published the container image corresponding to this commit tagged as
`ghcr.io/hyperledger/cactus-cmd-api-server:2023-11-16-issue2894`

[skip ci]

Fixes #2894

Signed-off-by: Peter Somogyvari <[email protected]>
sandeepnRES pushed a commit to sandeepnRES/cacti that referenced this issue Dec 21, 2023
@petermetz @sandeepnRES
fix(cmd-api-server): healthcheck broken due to missing wget binary
05d625c 
1. wget now gets installed explicitly
2. upgraded to ubuntu 22.04 LTS for the base image
3. upgraded nodejs to v20 LTS and npm to v10
3. Published the container image corresponding to this commit tagged as
`ghcr.io/hyperledger/cactus-cmd-api-server:2023-11-16-issue2894`

[skip ci]

Fixes hyperledger-cacti#2894

Signed-off-by: Peter Somogyvari <[email protected]>
@petermetz petermetz mentioned this issue Jun 14, 2024
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@petermetz petermetz

Labels
bug Something isn't working Developer_Experience P1 Priority 1: Highest
Projects
None yet
Milestone
v2.0.0
1 participant
@petermetz