Re: Review Request 25158: Aurora Update UI

2014-09-02 Thread Bill Farner 


> On Aug. 28, 2014, 6:51 p.m., Joshua Cohen wrote:
> > This looks nice! I'm assuming the grid is changes in real time as instances 
> > are updated?
> > 
> > I think I prefer the big preview (the one with the grid as opposed to the 
> > progress bar).
> > 
> > What are your thoughts on including the update summary from the finished 
> > view on the in progress view? I think that'd be useful information on an 
> > in-flight deploy, especially who kicked it off, when it started, and an 
> > elapsed timer...)
> 
> Maxim Khutornenko wrote:
> I actually like the small preview with progress bar more. Having instance 
> break-down could be overwhelming for larger jobs. Owner, progress and elapsed 
> time should be enough here saving the instance details "wow" factor for the 
> update details page.
> 
> Also, is there a sample page for an update with added instances?
> 
> David McLaughlin wrote:
> Re: Joshua's points
> 
> In-flight deploy does contain the username and when it was started? It's 
> just not as fancy because that display style only makes sense as a range. 
> 
> 
> Re: Maxim
> 
> No page for added instances - added instances are not a terminal state. 
> They are just treated as pending update targets.

+1 for the small rather than large preview.  The box grid for showing details 
of an update is very useful, but the progress bar looks like the sweet spot for 
at-a-glance status.


- Bill


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25158/#review51808
---


On Aug. 28, 2014, 6:13 p.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25158/
> ---
> 
> (Updated Aug. 28, 2014, 6:13 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, Maxim Khutornenko, and Bill Farner.
> 
> 
> Bugs: AURORA-614
> None
> 
> 
> Description
> ---
> 
> Still working on trying to make the event timeline work, but going to show 
> what I've got so far to get feedback. Will tidy up the code and post a review 
> for it once I get a ship it for this. 
> 
> The main thing here is the instance status visualisation. I've mocked out 
> data to show some use cases.
> 
> Note: I have two job page versions here with a preview of an update in 
> progress. Would be interested in getting feedback on which one people prefer.
> 
> 
> Diffs
> -
> 
> 
> Diff: https://reviews.apache.org/r/25158/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Job Page with in progress update, using same template as update page
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/332fd237-eba5-4204-93e5-4e271f8b4f8c__job-page-in-progress-big-preview.png
> Job Update with smaller in progress update, and more conventional progress 
> bar.
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/39f78052-348c-458c-956b-c3ba8ab1f642__job-progress-small-preview.png
> Update Page - medium sized job in progress
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/e6e7e5db-c5ff-4e68-a817-552243a706f5__update-page-in-progress-medium-job.png
> Update Page - Update completed view
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/fb8ed1c1-c90c-4c13-8464-6876ca811ad1__update-page-finished.png
> Update Page - partial update / canary view
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/ada5349c-84be-4838-9c20-cc99d5db3779__update-page-partial-canary.png
> Update Page - Update that has failed instances
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/7a8983f5-cc36-4219-ab9e-237680566c11__update-page-failures.png
> Update with skipped instances
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/d4283788-45b6-4564-9e9d-81327b8d3e06__update-page-skipped.png
> Update that had to be rolled back
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/0a968616-4199-4e34-bc37-3b66fff03eef__update-page-rolled-back.png
> Update where instances were removed
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/ca132beb-fcd4-4fe1-b682-6e229f632f45__update-page-instances-removed.png
> Update page - HUGE job, smaller viz
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/8e4d59f9-4644-46b7-a4b1-1a5de9538a80__update-page-huge-job.png
> Update Page - very small job
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/356db8be-e3d1-430a-ac35-3ff7f5855467__update-page-small-job.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>

Re: Review Request 25158: Aurora Update UI

2014-08-29 Thread David McLaughlin 


> On Aug. 28, 2014, 6:51 p.m., Joshua Cohen wrote:
> > This looks nice! I'm assuming the grid is changes in real time as instances 
> > are updated?
> > 
> > I think I prefer the big preview (the one with the grid as opposed to the 
> > progress bar).
> > 
> > What are your thoughts on including the update summary from the finished 
> > view on the in progress view? I think that'd be useful information on an 
> > in-flight deploy, especially who kicked it off, when it started, and an 
> > elapsed timer...)
> 
> Maxim Khutornenko wrote:
> I actually like the small preview with progress bar more. Having instance 
> break-down could be overwhelming for larger jobs. Owner, progress and elapsed 
> time should be enough here saving the instance details "wow" factor for the 
> update details page.
> 
> Also, is there a sample page for an update with added instances?

