Continuum

Continuum ignores duplicate projects in forced and scheduled builds

Details

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

Description

When there are duplicate projects being built at one time, Continuum ignores the duplicates and only builds one of them. (See ProjectSorter.java) It prints a log message, but there is nothing visible in the web ui.

Continuum considers projects with the same groupId:artifactId:version as duplicates, so for example you can't built both the trunk and a branch with the same version in the one group (or even in different groups if they are on the same build schedule.)

It should probably warn if you add duplicates to a group.

If you click 'build all' and it is not going to build everything, it should tell you.

For scheduled builds there should be some indication of why a project was not built.

Activity

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

I think we should just prevent adding duplicates to a group. WDYT?

Show
Maria Catherine Tan added a comment - I think we should just prevent adding duplicates to a group. WDYT?
Hide
Permalink
Wendy Smoak added a comment -

Preventing addition is fine with me.

Do we need to do anything about existing instances that already have duplicates? A user who upgrades is still going to have this problem... maybe a mention in the release notes is sufficient.

(Actually I was thinking recently that we need a "health check" utility. I'll open a separate issue for that.)

Show
Wendy Smoak added a comment - Preventing addition is fine with me. Do we need to do anything about existing instances that already have duplicates? A user who upgrades is still going to have this problem... maybe a mention in the release notes is sufficient. (Actually I was thinking recently that we need a "health check" utility. I'll open a separate issue for that.)
Hide
Permalink
Maria Catherine Tan added a comment -

I don't think we need to do anything anymore with existing instances. Will just mention this in the release notes.

Show
Maria Catherine Tan added a comment - I don't think we need to do anything anymore with existing instances. Will just mention this in the release notes.
Hide
Permalink
Maria Catherine Tan added a comment -

Fixed in
r777411 of 1.3.x branch
r777412 of trunk

  • prevent addition of duplicate projects to same group
  • allow duplicate projects from different group to be added in build queue during scheduled builds
  • log why project is not building during forced/scheduled build
Show
Maria Catherine Tan added a comment - Fixed in r777411 of 1.3.x branch r777412 of trunk prevent addition of duplicate projects to same group allow duplicate projects from different group to be added in build queue during scheduled builds log why project is not building during forced/scheduled build
Hide
Permalink
Wendy Smoak added a comment -

Has the documentation already been updated for this behavior change, or is there a separate issue for that?

Show
Wendy Smoak added a comment - Has the documentation already been updated for this behavior change, or is there a separate issue for that?
Hide
Permalink
Maria Catherine Tan added a comment -

It's a separate issue and i think jev already did those

Show
Maria Catherine Tan added a comment - It's a separate issue and i think jev already did those
Hide
Permalink
Wendy Smoak added a comment -

Jev, can you please find the issue and link it here?

When I search for this issue key in my email archives, I don't see any docs touched, and there are no related issues.

Show
Wendy Smoak added a comment - Jev, can you please find the issue and link it here? When I search for this issue key in my email archives, I don't see any docs touched, and there are no related issues.

People

  • Assignee:
    Maria Catherine Tan
    Reporter:
    Wendy Smoak
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: