Merge lp:~andreserl/maas/maas_preseed_userdata_hostname into lp:maas/trunk
| Status: | Merged |
|---|---|
| Approved by: | Andres Rodriguez on 2012-08-31 |
| Approved revision: | 957 |
| Merged at revision: | 957 |
| Proposed branch: | lp:~andreserl/maas/maas_preseed_userdata_hostname |
| Merge into: | lp:maas/trunk |
| Diff against target: |
15 lines (+4/-1) 1 file modified
contrib/preseeds_v2/enlist_userdata (+4/-1) |
| To merge this branch: | bzr merge lp:~andreserl/maas/maas_preseed_userdata_hostname |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Scott Moser | 2012-08-31 | Approve on 2012-08-31 | |
|
Review via email:
|
|||
Commit Message
During the enlistment process, we need to try to detect whether a hostname/DNS name is being set for the machine. This is specifically useful when using an external DNS server. If a hostname is found, the machine will be enlisted with this hostname.
This maintains compatibility to continue to do the same that what the installer was doing during the enlistment process.
Thanks to Scott Moser.
| Scott Moser (smoser) wrote : | # |
| Andres Rodriguez (andreserl) wrote : | # |
Agreed.
However, it will obtain the same hostname from the DNS server which is the important factor.
| Scott Moser (smoser) wrote : | # |
> This looks reasonable. It has potential errors, and isn't going to guarantee
> to produce the same hostname as was provided by the dhcp server, but it is
> *something*.
| Julian Edwards (julian-edwards) wrote : | # |
The problem is that MAAS won't have set up DNS for the node yet so I doubt
this will work at all. MAAS needs to have completed enlistment before it
knows the IP address and thus set up DNS.
| Andres Rodriguez (andreserl) wrote : | # |
Hi Julian,
This is to maintain compatibility with the older enlistment process.
Consider the fact that there's is external DNS/dhcp this process will
obtain the DNS name and enlist with Maas. Otherwise we have a "broken" Maas
and we won't be able to use juju out of the box having external DNS/dchp.
This has been tested and works as expectes
On Sep 2, 2012 9:07 PM, "Julian Edwards" <email address hidden>
wrote:
> The problem is that MAAS won't have set up DNS for the node yet so I doubt
> this will work at all. MAAS needs to have completed enlistment before it
> knows the IP address and thus set up DNS.
>
> --
>
> https:/
> You are the owner of lp:~andreserl/maas/maas_preseed_userdata_hostname.
>
| Julian Edwards (julian-edwards) wrote : | # |
On Monday 03 September 2012 01:19:18 you wrote:
> Hi Julian,
>
> This is to maintain compatibility with the older enlistment process.
> Consider the fact that there's is external DNS/dhcp this process will
> obtain the DNS name and enlist with Maas. Otherwise we have a "broken" Maas
> and we won't be able to use juju out of the box having external DNS/dchp.
> This has been tested and works as expectes
Ah ok, thanks for clarifying.


This looks reasonable. It has potential errors, and isn't going to guarantee to produce the same hostname as was provided by the dhcp server, but it is *something*.