Debian / Ubuntu compability changes#1
Open
prasmussen wants to merge 5 commits into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I could not get vmtools to work on the latest ubuntu and debian releases. I've made som changes to get them to work. Tested on Ubuntu 12.04 and Debian 6.0.5. Changes explained below.
2c71461
rc.local was not installed on debian based systems. rc.local is needed to regenerate the ssh keys at first boot.
3c0654f
rc.local on Ubuntu did not regenerate the ssh keys. Now it does.
0283548
The prepare image script checks if the root password is empty. The check was done one the shadow backup file, the check should probably be done on the main shadow file. Also, Ubuntu has '!' has a password hash for the root user as default which should be accepted as an empty password.
87a577c
The networking-interfaces-config.conf script did not work on Ubuntu. As far as i could figure out, upsert scripts is run by sh, and the array expansion is invalid in sh.
c181d82
I've added a section on the metadata api in the readme.