D'oh I spoke too soon. Please only remove the option parser version strings. There is an arbitrary version number the webserver and AI client use when negotiating the criteria manifest. I've filed 7383 to track needing much more rigorous versioning of the XML bits.
Thank you, Clay On Fri, 13 Mar 2009, Clay Baenziger wrote: > Hi Glenn, > Please remove any version 0.5 references in the ai-webserver > directory as well. > Thank you, > Clay > > On Fri, 13 Mar 2009, Sundar Yamunachari wrote: > >> Glenn Lagasse wrote: >>> All, >>> >>> I've got what I believe to be the required changes for 6699. I've >>> addressed what I believe was the intent in the bug originally. Which is >>> to say that I've removed any code and reference to installadm displaying >>> a version number. >>> >>> They are located at: >>> >>> http://cr.opensolaris.org/~glagasse/slim_6699 >>> >>> http://defect.opensolaris.org/bz/show_bug.cgi?id=6699 >>> >>> That said, as part of some due diligence on my part I trolled around for >>> other 'version' references related to AI. I don't *think* what I've >>> found is related, but I'd like to get feedback from others who are >>> probably more familiar. >>> >>> Instances: >>> >>> usr/src/cmd/installadm/docs/manifest.html >>> 283: service name='ai_properties' version='1' type='service'> >>> >> This is official document. We will look at it when we review the document. >> These doc files will be replaced for June release. >>> usr/src/cmd/auto-install/svc/auto-installer.xml >>> 38: version='1'> >>> >> This is the version of the SMF service and it has nothing to do with >> installadm version. >>> usr/src/cmd/auto-install/ai_get_manifest.py >>> 783: <Version Number="0.5"> >>> >> This is just a comment. >>> usr/src/cmd/ai-webserver: >>> *.py files reference a version number of some sort which appears >>> different than what installadm was reporting (1.0 for installadm, 0.5 >>> for the .py files) >>> >> 0.5 appears to be the version of the module OptionParser. It has nothing to >> do with installadm version. Can you check with Clay about OptionParser? >>> Are any of these additional references relevant to the bug at hand? >>> They don't appear to be from my casual observation. >>> >> No. >> >> Thanks, >> Sundar >>> Thanks, >>> >>> Glenn >>> _______________________________________________ >>> caiman-discuss mailing list >>> caiman-discuss at opensolaris.org >>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >>> >> >> _______________________________________________ >> caiman-discuss mailing list >> caiman-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >> >
