Skip to content

Event advertisement IP config#604

Merged
ghcs27 merged 2 commits intoSoCo:masterfrom
bjw-s:advertise_ip
May 22, 2018
Merged

Event advertisement IP config#604
ghcs27 merged 2 commits intoSoCo:masterfrom
bjw-s:advertise_ip

Conversation

@bjw-s
Copy link
Copy Markdown
Contributor

@bjw-s bjw-s commented Mar 15, 2018

Recreated #450 to allow configuration of event advertisement IP.

@adamcstephens
Copy link
Copy Markdown

This is very useful for those running in docker (not using net=host).

@stefankoegl
Copy link
Copy Markdown
Member

Looks good :) If there are no objections, I'll merge in a few days.

@bjw-s
Copy link
Copy Markdown
Contributor Author

bjw-s commented Apr 19, 2018

Don't think there seem to be any objections 😇

@adamcstephens
Copy link
Copy Markdown

Looking forward to this in fact.

Without this I’m stuck with the home assistant polling fallback, which can take longer to trigger automations.

Copy link
Copy Markdown
Member

@ghcs27 ghcs27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just marking this as approved (@stefankoegl already commented).
This should really be unproblematic.

@ghcs27
Copy link
Copy Markdown
Member

ghcs27 commented May 15, 2018

If there are no objections, I will merge this next week.
If you agree with the PR, please feel free to merge it earlier.

@ghcs27 ghcs27 merged commit 2c8ee9b into SoCo:master May 22, 2018
@ghcs27
Copy link
Copy Markdown
Member

ghcs27 commented May 23, 2018

Merged. Thank you for your contribution!
Please add a note about the change to the release notes thread (#614).

@bjw-s bjw-s deleted the advertise_ip branch May 24, 2018 06:55
@bjw-s bjw-s mentioned this pull request May 24, 2018
@bjw-s
Copy link
Copy Markdown
Contributor Author

bjw-s commented May 24, 2018

Thanks! I've added a post to the release notes thread.

@stefankoegl stefankoegl added this to the 0.16 milestone Aug 15, 2018
bjw-s added a commit to bjw-s/hass-config that referenced this pull request Feb 7, 2020
bjw-s added a commit to bjw-s/hass-config that referenced this pull request Feb 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants