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

Trying with docker tmpfs mount #1254

Merged
merged 6 commits into from
Jul 17, 2024

Conversation

mira-miracoli
Copy link
Contributor

Should be only a temporary solution, because the tmp in docker is written to memory.

Should be only a temporary solution, because the tmp in docker is written to memory
@sanjaysrikakulam sanjaysrikakulam dismissed their stale review July 9, 2024 07:35

Require changes

set the tmpfs size to 2GiB
Copy link
Member

@sanjaysrikakulam sanjaysrikakulam left a comment

Choose a reason for hiding this comment

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

Missing inherits: basic_docker_tool in toolshed.g2.bx.psu.edu/repos/iuc/funannotate_annotate/funannotate_annotate/.*: and the scheduling section needs to be removed if we want to run the tool with docker

Copy link
Member

@sanjaysrikakulam sanjaysrikakulam left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@mira-miracoli mira-miracoli marked this pull request as ready for review July 17, 2024 13:25
@mira-miracoli mira-miracoli merged commit 2711b7b into usegalaxy-eu:master Jul 17, 2024
4 checks passed
@mira-miracoli mira-miracoli deleted the tmp-tmp-fix branch July 17, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants