DataSourceOpenNebula's get_mask() and get_network() return empty string

[Akihiko Ota]

In DataSourceOpenNebula, get_mask() and get_network() check keys of CONTEXT Attributes, but doesn't check its values. If CONTEXT Attribute key ETH0_MASK and/or ETH0_NETWORK have no value, their methods should return default values, however actually return empty string('').

The problem occures when OpenNebula creats instances with Virtual Network resources which CONTEXT Attributes NETWORK_ADDRESS and/or NETWORK_MASK are not defined.

It seems that get_ip() also has the same bug.

I attached a patch for this problem.

[Scott Moser (smoser)]

Akihiko,
Thanks for filing a bug and even submitting a fix!

We use git and launchpad to do merge proposals. This is described at
 http://cloudinit.readthedocs.io/en/latest/topics/hacking.html

Please note that in order to accept your fix you'll have to sign the Contributors Agreement. That process is also described in the hacking link above.

Please feel free to reach out if you have any questions.

Thanks!

[Akihiko Ota (sw37th)]

Scott,

OK, I have created a merge request. Please review it.

[Chad Smith (chad.smith)]

Hello Akihiko Ota, or anyone else affected,

As part of a stable release update, this changeset is committed and accepted cloud-init into artful-proposed and xenial-proposed as cloud-init version 17.2-35-gf576b2a2-0ubuntu1. The package is now available in the specific -proposed repository and can be installed and tested on artful and xenial instances with something like the following:

sed -i 's/ xenial / xenial-proposed /' /etc/apt/sources
sudo apt-get update
sudo apt-get install cloud-init;
sudo cloud-init clean --logs --reboot; # For a fresh install run

If there are concerns please comment on LP: #1747059.

[Scott Moser (smoser)]

This bug is believed to be fixed in cloud-init in 18.1. If this is still a problem for you, please make a comment and set the state back to New

Thank you.

[James Falcon (falcojr)]

Tracked in Github Issues as https://github.com/canonical/cloud-init/issues/3079