That's right - this sends log messages to the gui, but since there's no gui
in destiny yet anyway I just commented it out for now
Should be ok now


On Tue, Aug 31, 2010 at 8:15 PM, James "Murphy" McCauley <jam...@nau.edu>wrote:

> This is a destiny-specific bug and was introduced in commit
> 58547d74330eb0511cdddbbeb4f3017278a50d53.
>
> I am not set up for commits here, but I've attached a patch.  I think there
> are actually a number of improvements that could be made to the socket-based
> logging, but that's a project for another day.  At least this stops it from
> crashing.
>
> KK, if you commit it, I suggest the log message:
>  Truncate long log messages when writing to logging socket.
>
> (If you don't, I'll commit it when I get back to California.)
>
> -- Murphy
>
>
>
> On Aug 31, 2010, at 1:51 AM, kk yap wrote:
>
> > Hi Niky,
> >
> > This is not an expected behavior.  What str(stats) JSON or XML in NOX
> > 0.6?  I had problem with long log messages a while ago, but did not
> > check if the previous version of NOX is okay with it.
> >
> > In this case, you would need to provide a more precise and simple way
> > for us to replicate the bug before we can go ahead to hack it out of
> > NOX.  Could you do that?  Many thanks.
> >
> > Regards
> > KK
> >
> > On 31 August 2010 00:44, Niky Riga <nr...@bbn.com> wrote:
> >> Hi KK,
> >>
> >> Thanks for getting back to me. The problem only appears in the destiny
> >> branch, I tried it again in the 0.6 version and the log message is
> printed
> >> normally, is this the expected behavior for the destiny branch?
> >>
> >> --niky
> >>
> >> kk yap wrote:
> >>>
> >>> Hi Niky,
> >>>
> >>> I believe this is because vlog does not support very long string.  The
> >>> correct patch (IMHO) is to truncate messages that are too long.
> >>> Unfortunately, I cannot get to a patch until later.  If anyone send us
> >>> a patch, one of us can push it.  Else, we will try to get to it.
> >>> Thanks.
> >>>
> >>> Regards
> >>> KK
> >>>
> >>> 2010/8/30 Niky Riga <nr...@bbn.com>:
> >>>
> >>>>
> >>>> Hi,
> >>>>
> >>>> I have recently ported my code from nox version 0.6 to nox destiny. I
> >>>> have
> >>>> checked out the head of the destiny branch some time last week.
> >>>>
> >>>> When a new datapath joins, I log the stats for debugging reasons. In
> nox
> >>>> destiny this causes a crash.
> >>>>
> >>>> The last command that is executed in my python module is :
> >>>>   logger.info('stats for switch : %s' % str(stats))
> >>>>
> >>>> The abort message is attached.
> >>>>
> >>>> Thanks,
> >>>> niky
> >>>>
> >>>> _______________________________________________
> >>>> nox-dev mailing list
> >>>> nox-dev@noxrepo.org
> >>>> http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org
> >>>>
> >>>>
> >>>>
> >>
> >>
> >
> > _______________________________________________
> > nox-dev mailing list
> > nox-dev@noxrepo.org
> > http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org
>
>
> _______________________________________________
> nox-dev mailing list
> nox-dev@noxrepo.org
> http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org
>
>
_______________________________________________
nox-dev mailing list
nox-dev@noxrepo.org
http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org

Reply via email to