Discussion:
[Freeipa-devel] [freeipa PR#529][opened] installer: update time estimates
tomaskrizek
2017-03-01 16:40:56 UTC
Permalink
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
stlaz
2017-03-07 16:17:37 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Title: #529: installer: update time estimates

stlaz commented:
"""
This will say:
```
Configuring certificate server (pki-tomcatd). Estimated time: 3 minutes
[1/2]: configure certmonger for renewals
[2/2]: Importing RA key
```
but the operation lasts ~5 seconds at most.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/529#issuecomment-284770027
stlaz
2017-03-07 16:17:58 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Title: #529: installer: update time estimates

stlaz commented:
"""
This will say:
```
Configuring certificate server (pki-tomcatd). Estimated time: 3 minutes
[1/2]: configure certmonger for renewals
[2/2]: Importing RA key
```
but the operation lasts ~5 seconds at most.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/529#issuecomment-284770027
tomaskrizek
2017-03-07 17:51:49 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Author: tomaskrizek
Title: #529: installer: update time estimates
Action: synchronized

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
tomaskrizek
2017-03-07 17:54:34 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Title: #529: installer: update time estimates

tomaskrizek commented:
"""
@stlaz That estimate was a bit off :) Thanks for noticing!
"""

See the full comment at https://github.com/freeipa/freeipa/pull/529#issuecomment-284802644
tomaskrizek
2017-03-08 09:06:54 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Author: tomaskrizek
Title: #529: installer: update time estimates
Action: synchronized

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
stlaz
2017-03-10 13:35:33 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Title: #529: installer: update time estimates

stlaz commented:
"""
Please fix the little issue and we will push this.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/529#issuecomment-285670167
tomaskrizek
2017-03-10 13:47:06 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Author: tomaskrizek
Title: #529: installer: update time estimates
Action: synchronized

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
stlaz
2017-03-10 14:30:43 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Title: #529: installer: update time estimates

Label: +ack
MartinBasti
2017-03-10 14:47:04 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Title: #529: installer: update time estimates

Label: +pushed
MartinBasti
2017-03-10 14:47:06 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Title: #529: installer: update time estimates

MartinBasti commented:
"""
master:

* 09c6b7578046fed0824fc0f0d9040be69c0f0eb6 installer: update time estimates
"""

See the full comment at https://github.com/freeipa/freeipa/pull/529#issuecomment-285686947
MartinBasti
2017-03-10 14:47:07 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/529
Author: tomaskrizek
Title: #529: installer: update time estimates
Action: closed

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

Loading...