Initially I thought it was only about adding some sort of "alias", such
that `ubuntu_pro` would also be recognized alongside `ubuntu_advantage`.
But when looking at the diff, I see that it's actually replacing one
cloud-init module with another. I see this was also hinted at in the
"where problems could occurr" section of the bug description.

My concerns with this approach are as follows:

a) the test plan doesn't seem to cover all other changes brought in via the new 
ubuntu_pro module. Not only new features, but also that existing ones don't 
regress behavior if the module is invoked via the old key `ubuntu_advantage`.
b) this doesn't feel like a minimal fix anymore
c) on the justification part, it was said that users should be able to use the 
same `ubuntu_pro` module configuration for x/b and later releases. But 
cloud-init SRUs for x/b are no longer being done. What if focal, for example, 
fixes a bug in the ubuntu_pro module there, would that also be backported to 
x/b from now on?
d) since this is replacing one module with another, it's very difficult to 
review the change. What else is the ubuntu_pro module bringing down? How can we 
be sure it's not changing behavior, specially if being invoked by the (now 
deprecated) `ubuntu_advantage` key?
e) this x/b SRU will require new images to be built for those releases, outside 
of ESM. In internal conversations, that seems it's doable, but has there been a 
formal agreement that it will be done for this SRU? And, considering point (c) 
above, what about future SRUs?

Is the "users can now use the exact same cloud-init config all the way
from x to noble" really true, or would it just apply to Pro
configuration? Note that bionic and xenial have very different cloud-
init versions:

xenial: 21.1-19-gbad84ad4
bionic: 23.1.2
focal and later: 24.2 (with 24.3.1 in proposed)


Considering all of the above, do you think it's still worth to pursue this SRU? 

** Changed in: cloud-init (Ubuntu Bionic)
       Status: New => Incomplete

** Changed in: cloud-init (Ubuntu Xenial)
       Status: New => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2067660

Title:
  deprecate ubuntu_advantage config key from user-data in favor of
  ubuntu_pro

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/2067660/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to