Skip to content

Commit

Permalink
Dashboard should only be installed on the data node
Browse files Browse the repository at this point in the history
Fixes ESGF#458
  • Loading branch information
chrisacbr committed Sep 17, 2018
1 parent ec1f07e commit ec426a4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions esg-node
Original file line number Diff line number Diff line change
Expand Up @@ -7164,9 +7164,9 @@ main() {
[ $((sel & TEST_BIT)) != 0 ] && test_tomcat
[ $((sel & INSTALL_BIT)) != 0 ] && [ $((sel & DATA_BIT+COMPUTE_BIT)) != 0 ] && setup_tds ${upgrade_mode}
[ $((sel & TEST_BIT)) != 0 ] && [ $((sel & DATA_BIT+COMPUTE_BIT)) != 0 ] && test_tds
[ $((sel & INSTALL_BIT)) != 0 ] && setup_subsystem node-manager esgf-node-manager #(tomcat off)
[ $((sel & INSTALL_BIT)) != 0 ] && [ $((sel & IDP_BIT+DATA_BIT)) != 0 ] && setup_subsystem node-manager esgf-node-manager #(tomcat off)

[ $((sel & INSTALL_BIT)) != 0 ] && setup_subsystem dashboard esgf-dashboard #(tomcat off)
[ $((sel & INSTALL_BIT)) != 0 ] && [ $((sel & DATA_BIT)) != 0 ] && setup_subsystem dashboard esgf-dashboard #(tomcat off)

#[ $((sel & INSTALL_BIT)) != 0 ] && [ $((sel & DATA_BIT)) != 0 ] && setup_subsystem desktop esgf-desktop #(tomcat off)

Expand Down

1 comment on commit ec426a4

@pchengi
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chrisacbr , I see that this was never submitted as a PR; also, why are you adding the IDP+Data requirement for the node manager setup? That's not related to the dashboard issue, is it?

Please sign in to comment.