-
Notifications
You must be signed in to change notification settings - Fork 64
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
Failing gracefully on errors #4
Comments
I would say there is no real graceful way of failing, except maybe tearing down everything, but then we risk touching stuff set-up independent of sqm-scripts (not nice). |
Hi Toke, I am thinking about taking inspiration from luci's diagnostics page and display the output of "logread | grep -e SQM" (as first approximation, we can also modify sqm_logger to dump all messages into a temporarily file and just display that) in an output pane somewhere on the first sqm tab (details might contain traces of devil, as always). That way we can pipe a lot of information and diagnostics back to the user, if we just end with a noticeable "THIS SQM INVOCATION WORKED LIKE A CHARME" or so we can even signal success back to the user (and probably also the lack of success by other means). |
I am thinking about taking inspiration from luci's diagnostics page and display
the output of "logread | grep -e SQM" (as first approximation, we can also
modify sqm_logger to dump all messages into a temporarily file and just display
that) in an output pane somewhere on the first sqm tab (details might contain
traces of devil, as always). That way we can pipe a lot of information and
diagnostics back to the user, if we just end with a noticeable "THIS SQM
INVOCATION WORKED LIKE A CHARME" or so we can even signal success back to the
user (and probably also the lack of success by other means).
If that works we can also signal back information about errors and potential
remedies to the user, which is a first step in failing moire gracefully ;)
This sound reasonable; let's see how it turns out in practice :)
…-Toke
|
Hi Toke, On Aug 26, 2015, at 18:53 , Toke H�iland-J�rgensen notifications@github.com wrote:
Best Regards
|
What if you exported the interface or pseudo-interface that you will build the shaper and qdisc on? Write to two tmpfs files |
Currently we mostly don't check commands for errors. We probably should,
and then fail gracefully if something critical fails...
This includes in particular insmod, (see #3), but also other things like
iptables setup and chains of tc commands...
The text was updated successfully, but these errors were encountered: