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

CLOUD-724: copy_out_guest_install_log #57

Open
wants to merge 8 commits into
base: release/v1.7.0
Choose a base branch
from

Conversation

NorseGaud
Copy link
Member

Pull Request

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Build related changes
  • Documentation content changes
  • Other: (describe here)

Proposed Change

Before VM deletion/cleanup, copy out the /var/log/install.log file from the guest into the CWD on the host.

Breaking Change?

  • Yes
  • No

Checklist

Please validate that your PR fulfills the following requirements:

  • I have read the CONTRIBUTING.md documentation
  • I have branched from the master branch
  • I have checked that a similar PR does not exist or has been rejected in the past
  • PR changes are specific to a feature or bug
  • No style/format/cosmetic changes included
  • I have used existing style and naming patterns in my changes
  • Tests for the changes have been added/updated (if relevant)
  • Built/compiled and tested locally
  • Docs have been added/updated (if relevant)
  • I have squashed my changes into a single commit

Additional Information

Requires CLOUD-689

tduffield and others added 6 commits November 25, 2020 09:15
`anka cp` has been show to be equally performant as mount+cp. This
ensures that even if we don't have mount capability, we can still
upload files and build them.

Tested with Anka Build 2.3.0 beta and Big Sur 11.0.1

Additional tweaks:
  * Save the install.log to disk prior to exiting

Signed-off-by: Tom Duffield <tom@chef.io>
Signed-off-by: Tom Duffield <tom@chef.io>
CLOUD-714: base vm wasn't receiving resource modifications due to missing options
@NorseGaud NorseGaud changed the base branch from master to release/v1.7.0 December 31, 2020 17:28
@NorseGaud NorseGaud self-assigned this Dec 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants