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

Follow-up from "Added input type format (per discussion with Andrey)" #52

Closed
3 of 5 tasks
vvv opened this issue Jul 23, 2019 · 0 comments
Closed
3 of 5 tasks

Follow-up from "Added input type format (per discussion with Andrey)" #52

vvv opened this issue Jul 23, 2019 · 0 comments

Comments

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

The following discussions from #49 should be addressed:

  • @andriy.tkachuk started a discussion:

    From /? No. From bq/ - maybe. But not atm.

  • @andriy.tkachuk started a discussion:

    add: "(provided by Consul)".

  • @andriy.tkachuk started a discussion: (+1 comment)

    Not "registered". It will just listen to HTTP POSTs from the Consul's service-type watch.

  • @andriy.tkachuk started a discussion:

    Service FID.

  • @andriy.tkachuk started a discussion:

    All these must be "passing" to regard the service Online on the node. Otherwise - it is Offline.

@vvv
Copy link
Contributor Author

vvv commented Jul 23, 2019

mentioned in merge request #49

@vvv
Copy link
Contributor Author

vvv commented Jul 23, 2019

marked the task @andriy.tkachuk started a discussion: as completed

@vvv
Copy link
Contributor Author

vvv commented Jul 23, 2019

marked the task @andriy.tkachuk started a discussion: as completed

@vvv
Copy link
Contributor Author

vvv commented Jul 23, 2019

marked the task @andriy.tkachuk started a discussion: (+1 comment) as completed

@vvv
Copy link
Contributor Author

vvv commented Jul 23, 2019

First 3 items resolved by commit e50a1df.

@vvv
Copy link
Contributor Author

vvv commented Aug 2, 2019

There was no activity on this issue for a while, so I'm closing it. Feel free to reopen.

@vvv
Copy link
Contributor Author

vvv commented Aug 2, 2019

closed

This issue was closed.
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

2 participants