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

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

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?

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.

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>

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!

updated patch. thanks for pointing that out wendy.

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