Re: Joshua's points

In-flight deploy does contain the username and when it was started? It's just 
not as fancy because that display style only makes sense as a range. 


Re: Maxim

No page for added instances - added instances are not a terminal state. They 
are just treated as pending update targets.


> On Aug. 28, 2014, 6:51 p.m., Joshua Cohen wrote:
> > File Attachment: Update page - HUGE job, smaller viz - 
> > update-page-huge-job.png
> > 
> >
> > Beyond a certain point, instead of making the boxes smaller and smaller 
> > (or having this section take up more and more real estate), does it make 
> > sense to set a lower bound for size (and an upper bound for overall 
> > vertical hiehgt) and simply let each box represent multiple instances?

Just to be clear - there are only three classes of size and it's a pretty 
simple heuristic. The 'big' boxes (the one with instance ids displayed) are for 
jobs with less than 20 instances. Between 20 and 1000 you get the medium sized 
visualisation, and then 1000+ you get the smallest boxes (as shown here). 

I did make probably a bad assumption that 3~5k was hitting the upper bound of 
job size. At the scale of thousands of instances, it's definitely going to be 
better to skip any kind of one-size-fits-all viz and maybe only show 
'interesting' instances.


> On Aug. 28, 2014, 6:51 p.m., Joshua Cohen wrote:
> > File Attachment: Update with skipped instances - update-page-skipped.png
> > 
> >
> > This brighter green for skipped feels off to me. It looks *more* 
> > successful than the actual successful updates.

Yeah, agreed. I'll tinker with that color when posting the code review.


- David


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25158/#review51808
---


On Aug. 28, 2014, 6:13 p.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25158/
> ---
> 
> (Updated Aug. 28, 2014, 6:13 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, Maxim Khutornenko, and Bill Farner.
> 
> 
> Bugs: AURORA-614
> None
> 
> 
> Description
> ---
> 
> Still working on trying to make the event timeline work, but going to show 
> what I've got so far to get feedback. Will tidy up the code and post a review 
> for it once I get a ship it for this. 
> 
> The main thing here is the instance status visualisation. I've mocked out 
> data to show some use cases.
> 
> Note: I have two job page versions here with a preview of an update in 
> progress. Would be interested in getting feedback on which one people prefer.
> 
> 
> Diffs
> -
> 
> 
> Diff: https://reviews.apache.org/r/25158/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Job Page with in progress update, using same template as update page
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/332fd237-eba5-4204-93e5-4e271f8b4f8c__job-page-in-progress-big-preview.png
> Job Update with smaller in progress update, and more conventional progress 
> bar.
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/39f78052-348c-458c-956b-c3ba8ab1f642__job-progress-small-preview.png
> Update Page - medium sized job in progress
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/e6e7e5db-c5ff-4e68-a817-552243a706f5__update-page-in-progress-medium-job.png
> Update Page - Update completed view
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/fb8ed1c1-c90c-4c13-8464-6876ca811ad1__update-page-finished.png
> Update Page - partial update / canary view
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/ada5349c-84be-4838-9c20-cc99d5db3779__update-page-partial-canary.png
> Update Page - Update that has failed instances

Re: Review Request 25158: Aurora Update UI

2014-08-28 Thread Maxim Khutornenko 


> On Aug. 28, 2014, 6:51 p.m., Joshua Cohen wrote:
> > This looks nice! I'm assuming the grid is changes in real time as instances 
> > are updated?
> > 
> > I think I prefer the big preview (the one with the grid as opposed to the 
> > progress bar).
> > 
> > What are your thoughts on including the update summary from the finished 
> > view on the in progress view? I think that'd be useful information on an 
> > in-flight deploy, especially who kicked it off, when it started, and an 
> > elapsed timer...)

