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

Vagrant A2 server and instructions for restoring from Chef Infra Server to A2 with Embedded Infra #149

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

smackmybitsup
Copy link

Vagrant A2 server and instructions for restoring from Chef Infra Server to A2 with Embedded Infra

Description

  • instructions and links to helpful resources
  • create Vagrant A2 Server
  • install needed software for restore
  • perform restore
  • validate

Related Issue

may help with #78

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

  • [ x] I have read the CONTRIBUTING document.
  • [ x] I have run the pre-merge tests locally and they pass.
  • [x ] I have updated the documentation accordingly.
  • [ x] I have added tests to cover my changes.
  • [ x] All new and existing tests passed.
  • [ x] All commits have been signed-off for the Developer Certificate of Origin.

@smackmybitsup smackmybitsup requested a review from a team as a code owner October 22, 2020 16:24

1. Put a copy of your .chef folder from your workstation onto the Vagrant A2 server

> ⚠️ PLEASE be CAREFUL: understand that at this point your `knife.rb` to points to your current **Chef Infra server**. ⚠️

Choose a reason for hiding this comment

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

It would be great to explain what will happen if you do this wrong.

Copy link

@kagarmoe kagarmoe left a comment

Choose a reason for hiding this comment

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

I have a couple of very minor comments. The process you describe looks plausible, but I wasn't able to try it out.


config.vm.box = "bento/ubuntu-18.04"
config.vm.synced_folder ".", "/opt/a2-testing", create: true
config.vm.hostname = 'learn-chef.auto'

Choose a reason for hiding this comment

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

Unless we're spinning up the learn chef site, let's use a different hostname.

require File.expand_path(
"rgloader/loader", ENV["VAGRANT_INSTALLER_EMBEDDED_DIR"])
else
raise "Encoded files can't be read outside of the Vagrant installer."

Choose a reason for hiding this comment

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

Would be great to add what the user should do next. The passive voice ("files can't be read") left me confused as to what is reading the files and what I need to do to fix the problem. Still, any error message is better than no error message, so thanks!

Suggested change
raise "Encoded files can't be read outside of the Vagrant installer."
raise "The Vagrant installer can't read outside encrypted files"

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.

2 participants