[Maas-devel] xml-to-yaml

[Jeffrey Lane]

On Mon, Aug 25, 2014 at 8:58 AM, Andres Rodriguez
<andres.rodriguez at canonical.com> wrote:
> Instead of providing totally different pages for the commissioning results
> per node (xml_lshw, yaml_lshw), I'd suggest you provide just one page with
> Tabs to differentiate between the output type (Exactly the same way as in
> the Node View)


My only comment is a question about why show both at all?  Just IMO,
but as a user:
1: I HATE having to read XML
2: If XML is provided, how about just as a download option so I can
parse or feed it into whatever tool I have.
3: Since you're already converting it to YAML, which is a good bit
easier to read, why also have an XML view of the same data?

I guess the point to 3 is that, why provide two different views of the
exact same data, one that is a headache to read and one that is
slightly better/easier to read?

IMO, As Andres said, a simple OK link to the results would be best,
but I'd change it so that instead of tabs, you simply displayed the
YAML converted data and provide a download button for either the YAML
or raw XML.

Anyway, I don't have anything more useful than that to add, Andres had
some very good points.

Perhaps one other question, since you're going to the trouble of
parsing the XML into YAML to make it readable, why not just parse it
the rest of the way and provide really readable output instead?  In
reality, you're really just swapping one markup language for another,
it's just that the new one happens to be more friendly to the eyes
than the original one.  But, that is just a curiosity point, and maybe
a suggestion for the next iteration of this output display, once they
YAML bits are solidified and merged.

Either way, looks like a great start!

Jeff

-- 
"Entropy isn't what it used to be."

Jeff Lane - Server Certification Team Lead, Tools Developer, Warrior
Poet, Lover of Pie
Phone: 919-442-8649
Ubuntu Ham: W4KDH                          Freenode IRC: bladernr or bladernr_
gpg: 1024D/3A14B2DD 8C88 B076 0DD7 B404 1417  C466 4ABD 3635 3A14 B2DD