Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

spring-boot-healthcheck adds probes to initContainers #1178

Closed
john-delivuk opened this issue Feb 8, 2018 · 7 comments
Closed

spring-boot-healthcheck adds probes to initContainers #1178

john-delivuk opened this issue Feb 8, 2018 · 7 comments
Labels
cat/bug Bug which needs fixing status/stale Issue/PR considered to be stale

Comments

@john-delivuk
Copy link

Description

With spring-boot-healthcheck enricher enabled, you'll get an invalid deployment spec because it will add livelinessProbes and readinessProbes to initContainers.

Info

  • f-m-p version : 3.5.34

  • Maven version (mvn -v) : 3.5.2

  • Kubernetes / OpenShift setup and version :
    kubernetes 1.8.6

  • If it's a bug, how to reproduce :
    Create a deployment with an init container and using springboot.

      initContainers:
      - command: "run.sh"
        image: someTaskThatRunsOnce:latest
@salaboy
Copy link

salaboy commented Feb 20, 2018

@john-delivuk are you testing with Spring Boot 2? I found some issues with it because now the health check endpoints are in a different path

@nicolaferraro
Copy link
Member

There are 2 different issues:

  1. This one is probably fixed by Improve support for sidecar containers #1202
  2. Spring-boot 2 is not yet supported. Actuator endpoints for health check are different

@salaboy
Copy link

salaboy commented Feb 20, 2018

@nicolaferraro can you point me out to where I can update that one? it should be great to configure with something like this:

<enricher>
            <config>
              <spring-boot-health-check>
                <path>/actuator/health</path>
              </spring-boot-health-check>
            </config>
          </enricher>

I'm more than happy to send a PR if you give me some pointers.
I've just tested updating the deployment descriptor from /health to /actuator/health and it works. If you can point me to where I can add that enricher configuration I will submit a PR as a temporary fix.

@nicolaferraro
Copy link
Member

@salaboy it would be awesome if you can work on a PR to change the plugin behavior according to the spring-boot version detected.

I've created issue #1206 to track (with info on how to implement it).

@rohanKanojia
Copy link
Member

Marking this as duplicate of #1283, please revert if not understood right.

@rohanKanojia
Copy link
Member

Duplicate of #1283

@rohanKanojia rohanKanojia marked this as a duplicate of #1283 Jul 26, 2018
@rohanKanojia rohanKanojia added the cat/bug Bug which needs fixing label Jul 26, 2018
@stale
Copy link

stale bot commented Oct 24, 2018

This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!

@stale stale bot added the status/stale Issue/PR considered to be stale label Oct 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cat/bug Bug which needs fixing status/stale Issue/PR considered to be stale
Projects
None yet
Development

No branches or pull requests

4 participants