URL: https://github.com/freeipa/freeipa/pull/529 Author: tomaskrizek Title: #529: installer: update time estimates Action: opened
PR body: """ Time estimates have been updated to be more accurate. Only tasks that are estimated to take longer than 10 seconds have the estimate displayed. https://pagure.io/freeipa/issue/6596 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/529/head:pr529 git checkout pr529
From 1ec69e69efe4422bcaf17aa874c53217cd605390 Mon Sep 17 00:00:00 2001 From: Tomas Krizek <tkri...@redhat.com> Date: Wed, 1 Mar 2017 17:35:56 +0100 Subject: [PATCH] installer: update time estimates Time estimates have been updated to be more accurate. Only tasks that are estimated to take longer than 10 seconds have the estimate displayed. https://pagure.io/freeipa/issue/6596 --- ipaserver/install/cainstance.py | 2 +- ipaserver/install/dsinstance.py | 6 +++--- ipaserver/install/httpinstance.py | 2 +- ipaserver/install/krainstance.py | 2 +- ipaserver/install/krbinstance.py | 4 ++-- ipaserver/install/upgradeinstance.py | 3 ++- 6 files changed, 10 insertions(+), 9 deletions(-) diff --git a/ipaserver/install/cainstance.py b/ipaserver/install/cainstance.py index 6e3f995..06c9531 100644 --- a/ipaserver/install/cainstance.py +++ b/ipaserver/install/cainstance.py @@ -450,7 +450,7 @@ def configure_instance(self, host_name, dm_password, admin_password, self.__add_lightweight_ca_tracking_requests) try: - self.start_creation(runtime=210) + self.start_creation(runtime=180) finally: self.clean_pkispawn_files() diff --git a/ipaserver/install/dsinstance.py b/ipaserver/install/dsinstance.py index bf80ae0..6a4efcb 100644 --- a/ipaserver/install/dsinstance.py +++ b/ipaserver/install/dsinstance.py @@ -349,7 +349,7 @@ def create_instance(self, realm_name, fqdn, domain_name, self.__common_post_setup() - self.start_creation(runtime=60) + self.start_creation(runtime=30) def enable_ssl(self): self.steps = [] @@ -358,7 +358,7 @@ def enable_ssl(self): self.step("restarting directory server", self.__restart_instance) self.step("adding CA certificate entry", self.__upload_ca_cert) - self.start_creation(runtime=10) + self.start_creation() def create_replica(self, realm_name, master_fqdn, fqdn, domain_name, dm_password, @@ -411,7 +411,7 @@ def create_replica(self, realm_name, master_fqdn, fqdn, self.__common_post_setup() - self.start_creation(runtime=60) + self.start_creation(runtime=30) def __setup_replica(self): diff --git a/ipaserver/install/httpinstance.py b/ipaserver/install/httpinstance.py index 7979ca1..610c54a 100644 --- a/ipaserver/install/httpinstance.py +++ b/ipaserver/install/httpinstance.py @@ -185,7 +185,7 @@ def create_instance(self, realm, fqdn, domain_name, pkcs12_info=None, self.step("configuring httpd to start on boot", self.__enable) self.step("enabling oddjobd", self.enable_and_start_oddjobd) - self.start_creation(runtime=60) + self.start_creation() def __start(self): self.backup_state("running", self.is_running()) diff --git a/ipaserver/install/krainstance.py b/ipaserver/install/krainstance.py index d7ab6fd..13b4c4a 100644 --- a/ipaserver/install/krainstance.py +++ b/ipaserver/install/krainstance.py @@ -135,7 +135,7 @@ def configure_instance(self, realm_name, host_name, dm_password, self.step("enabling KRA instance", self.__enable_instance) try: - self.start_creation(runtime=126) + self.start_creation(runtime=120) finally: self.clean_pkispawn_files() diff --git a/ipaserver/install/krbinstance.py b/ipaserver/install/krbinstance.py index 44b3821..2a390ce 100644 --- a/ipaserver/install/krbinstance.py +++ b/ipaserver/install/krbinstance.py @@ -164,7 +164,7 @@ def create_instance(self, realm_name, host_name, domain_name, admin_password, ma self.step("installing X509 Certificate for PKINIT", self.setup_pkinit) - self.start_creation(runtime=30) + self.start_creation() self.kpasswd = KpasswdInstance() self.kpasswd.create_instance('KPASSWD', self.fqdn, self.suffix, @@ -189,7 +189,7 @@ def create_replica(self, realm_name, self.__common_post_setup() - self.start_creation(runtime=30) + self.start_creation() self.kpasswd = KpasswdInstance() self.kpasswd.create_instance('KPASSWD', self.fqdn, self.suffix) diff --git a/ipaserver/install/upgradeinstance.py b/ipaserver/install/upgradeinstance.py index fca4226..e5bc8a2 100644 --- a/ipaserver/install/upgradeinstance.py +++ b/ipaserver/install/upgradeinstance.py @@ -120,7 +120,8 @@ def create_instance(self): if ds_running: self.step("starting directory server", self.__start) self.start_creation(start_message="Upgrading IPA:", - show_service_name=False) + show_service_name=False, + runtime=90) def __save_config(self): shutil.copy2(self.filename, self.savefilename)
-- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
