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

Use islandora/8 to pull a fully built box from vagrant cloud #179

Merged
merged 2 commits into from
May 5, 2020

Conversation

dannylamb
Copy link
Member

GitHub Issue: Islandora/documentation#1494

What does this Pull Request do?

I'm uploading a box to vagrant cloud as islandora/8. This PR points to this box by default, so that when people hit vagrant up, it will download a fully built box and "just work", as opposed to try and provision with ansible. If you decare the islandora_distro variable to be either centos/7 or ubuntu/bionic64, you'll get the usual dev build.

How should this be tested?

I'll comment on this issue when the upload to vagrant cloud has finished. Once that's done, pull in this PR and vagrant up. Vagrant should download the islandora/8 box and boot up, without provisioning.

Make sure you can vagrant ssh into the box and give it a whirl.

It's the 1.1.0 build, FYI.

Interested parties

@DonRichards @wgillingham @Islandora-Devops/committers

@DonRichards
Copy link
Member

Just started the process and it's pulling https://vagrantcloud.com/islandora/boxes/8/versions/1.1.0/providers/virtualbox.box
I'd review this but I don't have permissions to approve on the repo

@dannylamb
Copy link
Member Author

@DonRichards I'm re-uploading a new box. That one is fine and should totally work, but you can't vagrant ssh into it. I added Vagrant's "insecure key" to the box that's getting uploaded now and will let you know when it's done.

I'll also see what it takes to get you to approve.

@dannylamb
Copy link
Member Author

@DonRichards I sent you an invite to the islandora-devops org. You should have approval permissions once you accept.

@DonRichards
Copy link
Member

Odd problem Screen Shot 2020-04-27 at 11 19 42 AM
Something is hard coded in the VM "/home/daniel/Code/Environments/islandora-playbook/ubuntu-bionic-18.04-cloudimg-console.log"

@dannylamb
Copy link
Member Author

ah dangit, we have the same issue when we pass around ova's

@dannylamb
Copy link
Member Author

@DonRichards I uploaded a new version of the base box overnight and removed the serial port / logging stuff that I know causes the issue you ran into. Want to try it again? You'll have to delete or update the old base box in order to get the newest one.

@DonRichards
Copy link
Member

I'll put my finding here as well

# This line is a suggestion but doesn't impact this. Increases the timeout to 30 minutes.
config.vm.boot_timeout = 1800


config.vm.provider "virtualbox" do |vb|
    ...
     # These 2 are needed to disable the serial port
    vb.customize ["modifyvm", :id, "--uart1", "0x3F8", "4"]
    vb.customize ["modifyvm", :id, "--uartmode1", "disconnected" ]

@DonRichards
Copy link
Member

I ran the following

$ vagrant destroy -f
$ vagrant box remove islandora/8
$ git pull
# switch to PR-179
$ time vagrant up

....
   vagrant up  4.88s user 1.88s system 13% cpu 50.410 total

It took 50 seconds to build. And looks good.

@DonRichards DonRichards self-requested a review April 30, 2020 13:42
Copy link
Member

@DonRichards DonRichards left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected.

@DonRichards DonRichards merged commit 21cb8a3 into Islandora-Devops:dev May 5, 2020
@mjordan
Copy link
Contributor

mjordan commented May 6, 2020

I'd like to see instructions on how to build a dev box ("If you decare the islandora_distro variable to be either centos/7 or ubuntu/bionic64, you'll get the usual dev build.") in the README. Building a 1.1.0 box by default is awesome but devs might not want that. I am happy to open a PR.

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.

3 participants