I actually like the small preview with progress bar more. Having instance 
break-down could be overwhelming for larger jobs. Owner, progress and elapsed 
time should be enough here saving the instance details "wow" factor for the 
update details page.

Also, is there a sample page for an update with added instances?


- Maxim


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25158/#review51808
---


On Aug. 28, 2014, 6:13 p.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25158/
> ---
> 
> (Updated Aug. 28, 2014, 6:13 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, Maxim Khutornenko, and Bill Farner.
> 
> 
> Bugs: AURORA-614
> None
> 
> 
> Description
> ---
> 
> Still working on trying to make the event timeline work, but going to show 
> what I've got so far to get feedback. Will tidy up the code and post a review 
> for it once I get a ship it for this. 
> 
> The main thing here is the instance status visualisation. I've mocked out 
> data to show some use cases.
> 
> Note: I have two job page versions here with a preview of an update in 
> progress. Would be interested in getting feedback on which one people prefer.
> 
> 
> Diffs
> -
> 
> 
> Diff: https://reviews.apache.org/r/25158/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Job Page with in progress update, using same template as update page
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/332fd237-eba5-4204-93e5-4e271f8b4f8c__job-page-in-progress-big-preview.png
> Job Update with smaller in progress update, and more conventional progress 
> bar.
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/39f78052-348c-458c-956b-c3ba8ab1f642__job-progress-small-preview.png
> Update Page - medium sized job in progress
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/e6e7e5db-c5ff-4e68-a817-552243a706f5__update-page-in-progress-medium-job.png
> Update Page - Update completed view
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/fb8ed1c1-c90c-4c13-8464-6876ca811ad1__update-page-finished.png
> Update Page - partial update / canary view
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/ada5349c-84be-4838-9c20-cc99d5db3779__update-page-partial-canary.png
> Update Page - Update that has failed instances
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/7a8983f5-cc36-4219-ab9e-237680566c11__update-page-failures.png
> Update with skipped instances
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/d4283788-45b6-4564-9e9d-81327b8d3e06__update-page-skipped.png
> Update that had to be rolled back
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/0a968616-4199-4e34-bc37-3b66fff03eef__update-page-rolled-back.png
> Update where instances were removed
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/ca132beb-fcd4-4fe1-b682-6e229f632f45__update-page-instances-removed.png
> Update page - HUGE job, smaller viz
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/8e4d59f9-4644-46b7-a4b1-1a5de9538a80__update-page-huge-job.png
> Update Page - very small job
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/356db8be-e3d1-430a-ac35-3ff7f5855467__update-page-small-job.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>

Re: Review Request 25158: Aurora Update UI

2014-08-28 Thread Joshua Cohen 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25158/#review51808
---


This looks nice! I'm assuming the grid is changes in real time as instances are 
updated?

I think I prefer the big preview (the one with the grid as opposed to the 
progress bar).

What are your thoughts on including the update summary from the finished view 
on the in progress view? I think that'd be useful information on an in-flight 
deploy, especially who kicked it off, when it started, and an elapsed timer...)


File Attachment: Update with skipped instances - update-page-skipped.png


This brighter green for skipped feels off to me. It looks *more* successful 
than the actual successful updates.



File Attachment: Update page - HUGE job, smaller viz - update-page-huge-job.png


Beyond a certain point, instead of making the boxes smaller and smaller (or 
having this section take up more and more real estate), does it make sense to 
set a lower bound for size (and an upper bound for overall vertical hiehgt) and 
simply let each box represent multiple instances?


- Joshua Cohen


