Jack,

I'll take group A and if I have some time, group B as well.

-Drew

On 03/11/11 20:36, Jack Schwartz wrote:
 Hi everyone.

Here is the first of two code review wads for the Derived Manifests project.

This wad includes the AI client side support for running a script to derive manifests. A subsequent (much smaller) one coming within a few days will be for improved AI server support of default manifest management.

The focus of this code review is on the following:
- the Derived Manifest Module checkpoint
- the aimanifest command
- the Manifest Input Module (which supports aimanifest)

http://cr.opensolaris.org/~schwartz/110311.1/webrev/index.html

Please take a group of files and review by Weds 3/23 COB. Please let me know ASAP what you are able to review.

I'll be setting up an info session for Tuesday morning from 9-10 PST to help *jumpstart* the process. (Sorry, couldn't resist) Logistic info to follow.

I suggest the following groups:

(A) The Derived Manifest Module
   usr/src/cmd/auto-install/checkpoints/dmm/Makefile
   usr/src/cmd/auto-install/checkpoints/dmm/__init__.py
   usr/src/cmd/auto-install/checkpoints/dmm/dmm.py
   usr/src/cmd/auto-install/checkpoints/dmm/dmm_errors.py
   usr/src/cmd/auto-install/checkpoints/dmm/test/Makefile
   usr/src/cmd/auto-install/checkpoints/dmm/test/dmm_build_test.py
   usr/src/cmd/auto-install/checkpoints/dmm/test/dmm_env_test.py

(B) The Manifest Input Module, focus on overlay
   usr/src/lib/install_manifest_input/Makefile
   usr/src/lib/install_manifest_input/__init__.py
   usr/src/lib/install_manifest_input/process_dtd.py
   usr/src/lib/install_manifest_input/test/test_manifest_input_overlay.py
   usr/src/lib/install_manifest_input/test/test_process_dtd.py

(C) The Manifest Input Module, the rest
   usr/src/lib/install_manifest_input/mim.py
   usr/src/lib/install_manifest_input/mim_utils.py
   usr/src/lib/install_manifest_input/mim_errors.py
   usr/src/lib/install_manifest_input/test/test_manifest_input_pathing.py
usr/src/lib/install_manifest_input/test/test_manifest_input_set_get_add.py usr/src/lib/install_manifest_input/test/test_manifest_input_validate_commit.py

(D) aimanifest command, AI environment prep, packaging and other files:
   usr/src/cmd/aimanifest/Makefile
   usr/src/cmd/aimanifest/aimanifest.py
   usr/src/Makefile.master
   usr/src/Targetdirs
   usr/src/cmd/Makefile
   usr/src/cmd/Makefile.cmd
   usr/src/cmd/Makefile.targ
   usr/src/cmd/auto-install/auto_install.py
   usr/src/cmd/auto-install/checkpoints/Makefile
   usr/src/cmd/auto-install/config/get_manifest
   usr/src/cmd/auto-install/svc/auto-installer
   usr/src/cmd/auto-install/svc/manifest-locator
   usr/src/lib/Makefile
   usr/src/lib/Makefile.targ
   usr/src/lib/install_target/Makefile
   usr/src/lib/install_target/shadow/Makefile
usr/src/pkg/manifests/system-install-auto-install-auto-install-common.mf
   usr/src/pkg/manifests/system-install-auto-install.mf
   usr/src/tools/tests/tests.nose
   usr/src/cmd/rbac/Makefile
   usr/src/cmd/rbac/prof_attr.system%2Finstall%2Fauto-install
   usr/src/cmd/rbac/user_attr.system%2Finstall%2Fauto-install
   usr/src/cmd/auto-install/ai_get_manifest.py

    Thanks,
    Jack

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to