Skip to content

Conversation

@bernt-matthias
Copy link
Collaborator

@bernt-matthias bernt-matthias commented Oct 22, 2021

most importantly this will set the galaxy dir and its files dir to read only (for dockerized testing). which allows planemo test to find cases of tools that write into galaxy's files dir.

see also here https://github.com/galaxyproject/galaxy/blob/5dc9111f45b4ed5012575b31e199a591cfad6d7a/lib/galaxy/config/sample/job_conf.xml.sample_advanced#L483

related: #1190

@bernt-matthias bernt-matthias force-pushed the topic/output_to_working_dir branch from c360320 to 0bc81f8 Compare November 15, 2021 11:09
most importantly this will set the galaxy dir and its files dir
to read only. which allows planemo test to find cases of tools
that write into galaxy's files dir.

see also here https://github.com/galaxyproject/galaxy/blob/5dc9111f45b4ed5012575b31e199a591cfad6d7a/lib/galaxy/config/sample/job_conf.xml.sample_advanced#L483
@bernt-matthias bernt-matthias force-pushed the topic/output_to_working_dir branch from 0bc81f8 to 4b699cb Compare November 15, 2021 16:04
@nsoranzo

This comment was marked as resolved.

@nsoranzo nsoranzo changed the title let Galaxy use output_to_working dir per default Enable Galaxy outputs_to_working_directory by default May 16, 2022
@nsoranzo nsoranzo merged commit 27c256b into galaxyproject:master May 16, 2022
@bernt-matthias bernt-matthias deleted the topic/output_to_working_dir branch May 16, 2022 15:11
bernt-matthias added a commit to bernt-matthias/tools-iuc that referenced this pull request Jun 21, 2022
because a .fai is written next to it

leads to a failing ci job due to galaxyproject/planemo#1199
bernt-matthias added a commit to galaxyproject/tools-iuc that referenced this pull request Jun 21, 2022
* allow 4 diff for bam files

an additional diff line is due to more processors used

xref #4490

* link fasta file

because a .fai is written next to it

leads to a failing ci job due to galaxyproject/planemo#1199
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