<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <div class="moz-cite-prefix">On 8/17/2020 11:34 AM, Ed Tanous wrote:
    </div>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">
"Ed, what would it take for you to switch to using webui-vue?"</pre>
    </blockquote>
    <br>
    <span data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW55178573 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW55178573 BCX2" style="background-color:
        inherit;"><span data-contrast="auto" style="font-size: 12pt;
          line-height: 20.5042px;" class="TextRun SCXW112587793 BCX2"
          lang="EN-US"><span class="NormalTextRun SCXW112587793 BCX2"
            style="background-color: inherit;">Hi Ed, </span></span></span></span><br>
    <span data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW55178573 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW55178573 BCX2" style="background-color:
        inherit;"></span></span><br>
    <span data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW55178573 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW55178573 BCX2" style="background-color:
        inherit;">Thank you for your feedback.</span></span>
    <p><span data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW55178573 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW55178573 BCX2"
          style="background-color: inherit;"><br>
        </span></span></p>
    <p><span data-contrast="auto" style="font-size: 12pt; font-family:
        Calibri, Calibri_MSFontService, sans-serif; line-height:
        20.5042px;" class="TextRun SCXW55178573 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW55178573 BCX2"
          style="background-color: inherit;"></span></span>
    </p>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">What follows is my first impressions on an answer:

1. size parity with phosphor-webui.
On the current master, webui-vue has a 40% larger binary footprint
(614kb vs 440kb) and is still missing features that will add size.
Lots of bmcs run on 32MB of flash, so every kb matters, even if it
doesn't matter for the newer platforms with eMMC or larger SPI flash.
webui-vue needs to sort out where that extra heft came from, and
eradicate it.</pre>
    </blockquote>
    <span data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW70164443 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW70164443 BCX2" style="background-color:
        inherit;">It was smaller than phosphor-</span></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW70164443 BCX2" lang="EN-US"><span
        class="SpellingError SCXW70164443 BCX2" style="background-color:
        inherit;">webui</span></span><span data-contrast="auto"
      style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
      SCXW70164443 BCX2" lang="EN-US"><span class="NormalTextRun
        SCXW70164443 BCX2" style="background-color: inherit;"> until
        recently. </span></span><span class="LineBreakBlob BlobObject
      DragDrop SCXW70164443 BCX2" style="font-size: 12pt; line-height:
      20.5042px;"><span class="SCXW70164443 BCX2"> </span></span><br
      class="SCXW70164443 BCX2">
    <span class="LineBreakBlob BlobObject DragDrop SCXW70164443 BCX2"
      style="font-size: 12pt; line-height: 20.5042px;"></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun EmptyTextRun SCXW70164443 BCX2"></span><a
      class="Hyperlink SCXW70164443 BCX2" style="text-decoration: none;
      color: inherit;"
      href="https://gerrit.openbmc-project.xyz/c/openbmc/webui-vue/+/35696"
      target="_blank" rel="noreferrer noopener"><span style="color:
        rgb(5, 99, 193); text-decoration: underline; font-size: 12pt;
        line-height: 20.5042px;" data-contrast="none" class="TextRun
        Underlined SCXW70164443 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW70164443 BCX2"
          style="background-color: inherit;"
          data-ccp-charstyle="Hyperlink">https://gerrit.openbmc-project.xyz/c/openbmc/webui-vue/+/35696</span></span><span
        class="LineBreakBlob BlobObject DragDrop SCXW70164443 BCX2"
        style="font-size: 12pt; line-height: 20.5042px; color: rgb(5,
        99, 193);"><span class="SCXW70164443 BCX2"> </span></span></a><br
      class="SCXW70164443 BCX2">
    <a class="Hyperlink SCXW70164443 BCX2" style="text-decoration: none;
      color: inherit;"
      href="https://gerrit.openbmc-project.xyz/c/openbmc/webui-vue/+/35696"
      target="_blank" rel="noreferrer noopener"><span
        class="LineBreakBlob BlobObject DragDrop SCXW70164443 BCX2"
        style="font-size: 12pt; line-height: 20.5042px; color: rgb(5,
        99, 193);"></span></a><span data-contrast="auto"
      style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
      SCXW70164443 BCX2" lang="EN-US"><span class="NormalTextRun
        SCXW70164443 BCX2" style="background-color: inherit;">Gets us
        back under the size of phosphor-</span><span
        class="SpellingError SCXW70164443 BCX2" style="background-color:
        inherit;">webui</span><span class="NormalTextRun SCXW70164443
        BCX2" style="background-color: inherit;">.</span></span><span
      class="LineBreakBlob BlobObject DragDrop SCXW70164443 BCX2"
      style="font-size: 12pt; line-height: 20.5042px;"><span
        class="SCXW70164443 BCX2"> </span></span><br class="SCXW70164443
      BCX2">
    <span class="LineBreakBlob BlobObject DragDrop SCXW70164443 BCX2"
      style="font-size: 12pt; line-height: 20.5042px;"></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun EmptyTextRun SCXW70164443 BCX2"
      lang="EN-US"></span><span class="EOP SCXW70164443 BCX2"
      style="font-size: 12pt; line-height: 20.5042px;"
      data-ccp-props="{}"> </span>
    <div class="OutlineElement Ltr BCX2 SCXW70164443" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW70164443 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW70164443 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted">If we care about flash,</span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW70164443 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW70164443 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted"> we should look at removing or reducing the size
          of</span></span><span class="LineBreakBlob BlobObject DragDrop
        SCXW70164443 BCX2" style="font-size: 12pt; line-height:
        20.5042px;"><span class="SCXW70164443 BCX2"> </span></span><br
        class="SCXW70164443 BCX2">
      <span class="LineBreakBlob BlobObject DragDrop SCXW70164443 BCX2"
        style="font-size: 12pt; line-height: 20.5042px;"></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW70164443 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW70164443 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted">76.0K   /</span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW70164443 BCX2" lang="EN-US"><span
          class="SpellingError SCXW70164443 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted">usr</span></span><span data-contrast="auto"
        style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
        SCXW70164443 BCX2" lang="EN-US"><span class="NormalTextRun
          SCXW70164443 BCX2" style="background-color: inherit;"
          data-ccp-parastyle="HTML Preformatted">/share/www/DMTF_Redfish_logo_2017.svg</span></span><span
        class="EOP SCXW70164443 BCX2" style="font-size: 12pt;
        line-height: 20.5042px; font-family: Calibri,
        Calibri_MSFontService, sans-serif;"
data-ccp-props="{"469777462":[916,1832,2748,3664,4580,5496,6412,7328,8244,9160,10076,10992,11908,12824,13740,14656],"469777927":[0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],"469777928":[1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1]}">
      </span><br>
      <span class="EOP SCXW70164443 BCX2" style="font-size: 12pt;
        line-height: 20.5042px; font-family: Calibri,
        Calibri_MSFontService, sans-serif;"
data-ccp-props="{"469777462":[916,1832,2748,3664,4580,5496,6412,7328,8244,9160,10076,10992,11908,12824,13740,14656],"469777927":[0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],"469777928":[1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1]}"></span>
      <p class="Paragraph SCXW70164443 BCX2" style="font-weight: normal;
        font-style: normal; vertical-align: baseline; background-color:
        transparent; color: windowtext; text-align: left; margin-left:
        0px; margin-right: 0px; padding-left: 0px; padding-right: 0px;
        text-indent: 0px;"><span class="EOP SCXW70164443 BCX2"
          style="font-size: 12pt; line-height: 20.5042px; font-family:
          Calibri, Calibri_MSFontService, sans-serif;"
data-ccp-props="{"469777462":[916,1832,2748,3664,4580,5496,6412,7328,8244,9160,10076,10992,11908,12824,13740,14656],"469777927":[0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],"469777928":[1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1]}"><br>
        </span></p>
    </div>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">2. A more thorough list of deficiencies in comparison to phosphor-webui.
