Bug 200057 – [ews-build] Hide build steps to compress, upload and unzip layout-test-results

• Component: Tools / Tests
• Status: RESOLVED WORKSFORME

[Aakash Jain]

Build steps to compress, upload and unzip layout-test-results are just implementation details, and the users don't really need to worry about these steps. We should hide these steps from the UI when these steps are completed successfully. In case of layout-test failure, these three steps are run three times each (for first-run, second-run, and clean-tree-run), so total 9 steps. Hiding these would make the build page less cluttered.

[Aakash Jain]

Created attachment 374729 [details]
Patch

[Aakash Jain]

Sample runs:
https://ews-build.webkit-uat.org/#/builders/34/builds/57
https://ews-build.webkit-uat.org/#/builders/35/builds/153

[Alexey Proskuryakov]

I'm skeptical about this one. It's good to have the buildbot page as nice as possible, but we will never make it user friendly. So hiding things that are necessary for understanding and debugging EWS seems like over-shooting.

[Jonathan Bedard]

I'd like to see an example of the old 'cluttered' look for comparison.

I also don't see an obvious way to reveal hidden steps if we wanted to, am I missing something in the buildbot UI?

[Aakash Jain]

(In reply to Jonathan Bedard from comment #4)
> I'd like to see an example of the old 'cluttered' look for comparison.
https://ews-build.webkit-uat.org/#/builders/34/builds/35

> I also don't see an obvious way to reveal hidden steps if we wanted to, am I missing something in the buildbot UI?
Nope, you are right, there is no obvious way to reveal hidden step. For debugging you can get the info through API though.

[Jonathan Bedard]

(In reply to Aakash Jain from comment #5)
> (In reply to Jonathan Bedard from comment #4)
> > I'd like to see an example of the old 'cluttered' look for comparison.
> https://ews-build.webkit-uat.org/#/builders/34/builds/35
> 
> > I also don't see an obvious way to reveal hidden steps if we wanted to, am I missing something in the buildbot UI?
> Nope, you are right, there is no obvious way to reveal hidden step. For
> debugging you can get the info through API though.

It's a shame we can't just collapse steps and make them visible on request.

I don't think these steps actually provide any information, so I don't see a problem hiding then, even if there isn't a way to expose them in the UI. I don't see these steps ever being needed to understand and debug EWS, but perhaps there is an issue don't have in mind.

[Aakash Jain]

> I don't think these steps actually provide any information
Agree, that's what motivated me to write this patch.

> , so I don't see a problem hiding then, even if there isn't a way to expose them in the UI.
> I don't see these steps ever being needed to understand and debug EWS, but
> perhaps there is an issue don't have in mind.
Note that these step would be hidden only when they PASS, so any failures will be visible. 

One issue I can think of is slow (but still working) upload. The step would be visible while it's running, so if it's too slow we could notice it while it's running.

[Aakash Jain]

> I don't think these steps actually provide any information
Agree, that's what motivated me to write this patch.

> , so I don't see a problem hiding then, even if there isn't a way to expose them in the UI.
> I don't see these steps ever being needed to understand and debug EWS, but
> perhaps there is an issue don't have in mind.
Note that these step would be hidden only when they PASS, so any failures will be visible. 

One issue I can think of is slow (but still working) upload. The step would be visible while it's running, so if it's too slow we could notice it while it's running.

[Aakash Jain]

(In reply to Alexey Proskuryakov from comment #3)
> I'm skeptical about this one. It's good to have the buildbot page as nice as
> possible, but we will never make it user friendly. So hiding things that are
> necessary for understanding and debugging EWS seems like over-shooting.
Yeah, this one is just to make the buildbot page nicer (hopefully without affecting our ability to debug any issues). We will still be developing another page in https://bugs.webkit.org/show_bug.cgi?id=197522

[Alexey Proskuryakov]

I guess we just disagree about whether hiding information on a page that's primarily meant for debugging is desirable. If this ever confuses anyone even for five minutes, then it doesn't seem worth it to me.

[Aakash Jain]

(In reply to Alexey Proskuryakov from comment #10)
> I guess we just disagree about whether hiding information on a page that's
> primarily meant for debugging is desirable. If this ever confuses anyone
> even for five minutes, then it doesn't seem worth it to me.
ok, let's just skip this patch. The benefit isn't much, and we know atleast one scenario (slow upload) in which this can make it harder to debug issues.