-
Notifications
You must be signed in to change notification settings - Fork 8
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
using snakedeploy with repos not on github #26
Comments
@nate-d-olson I don't see that this is publicly available - do you have a public repo at the same url to test? If it's an issue of permissions we will need to add that. If it's an issue of parsing the gitlab url we can test that, but I'd like to figure out if it's just that it's private first. If it's the URL, then what I'll need to help and test is a public GitLab repository to develop from. |
@vsoch Thank you for the quick response. Unfortunately, all the repositories or https://gitlab.nist.gov/gitlab, are not publically accessible as this gitlab instance is hosted on an internal server. I am happy to help update and test the codebase so that the tool works with similar internal gitlab servers. Using a local file path works for us as well and maybe easier to implement and test and useful to more users. |
okay - I can't work on this during the workday but let me give a shot at adding a GitLab provider. It might be that simple! I should have some time this evening. |
See #27 |
Hi,
All the syntaxes above generate the same error message (displayed first by @nate-d-olson)
Best, |
Ah I don't think we parse local repos - that can be added! Could someone please test #27 before I do another PR? |
Ok sorry @vsoch, I based my comment on the first message of @nate-d-olson which mentioned local repos. Unfortunately I do not have a gitlab repo to test #27, when I use the original code from @nate-d-olson with your PR (snakedeploy v0.3.0+1.g6e83087):
I do not have the access as well so ended up with a time out:
When used locally, I have the error with |
@cpauvert it's no worry! I think we need @nate-d-olson to test since he has access to clone the repo, and if we need to add credentials in there somewhere we can discuss that! This is an extra project I work on in free time so I move a bit serially with respect to feature request -> testing -> merge (and then onto the next!) |
@vsoch and @cpauvert https://gitlab.nist.gov/gitlab/njd2/giab-asm-bench-whole-genome is behind an internal firewall and password protected. This is probably a rare use case. I am happy to help with testing but think implementing functionality for using snakedeploy with local repositories would help more snakemake deploy users. Thanks for your help! |
@nate-d-olson could you point me on an open GitLab repo to test? I don't really use GitLab beyond a few repos. |
@vsoch sorry for the back and forth. I cannot create an open GitLab.nist.gov repository. I set up a free gitlab account and want to set up a snakemake repo for testing can you recommend a simple snakemake pipeline repository for me to fork for use in testing snakedeploy? |
Sure thing, and that would be so great! So there is the workflow catalog that has a ton: https://snakemake.github.io/snakemake-workflow-catalog/ What I usually do is use the default tutorial on the website. What you can do that might be easiest is to start with my spack + snakemake tutorial: https://github.com/snakemake/snakemake-spack-tutorial but:
You can also just start from the tutorial repo and grab that same Snakefile: https://github.com/snakemake/snakemake-tutorial-data |
@vsoch Do I open a new issue regarding the local repos to not mess up the work and discussions here? |
A local repo is much easier to test so I can probably just take a shot at this point and open another PR - will add to my TODO list for the weekend. |
@vsoch here is the repo I set up for testing, https://gitlab.com/natedolson/snaketestworkflow. I am happy to run some tests myself or help you setup a testing framework using this example public gitlab repo. Thanks! |
perfecto! I'll work on these both this evening and weekend. Happy Friday y'all! |
@nate-d-olson can you double check if that repo is public? It's asking me to login to see it! |
Sorry, the gitlab repos are private by default. I recreated the repo as public, let me know if you are still unable to access it. https://gitlab.com/nate-d-olson/snaketestworkflow.git |
okay I see it now! Thank you! |
hey @nate-d-olson ! Okay I think I'm almost done, but I did mess up in my instructions to you. I'm looking at the https://snakemake.github.io/snakemake-workflow-catalog/ and I think an "official" snakemake workflow has a workflow/ and config/ directory - that seems to be the pattern we expect when we generate the deployment. I'm wondering if snakedeploy could also support a Snakefile in the root - @johanneskoester what do you think? |
I don't see any reason we couldn't find the Snakefile in the repo and customize the path for that - I'm going to give it a shot and see what happens :) |
Okay so if I'm understanding this correctly, snakedeploy is only going to work for workflows for which the data / assets outside of the snakefile and config are at remotes (e.g., no local data directories). I guess I'm wondering what the advantage is over just a clone and then run with snakemake? |
okay refactor is done for GitLab and local #27 but we do need a bit of discussion about the details. We probably need to pull in @johanneskoester for some of the discussion - if he doesn't hop into discussion by next weekend I'll ping him via email. |
IMHO, you can use a whole chunk of the workflow and include it in your own workflow in a modular way to do something with the output of the worklow without having to have all the directory/file structure (envs, schemas, scripts, rules etc.). But it depends on the case I guess. |
Description
Would like to use snakedelopy with either a local git repository or an internal gitlab server.
I get the following error that the object has no attribute
get_repo_name
I understand that this repo is under activate development and you might not be able to support this functionality at this time.
What I Did
When I tried to use the gitlab url
When I tried to use a local path
The text was updated successfully, but these errors were encountered: