Skip to content

enhance 'completed' message with how much time was needed the installation#2956

Merged
wpoely86 merged 3 commits intoeasybuilders:developfrom
boegel:took_time
Aug 14, 2019
Merged

enhance 'completed' message with how much time was needed the installation#2956
wpoely86 merged 3 commits intoeasybuilders:developfrom
boegel:took_time

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Aug 6, 2019

Before:

== COMPLETED: Installation ended successfully

now:

== COMPLETED: Installation ended successfully (took 5 sec)

For long-running builds the time will be shown in minutes/seconds or hours/minutes/seconds (not days), see example outputs in tests.

@boegel boegel added this to the 3.x milestone Aug 6, 2019
@boegel boegel requested a review from vanzod August 6, 2019 13:14
Comment thread test/framework/options.py Outdated
Comment thread test/framework/options.py Outdated
Comment thread test/framework/options.py Outdated
Comment thread test/framework/options.py Outdated
if not isinstance(delta, datetime.timedelta):
raise EasyBuildError("Incorrect value type provided to time2str, should be datetime.timedelta: %s", type(delta))

delta_secs = delta.days * 3600 * 24 + delta.seconds + delta.microseconds / 10**6
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

total_seconds() ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A py2.7 only, nevermind

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jup, indeed... :)

@wpoely86 wpoely86 merged commit b228031 into easybuilders:develop Aug 14, 2019
@boegel boegel deleted the took_time branch August 14, 2019 08:36
@migueldiascosta migueldiascosta modified the milestones: 3.x, 3.9.4 Aug 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants