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

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

Closed
ydahhrk opened this issue Aug 19, 2015 · 3 comments
Closed
Assignees
Milestone

Comments

@ydahhrk
Copy link
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
Copy link

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
Copy link
Member Author

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
Copy link
Member Author

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
Projects
None yet
Development

No branches or pull requests

3 participants