search

tobilg / marathon-slack

Integration for Marathon's Event Bus with Slack
MIT License
42 stars 23 forks source link

Task Id in `health_status_changed_event` event is undefined #20

Closed idanto closed 6 years ago

idanto commented 6 years ago

In marathon 1.4.x there was an API change in health_status_changed_event event - Instead of taskId property there is instanceId property.

Marathon 1.4 docs Marathon 1.3 docs Latest Marathon Docs

tobilg commented 6 years ago

Thanks for reporting, I’ll have a look today.

idanto commented 6 years ago

@tobilg I already have a fix but in order to test it I have to open another pull request for you in https://github.com/tobilg/marathon-event-bus-mock

If it is ok from your side I will open both pull requests in the next hours

tobilg commented 6 years ago

Sure, please do so, as it'd save a lot of time for me. Thanks a lot in advance!