~harlowja/cloud-init/ensure-sudo-dir-ready : changes

~harlowja/cloud-init/ensure-sudo-dir-ready » Changes from revision 679

From Revision 679 to 660

Rev	Summary	Authors	Date
679	Ensure that the include dir starts the line Ensure that the include dir starts the line and is not a part of a comment or other part of the line.	Joshua Harlow	11 years ago
678	Update the log statement used here to be Update the log statement used here to be a little more relevant.	Joshua Harlow	11 years ago
677	Dir should be 0755, not 0440 Dir should be 0755, not 0440	Joshua Harlow	11 years ago
676	Ensure that the directory where the sudoers Ensure that the directory where the sudoers file is being added actually exists before it is written into and ensure that the directory is included in the main sudoers file.	Joshua Harlow	11 years ago
675	make DataSourceMAAS 'main()' use load_yaml make DataSourceMAAS 'main()' use load_yaml	Scott Moser	11 years ago
674	fix pep8 and pylint fix pep8 and pylint	Scott Moser	11 years ago
673	add 'safeyaml' to cloudinit add 'safeyaml' to cloudinit In 0.7.0 we started using yaml.safe_load to load data rather than yaml.load. Some producers (namely, ubuntu MAAS created) have produced cloud-config data in the past that included python unicode types. This creates a specialized safe_loader that is basically safe_load + support for python unicode.	Scott Moser	11 years ago
672	rpm spec and rpm build, better handle case with no package_m... rpm spec and rpm build, better handle case with no package_mirrors	Joshua Harlow	11 years ago
671	restart salt-minion instead of start. restart salt-minion instead of start. Packages on debian/ubuntu should start on installation. As a result, if we want to get config changes we've inserted to be read, we need to restart. Note one interesting thing here. upstart considers 'restart' as "restart only if currently running", while 'service' considers restart to be "stop if running, then start". So the use of 'service' here is important, rather than just 'restart'	Scott Moser	11 years ago
670	cleanup the user/group lists cleanup the user/group lists The primary utility here is normalize_user_groups, which would be called by config modules to get a list of users or groups. This centralizes what was copied code into this one location.	Joshua Harlow	11 years ago
669	add tests for netcfg code add tests for netcfg code	Joshua Harlow	11 years ago
668	cleanups for rhel network config cleanups for rhel network config Rework the rhel sysconfig writing/updating so that it goes through a single function which helps ensure correctness. Also write to /etc/sysconfig/network when we have written out devices to ensure that networking is on.	Joshua Harlow	11 years ago
667	handle ifup on RH distros by iterating over 'ifup devname' handle ifup on RH distros by iterating over 'ifup devname' Fix the ifup so that if a list of devices is provided then each interface is brought up individually instead of using the '--all' which isn't on rhel. The default debian behavior will be to use this still though as it overrides the new bring up interfaces function for this case.	Joshua Harlow	11 years ago
666	DataSourceMAAS: if a oauth request fails due to 403 try upda... DataSourceMAAS: if a oauth request fails due to 403 try updating local time In the event of a 403 (Unauthorized) in oauth, try set a 'oauth_clockskew' variable. In future headers, use a time created by 'time.time() + self.oauth_clockskew'. The idea here is that if the local time is bad (or even if the server time is bad) we will essentially use something that should be similar to the remote clock. This fixes LP: #978127.	Scott Moser	11 years ago
665	fix pylint fix pylint	Scott Moser	11 years ago
664	fix make pep8 fix make pep8	Scott Moser	11 years ago
663	write-ssh-key-fingerprints: do not send HOST KEYS through lo... write-ssh-key-fingerprints: do not send HOST KEYS through logger In the previous commit to htis file I had wrapped the writing of 'BEGIN SSH HOST KEY KEYS' to go through logger. This would cause the keys to be prefixed with 'ec2:' which, previously they were not. That would break existing users and make it more difficult to consume that data, which was explicitly added to be easy to consume.	Scott Moser	11 years ago
662	do not create 'sems' directory. 'sem' is proper instance/ pa... do not create 'sems' directory. 'sem' is proper instance/ path	Scott Moser	11 years ago
661	send stderr from write-ssh-key-fingerprints to stdout send stderr from write-ssh-key-fingerprints to stdout This changes all output write-ssh-key-fingerprints to go to its stdout by redirecting stderr to stdout. The reason for this is that cc_keys_to_console.py was swallowing stderr and not replaying it to /dev/console. Ideally, we'd have a way in 'util.subp' to do effectively the same thing as we're doing here in the shell script.	Scott Moser	11 years ago
660	fix permissions on /etc/sudoers.d file to be 0440 not 0644 fix permissions on /etc/sudoers.d file to be 0440 not 0644 sudo complains about the permissions on the sudoers config file that cloud-init writes: [ec2-user@ip-10-166-110-107 ~]$ sudo -s sudo: /etc/sudoers.d/90-cloud-init-users is mode 0644, should be 0440 This patch makes the file's permissions match sudo's recommendation. Note: ubuntu's sudo doesn't seem to complain about 644, but 440 is probably better.	Garrett Holmstrom	11 years ago

Older »