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

Added input type format (per discussion with Andrey) #49

Merged
1 commit merged into from
Jul 23, 2019
Merged

Conversation

typeundefined
Copy link

hax can ingest the raw messages directly from consul. Format of the expected messages is documented.
cc @andriy.tkachuk

@andriytk
Copy link
Collaborator

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

@andriytk
Copy link
Collaborator

add: "(provided by Consul)".

@andriytk
Copy link
Collaborator

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

@andriytk
Copy link
Collaborator

Add also that service can be regarded Online only if all the Checks from the above array are "passing". If any of them is not "passing" - the service is Offline.

@andriytk
Copy link
Collaborator

Service FID.

@andriytk
Copy link
Collaborator

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

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

resolved all discussions

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

mentioned in issue #52

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

created #52 to continue this discussion

4 similar comments
@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

created #52 to continue this discussion

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

created #52 to continue this discussion

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

created #52 to continue this discussion

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

created #52 to continue this discussion

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

added 13 commits

  • c254fae7...5a597296 - 12 commits from branch master
  • 3b2c853 - Added input type format (per discussion with Andrey)

Compare with previous version

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

I'm merging the patch.

@konstantin.nekrasov, the issues raised by @andriy.tkachuk are collected in http://gitlab.mero.colo.seagate.com/mero/hare/issues/2. Please address those in a follow-up patch.

@vvv
Copy link
Contributor

vvv commented Jul 23, 2019

merged

This pull request 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

Successfully merging this pull request may close these issues.

None yet

3 participants