Skip to content
This repository has been archived by the owner on Aug 29, 2019. It is now read-only.

Take a look at "condition: service_healthy" for starting invoicer if its not connecting to LND #15

Closed
nolim1t opened this issue Jan 9, 2019 · 3 comments
Assignees

Comments

@nolim1t
Copy link
Member

nolim1t commented Jan 9, 2019

May be possibly required because invoicer needs to check a few things.

Should possibly build in lnd / lncli too?

example

db:
    image: mysql:5.7.20
    healthcheck:
      test: >
        mysql \
          --host='localhost' \
          --user='${mysql_database_user}' \
          --password='${mysql_database_password}' \
          --execute='USE ${mysql_database_name}' \
      interval: ${healthcheck_interval}
      timeout: ${healthcheck_timeout}
      retries: ${healthcheck_retries}
...
@nolim1t nolim1t self-assigned this Jan 10, 2019
@nolim1t nolim1t changed the title Take a look at "condition: service_healthy" Take a look at "condition: service_healthy" for starting invoicer Jan 10, 2019
@nolim1t
Copy link
Member Author

nolim1t commented Jan 13, 2019

Will revisit this because GRPC_GO_RETRY=on variable doesn't seem to work. A possible solution is to look for the error message in the healthcheck from a curl request and then get docker-compose to restart the container.

@nolim1t nolim1t changed the title Take a look at "condition: service_healthy" for starting invoicer Take a look at "condition: service_healthy" for starting invoicer if its not connecting to LND Jan 13, 2019
nolim1t added a commit to nolim1t/invoicer that referenced this issue Jan 15, 2019
nolim1t added a commit to nolim1t/invoicer that referenced this issue Jan 15, 2019
nolim1t added a commit that referenced this issue Jan 15, 2019
@nolim1t
Copy link
Member Author

nolim1t commented Jan 15, 2019

current status: Building and pushing to dockerhub to see if this actually fixes the problem

@nolim1t
Copy link
Member Author

nolim1t commented Jan 16, 2019

Pushed to dockerhub, however healthcheck doesn't fully resolve the reconnecting issue :/

@nolim1t nolim1t closed this as completed Jan 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant