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

Missing novaseq config from main.yml #36

Open
jethror1 opened this issue Mar 2, 2021 · 0 comments
Open

Missing novaseq config from main.yml #36

jethror1 opened this issue Mar 2, 2021 · 0 comments

Comments

@jethror1
Copy link
Contributor

jethror1 commented Mar 2, 2021

During the ansible set up the config in the playbook is copied to the user config file but the novaseq option is missing from here.

If monitor_runs.py is run via the cmd line and the novaseq key added manually to the config file then this is correctly added and CopyComplete.txt is picked up, but when running via Ansible this is not used and the run never completes uploading.

I'm not sure as to the logic of having a separate 'novaseq' option, could you not just check for either RTAComplete.txt or CopyComplete.txt in check_local_runs() of monitor_runs.py and not need to specify novaseq as an input? This seems simpler and would achieve the same result of handling novaseq runs.

Also this novaseq option is missing from the readme in the playbook options.

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

No branches or pull requests

1 participant