Code review comment for lp:~lamont/maas/xenial-has-released

Revision history for this message
LaMont Jones (lamont) wrote :

> > We are hardcoding this for a reason. This is to prevent that the next LTS
> > release cause the same issues as it caused when Xenial was out.
>
> The code being removed literally says "if trusty is the current LTS, that's a
> lie, it's xenial" -- that situation will not reoccur for at least 26 years,
> and requires that we be silly enough to reuse both "trusty" and "xenial"....
>
> If we want to keep the code, it should probably turn into something that says
> "use the current devel release if it's LTS" -- that would need a prod vs dev
> switch, but is very doable. (the new release gets added to distro-info shortly
> after it opens, and we'd just need to see that devel opened and that it's an
> LTS.)

Also, it's trivial for devs to override the LTS release name in the database. (Which I did for a while when I was using yakkety to test ipv6, because we had the packages there, but not in xenial yet.)

« Back to merge proposal