Skip to content
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

fix easyconfig + extensions progress bars when --skip is used #3882

Merged
merged 3 commits into from
Oct 28, 2021

Conversation

boegel
Copy link
Member

@boegel boegel commented Oct 28, 2021

Two problems being fixed here:

  • Already installed extensions that were filtered out were still being counted to determine the total size of the extensions progress bar
    • => fixed in 8d1d8c6 by updating the total count after skipping already installed extensions
  • Determining the total step count by checking which steps were being skipped didn't actually work, because the self.skip_step method actually only works after the check_readiness_step method has been run
    • => fixed in fe9b9b1 by not trying to exclude skipped steps when determining the size of the easyconfigs progress bar, but always updating it after a step was run or skipped

… update easyconfig progress bar), self.skip_step only really works after check_readiness_step has been run...
@boegel boegel added the bug fix label Oct 28, 2021
@boegel boegel added this to the 4.5.0 (next release) milestone Oct 28, 2021
Copy link
Member

@branfosj branfosj left a comment

Choose a reason for hiding this comment

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

Fairly minor change as skippable is no longer user.

And the tests are hitting that download issue we've been seeing today :(

easybuild/framework/easyblock.py Outdated Show resolved Hide resolved
@boegel
Copy link
Member Author

boegel commented Oct 28, 2021

And the tests are hitting that download issue we've been seeing today :(

Fix for that annoying test in #3883, hopefully relying on sources.easybuild.io will prove to be more stable than https://ftp.gnu.org...

@branfosj branfosj self-requested a review October 28, 2021 18:09
@branfosj branfosj merged commit 6d5894e into easybuilders:develop Oct 28, 2021
@boegel boegel deleted the fix_progress_bar_skipped branch October 28, 2021 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants