http://defect.opensolaris.org/bz/show_bug.cgi?id=9947


Anurag S. Maskey <Anurag.Maskey at Sun.COM> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ACCEPTED                    |CAUSEKNOWN


--- Comment #6 from Anurag S. Maskey <Anurag.Maskey at Sun.COM> 2009-10-22 
20:52:58 UTC ---
In bfu.sh, the dlmgmt.xml manifest is imported during bfu.

   2352     #
   2353     # Import the datalink-management service.
   2354     #
   2355     smf_import_service network/dlmgmt.xml \
   2356         svc:/network/datalink-management:default

The specifying of the datalink-management FMRI suggests smf_import_service() to
enable the service only after reboot.

    529 # $1: the path of the xml file (the related path to /var/svc/manifest)
    530 # $2: the service name - specified only if the service is enabled after
reboot.
    531 #
    532 smf_import_service() {
    533

Then, code execution reaches 

    541         elif [[ -x /tmp/bfubin/svccfg ]]; then
    542             if [[ "${2}a" == a ]]; then
    543                 eval $BFUSVCCFG import /var/svc/manifest/$1
    544             else
    545                 tmpfile=/tmp/`echo "$1" | tr / :`.$$
    546                 sed -e "s/enabled='true'/enabled='false'/" \
    547                     /var/svc/manifest/$1 > "$tmpfile"
    548                 eval $BFUSVCCFG import "$tmpfile"
    549                 #
    550                 # Make sure the service is enabled after reboot.
    551                 #
    552                 enable_next_boot $2
    553             fi
    554         fi

Because the FMRI is specified in $2, line 546 explicitly changed the
"enabled=true" to false and then imports the manifest.  Also,
enable_next_boot() temporarily disables the service and changes enabled to true
so that on reboot the service is enabled..

    514 enable_next_boot () {
    515     if [ -x /tmp/bfubin/svccfg ]; then
    516         svcadm disable -t $1
    517         [ $? = 0 ] || echo "warning: unable to temporarily disable $1"
    518         eval $BFUSVCCFG -s $1 setprop general/enabled = true
    519         [ $? = 0 ] || echo "warning: unable to enable $1 for next boot"
    520     fi
    521 }

It appears the fix here might be to not pass the datalink-management FMRI to
smf_service_import() so that just the dlmgmt.xml manifest is imported.

-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.

Reply via email to