Skip to content

fix WOL in docker/jail#6810

Merged
fabaff merged 2 commits into
home-assistant:devfrom
goto100:patch-1
Mar 27, 2017
Merged

fix WOL in docker/jail#6810
fabaff merged 2 commits into
home-assistant:devfrom
goto100:patch-1

Conversation

@goto100
Copy link
Copy Markdown
Contributor

@goto100 goto100 commented Mar 27, 2017

Description:

in docker/jail, WOL doesn't works due to subnet multicast issues.

add ip_address parameter to send_magic_packet if host specific.

add ip_address parameter to send_magic_packet if host specific.

in docker/jail, WOL doesn't works due to subnet broadcast issues.
@mention-bot
Copy link
Copy Markdown

@goto100, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fabaff, @Chris-V and @iamtpage to be potential reviewers.

@homeassistant
Copy link
Copy Markdown
Contributor

Hi @goto100,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

"""Turn the device on."""
self._wol.send_magic_packet(self._mac_address)
if self._host:
self._wol.send_magic_packet(self._mac_address, ip_address=self._host)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

line too long (81 > 79 characters)

@goto100
Copy link
Copy Markdown
Contributor Author

goto100 commented Mar 27, 2017

Copy link
Copy Markdown
Member

@pvizeli pvizeli left a comment

Choose a reason for hiding this comment

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

Please fix lint too

Copy link
Copy Markdown
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

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

🐦

@fabaff fabaff merged commit b8c1bc9 into home-assistant:dev Mar 27, 2017
@fabaff fabaff mentioned this pull request Apr 6, 2017
@aneisch
Copy link
Copy Markdown
Contributor

aneisch commented Apr 11, 2017

This commit seems to have contributed to #7026

@goto100
Copy link
Copy Markdown
Contributor Author

goto100 commented Apr 19, 2017

I'm sorry about this, this patch only works when router IP lease hasn't expired, if the lease expired, the WOL function won't works in any environment (include jail/docker), this causes the bug #7026, which been fixed by #7144.

Sorry again, I haven't been tested the functional fully.

@goto100 goto100 deleted the patch-1 branch April 20, 2017 07:46
@home-assistant home-assistant locked and limited conversation to collaborators Aug 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants