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

Improve visibility of friendly error messages due to bad configuration #169

Closed
ydahhrk opened this Issue Aug 19, 2015 · 3 comments

Comments

Projects
None yet
3 participants
@ydahhrk
Member

ydahhrk commented Aug 19, 2015

$ sudo jool -bat 192.0.2.1#19221 2001:db8::1#2181
Invalid input data or parameter (System error -7)
$ # ?
$
$ dmesg -t | tail -1
NAT64 Jool ERROR (add_static_route): The transport address (192.0.2.1#19221) does not
belong to the IPv4 pool. Please add it there first.

It would be better if it did this:

$ sudo jool -ba 192.0.2.1#19221 2001:db8::1#2181
The transport address (192.0.2.1#19221) does not belong to the IPv4 pool. Please add it
there first.

Well, granted, if the console is listening to kernel messages, it does something quite acceptable:

$ sudo jool -bat 192.0.2.1#19221 2001:db8::1#2181
NAT64 Jool ERROR (add_static_route): The transport address (192.0.2.1#19221) does not
belong to the IPv4 pool. Please add it there first.
Invalid input data or parameter (System error -7)

But not all consoles do that by default.

This has extremely low priority because

  1. Nobody has asked for or bumped it.
  2. It has a simple workaround (dmesg).
@kpfleming

This comment has been minimized.

Show comment
Hide comment
@kpfleming

kpfleming Sep 18, 2015

I ran into this problem this past week, and it certainly would have been helpful if the error message was passed back to the userspace tool.

kpfleming commented Sep 18, 2015

I ran into this problem this past week, and it certainly would have been helpful if the error message was passed back to the userspace tool.

@ydahhrk

This comment has been minimized.

Show comment
Hide comment
@ydahhrk

ydahhrk Sep 18, 2015

Member

Thanks! Assigned.

I'm not really sure if it's going to be ready by version 3.4.0, but it's a good not-so-introductory exercise for @rolivasnic.

Member

ydahhrk commented Sep 18, 2015

Thanks! Assigned.

I'm not really sure if it's going to be ready by version 3.4.0, but it's a good not-so-introductory exercise for @rolivasnic.

@ydahhrk

This comment has been minimized.

Show comment
Hide comment
@ydahhrk

ydahhrk Sep 22, 2015

Member

Huh, he already finished this even though I'm still not done tweaking pool4, so I guess this will make it into 3.4.0.

Member

ydahhrk commented Sep 22, 2015

Huh, he already finished this even though I'm still not done tweaking pool4, so I guess this will make it into 3.4.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment