Cloud-init inject hostname should support dot and without .novalocal

[Sibiao Luo]

Description of problem:
Cloud-init inject hostname should support dot and without .novalocal.

Version-Release number of selected component (if applicable):
cloud-init 0.7.9

How reproducible:
100%

Steps to Reproduce:
1.create a RHEL7.5/7.4 instance specified the hostname with OpenStack API
2.login VM with hostnamectl to check the hostname

Actual results:
API: hostname=ecs-rhel7.4-instance --------> VM: hostname=ecs-rhel7.novanoval
API: hostname=ecs-rhel74-instance --------> VM: ecs-rhel74-instance.novanoval

Expected results:
The VM hostname is the same as specified in OpenStack API or ECS Console.

Additional info:
# diff __init__.py
usr/lib/python2.7/site-packages/cloudinit/sources/__init__.py
205c205
< toks = lhost.split(".novalocal")
---
> toks = lhost.split(".")
209c209
< #domain = '.'.join(toks[1:])
---
> domain = '.'.join(toks[1:])

[Scott Moser (smoser)]

Hi,
Could you please test to see if this is still a problem with trunk?

We have daily copr builds at
 https://copr.fedorainfracloud.org/coprs/g/cloud-init/cloud-init-dev/

Please test with the builds there and report back.

If it is still a problem, please attach the output of 'cloud-init collect-logs'.

After you've done so, please set the status to 'Confirmed'.

Thanks.

[Sibiao Luo (sibiaoluo)]

Try this:
https://copr-be.cloud.fedoraproject.org/results/%40cloud-init/cloud-init-dev/epel-7-x86_64/00788016-cloud-init/cloud-init-18.3+24.gf6249277-1.el7.noarch.rpm

It support donot now, but still append the .novalocal problem.

You shoud split it by .novalocal which can make success.

# diff __init__.py
usr/lib/python2.7/site-packages/cloudinit/sources/__init__.py
205c205
< toks = lhost.split(".novalocal")
---
> toks = lhost.split(".")
209c209
< #domain = '.'.join(toks[1:])
---
> domain = '.'.join(toks[1:])

[Scott Moser (smoser)]

Hi,
can you please attach the output of 'cloud-init collect-logs'.

Thank you.
Scott

[Launchpad Janitor (janitor)]

[Expired for cloud-init because there has been no activity for 60 days.]

[James Falcon (falcojr)]

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