New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
installer: update time estimates #529
installer: update time estimates #529
Conversation
This will say during replica installation (DL1):
but the operation lasts ~5 seconds at most. |
1ec69e6
to
3ee3251
Compare
ipaserver/install/krainstance.py
Outdated
@@ -134,8 +134,13 @@ def configure_instance(self, realm_name, host_name, dm_password, | |||
|
|||
self.step("enabling KRA instance", self.__enable_instance) | |||
|
|||
if ra_only: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this is necessary. Is kra install ever executed with ra_only=True
option?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase on current master
edit: never mind, you are rebased. I have no idea why you added this.
@stlaz That estimate was a bit off :) Thanks for noticing! |
3ee3251
to
d2ba0a1
Compare
ipaserver/install/service.py
Outdated
@@ -401,7 +401,7 @@ def start_creation(self, start_message=None, end_message=None, | |||
else: | |||
end_message = "Done configuring %s." % self.service_desc | |||
|
|||
if runtime > 0: | |||
if runtime and runtime > 0: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The correct condition is if runtime is not None and and runtime > 0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SyntaxError: invalid syntax
:-P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤕
Please fix the little issue and we will push this. |
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
d2ba0a1
to
adfafc3
Compare
master:
|
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