On Aug. 28, 2014, 6:13 p.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25158/
> ---
> 
> (Updated Aug. 28, 2014, 6:13 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, Maxim Khutornenko, and Bill Farner.
> 
> 
> Bugs: AURORA-614
> None
> 
> 
> Description
> ---
> 
> Still working on trying to make the event timeline work, but going to show 
> what I've got so far to get feedback. Will tidy up the code and post a review 
> for it once I get a ship it for this. 
> 
> The main thing here is the instance status visualisation. I've mocked out 
> data to show some use cases.
> 
> Note: I have two job page versions here with a preview of an update in 
> progress. Would be interested in getting feedback on which one people prefer.
> 
> 
> Diffs
> -
> 
> 
> Diff: https://reviews.apache.org/r/25158/diff/
> 
> 
> Testing
> ---
> 
> 
> File Attachments
> 
> 
> Job Page with in progress update, using same template as update page
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/332fd237-eba5-4204-93e5-4e271f8b4f8c__job-page-in-progress-big-preview.png
> Job Update with smaller in progress update, and more conventional progress 
> bar.
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/39f78052-348c-458c-956b-c3ba8ab1f642__job-progress-small-preview.png
> Update Page - medium sized job in progress
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/e6e7e5db-c5ff-4e68-a817-552243a706f5__update-page-in-progress-medium-job.png
> Update Page - Update completed view
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/fb8ed1c1-c90c-4c13-8464-6876ca811ad1__update-page-finished.png
> Update Page - partial update / canary view
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/ada5349c-84be-4838-9c20-cc99d5db3779__update-page-partial-canary.png
> Update Page - Update that has failed instances
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/7a8983f5-cc36-4219-ab9e-237680566c11__update-page-failures.png
> Update with skipped instances
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/d4283788-45b6-4564-9e9d-81327b8d3e06__update-page-skipped.png
> Update that had to be rolled back
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/0a968616-4199-4e34-bc37-3b66fff03eef__update-page-rolled-back.png
> Update where instances were removed
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/ca132beb-fcd4-4fe1-b682-6e229f632f45__update-page-instances-removed.png
> Update page - HUGE job, smaller viz
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/8e4d59f9-4644-46b7-a4b1-1a5de9538a80__update-page-huge-job.png
> Update Page - very small job
>   
> https://reviews.apache.org/media/uploaded/files/2014/08/28/356db8be-e3d1-430a-ac35-3ff7f5855467__update-page-small-job.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>

Review Request 25158: Aurora Update UI

2014-08-28 Thread David McLaughlin 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25158/
---

Review request for Aurora, Joshua Cohen, Maxim Khutornenko, and Bill Farner.


Bugs: AURORA-614
None


Description
---

Still working on trying to make the event timeline work, but going to show what 
I've got so far to get feedback. Will tidy up the code and post a review for it 
once I get a ship it for this. 

The main thing here is the instance status visualisation. I've mocked out data 
to show some use cases.

Note: I have two job page versions here with a preview of an update in 
progress. Would be interested in getting feedback on which one people prefer.


Diffs
-


Diff: https://reviews.apache.org/r/25158/diff/


Testing
---


File Attachments


Job Page with in progress update, using same template as update page
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/332fd237-eba5-4204-93e5-4e271f8b4f8c__job-page-in-progress-big-preview.png
Job Update with smaller in progress update, and more conventional progress bar.
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/39f78052-348c-458c-956b-c3ba8ab1f642__job-progress-small-preview.png
Update Page - medium sized job in progress
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/e6e7e5db-c5ff-4e68-a817-552243a706f5__update-page-in-progress-medium-job.png
Update Page - Update completed view
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/fb8ed1c1-c90c-4c13-8464-6876ca811ad1__update-page-finished.png
Update Page - partial update / canary view
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/ada5349c-84be-4838-9c20-cc99d5db3779__update-page-partial-canary.png
Update Page - Update that has failed instances
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/7a8983f5-cc36-4219-ab9e-237680566c11__update-page-failures.png
Update with skipped instances
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/d4283788-45b6-4564-9e9d-81327b8d3e06__update-page-skipped.png
Update that had to be rolled back
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/0a968616-4199-4e34-bc37-3b66fff03eef__update-page-rolled-back.png
Update where instances were removed
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/ca132beb-fcd4-4fe1-b682-6e229f632f45__update-page-instances-removed.png
Update page - HUGE job, smaller viz
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/8e4d59f9-4644-46b7-a4b1-1a5de9538a80__update-page-huge-job.png
Update Page - very small job
  
https://reviews.apache.org/media/uploaded/files/2014/08/28/356db8be-e3d1-430a-ac35-3ff7f5855467__update-page-small-job.png


Thanks,

David McLaughlin