There's a "feature parity" list on the readme that makes it seem like
it's closer than it is in practice.  In the course of writing the
patchset above I found that webui-vue doesn't support Mutual TLS,
doesn't support CSRF allow list (a security feature), and doesn't
support the "next" url forward.  None of these are listed in the
feature parity list.  Considering that's just what I found in the
initial look for the above patchset, I'm guessing there's more use
cases that got overlooked.  If we're dropping use cases, we need to be
explicit about it, and document why.</pre>
    </blockquote>
    <span data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW223741068 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted">Yeah, we clearly missed some features that were in
        phosphor-</span></span><span data-contrast="auto"
      style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
      SCXW223741068 BCX2" lang="EN-US"><span class="SpellingError
        SCXW223741068 BCX2" style="background-color: inherit;"
        data-ccp-parastyle="HTML Preformatted">webui</span></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW223741068 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted"> with the rewrite. I opened some issues for these
        and the others we know about. IBM will work on</span></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW223741068 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted"> the "next" URL forward. For features implemented
        in phosphor-</span></span><span data-contrast="auto"
      style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
      SCXW223741068 BCX2" lang="EN-US"><span class="SpellingError
        SCXW223741068 BCX2" style="background-color: inherit;"
        data-ccp-parastyle="HTML Preformatted">webui</span></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW223741068 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted"> that we don’t plan to support will need to
        reimplemented by the community in </span></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW223741068 BCX2" lang="EN-US"><span
        class="SpellingError SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted">webui-vue</span></span><span data-contrast="auto"
      style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
      SCXW223741068 BCX2" lang="EN-US"><span class="NormalTextRun
        SCXW223741068 BCX2" style="background-color: inherit;"
        data-ccp-parastyle="HTML Preformatted"> but I don’t think there
        should be many and for joining us on </span></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW223741068 BCX2" lang="EN-US"><span
        class="SpellingError SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted">webui-vue</span></span><span data-contrast="auto"
      style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
      SCXW223741068 BCX2" lang="EN-US"><span class="NormalTextRun
        SCXW223741068 BCX2" style="background-color: inherit;"
        data-ccp-parastyle="HTML Preformatted"> you get a </span><span
        class="SpellingError SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted">theme-able</span><span class="NormalTextRun
        SCXW223741068 BCX2" style="background-color: inherit;"
        data-ccp-parastyle="HTML Preformatted">, translatable, fully
        Redfish, and actively developed GUI. </span></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW223741068 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted">😊<br>
      </span></span><br>
    <span data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW223741068 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted">Can you further explain what CSRF allow list
        feature we had in phosphor-webui?</span></span>
    <p><span data-contrast="auto" style="font-size: 12pt; font-family:
        Segoe UI Emoji, Segoe UI Emoji_MSFontService, sans-serif;
        line-height: 20.5042px;" class="TextRun SCXW223741068 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW223741068 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted"><br>
        </span></span></p>
    <span data-contrast="auto" style="font-size: 12pt; font-family:
      Segoe UI Emoji, Segoe UI Emoji_MSFontService, sans-serif;
      line-height: 20.5042px;" class="TextRun SCXW223741068 BCX2"
      lang="EN-US"><span class="NormalTextRun SCXW223741068 BCX2"
        style="background-color: inherit;" data-ccp-parastyle="HTML
        Preformatted"></span></span>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">3. Chunked payloads
While Phosphor-webui opted for a single, very large javascript bundle,
webui-vue opts for multiple chunked bundles.  In phosphor, this single
bundle was done on purpose.  Chunking works great for CDNs and
multithreaded webservers, but tends to cause slower page loads when
done on a bmc, as bmcweb is largely optimized for single connection
single client single request.  Although it's able to handle multiple
clients and multiple connections, the bmc NIC tends to get "starved"
of bandwidth for other things, which can cause performance degradation
if you have lots of things going on in parallel.  I'd recommend going
back to the old paradigm, unless there's a good reason to chunk from
the BMC.  Note the bmcweb router holds all static routes in memory
under the assumption that there won't be very many of them.  With
chunking, that's no longer true, and probably causes some unnecessary
increased memory usage.</pre>
    </blockquote>
    <span data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW68329248 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted">After some
        discussion, we agree and </span></span><a class="Hyperlink
      SCXW68329248 BCX2" style="text-decoration: none; color: inherit;"
href="https://gerrit.openbmc-project.xyz/c/openbmc/webui-vue/+/35696"
      target="_blank" rel="noreferrer noopener"><span style="color:
        rgb(5, 99, 193); text-decoration: underline; font-size: 12pt;
        line-height: 20.5042px;" data-contrast="none" class="TextRun
        Underlined SCXW68329248 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW68329248 BCX2"
          style="background-color: inherit;"
          data-ccp-charstyle="Hyperlink">https://gerrit.openbmc-project.xyz/c/openbmc/webui-vue/+/35696</span></span></a><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW68329248 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted"> moves us to a
        single, large </span></span><span data-contrast="auto"
      style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
      SCXW68329248 BCX2" lang="EN-US"><span class="SpellingError
        SCXW68329248 BCX2" style="background-color: inherit;"
        data-ccp-parastyle="HTML Preformatted">javascript</span></span><span
      data-contrast="auto" style="font-size: 12pt; line-height:
      20.5042px;" class="TextRun SCXW68329248 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted"> bundle. This
        should have the same number of </span><span
        class="SpellingError SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted">js</span><span
        class="NormalTextRun SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted">, html, and </span><span
        class="SpellingError SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted">css</span><span
        class="NormalTextRun SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted"> files as </span><span
        class="SpellingError SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted">phoshor-webui</span><span
        class="NormalTextRun SCXW68329248 BCX2" style="background-color:
        inherit;" data-ccp-parastyle="HTML Preformatted">.</span></span>
    <p><br>
    </p>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">4. A more stable migration strategy
The changeover to vue got started completely from scratch, and got
mixed with DOM changes that functionally changed the UI.  If this was
to learn vue, and build a toy UI, that would've been fine, but the
fact that it's now its own full repo means we have fork problems, give
that there is no stable and specified bmcweb->phosphor-webui
interface.  It would've been relatively straightforward to move
phosphor-webui over to vue, by keeping the existing DOM, CSS, and
layout, while replacing the templating and router with vue, but the
decision was start over completely from scratch, and now openbmc has 2
"official" webuis.  I probably missed the discussion on why a hard
throwaway was needed here, but it seems like a series of patches that
ONLY moved over to vue would've been much easier to manage here for
the community as a whole, as each step of the way we can verify
feature parity when reviewing patchsets. </pre>
    </blockquote>
    <div class="OutlineElement Ltr BCX2 SCXW198745046" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW198745046 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW198745046 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted">In theory but we don’t think it would have been
          straightforward. </span></span><span data-contrast="auto"
        style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
        SCXW198745046 BCX2" lang="EN-US"><span class="NormalTextRun
          SCXW198745046 BCX2" style="background-color: inherit;"
          data-ccp-parastyle="HTML Preformatted">Had we gone with a
          transition it would have forced all users into this </span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW198745046 BCX2" lang="EN-US"><span
          class="SpellingError SCXW198745046 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted">vue</span></span><span data-contrast="auto"
        style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
        SCXW198745046 BCX2" lang="EN-US"><span class="NormalTextRun
          SCXW198745046 BCX2" style="background-color: inherit;"
          data-ccp-parastyle="HTML Preformatted"> rewrite, potentially
          some in-between larger GUI while it was in transition.
          Phosphor-</span></span><span data-contrast="auto"
        style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
        SCXW198745046 BCX2" lang="EN-US"><span class="SpellingError
          SCXW198745046 BCX2" style="background-color: inherit;"
          data-ccp-parastyle="HTML Preformatted">webui</span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW198745046 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW198745046 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted"> suffers from some anti-patterns (some of the
          reason why it took such crazy large commits to do any
          theming), we don’t think we could have reached where we are
          today with a transition approach. </span></span><span
        class="EOP SCXW198745046 BCX2" style="font-size: 12pt;
        line-height: 20.5042px;"
