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

Warning: vapor_rosnode validation failed: touched.ipv4: Path touched.ipv4 is required. #28

Open
HWiese1980 opened this issue Jan 22, 2019 · 5 comments

Comments

@HWiese1980
Copy link

This is probably just a minor annoyance but I feel like feedback like this is appreciated, so...

I have a warning that frequently pops up during startup of some nodes. I cannot really pinpoint what's causing them but they appear to be reproducible.

Here's the warning text:

(node:10869) UnhandledPromiseRejectionWarning: ValidationError: vapor_rosnode validation failed: touched.ipv4: Path `touched.ipv4` is required.
    at new ValidationError (/home/user/catkin_ws/src/vapor_master/node_modules/mongoose/lib/error/validation.js:27:11)
    at model.Document.invalidate (/home/user/catkin_ws/src/vapor_master/node_modules/mongoose/lib/document.js:1882:32)
    at p.doValidate.skipSchemaValidators (/home/user/catkin_ws/src/vapor_master/node_modules/mongoose/lib/document.js:1750:17)
    at /home/user/catkin_ws/src/vapor_master/node_modules/mongoose/lib/schematype.js:824:9
    at _combinedTickCallback (internal/process/next_tick.js:132:7)
    at process._tickCallback (internal/process/next_tick.js:181:9)
(node:10869) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 33)
@sevenbitbyte
Copy link
Member

Hmm sounds like an IPv6 handling error of some sort. I've tried several IPv6 scenarios but can't seem to get this same error:

@HWiese1980 could you run with debugging turned on you can do that be prefixing DEBUG=* prior to your start command:

DEUBG=* snap run vapor-master

or if you're running from the npm package you can simply:

yarn watch

@NickZ
Copy link

NickZ commented Jan 30, 2019

@HWiese1980 This should have been fixed by #29, can you please test to see if it has?

We've pushed a patched build to the edge channel in the snap store. You can try it out with: sudo snap install vapor-master --edge

@HWiese1980
Copy link
Author

I'll test it sometime this week, thanks

@HWiese1980
Copy link
Author

Oh, by the way, since snap has some major issues with unusual home paths (home directories in a different place than /home/user) on Ubuntu 16.04 I cannot use snap. I'm bound to stick to building from source.

@HWiese1980
Copy link
Author

I've upgraded my local repository to commit 73478c3 now. It seems like the warning persists, though.

I'm seeing a bunch of new error messages now too. I guess though, it's probably better to open issues for them separately.

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

No branches or pull requests

3 participants