-
Notifications
You must be signed in to change notification settings - Fork 53
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
install for content-modeing-overhaul branches of everything #66
install for content-modeing-overhaul branches of everything #66
Conversation
Not sure if this is due to the vagrant 2.0.1 versus 2.0.3 differences, but vagrant and ansible are not able to login with this PR.
Then it seems to continue building the box, but then once it is Ansible time...
I can't run the ansible-playbook command manually as it has the same problem. I note that the port has changed from |
Also 2.1.0 is out, maybe that is a fix version, worth testing out. It is also worth resetting all the environment vars/caches, a.k.a vagrant destroy && vagrant up when you update since many of the plugins and cached sessions could be outdated? |
@whikloj Definitely should check that port. I noticed this morning I was getting the same 'Connection reset' loop when vagrant ssh'ing into a box I built last week. I just gave it a while and it eventually sorted itself out. 🤷♂️ |
Yeah it is the |
@dannylamb I'm going to hold on to this until packagist is ready and then you can uncomment those necessary lines. |
…ybook into content-modeling-overhaul
ok, i think that's all the PRs associated with this issue. let me update this PR. |
@dannylamb Once this PR is updated, would it be reasonable to try a full install (using this Ansible gear)? |
@ajs6f Yes, it should work for ubuntu with both mysql and pgsql. I've hit a minor hurdle releasing Alpaca so the artifact is available to the installer, but once that gets sorted out it should work np. |
@dannylamb Okay, cool, I'll take it out for my usual high-speed low -skill test drive on CentOS once I see a release of Alpaca (as a snapshot to Maven, I assume?). |
# Comment this back in once code gets merged and is available on packagist | ||
# - "islandora/islandora:dev-8.x-1.x" | ||
# - "islandora/openseadragon:dev-8.x-1.x" | ||
# - "islandora/islandora_image:dev-8.x-1.x" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dannylamb I think we are up-to-date in packagist. These can be uncommented now.
# Comment this back in once code gets merged and is available on packagist | ||
# - islandora_feature | ||
# - islandora_openseadragon | ||
# - islandora_image_feature |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto
Haha I think the comment
Made Github confused. |
#- name: Import features | ||
# command: "{{ drush_path }} -y fim --bundle=islandora islandora_feature,islandora_openseadragon,islandora_image_feature" | ||
# args: | ||
# chdir: "{{ drupal_core_path }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should/Can this be re-enabled?
#- name: Index tags | ||
# command: "{{ drush_path }} -y php:eval 'array_walk(\\Drupal::service('entity_type.manager')->getStorage('taxonomy_term')->loadMultiple(), function($term) { $term->save(); });'" | ||
# args: | ||
# chdir: "{{ drupal_core_path }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto re: re-enabling.
Sweet. We're end of day over here, but I'll try and push up the actual changes ASAP so we can test. |
|
Ok, so I'm updating this to make the migrations go at the very end. Everything needs to be in place so stuff can get indexed appropriately, etc... |
@@ -1,5 +1,6 @@ | |||
--- | |||
|
|||
apache_listen_ip: localhost |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding this somehow messes with the virtual host file and now root is /var/www/html instead of /var/www/html/drupal/web
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Likely a geerlingguy artefact. Any reason why we want it there in the first place? Not having it in the past hasn't caused any trouble that I'm aware of.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this will come up fine if we change this to:
apache_listen_ip: "*"
which is the default, so we could probably not set it as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I stand corrected, we do need the variable set for this PR. I'm testing @jonathangreen 's suggestion now.
To fix an existing box, goto /etc/apache2/conf-enabled/islandora.conf and
change the Virtualhost from localhost to _default_
…On Tue, 19 Jun 2018, 18:10 Jonathan Green, ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In inventory/vagrant/group_vars/webserver/apache.yml
<#66 (comment)>
:
> @@ -1,5 +1,6 @@
---
+apache_listen_ip: localhost
I think this will come up fine if we change this to:
apache_listen_ip: "*"
which is the default, so we could probably not set it as well.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#66 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACua4d39ZR4REhBdrX1RAkEDRuF8WEUgks5t-TCRgaJpZM4TvzHd>
.
|
Most recent run using
|
Yeah, using * blows up the migrations. That's where I'm stuck. I don't want to hard code localhost or 127.0.0.1 into the task, but can't seem to get the vhost to comply. At least not without postprocessing... I may just add another task to the post-install, but that just seems layering more crap on top of something already not quite working right. |
Well, we should be able to use the migrate command without the uri switch, as it is theoretically only necessary for multi-site installations. (Can we make that assumption with the playbook?) I can't test removing the option this morning (out of the office and won't make today's call) but I can look into it this afternoon if you don't beat me to it. |
Or, could we use I need to stop thinking about this away from work and my dev env... |
This is so close. We need 2 changes in the requirements.yml: - src: https://github.com/Islandora-Devops/ansible-role-alpaca
name: Islandora-Devops.alpaca
version: 0.0.2 and - src: https://github.com/Islandora-Devops/ansible-role-crayfish
name: Islandora-Devops.crayfish
version: 0.0.2 So we pull in the correct requirements from crayfish and alpaca. |
It's alive! Note: I just ran it using CentOS which requires a few version updates (ansible-role-crayfish: master, ansible-role-grok: 0.0.2, and ansible-role-karaf: master). |
I sliced a new version for ansible-role-karaf so we can update to 0.0.2 for cent. |
@jonathangreen thanks. Could you also merge islandora-deprecated/ansible-role-crayfish/pull/7 and cut a new ansible-role-crayfish version, too, while you're at it? |
Fired up into a VM and seems good (i.e loads it at 8000, syncs to feodra and indexes in triplestore). 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@seth-shaw-unlv I merged islandora-deprecated/ansible-role-crayfish#7 and sliced a release for 0.0.3. I figured I'd merge this PR since it was working for me on Ubuntu. Want to make a new PR to update the roles as we need for Cent and we'll work on getting that merged? |
Sure. I'm going to do it as part of Islandora/documentation/issues/809. I have a local branch with updated requirements I'm testing right now. |
Just spun up a fresh box and looks like islandora-connector-houdini is available in the feature but not installed in Karaf. |
Ok, I see what I need to do. In the meantime, just |
This PR is for installing all the required changes for testing Islandora/documentation#86.
A lot of it is commented out becaues it depends on code being in packagist, so once everything is tested and PRs are merged, we'll comment it back in. In the meantime, I have a gist that bridges the gap until then.