Continuum

Configuration for Distributed Build should be visible only if Distributed Build is enabled

Details

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

Description

The "Build Agents" and "Releases" menu items are only relevant if Distributed Builds are enabled and should not be visible otherwise.

I also noticed that the Build Agent Group selection is visible in the Build Environment page, even when Distributed Build is not enabled.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
jan ancajas added a comment -

Based from the discussion :
http://www.nabble.com/Re%3A-svn-commit%3A-r751678---in--continuum-trunk%3A-continuum-api-src-main-java-org-apache-continuum-release-distributed-manager---continuum-buildagent-continuum-buildagent-api-src-main-java-org-apache-continuum-buildagent---continuum-buildagent-continuum-b-td22469769.html

summarized the proposed changes :

1. Add security role "continuum-release-view"
2. Add Distributed section in Menu
3. Move "Releases" and "Build Agents" buttons under Distributed section
4. Releases will show only for those who has a continuum-release-view, whom for now is the sysad only
5. Show distributed section in menu if distributed build is enabled

Show
jan ancajas added a comment - Based from the discussion : http://www.nabble.com/Re%3A-svn-commit%3A-r751678---in--continuum-trunk%3A-continuum-api-src-main-java-org-apache-continuum-release-distributed-manager---continuum-buildagent-continuum-buildagent-api-src-main-java-org-apache-continuum-buildagent---continuum-buildagent-continuum-b-td22469769.html summarized the proposed changes : 1. Add security role "continuum-release-view" 2. Add Distributed section in Menu 3. Move "Releases" and "Build Agents" buttons under Distributed section 4. Releases will show only for those who has a continuum-release-view, whom for now is the sysad only 5. Show distributed section in menu if distributed build is enabled
Hide
Permalink
Wendy Smoak added a comment -

Thanks, Jan. Here are a few more:

6. only show the Build Agent Group selection on the Build Environment page if distributed build is enabled.

Is there any other distributed build-related configuration that needs to be suppressed to avoid confusion?

And the reverse situation:

7. The Build Queue button should not appear when distributed build is enabled.
(How about a 'Parallel Builds' menu section for this which appears when Distributed Build is not enabled?)

8. The 'Add Build Queue' selection on the Schedule should not appear when distributed build is enabled.

Is there any other parallel build-related configuration that needs to be suppressed?

Show
Wendy Smoak added a comment - Thanks, Jan. Here are a few more: 6. only show the Build Agent Group selection on the Build Environment page if distributed build is enabled. Is there any other distributed build-related configuration that needs to be suppressed to avoid confusion? And the reverse situation: 7. The Build Queue button should not appear when distributed build is enabled. (How about a 'Parallel Builds' menu section for this which appears when Distributed Build is not enabled?) 8. The 'Add Build Queue' selection on the Schedule should not appear when distributed build is enabled. Is there any other parallel build-related configuration that needs to be suppressed?
Hide
Permalink
jan ancajas added a comment -

6. only show the Build Agent Group selection on the Build Environment page if distributed build is enabled.

Is there any other distributed build-related configuration that needs to be suppressed to avoid confusion?

i think we've got it all covered.

7. The Build Queue button should not appear when distributed build is enabled.
(How about a 'Parallel Builds' menu section for this which appears when Distributed Build is not enabled?)

It makes sense since parallel and distributed features are mutually exclusive.

8. The 'Add Build Queue' selection on the Schedule should not appear when distributed build is enabled.

Is there any other parallel build-related configuration that needs to be suppressed?

i think we've got it all covered.

Show
jan ancajas added a comment -
6. only show the Build Agent Group selection on the Build Environment page if distributed build is enabled. Is there any other distributed build-related configuration that needs to be suppressed to avoid confusion?
i think we've got it all covered.
7. The Build Queue button should not appear when distributed build is enabled. (How about a 'Parallel Builds' menu section for this which appears when Distributed Build is not enabled?)
It makes sense since parallel and distributed features are mutually exclusive.
8. The 'Add Build Queue' selection on the Schedule should not appear when distributed build is enabled. Is there any other parallel build-related configuration that needs to be suppressed?
i think we've got it all covered.
Hide
Permalink
jan ancajas added a comment -

attached patch.
Created custom tag "ifBuildTypeEnabled" for portion in the jsp pages we want to hide/show if parallel/distributed build is enabled.

sample use: 

<c1:ifBuildTypeEnabled buildType="parallel">
</c1:ifBuildTypeEnabled>
<c1:ifBuildTypeEnabled buildType="distributed">
</c1:ifBuildTypeEnabled>
Show
jan ancajas added a comment - attached patch. Created custom tag "ifBuildTypeEnabled" for portion in the jsp pages we want to hide/show if parallel/distributed build is enabled. sample use: 
<c1:ifBuildTypeEnabled buildType="parallel">
</c1:ifBuildTypeEnabled>
<c1:ifBuildTypeEnabled buildType="distributed">
</c1:ifBuildTypeEnabled>
Hide
Permalink
Wendy Smoak added a comment -

Did you catch the 'Build Agent Group' column on the Build Environments page (/continuum/admin/buildEnvList!list.action) ?

If not, then

9. Build Agent Group column on the Build Environments page should only be displayed if Distributed Build is enabled

Thanks!

Show
Wendy Smoak added a comment - Did you catch the 'Build Agent Group' column on the Build Environments page (/continuum/admin/buildEnvList!list.action) ? If not, then 9. Build Agent Group column on the Build Environments page should only be displayed if Distributed Build is enabled Thanks!
Hide
Permalink
jan ancajas added a comment -

updated patch. thanks for pointing that out wendy.

Show
jan ancajas added a comment - updated patch. thanks for pointing that out wendy.
Hide
Permalink
Maria Catherine Tan added a comment -

Fix in revision 755471. Thanks Jan!

added some fix to the patch

  • added continuum-manage-parallel-builds role
  • secure BuildQueue action
  • check individual permission for each distributed menu button
Show
Maria Catherine Tan added a comment - Fix in revision 755471. Thanks Jan! added some fix to the patch
  • added continuum-manage-parallel-builds role
  • secure BuildQueue action
  • check individual permission for each distributed menu button

People

Vote (0)
Watch (0)

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.