Skip to content

Conversation

@cl-k-takahashi
Copy link
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>
@rohityadavcloud
Copy link
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
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

@rohityadavcloud
Copy link
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.

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