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

add support for creating an index & using it when searching for easyconfigs #3210

Merged
merged 21 commits into from
Apr 8, 2020

Conversation

boegel
Copy link
Member

@boegel boegel commented Feb 21, 2020

With --create-index you can create an index file that significantly speeds up searching for (easyconfig) files (& patches) on slow filesystems. You can control how long the index is valid via --index-max-age (in seconds), where 1 week (604800 seconds) is the default, and 0 indicates that the index remains valid indefinitely.

If an index is available, it's used both for --search and --robot.

@boegel boegel added this to the 4.x milestone Feb 21, 2020
test/framework/filetools.py Outdated Show resolved Hide resolved
test/framework/filetools.py Outdated Show resolved Hide resolved
test/framework/filetools.py Outdated Show resolved Hide resolved
easybuild/tools/filetools.py Outdated Show resolved Hide resolved
test/framework/filetools.py Outdated Show resolved Hide resolved
test/framework/filetools.py Outdated Show resolved Hide resolved
test/framework/filetools.py Outdated Show resolved Hide resolved
easybuild/tools/filetools.py Outdated Show resolved Hide resolved
@smoors
Copy link
Contributor

smoors commented Feb 21, 2020

@boegel is there an easy way to remove the index as well?

@boegel
Copy link
Member Author

boegel commented Feb 22, 2020

@boegel is there an easy way to remove the index as well?

You mean rm? ;)

Or do you want a --remove-index?

@smoors
Copy link
Contributor

smoors commented Feb 23, 2020

I was thinking about how this is going to be used in practice.
I can imagine this feature being used a lot, but in some cases you really want to see the latest of the latest. maybe adding an option --ignore-index will do the trick?

@boegel
Copy link
Member Author

boegel commented Feb 24, 2020

@smoors Having an --ignore-index makes a lot of sense, I'll look into that...

@easybuilders easybuilders deleted a comment from coveralls Feb 29, 2020
@boegel boegel requested a review from smoors March 1, 2020 12:42
test/framework/options.py Show resolved Hide resolved
test/framework/options.py Outdated Show resolved Hide resolved
easybuild/tools/filetools.py Outdated Show resolved Hide resolved
# do not consider (certain) hidden directories
# note: we still need to consider e.g., .local !
# replace list elements using [:], so os.walk doesn't process deleted directories
# see http://stackoverflow.com/questions/13454164/os-walk-without-hidden-folders
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: use https

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

easybuild/tools/filetools.py Outdated Show resolved Hide resolved
@boegel boegel force-pushed the easyconfigs_index branch from ed0129b to b5c5b38 Compare March 8, 2020 09:35
@boegel boegel force-pushed the easyconfigs_index branch from b5c5b38 to 6bb2131 Compare March 8, 2020 09:37
@boegel boegel removed this from the 4.x milestone Mar 8, 2020
@boegel boegel added this to the next release (4.1.2?) milestone Mar 8, 2020
@easybuilders easybuilders deleted a comment from boegelbot Mar 8, 2020
test/framework/options.py Outdated Show resolved Hide resolved
smoors
smoors previously approved these changes Apr 8, 2020
Copy link
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

@smoors
Copy link
Contributor

smoors commented Apr 8, 2020

Going in, thanks @boegel!

@smoors smoors merged commit c30bedc into easybuilders:develop Apr 8, 2020
@boegel boegel deleted the easyconfigs_index branch April 8, 2020 12:23
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.

3 participants