Skip to content
This repository was archived by the owner on Jun 12, 2019. It is now read-only.

Conversation

@jsnshrmn
Copy link
Member

No description provided.

…e're adding a capability flag, don't have to hack pam nologin since we're removing problematic user stuff from systemd as part of the build..
…ms are correct even if twlight commands fail.
@jsnshrmn
Copy link
Member Author

jsnshrmn commented Feb 6, 2019

this will also close #7 when it gets merged.

@jsnshrmn
Copy link
Member Author

jsnshrmn commented Feb 7, 2019

an open question for @Samwalton9 and @uyscuti-wiki:
This PR should enable docker to work while preserving the functionality for virtualbox (which I can't test).
I've updated the README to talk through the docker configuration and don't mention virtualbox at all since it's not something I can directly support currently. Does that seem reasonable? We could have both documented, but then somebody using virtualbox would have to be able to support/troubleshoot workflow issues for this.

@jsnshrmn jsnshrmn changed the title wip -- support virtualbox and docker in the same vagrantfile. Support virtualbox and docker in the same vagrantfile. Feb 7, 2019
@uyscuti-wiki
Copy link
Contributor

Sounds reasonable. A couple of questions though:

  1. does this PR alter the way we've been doing Vagrant/VirtualBox installations so far?
  2. does this affect existing local vagrant environments?

@Samwalton9
Copy link
Member

Samwalton9 commented Feb 8, 2019

Received the following error when attempting to vagrant up from this branch:

There are errors in the configuration of this machine. Please fix
the following errors and try again:

vm:
* A box must be specified.

VirtualBox Provider:
* The following settings shouldn't exist: box

@jsnshrmn
Copy link
Member Author

jsnshrmn commented Feb 9, 2019

What vagrant providers do you have installed?

@Samwalton9
Copy link
Member

How do I check that?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants