823616 - Create a template for TreeStatus logs

Status: RESOLVED FIXED

(Release Engineering :: General, enhancement)

Description

[Ed Morley [:emorley]]

This is fairly low priority, but currently logs are output without a template (https://treestatus.mozilla.org/mozilla-central/logs) and as such don't really offer much over just using https://treestatus.mozilla.org/mozilla-central/logs?format=json

Also, https://treestatus.mozilla.org/mozilla-central shows log entries, so maybe we can just use that?

Comment 1

[shaktimaan]

Looks like https://treestatus.mozilla.org/mozilla-central and https://treestatus.mozilla.org/mozilla-central/logs output the same information, except that the former uses a template. Why not redirect /tree/logs to /tree? 

Am I overlooking something?

Comment 2

[Ed Morley [:emorley]]

So there is a difference in the json output:
https://treestatus.mozilla.org/mozilla-central?format=json
https://treestatus.mozilla.org/mozilla-central/logs?format=json

And I think that is desirable, since people can decide whether they just need the current state, or history.

I think really the end result should be:
"Add template to tree/logs, and make the main tree page only show the last N log entries"

This bug is the first part of that, we can either just do that for now (and have another bug for making the https://treestatus.mozilla.org/mozilla-central page only show a short summary of history (with https://treestatus.mozilla.org/mozilla-central/logs being the full history), or do both here - is up to you :-)

Comment 3

[shaktimaan]

Attachment: Bug 823616 - Create a template for TreeStatus logs; r=edmorley

Changes:
1. Created a template file 'treelogs.html' for HTML view for /tree/logs
2. Limited the number of log entries in /tree to 5. This could be changed if we think 5 is too low
3. Modified the view definition for /tree/logs to render the template

Comment 4

[Chris AtLee [:catlee]]

Comment on attachment 8523311 [details] [review]
Bug 823616 - Create a template for TreeStatus logs; r=edmorley

looks good to me - I merged it upstream

Comment 5

[Ed Morley [:emorley]]

https://github.com/mozilla/treestatus/commit/4464faa2631d48eaaf05535736cb6ef947076b7a

Comment 6

[Ed Morley [:emorley]]

This looks pretty good on dev:
https://treestatus-dev.allizom.org/mozilla-aurora
https://treestatus-dev.allizom.org/mozilla-aurora/logs

The only changes I think it might need, is to add a link to the /logs page from the tree page. ie: Below the 5 most recent changes, add a link called "Full log", "More..." or something similar.

I wonder as well whether it would be better to call the heading on the tree page "Recent Changes" rather than "Summary"?

Comment 7

[shaktimaan]

I think your suggestions would look good. How do I go about making the suggested changes when my PR is merged already?

Comment 8

[Ed Morley [:emorley]]

(In reply to shaktimaan from comment #7)
> I think your suggestions would look good. How do I go about making the
> suggested changes when my PR is merged already?

If you create another PR, and add the link to that as a second attachment to this bug, that would be great :-)

Comment 9

[shaktimaan]

Attachment: Bug 823616 - Create a template for TreeStatus logs; r=catlee

Comment 10

[Chris AtLee [:catlee]]

Comment on attachment 8527109 [details] [review]
Bug 823616 - Create a template for TreeStatus logs; r=catlee

merged upstream as https://github.com/mozilla/treestatus/commit/ae1c22ac018c61e915eced0e0eb4ddf0fedabc58

thanks!

Comment 11

[Ed Morley [:emorley]]

In production - thank you for the patches :-)