Ethan, Clay, Keith, Dave and Joe,
This is a request for a second review of the webserver design project.
I have updated the project according to the feedback provided.
Clay,
I did not untangle the Thread code within the AI_database.py file as
when I started it I felt that it was more involved then I could do within
the schedule. I will file a CR on the issue and address it that way. Is
that OK with you?
The new webrev is located at:
http://cr.opensolaris.org/~johnfisc/webserver/webserver-design-2/
http://cr.opensolaris.org/~johnfisc/webserver/webserver-design-diff-2/
This covers the design as specified in:
http://hub.opensolaris.org/bin/download/Project+caiman/auto_install/AI%2DWebserver%2DSpecification.pdf
I believe it covers the following CRs:
9538 AI webserver not restarting after server reboot
8282 Port sharing with multiple AI webservers
8708 SMF: ai-webserver not restarted if it crashes
8561 Blocker bug for AI Webserver design work
6047 HTTP_PORT should be obtained programmatically, rather than
defined
The design is fairly simple. The changes for the design include:
o replaces the cherrypy webserver with a cgi-bin script called
cgi_get_manifest.py.
- the scripts are very similar except without the cherrypy
specific code
o replacement of port reference for database with servicename
- an implication of the webserver consolidation, majority of the
important code
changes
o shell scripts no longer start/stop webserver
o compatibility for older services - configuration changes
o build changes - Makefile changes
There are a few additional changes from pep8 checks. I didn't
do the majority of the pylint changes because several of the files
require extensive changes.
Thanks,
John
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
