Skip to content

VMWARE: search the deployPkg plugin in multiarch dir#1061

Merged
TheRealFalcon merged 2 commits into
canonical:mainfrom
xiaofengw1205:add-search-dir-for-vmware-customization
Oct 12, 2021
Merged

VMWARE: search the deployPkg plugin in multiarch dir#1061
TheRealFalcon merged 2 commits into
canonical:mainfrom
xiaofengw1205:add-search-dir-for-vmware-customization

Conversation

@xiaofengw1205
Copy link
Copy Markdown
Contributor

@xiaofengw1205 xiaofengw1205 commented Oct 11, 2021

Proposed Commit Message

VMWARE: search the deployPkg plugin in multiarch dir

Due to multiarch, the libdeployPkgPlugin.so is deployed into dir
/usr/lib/<multiarch name>/open-vm-tools, we need to add this path
into search_paths.

LP: #1944946

Test Steps

  1. Install open-vm-tools in multiarch Ubuntu, verify the customization could executed sucessfully on it.

Checklist:

  • [ X] My code follows the process laid out in the documentation
  • [ X] I have updated or added any unit tests accordingly
  • I have updated or added any documentation accordingly

Copy link
Copy Markdown
Contributor

@TheRealFalcon TheRealFalcon left a comment

Choose a reason for hiding this comment

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

Thanks!

@xiaofengw-vmware just double checking, this is a fix for https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1944946 , correct?

@xiaofengw1205
Copy link
Copy Markdown
Contributor Author

Thanks!

@xiaofengw-vmware just double checking, this is a fix for https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1944946 , correct?

Yes, this is the fix to bug 1944946 .Thanks James for your quick response.

@xiaofengw1205
Copy link
Copy Markdown
Contributor Author

Thanks!

@xiaofengw-vmware just double checking, this is a fix for https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1944946 , correct?

Please help to approve and merge it into master branch. Thanks.

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