From: Daniel M. <dan...@hp...> - 2007-04-17 21:09:28
|
In order to make life easier for anybody trying to review these patches, I'm submitting them again in a more sane, complete format. My goal in making these changes is to allow the "SIM integration" work that's already been done to respond to more than one machine. In other words, to allow an administrator to specify multiple serial numbers. Eventually I hope this will be useful to help LinuxCOE couple with more complete manageability solutions like SIM so that a user can select a machine or group of machines in SIM and SIM can pass the list of serials to LinuxCOE. The actual change is to change the Serial Number field in Step1 of coe_bootimage to accept a file containing multiple serial numbers (one serial per line). Then I had to make small changes to each of nph-coe_image, nph-debian_image and nph-suse_image to have them upload and parse the uploaded file instead of getting a single serial from POST. I set them to use one of the serials as the "real" directory and create symlinks to it for all of the other serial numbers in $simdir. That also meant that I had to change the $simdir/$serial/$serial file to something more generic, I chose $simdir/$serial/CUSTOM_SETTINGS. This will require a change in SSSTK. The other files in here are some CGI::Test scripts that provide regression tests for the new functionality I've provided. I've also included some tests in test_coe_bootimage.pl for functionality I did not provide, hoping it will make a good rally point so that whatever unit tests other people develop will be similar. I've included a README file that describes how to run the tests for the time being, but I intend to write a wrapper to execute and tally all of the tests. |