Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[hueemulation] Fix broken fallback address #7305

Merged
merged 1 commit into from
Apr 5, 2020

Conversation

wborn
Copy link
Member

@wborn wborn commented Apr 5, 2020

With the changes in #6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

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

lgtm

@kaikreuzer kaikreuzer merged commit 2f2736b into openhab:2.5.x Apr 5, 2020
@wborn wborn deleted the hueemulation-fix-broken-fallback branch April 5, 2020 16:08
@cpmeister cpmeister added the bug An unexpected problem or unintended behavior of an add-on label Apr 5, 2020
@cpmeister cpmeister added this to the 2.5.4 milestone Apr 5, 2020
Hans-Reiner pushed a commit to Hans-Reiner/openhab2-addons that referenced this pull request Apr 11, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
Signed-off-by: Hans-Reiner Hoffmann <[email protected]>
yfre pushed a commit to yfre/openhab-addons that referenced this pull request Apr 27, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
Signed-off-by: Eugen Freiter <[email protected]>
markus7017 pushed a commit to markus7017/openhab-addons that referenced this pull request May 29, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
LoungeFlyZ pushed a commit to LoungeFlyZ/openhab2-addons that referenced this pull request Jun 8, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
J-N-K pushed a commit to J-N-K/openhab-addons that referenced this pull request Jul 14, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
CSchlipp pushed a commit to CSchlipp/openhab-addons that referenced this pull request Jul 26, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
Signed-off-by: CSchlipp <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Aug 31, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
DaanMeijer pushed a commit to DaanMeijer/openhab-addons that referenced this pull request Sep 1, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
Signed-off-by: Daan Meijer <[email protected]>
wborn added a commit to wborn/openhab-addons that referenced this pull request Sep 10, 2020
Upgrades the Travis CI build environment to Ubuntu 18.04 (Bionic Beaver).
The openjdk8 package is unsupported in newer Ubuntu versions.

Also fixes the following Travis configuration validation warnings:

* deprecated key sudo (The key `sudo` has no effect anymore.)
* missing os, using the default linux

Removes $HOME/.bnd/cache which does not exist after builds.

It's also no longer necessary to disable IPv6 with builds.
This was an actual issue in the hueemulation add-on which was fixed by openhab#7305.

Signed-off-by: Wouter Born <[email protected]>
wborn added a commit to wborn/openhab-addons that referenced this pull request Sep 10, 2020
Upgrades the Travis CI build environment to Ubuntu 20.04 (Focal Fossa).

Also fixes the following Travis configuration validation warnings:

* deprecated key sudo (The key `sudo` has no effect anymore.)
* missing os, using the default linux

Removes $HOME/.bnd/cache which does not exist after builds.

It's also no longer necessary to disable IPv6 with builds.
This was an actual issue in the hueemulation add-on which was fixed by openhab#7305.

Signed-off-by: Wouter Born <[email protected]>
Hilbrand pushed a commit that referenced this pull request Sep 10, 2020
Upgrades the Travis CI build environment to Ubuntu 20.04 (Focal Fossa).

Also fixes the following Travis configuration validation warnings:

* deprecated key sudo (The key `sudo` has no effect anymore.)
* missing os, using the default linux

Removes $HOME/.bnd/cache which does not exist after builds.

It's also no longer necessary to disable IPv6 with builds.
This was an actual issue in the hueemulation add-on which was fixed by #7305.

Signed-off-by: Wouter Born <[email protected]>
Hilbrand pushed a commit that referenced this pull request Sep 10, 2020
Upgrades the Travis CI build environment to Ubuntu 18.04 (Bionic Beaver).
The openjdk8 package is unsupported in newer Ubuntu versions.

Also fixes the following Travis configuration validation warnings:

* deprecated key sudo (The key `sudo` has no effect anymore.)
* missing os, using the default linux

Removes $HOME/.bnd/cache which does not exist after builds.

It's also no longer necessary to disable IPv6 with builds.
This was an actual issue in the hueemulation add-on which was fixed by #7305.

Signed-off-by: Wouter Born <[email protected]>
CSchlipp pushed a commit to CSchlipp/openhab-addons that referenced this pull request Sep 12, 2020
…ab#8442)

Upgrades the Travis CI build environment to Ubuntu 18.04 (Bionic Beaver).
The openjdk8 package is unsupported in newer Ubuntu versions.

Also fixes the following Travis configuration validation warnings:

* deprecated key sudo (The key `sudo` has no effect anymore.)
* missing os, using the default linux

Removes $HOME/.bnd/cache which does not exist after builds.

It's also no longer necessary to disable IPv6 with builds.
This was an actual issue in the hueemulation add-on which was fixed by openhab#7305.

Signed-off-by: Wouter Born <[email protected]>
markus7017 pushed a commit to markus7017/openhab-addons that referenced this pull request Sep 19, 2020
With the changes in openhab#6967 the HueEmulationServiceOSGiTest always fails for me.
The reason is that if the discoveryIps are not configured, it is no longer guaranteed it will use the primary IPv4 host address as fallback.
In my case the iterator on the HashSet would return an IPv6 address as first address.
By using a LinkedHashSet the order is preserved and it will again use the primary IPv4 host address as fallback.

Signed-off-by: Wouter Born <[email protected]>
markus7017 pushed a commit to markus7017/openhab-addons that referenced this pull request Sep 19, 2020
…ab#8442)

Upgrades the Travis CI build environment to Ubuntu 18.04 (Bionic Beaver).
The openjdk8 package is unsupported in newer Ubuntu versions.

Also fixes the following Travis configuration validation warnings:

* deprecated key sudo (The key `sudo` has no effect anymore.)
* missing os, using the default linux

Removes $HOME/.bnd/cache which does not exist after builds.

It's also no longer necessary to disable IPv6 with builds.
This was an actual issue in the hueemulation add-on which was fixed by openhab#7305.

Signed-off-by: Wouter Born <[email protected]>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Oct 8, 2020
…ab#8442)

Upgrades the Travis CI build environment to Ubuntu 18.04 (Bionic Beaver).
The openjdk8 package is unsupported in newer Ubuntu versions.

Also fixes the following Travis configuration validation warnings:

* deprecated key sudo (The key `sudo` has no effect anymore.)
* missing os, using the default linux

Removes $HOME/.bnd/cache which does not exist after builds.

It's also no longer necessary to disable IPv6 with builds.
This was an actual issue in the hueemulation add-on which was fixed by openhab#7305.

Signed-off-by: Wouter Born <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants