summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Break out the gpg key validation into its own methodapt_repo_cleanupTim Smith2018-02-262-8/+38
| | | | | | This logic was entirely different than the key validation and doing them in the same block didn't make any sense. This makes it a lot easier to follow as well since there isn't the cmd variable that requires searching the code to see what we're actually running Signed-off-by: Tim Smith <tsmith@chef.io>
* Update specsTim Smith2018-02-261-30/+33
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Yard improvementsTim Smith2018-02-261-5/+27
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Don't check that we're a PPA URL twiceTim Smith2018-02-261-1/+0
| | | | | | We check that we're a PPA URL before calling the method. No need to check again within the method. Signed-off-by: Tim Smith <tsmith@chef.io>
* Test is_ppa_url and repo_componentsTim Smith2018-02-261-0/+29
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Remove "should" wordingTim Smith2018-02-261-6/+6
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Test cookbook_name and key_type methodsTim Smith2018-02-261-1/+23
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* More YARDTim Smith2018-02-261-0/+21
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Add debugging for when we do nothing on the remove actionTim Smith2018-02-261-1/+2
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Extract repo components logic into its own methodTim Smith2018-02-261-7/+15
| | | | | | Same old: Easier to read, easier to test Signed-off-by: Tim Smith <tsmith@chef.io>
* More YARDTim Smith2018-02-261-0/+4
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Split building the key import command into into its own methodTim Smith2018-02-261-2/+11
| | | | | | Easier to follow and easier to test Signed-off-by: Tim Smith <tsmith@chef.io>
* Add a debug line for skipping key importTim Smith2018-02-261-1/+3
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Add some commentsTim Smith2018-02-261-0/+13
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Collapse the not_if formatTim Smith2018-02-261-3/+1
| | | | | | Much easier to read Signed-off-by: Tim Smith <tsmith@chef.io>
* Break key type detection into its own methodTim Smith2018-02-261-8/+18
| | | | | | | A) Method was getting big B) We can test this now Signed-off-by: Tim Smith <tsmith@chef.io>
* Remove unnecessary run_command on the shell_outTim Smith2018-02-261-2/+0
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 14.0.84 by Expeditorv14.0.84Chef Expeditor2018-02-265-11/+12
|
* Merge pull request #5972 from nathwill/surface-guard-interpreter-errorsThom May2018-02-261-1/+3
|\ | | | | surface default guard interpreter errors
| * surface guard interpreter errors for debuggingNathan Williams2017-07-231-1/+3
| | | | | | | | Signed-off-by: Nathan Williams <nath.e.will@gmail.com>
* | Bump version to 14.0.83 by Expeditorv14.0.83Chef Expeditor2018-02-265-11/+12
| |
* | Merge pull request #6899 from chef/descriptions_v2Tim Smith2018-02-2624-52/+71
|\ \ | | | | | | Add more introduced and description fields to resources
| * | Add back the identity_attrTim Smith2018-02-261-0/+3
| | | | | | | | | | | | | | | | | | Turns out this is going to cause issues Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add more introduced and description fields to resourcesdescriptions_v2Tim Smith2018-02-2224-53/+69
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #6877 from chef/essentially_builtThom May2018-02-263-0/+148
|\ \ \ | | | | | | | | Add build_essential resource from the build-essential cookbook
| * | | Add build_essential resource from the build-essential cookbookessentially_builtTim Smith2018-02-213-0/+148
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This has the following modifications from the cookbook: Solaris 10 no longer warns since we don't support Solaris 10. We assume 11 Windows no longer includes the windows recipe. This really should not have been added since build-essential concept doesn't translate there macOS xcode setup is done inline vs. including the xcode command line setup resource from build-essential cookbook. It's a single execute resource and this seemed pretty silly Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Bump version to 14.0.82 by Expeditorv14.0.82Chef Expeditor2018-02-265-14/+15
| | | |
* | | | Merge pull request #6904 from chef/cronThom May2018-02-261-66/+10
|\ \ \ \ | | | | | | | | | | Convert parts of cron resource to use properties
| * | | | Convert parts of cron resource to use propertiescronTim Smith2018-02-221-66/+10
| | |/ / | |/| | | | | | | | | | | | | | | | | | Modernize this resource a bit. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Bump version to 14.0.81 by Expeditorv14.0.81Chef Expeditor2018-02-235-13/+14
| | | |
* | | | Merge pull request #6911 from chef/lcg/revert-lazy-attributesLamont Granquist2018-02-238-363/+153
|\ \ \ \ | | | | | | | | | | revert lazy attributes
| * | | | Revert "Per-container deep merge caching"lcg/revert-lazy-attributesLamont Granquist2018-02-238-309/+152
| | | | | | | | | | | | | | | | | | | | This reverts commit 0c29acc106ca774e230c8ef45694c8bffd166b69.
| * | | | Revert "turbo attribute merging"Lamont Granquist2018-02-231-16/+1
| | | | | | | | | | | | | | | | | | | | This reverts commit 5b17173847fe3e6d2a976fecad065952c801522e.
| * | | | Revert "turboize arrays"Lamont Granquist2018-02-231-36/+10
| | | | | | | | | | | | | | | | | | | | This reverts commit e24565b571e97275d39fcf7c97afc7dd6e1a37c4.
| * | | | Revert "remove comments + debugging"Lamont Granquist2018-02-232-0/+3
| | | | | | | | | | | | | | | | | | | | This reverts commit 9fe57c38a57135c7bd29ff05a1c7839a4d30c38c.
| * | | | Revert "remove more debugging and revert no-longer necessary fix"Lamont Granquist2018-02-231-2/+4
| | | | | | | | | | | | | | | | | | | | This reverts commit 80b9d0448d9fb01a7188cbee6c9595bf0cf5b974.
| * | | | Revert "fix node assignment of ImmutableArrays to VividMashes/AttrArrays"Lamont Granquist2018-02-233-18/+1
|/ / / / | | | | | | | | | | | | This reverts commit e56f8d58b80334256085907961aac9ca9f56f125.
* | | | Bump version to 14.0.80 by Expeditorv14.0.80Chef Expeditor2018-02-235-11/+12
| | | |
* | | | Merge pull request #6901 from chef/powershell_package_nameThom May2018-02-231-7/+1
|\ \ \ \ | | | | | | | | | | Simplify how we define the powershell_package resource_name
| * | | | Simplify how we define the powershell_package resource_namepowershell_package_nameTim Smith2018-02-221-7/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | Use our DSL Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Bump version to 14.0.79 by Expeditorv14.0.79Chef Expeditor2018-02-235-11/+12
| | | |
* | | | Merge pull request #6905 from chef/mdadmThom May2018-02-231-81/+9
|\ \ \ \ | | | | | | | | | | Modernize mdadm
| * | | | Modernize mdadmmdadmTim Smith2018-02-221-81/+9
| |/ / / | | | | | | | | | | | | | | | | | | | | Use properties throughout instead of set_and_return Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Bump version to 14.0.78 by Expeditorv14.0.78Chef Expeditor2018-02-235-14/+15
| | | |
* | | | Merge pull request #6908 from chef/macos_serviceThom May2018-02-231-21/+3
|\ \ \ \ | |/ / / |/| | | Modernize macosx_service
| * | | Modernize macosx_servicemacos_serviceTim Smith2018-02-221-21/+3
|/ / / | | | | | | | | | | | | | | | Use properties throughout instead of set_or_return Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Bump version to 14.0.77 by Expeditorv14.0.77Chef Expeditor2018-02-225-11/+12
| | |
* | | Bump version to 14.0.76 by Expeditorv14.0.76Chef Expeditor2018-02-225-11/+12
| | |
* | | Bump version to 14.0.75 by Expeditorv14.0.75Chef Expeditor2018-02-225-11/+12
| | |
* | | Bump version to 14.0.74 by Expeditorv14.0.74Chef Expeditor2018-02-225-11/+12
| | |