I would like to get this patch included in upcoming 6.2, the issue is
quite serious.
---
Do not forget to add new line in updated /etc/sysconfig/network
configuration. Move the actual change of the hostname after the
user confirmation about proceeding with installation. It confused
users when the hostname change occurred before this prompt.

https://fedorahosted.org/freeipa/ticket/1724

>From cd85399448901a2ee23b4c73d2f8d7fa2536049a Mon Sep 17 00:00:00 2001
From: Martin Kosek <mko...@redhat.com>
Date: Mon, 29 Aug 2011 10:22:20 +0200
Subject: [PATCH] ipa-client-install breaks network configuration

Do not forget to add new line in updated /etc/sysconfig/network
configuration. Move the actual change of the hostname after the
user confirmation about proceeding with installation. It confused
users when the hostname change occurred before this prompt.

https://fedorahosted.org/freeipa/ticket/1724
---
 ipa-client/ipa-install/ipa-client-install |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/ipa-client/ipa-install/ipa-client-install b/ipa-client/ipa-install/ipa-client-install
index fe4e1e4dd19b9c81580654d8c3fd9996fb711e46..64c5bf2c653764d3e5465e879c5a4b8d3a64d95a 100755
--- a/ipa-client/ipa-install/ipa-client-install
+++ b/ipa-client/ipa-install/ipa-client-install
@@ -571,7 +571,7 @@ $)''', re.VERBOSE)
                     option, value = m.group('option', 'value')
                     if option is not None and option == 'HOSTNAME':
                         if value is not None and hostname != value:
-                            new_line = u'HOSTNAME=%s' % (hostname)
+                            new_line = u"HOSTNAME=%s\n" % (hostname)
                             statestore.backup_state('network', 'hostname', value)
                 new_config.write(new_line)
         new_config.flush()
@@ -779,10 +779,6 @@ def main():
     if hostname != hostname.lower():
         sys.exit('Invalid hostname \'%s\', must be lower-case.' % hostname)
 
-    if options.hostname:
-        # configure /etc/sysconfig/network to contain the hostname we set.
-        backup_and_replace_hostname(fstore, options.hostname)
-
     # Create the discovery instance
     ds = ipadiscovery.IPADiscovery()
 
@@ -879,6 +875,10 @@ def main():
     if not options.unattended and not user_input("Continue to configure the system with these values?", False):
         return 1
 
+    if options.hostname:
+        # configure /etc/sysconfig/network to contain the hostname we set.
+        backup_and_replace_hostname(fstore, options.hostname)
+
     if not options.unattended:
         if options.principal is None and options.password is None and options.prompt_password is False:
             options.principal = user_input("User authorized to enroll computers", allow_empty=False)
-- 
1.7.6

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to