Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign upUse external roles #43
Conversation
This comment has been minimized.
This comment has been minimized.
Let's see what happens. |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
Did you delete your roles/external directory? If not then you didn't have to re-download all of the geerlingguy roles and you only downloaded our 6 new roles. Hence not hitting the 21 limit. |
This comment has been minimized.
This comment has been minimized.
Ah, ok. Let me try that. |
This comment has been minimized.
This comment has been minimized.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Please ignore my previous comment. vagrant up works with this PR. No errors. Basic CRUD (create a resource, sync to Fedora, Triplestore) works as expected. |
This comment has been minimized.
This comment has been minimized.
Are we cool to move this in, then? |
This comment has been minimized.
This comment has been minimized.
@dannylamb Yes, Works as expected for me. In addition to this one, if 40 and 44 can be merged, then claw-vagrant can be decommissioned. |
This comment has been minimized.
This comment has been minimized.
@whikloj @Natkeeran @seth-shaw-unlv Ok, before this goes in then, let's take a look at what we've got going and see if there's other stuff we can sneak in first before I unleash some git-fu on all the repositories for the split out roles. |
This comment has been minimized.
This comment has been minimized.
Wait?! What git-fu are your doing? The plan (as I understood it) was to NOT do anything to claw-playbook until we got this merged and the external roles in use. Then I was going to start on PRs to clean up all the variable overlap that was happening in the various roles. |
This comment has been minimized.
This comment has been minimized.
So any PRs for the individual roles could be recreated on their specific repository (ie #44), but then it would be easier to test as we would actually be using the external roles here. |
This comment has been minimized.
This comment has been minimized.
@whikloj Yes, that approach seems good to me. |
This comment has been minimized.
This comment has been minimized.
I was totally going to just force push whatever changes have already rolled through to the individual repos and merge the PR after testing again. Just sense it's stuff we've already reviewed. Subsequent work would still go through the normal workflow for the applicable role(s). |
This comment has been minimized.
This comment has been minimized.
@dannylamb But you shouldn't need to force push anything. I rebuilt all the external role repositories as exact copies of what is currently in claw-playbook (I had to add the |
This comment has been minimized.
This comment has been minimized.
Just going to throw this here to tie the two discussions together. I can't get the roles to come in cleanly, same error as Jared had. I made a PR on his PR here: whikloj#1 that pulls them from git instead of galaxy and that works okay for me. I dunno if thats an okay solution for everyone. |
This comment has been minimized.
This comment has been minimized.
So this comes up fine for me after two things:
I guess I can't merge this anymore because I'm implicated in it now, but I'm satisfied and would merge this after the two PR above are merged. |
This comment has been minimized.
This comment has been minimized.
I'm going to have to rebase this PR after the FITS removal, so do we have any issues with switching the galaxy roles to Github repos? If we don't I can merge @jonathangreen's PR. Personally I'm a little concerned about the fact that Galaxy seems to be our failure point. |
whikloj
and others
added some commits
Dec 15, 2017
whikloj
force-pushed the
whikloj:issue-764
branch
from
b6df537
to
b57810e
Jan 12, 2018
This comment has been minimized.
This comment has been minimized.
@dannylamb I just went ahead and merged @jonathangreen's PR. It solves the problem for him and me (maybe a OSX issue) and now we are good to go. |
This comment has been minimized.
This comment has been minimized.
@dannylamb Can we merge this one! |
whikloj commentedDec 20, 2017
Resolves Islandora-CLAW/CLAW#764
This moves all the internal roles out to the external ones.
This has introduced a bug documented here where you (or at least I) can't download all the external roles without a failure from galaxy (or Github...which is my guess).
How To Test
vagrant up
. It will fail (or it always does for me on the 21st role)vagrant provision
You should have a working CLAW instance.