data-ccp-props="{"469777462":[916,1832,2748,3664,4580,5496,6412,7328,8244,9160,10076,10992,11908,12824,13740,14656],"469777927":[0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],"469777928":[1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1]}"> </span></div>
    <div class="OutlineElement Ltr BCX2 SCXW198745046" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW198745046 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW198745046 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted"> </span></span><span class="LineBreakBlob
        BlobObject DragDrop SCXW198745046 BCX2" style="font-size: 12pt;
        line-height: 20.5042px;"><span class="SCXW198745046 BCX2"> </span><br
          class="SCXW198745046 BCX2">
      </span><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW198745046 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW198745046 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted">A separate repo we felt was the safest bet. This
          two repo approach doesn’t limit the community from moving
          forward as the </span></span><span data-contrast="auto"
        style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
        SCXW198745046 BCX2" lang="EN-US"><span class="SpellingError
          SCXW198745046 BCX2" style="background-color: inherit;"
          data-ccp-parastyle="HTML Preformatted">webui-vue</span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW198745046 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW198745046 BCX2"
          style="background-color: inherit;" data-ccp-parastyle="HTML
          Preformatted"> is maturing. This approach has been used in the
          community before.  </span></span><span class="LineBreakBlob
        BlobObject DragDrop SCXW198745046 BCX2" style="font-size: 12pt;
        line-height: 20.5042px;"><span class="SCXW198745046 BCX2"> </span><br
          class="SCXW198745046 BCX2">
      </span><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun EmptyTextRun
        SCXW198745046 BCX2" lang="EN-US"></span><span class="EOP
        SCXW198745046 BCX2" style="font-size: 12pt; line-height:
        20.5042px;"
data-ccp-props="{"469777462":[916,1832,2748,3664,4580,5496,6412,7328,8244,9160,10076,10992,11908,12824,13740,14656],"469777927":[0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0],"469777928":[1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1]}"> </span></div>
    <div class="OutlineElement Ltr BCX2 SCXW198745046" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun EmptyTextRun
        SCXW198745046 BCX2"></span><a class="Hyperlink SCXW198745046
        BCX2" style="text-decoration: none; color: inherit;"
