-
Notifications
You must be signed in to change notification settings - Fork 837
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
gentoo: fix hostname rendering when value has a comment (#611)
Gentoo's hostname file format instead of being just the host name is hostname=thename". The old code works fine when the file has no comments but if there is a comment the line ``` gentoo_hostname_config = 'hostname="%s"' % conf ``` can render an invalid hostname file that looks similar to ``` hostname="#This is the host namehello" ``` The fix inserts the hostname in a gentoo friendly way so that it gets handled by HostnameConf as a whole and comments are handled and preserved
- Loading branch information
1 parent
404f0a4
commit b8bd081
Showing
3 changed files
with
33 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
# This file is part of cloud-init. See LICENSE file for license information. | ||
|
||
from cloudinit import util | ||
from cloudinit import atomic_helper | ||
from cloudinit.tests.helpers import CiTestCase | ||
from . import _get_distro | ||
|
||
|
||
class TestGentoo(CiTestCase): | ||
|
||
def test_write_hostname(self): | ||
distro = _get_distro("gentoo") | ||
hostname = "myhostname" | ||
hostfile = self.tmp_path("hostfile") | ||
distro._write_hostname(hostname, hostfile) | ||
self.assertEqual('hostname="myhostname"\n', util.load_file(hostfile)) | ||
|
||
def test_write_existing_hostname_with_comments(self): | ||
distro = _get_distro("gentoo") | ||
hostname = "myhostname" | ||
contents = '#This is the hostname\nhostname="localhost"' | ||
hostfile = self.tmp_path("hostfile") | ||
atomic_helper.write_file(hostfile, contents, omode="w") | ||
distro._write_hostname(hostname, hostfile) | ||
self.assertEqual('#This is the hostname\nhostname="myhostname"\n', | ||
util.load_file(hostfile)) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,6 +13,7 @@ johnsonshi | |
jqueuniet | ||
landon912 | ||
lucasmoura | ||
manuelisimo | ||
marlluslustosa | ||
matthewruffell | ||
nishigori | ||
|