All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rebecca Broekhuis <beccabroek@gmail.com>
To: geissonator@gmail.com
Cc: ed.tanous@intel.com, openbmc@lists.ozlabs.org
Subject: Re: SOL Logging
Date: Mon, 27 Aug 2018 09:21:28 -0500	[thread overview]
Message-ID: <CABzWfr=g7s7SMDqovsoRA1aX71ZpRaHZzPefM3vB40h5SUP86g@mail.gmail.com> (raw)
In-Reply-To: <CALLMt=onADDRBk1w-zkxSEq-8XQQxbOrkP-zAVP9x+pFbN1oAA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3839 bytes --]

I agree with what you're saying about the javascript implementation of this
being limited. Besides the session being destroyed upon leaving the page, I
am also seeing that we are losing console messages in the front end when
the websocket closes for a moment (which it is consistently doing during
power on). We lose those messages sent during the time it takes to re-open
the connection. They aren't displayed in the terminal emulator and
therefore aren't exported to the text file when the user clicks 'export'. I
could see that as another reason to implement the ability to download the
existing obmc-console.log, since the pure javascript way of exporting to a
text file could be an incomplete log in some cases.

There might be a another way to address missing messages as a result of the
websocket closing, but from my current understanding it seems like another
reason having the javascript export exclusively would be an unreliable
option.

I can also see how having both downloads available could be confusing to
the user -- I'd be happy to run this by the design team to see if they have
input.

-- Becca

On Mon, Aug 27, 2018 at 7:49 AM Andrew Geissler <geissonator@gmail.com>
wrote:

> On Fri, Aug 24, 2018 at 4:03 PM Tanous, Ed <ed.tanous@intel.com> wrote:
> >
> > I would like to start a discussion around the feature of SOL logging,
> and how to best implement it.
> >
> >
> >
> > There is a proposal here
> https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-webui/+/12063/
> that proposes dumping the SOL console text buffer from Javascript into a
> file, and presenting it to the user as a download.  On its face, it seems
> to work as designed.  This has some pretty severe limitations, in that you
> can only dump the console log from your session, and your session is
> essentially destroyed when you switch pages in the webui, or hit refresh.
> I think this is overly limiting, and of the production BMC stacks that I
> know of, none of them implement SOL logging in this way.
> >
> >
> >
> > Instead, other BMCs implement it as a circular buffer inside the BMC
> itself, which allows SOL to log data all the time, not just while the user
> is logged in.  I think architecting it this way would be much more useful
> for admins consuming OpenBMC, and make us more useful as a solution.  Doing
> it this way also can make the SOL log available to IPMI and Redfish, as
> well as a file download, which improves the usability quite a bit.  Most
> places I see the SOL logging requested is for audit type purposes, which
> the javascript based version can’t do.
>
> We use https://github.com/openbmc/obmc-console for this currently.  It
> creates a file at /var/log/obmc-console.log and even has a
> customizable size for the file to wrap at. I guess the advantage of
> having it in the GUI is the user could ensure a full dump of the
> console (vs. just grabbing the existing obmc-console.log which may
> have wrapped). Being able to view the SOL output has it happens in the
> GUI, then having an option to export it sounds kind of useful.  Also
> be nice to be able to retrieve and view the existing obmc-console.log
> from the system in case you missed it in the GUI.  Maybe that latter
> part is enough though, reduces GUI complexity and testing. Something
> good to run by our usability/design team Gunnar/Rebecca?
>
> >
> >
> > I’m looking for feedback from the community here.  As written, I don’t
> think the javascript console export is a useful feature given its
> limitations.  Do others agree?  Am I off base?  Are there use models where
> having a log of only the current session is useful?  Should we document the
> limitations so we can respond to bugs in the future?
> >
> >
> >
> > Thanks,
> >
> >
> >
> > -Ed
>

[-- Attachment #2: Type: text/html, Size: 4490 bytes --]

  reply	other threads:[~2018-08-27 14:21 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-24 21:02 SOL Logging Tanous, Ed
2018-08-25  0:32 ` Sai Dasari
2018-08-27 12:48 ` Andrew Geissler
2018-08-27 14:21   ` Rebecca Broekhuis [this message]
2018-08-27 17:00 ` Jeremy Kerr
2018-08-28 21:16   ` Rebecca Broekhuis
2018-08-28 23:00     ` Tanous, Ed
2018-08-29  2:13     ` Jeremy Kerr
2018-08-29 11:37 ` Alexander Amelkin
2018-08-29 20:08   ` Rebecca Broekhuis
2018-09-05  0:07     ` Emily Shaffer
2018-09-05  0:30       ` Ed Tanous
2018-09-05 18:23         ` Rebecca Broekhuis
2018-09-10  6:52         ` Stewart Smith
2018-09-14  2:58         ` Jeremy Kerr

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CABzWfr=g7s7SMDqovsoRA1aX71ZpRaHZzPefM3vB40h5SUP86g@mail.gmail.com' \
    --to=beccabroek@gmail.com \
    --cc=ed.tanous@intel.com \
    --cc=geissonator@gmail.com \
    --cc=openbmc@lists.ozlabs.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.