Continuum

Queues page should include the prepare build queue

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.3.2 (Beta)
  • Fix Version/s: 1.3.3 (Beta)
  • Component/s: Parallel Builds
  • Labels:
    None
  • Complexity:
    Intermediate
  • Number of attachments :
    1

Description

Currently when a forced or scheduled build kicks off, there is a delay before projects appear on the Queues page.

During this time according to the logs, they are in the 'prepare build' queue, but this is not visible through the web ui.

This causes users to think that nothing is happening.

The Queues page should also show what projects are in the 'prepare build' queue.

Issue Links

This issue relates to:
CONTINUUM-2229 Update the status icon of the project when it is added in build-prepare-queue Major Closed
This issue is related to:
CONTINUUM-2208 group scm updates make builds look like they are not queued Major Closed
CONTINUUM-2005 Go the "Project Build tab" from the "Build Queue" page Major Closed

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Maria Catherine Tan added a comment -

Fixed in
r776516 of 1.3.x branch
r776518 of trunk

Show
Maria Catherine Tan added a comment - Fixed in r776516 of 1.3.x branch r776518 of trunk
Hide
Permalink
Wendy Smoak added a comment - - edited

See screen shot, the message "Prepare Build Queue is empty" is displayed even when there is a project in the queue.

... it's a typo in this line:

> Modified: continuum/branches/continuum-1.3.x/continuum-webapp/src/main/webapp/WEB-INF/jsp/admin/buildQueueView.jsp
...
> + <c:if test="${empty prepaerBuildQueues}">

Show
Wendy Smoak added a comment - - edited See screen shot, the message "Prepare Build Queue is empty" is displayed even when there is a project in the queue. ... it's a typo in this line: > Modified: continuum/branches/continuum-1.3.x/continuum-webapp/src/main/webapp/WEB-INF/jsp/admin/buildQueueView.jsp ... > + <c:if test="${empty prepaerBuildQueues}">
Hide
Permalink
Wendy Smoak added a comment -

The order on the queues page is currently
prepare build
current build
checkout

I think the current build should be at the top. That's the most interesting/useful thing on the page, and when the page refreshes you can immediately see what's building.

That probably implies a reverse chronological order, so...

current build
checkout
prepare build

What do you think?

Show
Wendy Smoak added a comment - The order on the queues page is currently prepare build current build checkout I think the current build should be at the top. That's the most interesting/useful thing on the page, and when the page refreshes you can immediately see what's building. That probably implies a reverse chronological order, so... current build checkout prepare build What do you think?
Hide
Permalink
Wendy Smoak added a comment -

or... is the 'checkout queue' only used when you first add a project, and not at all during scheduled/forced builds? In that case, maybe it should be:

current build
prepare build
current checkout

Also, this change needs a documentation update, so let's leave the issue open until that's done.

I didn't think Queues was documented at all, but grep turned up user_guides/viewing_queues/index.apt which isn't listed on the menu!

Show
Wendy Smoak added a comment - or... is the 'checkout queue' only used when you first add a project, and not at all during scheduled/forced builds? In that case, maybe it should be: current build prepare build current checkout Also, this change needs a documentation update, so let's leave the issue open until that's done. I didn't think Queues was documented at all, but grep turned up user_guides/viewing_queues/index.apt which isn't listed on the menu!
Hide
Permalink
Maria Catherine Tan added a comment -

Fixed in
r776901 of 1.3.x branch
r776902 of trunk

Thanks Wendy!

Show
Maria Catherine Tan added a comment - Fixed in r776901 of 1.3.x branch r776902 of trunk Thanks Wendy!
Hide
Permalink
Wendy Smoak added a comment - - edited

The project groups column should be links instead of plain text, like they are in the other sections on the page.

There should be a cancel icon next to the current item in the prepare build queue.

(These can be handled as separate issues if you want... I know you're trying to get 1.3.3 out. Let me know and I'll open them.)

Show
Wendy Smoak added a comment - - edited The project groups column should be links instead of plain text, like they are in the other sections on the page. There should be a cancel icon next to the current item in the prepare build queue. (These can be handled as separate issues if you want... I know you're trying to get 1.3.3 out. Let me know and I'll open them.)
Hide
Permalink
Maria Catherine Tan added a comment -

The reason why I did not put a cancel icon next to the current item in prepare build queue is because it's doing an svn update, so I'd like to prevent having a corrupted working copy and will need the user to do an svn cleanup.

IIRC, previously we also don't allow cancellation while project is still updating?

Show
Maria Catherine Tan added a comment - The reason why I did not put a cancel icon next to the current item in prepare build queue is because it's doing an svn update, so I'd like to prevent having a corrupted working copy and will need the user to do an svn cleanup. IIRC, previously we also don't allow cancellation while project is still updating?
Hide
Permalink
Maria Catherine Tan added a comment -

Yes, please open another issue if you still think we should have a cancel icon for current prepare build.

Show
Maria Catherine Tan added a comment - Yes, please open another issue if you still think we should have a cancel icon for current prepare build.
Hide
Permalink
Wendy Smoak added a comment -

No problem if there's a reason not to have a way to cancel it, but everything else on the page has one so we should mention in the docs why it's not there.

Do you mean there's a time when you can't cancel a current build? It looks like you can cancel a checkout, doesn't that suffer from the same problem? I'm looking at http://continuum.apache.org/docs/1.3.3-SNAPSHOT/administrator_guides/queues.html

It just needs the links then, and this one can be done.

Show
Wendy Smoak added a comment - No problem if there's a reason not to have a way to cancel it, but everything else on the page has one so we should mention in the docs why it's not there. Do you mean there's a time when you can't cancel a current build? It looks like you can cancel a checkout, doesn't that suffer from the same problem? I'm looking at http://continuum.apache.org/docs/1.3.3-SNAPSHOT/administrator_guides/queues.html It just needs the links then, and this one can be done.
Hide
Permalink
Maria Catherine Tan added a comment -

added link in project group column and also reorder queues when distributed build is enabled

777753 of 1.3.x branch
777754 of trunk

Show
Maria Catherine Tan added a comment - added link in project group column and also reorder queues when distributed build is enabled 777753 of 1.3.x branch 777754 of trunk
Hide
Permalink
Maria Catherine Tan added a comment - - edited

need to mention in the docs why we can't cancel current prepare build

Show
Maria Catherine Tan added a comment - - edited need to mention in the docs why we can't cancel current prepare build

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.