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

No Ubuntu Required #41

Merged
merged 1 commit into from
Dec 10, 2017
Merged

Conversation

whikloj
Copy link
Member

@whikloj whikloj commented Dec 7, 2017

  • Remove dependency on ubuntu user
  • Fix Ubuntu bootstrap

https://www.youtube.com/watch?v=r0qBaBb1Y-U

Fix Ubuntu bootstrap
@MarcusBarnes
Copy link

Appears to build successfully. The Islandora namespace in Blazegraph was not selected by default, but this may not related to the changes in this pull-request.

@whikloj Any other things I should check for when testing and reviewing this pull-request? Thank you in advance.

@whikloj
Copy link
Member Author

whikloj commented Dec 8, 2017

Not really, this was just to ensure that the system worked without relying on the /home/ubuntu directory.

@dannylamb
Copy link
Member

@MarcusBarnes Turns out setting that default namespace for the web UI is a bit of an ordeal: Islandora/documentation#536

We settled on adding documentation around it in islandora-deprecated/claw_vagrant#54

It's probably worth adding that to claw-playbook documentation as well.

@MarcusBarnes
Copy link

@dannylamb Thanks for the clarification.

@whikloj
Copy link
Member Author

whikloj commented Dec 8, 2017

Oh 🤦‍♂️ I was thinking of @Natkeeran's work to add the islandora namespace...I wasn't thinking about it being auto-selected in Blazegraph. Yeah @MarcusBarnes that seems to be a browser/cookies thing and not easy to affect.

@Natkeeran Natkeeran merged commit d685df1 into Islandora-Devops:master Dec 10, 2017
@whikloj whikloj deleted the no-ubuntu-required branch September 24, 2021 16:21
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.

4 participants