Susan Sohn wrote:
> Sundar,
>
> Fixed. Webrev updated.
It looks good now.
- Sundar
>
> Sue
>
> On 04/03/09 13:13, Sundar Yamunachari wrote:
>> Sue,
>>
>> *usr/src/cmd/installadm/setup-dhcp.sh:*
>>
>> 115 echo "Use the following command if you are running Sun's
>> DHCP server"
>> 116 echo "to add the macro:"
>> 117 echo " $DHTADM -A -m ${macro} -d ${macvalue}"
>>
>> Can you change 115 to "If you are running Sun's DHCP server, use the
>> following command to add the DHCP macro ${macro}"
>>
>> Thanks,
>> Sundar
>>
>> Sue Sohn wrote:
>>> Hi Sarah, Sundar, and Jack,
>>>
>>> Thank you for the review and feedback.
>>>
>>> I have added the new output to the script and have updated the webrev.
>>>
>>> Thanks,
>>> Sue
>>>
>>> Jack Schwartz wrote:
>>>> HI everyone.
>>>>
>>>> On 04/01/09 05:54, Sarah Jelinek wrote:
>>>>> Sundar Yamunachari wrote:
>>>>>> Sarah Jelinek wrote:
>>>>>>> Hi Sue,
>>>>>>>
>>>>>>> In the case of needing to add a static ip for the client on the
>>>>>>> dhcp server, we need to run pntadm, correct? Assuming the user
>>>>>>> has not setup a set of addresses for clients in their dhcp server.
>>>>>>>
>>>>>>> Do we need to address this case in this bug? I ran in to this
>>>>>>> issue yesterday, specifically I ran the dhtadm commands but my
>>>>>>> client couldn't boot because the I didn't have the clients ip
>>>>>>> address setup. I didn't know this, and when I ran the dhtadm
>>>>>>> commands it didn't fail, so I was kind of clueless. Sundar
>>>>>>> helped me understand the issue.
>>>>>>>
>>>>>>> I am not sure it is feasible to provide this type of
>>>>>>> information, unless we check somehow. Is there a way to check?
>>>>>>> The dhtadm command will return 0 even in the case I describe above.
>>>>>> Sarah,
>>>>>>
>>>>>> pntadm uses to add, delete or modify IP address records in the
>>>>>> DHCP table. Since DHCP server is remote, we can not check what IP
>>>>>> addresses are available in the DHCP server or which address
>>>>>> should be assigned to the client. However, we could tell the user
>>>>>> that IP address assignment for the client or set of clients
>>>>>> should be done.
>>>>>>
>>>>> That seems like a reasonable compromise. That is letting the user
>>>>> know that the IP address assignment for the client should be done.
>>>> I think this is a great idea! This nicely handles what I was
>>>> talking about in yesterday's meeting.
>>>>
>>>> I suggest putting some kind of quick reference in the message, for
>>>> example:
>>>>
>>>> "Note: be sure to assign client IP address if needed (e.g. run
>>>> pntadm(1M))"
>>>>
>>>> Thanks,
>>>> Jack
>>>>>
>>>>>
>>>>> Thanks for the explanation.
>>>>>
>>>>> sarah
>>>>>> - Sundar
>>>>>>>
>>>>>>> thanks,
>>>>>>> sarah
>>>>>>> ***
>>>>>>>> Please review the changes for:
>>>>>>>>
>>>>>>>> 7747 Installadm should print actual commands needed for the
>>>>>>>> DHCP setup
>>>>>>>> http://defect.opensolaris.org/bz/show_bug.cgi?id=7747
>>>>>>>>
>>>>>>>> which are posted at:
>>>>>>>>
>>>>>>>> http://cr.opensolaris.org/~sohn/7747
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Sue
>>>>>>>> _______________________________________________
>>>>>>>> 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
>>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> 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
>>>
>>> _______________________________________________
>>> caiman-discuss mailing list
>>> caiman-discuss at opensolaris.org
>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>
>
