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
Updates to the Dev VM environment/tooling #3149
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
👷 Deploy Preview for chef-server processing. 🔨 Explore the source changes: e514877 🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-server/deploys/6203fdca5c078500078eed90 |
marcparadise
force-pushed
the
mp/dvm-updates
branch
from
February 7, 2022 17:29
8d4d1be
to
f92f2de
Compare
tas50
reviewed
Feb 7, 2022
tas50
reviewed
Feb 7, 2022
PrajaktaPurohit
approved these changes
Feb 8, 2022
If you need to work on an erlang project dependency (for example, you want to modify chef_authn) | ||
this can also be loaded into the VM with hot code loading support. After loading the service | ||
in the VM (eg. `dvm load oc_erchef`) , switch to the host and clone the dependency into `external-deps` | ||
directory. For example, to load `chef_authn` into a loaded and running `oc_erchef`: |
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.
Thank you for writing this up!
PrajaktaPurohit
approved these changes
Feb 8, 2022
jashaik
reviewed
Feb 9, 2022
Testing Report:
|
jashaik
approved these changes
Feb 9, 2022
This makes many changes to improve the Dev VM environment: * remove 'sync' tool, because native/direct file share with host is now performant enough on both Mac and Linux * removed unsupported scenarios and code from Vagrantfile * clarify Vagrantfile contents via renames and comments * fix project loading for erlang and ruby projects * fix project loading for cookbooks and ctl commands * fix erlang project depenency handling and hotloading * update README to reflect latest changes * install some additional tooling (tree, jq, ripgrep) * update distro to ubuntu 20.04 * remove 'quickstart' behaviors which weren't used and didn't always work very well. * set up a default org and admin user; ensure `knife` works out of the box for this configuration for root. * with the sync tool gone, remove dvmtools.rb and move the remaining code into the Vagrantfile Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
marcparadise
force-pushed
the
mp/dvm-updates
branch
from
February 9, 2022 17:45
dc40466
to
e514877
Compare
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This makes many changes to improve the Dev VM environment:
is now performant enough on both Mac and Linux
work very well.
knife
works out of thebox for this configuration for root.
code into the Vagrantfile
Signed-off-by: Marc A. Paradise marc.paradise@gmail.com