| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
we want attributes before methods
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|
|
|
| |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
|
|
|
|
|
|
| |
(This was always intended to be the case, and it's difficult
to do it yourself. We have been teaching people whyrun-safe
methods to make resources right alongside Custom Resources, as well.)
|
| |
|
| |
|
|
|
|
| |
chef config
|
| |
|
| |
|
|\
| |
| | |
derp fix for master breakage
|
| |
| |
| |
| |
| |
| | |
thought this was green, but it was clearly red on appveyor/windows
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|\ \
| |/
|/|
| |
| | |
MsysTechnologiesllc/dh/registry_resource_idempotent_issue
Fix for idempotent issue for some registry types in registry_key resource
|
| | |
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | | |
revert supports[:manage_home] behavior
|
| | |
| | |
| | |
| | | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |/
| |
| |
| |
| |
| |
| | |
even though i violently disagree that this is correct behavior and we're
just going to break everyone in one massive go when we hit Chef 13.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
| |
| |
| | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|/
|
|
|
|
|
| |
also fixes the fact that we've been using @new_resource.name instead of
@new_resource.path all along which was never what was intended.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
|
|\
| |
| |
| |
| | |
MsysTechnologiesllc/ali/allow_symlink_directory_delete_on_windows
Added code to handle deletion of directories on Windows that are symlinks.
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Hook up the recipe_file_loaded event which was defined but not actually called
|
| | |
| | |
| | |
| | | |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This also extends two of the recipe events to get the recipe name in addition to the path because that's
usually useful for display. These arguments are both added at the end so the auto-magic argument trimmer will keep backwards compat.
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
|
|\ \ \
| | | |
| | | | |
use threads to workaround Process.spawn buginess in chef-solo unit test
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
there's a WONTFIX closed bug in ruby-lang.org on this bug:
https://bugs.ruby-lang.org/issues/10583
they seem to be favoring speed over accuracy with the result
that if you're doing I/O synchronization (like our run_lock?)
inside of a Process.spawn chain like this that you have to
wrap the spawn calls inside of threads.
probably a best practice to wrap every spawn call with a
Thread.
this probably also explains why we had the silly-long 20
second timeout?
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
MsysTechnologiesllc/dh/add_timeout_during_package_removal
Added timeout during removing of windows package
|
| | | | |
|
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- fix the 'last' statements (my perl brain occassionally crops up
in ruby code) and replace with a raise. of course this just swaps
one exception for another, but this time its really intended to
work this way.
- swap the order of the expecations so that if we fails to see the
waiting on the other process statement, but we blow up in both
threads in the exception that was just added, we know we actually
have a legit run_lock race condition that this code just found.
does not fix this bug, just gets us a bit more information if it
reoccurs.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Added fix for spaces in profile identifiers
* Added an attempt to test spaces in identifier names
* Derp, bad merge
* Fixed osx profile spec test
|
|\ \ \
| | | |
| | | | |
rewrite linux_user provider check_lock
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
this removes the kinda brittle kinda insane parsing of the installed
rpm version. now we're just less strict about the exit 1 code and
fail on the output not being what is expected.
this also likely fixes a bug in rhel in why-run mode where shell_out!
would have failed on the user-not-found and raised an exception that
would have ended the run. the code before certainly looks buggy.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Use symbols instead of strings as keys for systemd user property related funcs
* Fix tests for provider/service/systemd for when user property is set
* Change hash in systemctl_opts to use symbols instead of strings
* Fix tests for chef/provider/systemd_unit.rb
|
|\ \ \ \
| | | | |
| | | | | |
Use upstart goal state as service status
|
| |/ / /
| | | |
| | | |
| | | | |
Solves #5204
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Some properties in custom resources may include sensitive data, such as a
password for a database server. When the Resource's state is built for use by
Data Collector or similar auditing tool, `Chef::Resource#state_for_resource_reporter`
builds a hash of all state properties for that resource and their values. This
leads to sensitive data being transmitted and potentially stored in the clear.
This change enhances properties with the ability to set an individual property
as sensitive and then have the value of that property suppressed when exporting
the Resource's state.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
userdel -r will not remove a homedir not owned by the user, have
to manually rm -rf it.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|\ \ \
| | | |
| | | | |
Rewrite linux useradd provider
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
-M is not valid for usermod
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|