yahoo-eng-team team mailing list archive

[Chris Glass <tribaal@xxxxxxxxx>]

Public bug reported:

While testing the Exoscale datasource for its inclusion in a SRU, it was
discovered that a cloud_config_module didn't work.

Passing user data such as:
https://gist.github.com/chrisglass/fb0cf860be8cf01f456dfff8e162e004
results in the "runcmd" stanza not to be executed.

(feel free to get in touch should you like to play with an instance
displaying the problem on Eoan)

Hypothesis:

The merge of the datasource's extra_config field (https://git.launchpad.net/cloud-init/tree/cloudinit/sources/DataSourceExoscale.py#n124) is erroneous: instead of *overwriting* the cloud_config_modules entry from the cloud.cfg file/user data, the cloud_config_modules should be *merged*. 
An additional difficulty being that we insert a two-elements list (["set-passwords", "always"]) and it needs to be merge with a list containing just "set-passwords".

** Affects: cloud-init
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to cloud-init.
https://bugs.launchpad.net/bugs/1841454

Title:
  Exoscale datasource overwrites *all* cloud_config_modules

Status in cloud-init:
  New

Bug description:
  While testing the Exoscale datasource for its inclusion in a SRU, it
  was discovered that a cloud_config_module didn't work.

  Passing user data such as:
  https://gist.github.com/chrisglass/fb0cf860be8cf01f456dfff8e162e004
  results in the "runcmd" stanza not to be executed.

  (feel free to get in touch should you like to play with an instance
  displaying the problem on Eoan)

  Hypothesis:

  The merge of the datasource's extra_config field (https://git.launchpad.net/cloud-init/tree/cloudinit/sources/DataSourceExoscale.py#n124) is erroneous: instead of *overwriting* the cloud_config_modules entry from the cloud.cfg file/user data, the cloud_config_modules should be *merged*. 
  An additional difficulty being that we insert a two-elements list (["set-passwords", "always"]) and it needs to be merge with a list containing just "set-passwords".

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