href="https://lists.ozlabs.org/pipermail/openbmc/2020-January/020205.html"
        target="_blank" rel="noreferrer noopener"><span style="color:
          rgb(5, 99, 193); text-decoration: underline; font-size: 12pt;
          line-height: 20.5042px;" data-contrast="none" class="TextRun
          Underlined SCXW198745046 BCX2" lang="EN-US"><span
            class="NormalTextRun SCXW198745046 BCX2"
            style="background-color: inherit;"
            data-ccp-charstyle="Hyperlink">https://lists.ozlabs.org/pipermail/openbmc/2020-January/020205.html</span></span></a><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun EmptyTextRun SCXW198745046 BCX2"
        lang="EN-US"></span><span class="EOP SCXW198745046 BCX2"
        style="font-size: 12pt; line-height: 20.5042px;"
        data-ccp-props="{}"> <br>
      </span></div>
    <p><br>
    </p>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap=""> As is (ignoring Kathys
patches for a moment) there are 4 patchsets open for phosphor-webui.
Is there a documented strategy for who is responsible for moving them
over to webui-vue?  I didn't see anything written down, which leads me
to believe there's no plan.</pre>
    </blockquote>
    <div class="OutlineElement Ltr BCX2 SCXW36054189" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW36054189 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW36054189 BCX2"
          style="background-color: inherit;">After “ignoring Kathys</span></span><span
        class="LineBreakBlob BlobObject DragDrop SCXW36054189 BCX2"
        style="font-size: 12pt; line-height: 20.5042px;"><span
          class="SCXW36054189 BCX2"> </span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW36054189 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW36054189 BCX2"
          style="background-color: inherit;">patches”, I don’t see any
          other commit except your “Add the option to use backend login
          routines”, that isn’t up for review (Virtual Media) or already
          in (“Relace node-sass with dart-sass" and “expired password”)
        </span></span><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW36054189 BCX2"
        lang="EN-US"><span class="SpellingError SCXW36054189 BCX2"
          style="background-color: inherit;">webui-vue</span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW36054189 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW36054189 BCX2"
          style="background-color: inherit;">. </span></span><span
        class="EOP SCXW36054189 BCX2" style="font-size: 12pt;
        line-height: 20.5042px;" data-ccp-props="{}"> <br>
      </span></div>
    <div class="OutlineElement Ltr BCX2 SCXW36054189" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun EmptyTextRun
        SCXW36054189 BCX2"></span><a class="Hyperlink SCXW36054189 BCX2"
        style="text-decoration: none; color: inherit;"
href="https://gerrit.openbmc-project.xyz/q/project:openbmc%252Fphosphor-webui+status:open"
        target="_blank" rel="noreferrer noopener"><span style="color:
          rgb(5, 99, 193); text-decoration: underline; font-size: 12pt;
          line-height: 20.5042px;" data-contrast="none" class="TextRun
          Underlined SCXW36054189 BCX2" lang="EN-US"><span
            class="NormalTextRun SCXW36054189 BCX2"
            style="background-color: inherit;"
            data-ccp-charstyle="Hyperlink">https://gerrit.openbmc-project.xyz/q/project:openbmc%252Fphosphor-webui+status:open</span></span></a><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW36054189 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW36054189 BCX2"
          style="background-color: inherit;"> </span></span><span
        class="EOP SCXW36054189 BCX2" style="font-size: 12pt;
        line-height: 20.5042px;" data-ccp-props="{}"> </span></div>
    <div class="OutlineElement Ltr BCX2 SCXW36054189" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun EmptyTextRun
        SCXW36054189 BCX2" lang="EN-US"></span><span
        class="LineBreakBlob BlobObject DragDrop SCXW36054189 BCX2"
        style="font-size: 12pt; line-height: 20.5042px;"><span
          class="SCXW36054189 BCX2"> </span><br class="SCXW36054189
          BCX2">
      </span><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW36054189 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW36054189 BCX2"
          style="background-color: inherit;">As mentioned, we know we
          are missing some feature parity and are looking for help from
          the community (raising any problems or contributing) but we
          have done a lot of work to get where we are, every page on the
          existing </span></span><span data-contrast="auto"
        style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
        SCXW36054189 BCX2" lang="EN-US"><span class="SpellingError
          SCXW36054189 BCX2" style="background-color: inherit;">webui</span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW36054189 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW36054189 BCX2"
          style="background-color: inherit;"> except SNMP and Virtual
          Media is in </span></span><span data-contrast="auto"
        style="font-size: 12pt; line-height: 20.5042px;" class="TextRun
        SCXW36054189 BCX2" lang="EN-US"><span class="SpellingError
          SCXW36054189 BCX2" style="background-color: inherit;">webui-vue</span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW36054189 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW36054189 BCX2"
          style="background-color: inherit;"> and </span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW36054189 BCX2" lang="EN-US"><span
          class="SpellingError SCXW36054189 BCX2"
          style="background-color: inherit;">webui-vue</span></span><span
        data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW36054189 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW36054189 BCX2"
          style="background-color: inherit;"> has a lot of advantages. </span></span><span
        class="EOP SCXW36054189 BCX2" style="font-size: 12pt;
        line-height: 20.5042px; font-family: Calibri,
        Calibri_MSFontService, sans-serif;" data-ccp-props="{}"> <br>
      </span></div>
    <div class="OutlineElement Ltr BCX2 SCXW36054189" style="direction:
      ltr;"><span class="EOP SCXW36054189 BCX2" style="font-size: 12pt;
        line-height: 20.5042px; font-family: Calibri,
        Calibri_MSFontService, sans-serif;" data-ccp-props="{}"><br>
        IBM has moved 2 of their systems to webui-vue<br>
        <br>
<a class="moz-txt-link-freetext" href="https://github.com/openbmc/openbmc/commit/4a3fa4d6d865b46ba54f2652c82f58a406455ebc">https://github.com/openbmc/openbmc/commit/4a3fa4d6d865b46ba54f2652c82f58a406455ebc</a><br>
        <br>
      </span></div>
    <p>
    </p>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">5. Missed opportunity with Redfish UI
If you're going to build a redfish first GUI, it seems like an
opportunity where we could've used the CSDL definitions to drive and
build a lot of the UI automatically.  Given that phosphor-webui
doesn't do this today, this wouldn't have kept me from using
webui-vue, but the idea that the UI can simply build itself for the
"easy" stuff would be a massive win on productivity, and would
probably make me more personally inclined to go help sort out 1-4
above, as I think the end would justify the means.  As is, it takes a
very similar "hardcode every page" approach that the phosphor-webui
did.</pre>
    </blockquote>
    <br>
    <div class="OutlineElement Ltr BCX2 SCXW101444828" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW101444828 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW101444828 BCX2"
          style="background-color: inherit;">We don’t see how we could
          reasonably do this. Would need more discussion around this. <br>
          We also think there would be some trade-offs e.g. those
          Redfish Schemas are quiet large and today are really only
          needed for Redfish validation, using them in the GUI puts
          another requirement on them.<br>
        </span></span></div>
    <div class="OutlineElement Ltr BCX2 SCXW101444828" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW101444828 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW101444828 BCX2"
          style="background-color: inherit;"><br>
        </span></span><span class="EOP SCXW101444828 BCX2"
        style="font-size: 12pt; line-height: 20.5042px; font-family:
        Arial, Arial_MSFontService, sans-serif;" data-ccp-props="{}"> </span>
    </div>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">Don't get me wrong here, I think all of the above can be fixed, and we
certainly could have a great Vue based webui that works for everyone,
but right now, the rather arbitrary "the project will stop supporting
on angularjs on X date" doesn't ring that important to me.  Javascript
frameworks and modules go unsupported all the time, and we're using
several "unsupported" modules today to no ill effect. </pre>
    </blockquote>
    <div class="OutlineElement Ltr BCX2 SCXW208621347" style="direction:
      ltr;"><span data-contrast="auto" style="font-size: 12pt;
        line-height: 20.5042px;" class="TextRun SCXW208621347 BCX2"
        lang="EN-US"><span class="NormalTextRun SCXW208621347 BCX2"
          style="background-color: inherit;">We disagree some, we do
          think there is ill effect of being on an unsupported
          JavaScript framework. Vue has new features released regularly
          and has an active community.   </span></span><span class="EOP
        SCXW208621347 BCX2" style="font-size: 12pt; line-height:
        20.5042px; font-family: Calibri, Calibri_MSFontService,
        sans-serif;" data-ccp-props="{}"> <br>
      </span></div>
    <p><br>
    </p>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">Theming is a
nice feature, but those are mostly DOM and CSS changes, and are
unrelated to a changeover to Vue.  Translations was attempted in
phosphor-webui in the past, but to quote the review:

"neither any clients nor any companies wanted OpenBMC translated."
<a class="moz-txt-link-freetext" href="https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-webui/+/17582">https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-webui/+/17582</a></pre>
    </blockquote>
    <span data-contrast="auto" style="font-size: 12pt; font-family:
      Calibri, Calibri_MSFontService, sans-serif; line-height:
      20.5042px;" class="TextRun SCXW151303975 BCX2" lang="EN-US"><span
        class="NormalTextRun SCXW151303975 BCX2"
        style="background-color: inherit;">IBM does have a requirement
        on translation and we have heard in the GUI workgroup meeting,
        other companies would be interested in translation as well. <br>
      </span></span>
    <p><span data-contrast="auto" style="font-size: 12pt; font-family:
        Calibri, Calibri_MSFontService, sans-serif; line-height:
        20.5042px;" class="TextRun SCXW151303975 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW151303975 BCX2"
          style="background-color: inherit;"><br>
        </span></span></p>
    <p><span data-contrast="auto" style="font-size: 12pt; font-family:
        Calibri, Calibri_MSFontService, sans-serif; line-height:
        20.5042px;" class="TextRun SCXW151303975 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW151303975 BCX2"
          style="background-color: inherit;"></span></span></p>
    <p><span data-contrast="auto" style="font-size: 12pt; font-family:
        Calibri, Calibri_MSFontService, sans-serif; line-height:
        20.5042px;" class="TextRun SCXW151303975 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW151303975 BCX2"
          style="background-color: inherit;"></span></span>
    </p>
    <blockquote type="cite"
cite="mid:CACWQX833imTWQxB2hDvzVo5mTQzggAVEieZf4TV8mu=y2uPajA@mail.gmail.com">
      <pre class="moz-quote-pre" wrap="">Also keep in mind, I have very little seat time in webui-vue, the
above is mostly first impressions in response to the ask from Gunnar.
Overall, if the above can be fixed, I'd probably move to webui-vue.

Thanks,

-Ed</pre>
    </blockquote>
    <p>Appreciate the feedback, was really helpful. <br>
      <br>
      <span data-contrast="auto" style="font-size: 12pt; line-height:
        20.5042px;" class="TextRun SCXW191667935 BCX2" lang="EN-US"><span
          class="NormalTextRun SCXW191667935 BCX2"
          style="background-color: inherit;">Gunnar, Yoshie, and Derick
        </span></span><br>
    </p>
    <br>
  </body>
</html>