Re: Review Request 25158: Aurora Update UI
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
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 https://reviews.apache.org/r/25158/#fcomment26 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 https://reviews.apache.org/r/25158/#fcomment27 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
--- 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 https://reviews.apache.org//r/25158/#fcomment24 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 https://reviews.apache.org//r/25158/#fcomment25 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
Re: Review Request 25158: Aurora Update UI
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
