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

Adding VBox and Vagrant max versions #1356

Closed
wants to merge 1 commit into from

Conversation

bmedx
Copy link
Contributor

@bmedx bmedx commented Nov 29, 2016

Ran into some issues with setting up devstack on the most recent Vagrant and VirtualBox versions. Looks like they are known issues, so just updating the docs to clarify working versions of those requirements.

[OPEN-1658] (https://openedx.atlassian.net/browse/OPEN-1658)

Reviewers

  • Doc team review (sanity check, copy edit, or dev edit?): @edx/doc
  • Product review: @jmbowman

Testing

  • Ran ./run_tests.sh without warnings or errors

Post-review

  • Add a comment with the description of this change or link this PR to the next release notes task.
  • Squash commits

Copy link
Contributor

@jmbowman jmbowman left a comment

Choose a reason for hiding this comment

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

Looks good to me. I did find https://openedx.atlassian.net/browse/OPEN-1658 , was that the problem you hit? If so, it may be worth referencing the ticket here

@bmedx
Copy link
Contributor Author

bmedx commented Nov 29, 2016

@jmbowman Ah nice, added to the PR and I'll comment on the issue referencing back here.

@nedbat
Copy link
Contributor

nedbat commented Nov 29, 2016

@bmedx are you sure Vagrant 1.8.7 doesn't work? I know 1.8.5 didn't, but thought that .7 had a fix.

@bmedx
Copy link
Contributor Author

bmedx commented Nov 29, 2016

@nedbat I had tested it both ways, but just re-ran with 1.8.7 and it worked. I don't have the logs around to see what happened, unfortunately, perhaps it was some other transient error. Vagrant changelogs don't seem to mention the issue I was seeing, which had to do with libcurl versioning, so I can't identify which version(s) may have been broken. Aside from 1.8.5 do you know of any others? I'll update this change tomorrow to be more accurate.

@nedbat
Copy link
Contributor

nedbat commented Nov 30, 2016

@bmedx OK, the libcurl problem is known in 1.8.7, and we've been solving it manually: hashicorp/vagrant#7997 . These guys went so far as to add a warning to their Vagrantfile: zulip/zulip@287eb2c

@bmedx
Copy link
Contributor Author

bmedx commented Dec 7, 2016

@nedbat Are you suggesting we add that information to the docs? I'm not sure what the right level of detail to go for there is.

@nedbat
Copy link
Contributor

nedbat commented Dec 7, 2016

Honestly, I'm not sure what the right recommendations are. I don't know what problems people have had with Virtualbox 5.1. I know I had a problem the other day that was only solved by upgrading to 5.1, so it all feels really really squishy and I hate it :)

@catong
Copy link
Contributor

catong commented Aug 29, 2017

@nedbat have you been able to confirm the supported versions?

@bmedx
Copy link
Contributor Author

bmedx commented Aug 29, 2017

I imagine that with the move to Docker this is no longer relevant, or at least won't be soon. I'm going to close it out.

@bmedx bmedx closed this Aug 29, 2017
@bmedx bmedx deleted the bmedx/install_prerequesites_update branch August 29, 2017 19:22
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