Skip to content

Inject keys into systemvm.iso using bsdtar (related to #3536)#4015

Closed
cl-k-takahashi wants to merge 2 commits into
apache:mainfrom
cl-k-takahashi:patch-injectkeys-bsdtar
Closed

Inject keys into systemvm.iso using bsdtar (related to #3536)#4015
cl-k-takahashi wants to merge 2 commits into
apache:mainfrom
cl-k-takahashi:patch-injectkeys-bsdtar

Conversation

@cl-k-takahashi

Copy link
Copy Markdown
Contributor

I fixed an issue of #3536 which was reverted on #3800

At first, systemvm_mnt dir is owned by root, this makes bsdtar unable to work.
By removing systemvm_mnt first, bsdtar can work properly.
systemvm_mnt can be removed by user cloud as /var/[lib/]cloudstack/management is owned by cloud.

Signed-off-by: Kai Takahashi k-takahashi@creationline.com

Description

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):

How Has This Been Tested?

Signed-off-by: Kai Takahashi <k-takahashi@creationline.com>
@yadvr

yadvr commented Jun 17, 2021

Copy link
Copy Markdown
Member

cc @weizhouapache if we pass the sshpublic key to VMware & XenServer systemvm/VR via the agent do we even need to patch the systemvm.iso like this at all?

@weizhouapache

Copy link
Copy Markdown
Member

cc @weizhouapache if we pass the sshpublic key to VMware & XenServer systemvm/VR via the agent do we even need to patch the systemvm.iso like this at all?

@rhtyd I think we can pass the encoded sshkey in cmdline instead of injecting it to the systemvm.iso. let's discuss later

@yadvr

yadvr commented Dec 30, 2021

Copy link
Copy Markdown
Member

Thanks @cl-k-takahashi for the PR, we've moved to a new patching approach that won't use systemvm.iso. Closing this due to the change.

@yadvr yadvr closed this Dec 30, 2021
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.

4 participants