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

Unhandled promise rejection while starting #107

Closed
nikolaykasyanov opened this issue Feb 22, 2020 · 4 comments
Closed

Unhandled promise rejection while starting #107

nikolaykasyanov opened this issue Feb 22, 2020 · 4 comments
Labels

Comments

@nikolaykasyanov
Copy link

Config

I'm running Room Assistant in Docker via the following docker-compose.yml:

...
  rass:
    image: mkerix/room-assistant:2.0.0
    network_mode: host
    volumes:
      - /var/run/dbus:/var/run/dbus
      - ./volumes/rass/config:/room-assistant/config

Room Assistant config:

global:
  instanceName: living-room
  integrations:
    - bluetoothClassic
    - homeAssistant
bluetoothClassic:
  minRssi: -20
  addresses:
    - 'redacted'
homeAssistant:
  mqttUrl: mqtt://127.0.0.1
  mqttOptions:
    username: redacted
    password: redacted

Starcktrace

rass_1         | (node:1) UnhandledPromiseRejectionWarning: Error: dns service error: unknown
rass_1         |     at new Advertisement (/usr/local/lib/node_modules/room-assistant/node_modules/mdns/lib/advertisement.js:56:10)
rass_1         |     at Object.create [as createAdvertisement] (/usr/local/lib/node_modules/room-assistant/node_modules/mdns/lib/advertisement.js:64:10)
rass_1         |     at ClusterService.startBonjourDiscovery (/usr/local/lib/node_modules/room-assistant/dist/cluster/cluster.service.js:112:35)
rass_1         |     at ClusterService.onApplicationBootstrap (/usr/local/lib/node_modules/room-assistant/dist/cluster/cluster.service.js:67:22)
rass_1         |     at MapIterator.iteratee (/usr/local/lib/node_modules/room-assistant/node_modules/@nestjs/core/hooks/on-app-bootstrap.hook.js:21:43)
rass_1         |     at MapIterator.next (/usr/local/lib/node_modules/room-assistant/node_modules/iterare/lib/map.js:13:39)
rass_1         |     at IteratorWithOperators.next (/usr/local/lib/node_modules/room-assistant/node_modules/iterare/lib/iterate.js:21:28)
rass_1         |     at Function.from (<anonymous>)
rass_1         |     at IteratorWithOperators.toArray (/usr/local/lib/node_modules/room-assistant/node_modules/iterare/lib/iterate.js:180:22)
rass_1         |     at callOperator (/usr/local/lib/node_modules/room-assistant/node_modules/@nestjs/core/hooks/on-app-bootstrap.hook.js:22:10)
@nikolaykasyanov
Copy link
Author

I should've searched first, the issue is mentioned here: #102 (comment). Should I keep it open for bookkeeping though, as it seems like beta 7 didn't fix it? cc @mKeRix

@mKeRix
Copy link
Owner

mKeRix commented Feb 22, 2020

Do you have Avahi running on your host machine? Otherwise that will lead to these errors and cluster auto discovery not working. I agree that the message is a bit cryptic, we should at least keep this issue open to improve on the message that is logged.

@nikolaykasyanov
Copy link
Author

@mKeRix I do, that explains it, thanks.

@mKeRix mKeRix closed this as completed in 31174db Feb 28, 2020
github-actions bot pushed a commit that referenced this issue Feb 28, 2020
# [2.1.0](v2.0.1...v2.1.0) (2020-02-28)

### Bug Fixes

* **bluetooth-classic:** enforce time limit for inquiries ([f9a1ae4](f9a1ae4))
* **bluetooth-classic:** give hciconfig proper permissions ([29c4406](29c4406))
* **bluetooth-classic:** request info only on current instance ([9ae5cca](9ae5cca))
* **cluster:** catch mdns discovery issues ([31174db](31174db)), closes [#107](#107)

### Features

* make hci device to be used configurable ([d8f9d59](d8f9d59))
@github-actions
Copy link

🎉 This issue has been resolved in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants