Code review comment for lp:~gmb/maas/add-ptr-records-bug-1382190

Revision history for this message
Julian Edwards (julian-edwards) wrote :

On Tuesday 28 Oct 2014 12:52:05 you wrote:
> > On Monday 27 Oct 2014 12:09:17 you wrote:
> > > I'm +1 on restricting the dynamic pool to the max size of a /16.
> >
> > I think we need to avoid doing this.
> >
> > For a /16 network, it added a huge amount of overhead and load onto the
> > process that writes the DNS zone files as it's now having to write the
> > pre-
> > generated entries *every time another entry changes*.
>
> I don't think that's actually true if you are using GENERATE (for IPv4) and
> wildcard (for IPv6), is it?

It should be OK for that, but I thought gmb had eschewed it in favour of
writing zone files? Or did that flip again.

> > 1. Juju starts using the API to get a static IP and passes it to
> > cloud-init.
> We considered this, and it should happen, but it won't happen for 1.7.

Right. I am worried as to how long we'll have to support this legacy GENERATE
stuff.

>
> > 2. Use DDNS on the DHCPD to update our own DNS server directly, since MAAS
> > doesn't need to know the IPs itself for LXCs.
> >
> > I think #2 is a trivial solution here, it's a simple piece of config.
>
> This may be a good solution, but have we ever seen that working?

As per my other emails, it's a PITA to get working.

« Back to merge proposal