I made another round of install, few more small remarks:
- qserv-configure.py produces a lot of output which could be confusing if people try to look at it and understand it. It may be better to reduce it to something that just indicates that each step is completed successfully. If we want verbose output it could be redirected to a log file. This could go into separate ticket.
- qserv-configure.py managed to confuse me with this warning/question:
WARNING : Do you want to update client configuration currently pointing on /usr/local/home/salnikov/.lsst/qserv.conf [y/n]
I guess the intent was to ask if user config file needs to be updated. Maybe the question needs to be re-phrased and made less alarming? Something like:
Do you want to update user configuration file ($HOME/.lsst/qserv.conf) for new run directory? [y/n]
- Making $HOME/.lsst/qserv.conf a symlink is probably OK for now in "mono" setup, but could be problematic in more general case because run directory does not have to be accessible from all machines. We need to think how to handle config files better.
- README file still says to run qserv-start.sh without full path name which assumes it's in a $PATH. Needs to be updated
If you can fix 2 and 4 quickly I promise I will close review (I'm traveling back to US tomorrow).