summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix svcadm clear to only run in maintenance statejsinha/fix_svcadmJaymala Sinha2017-12-072-5/+11
| | | | | Signed-off-by: Jaymala Sinha <jsinha@chef.io> Signed-off-by: Patrick Wright <patrick@chef.io>
* Merge pull request #6620 from chef/improved_task_loggingTim Smith2017-12-073-45/+49
|\ | | | | Additional logging improvements to windows_task
| * Improve the passwordless user error, logic, and testsimproved_task_loggingTim Smith2017-12-063-21/+25
| | | | | | | | | | | | Specify exactly what we're checking for in the error. This needs to go into the docs site as well. Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update resource specsTim Smith2017-12-061-14/+14
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Attribute -> Property and remove double spacesTim Smith2017-12-061-10/+10
| | | | | | | | | | | | These are properties not attributes. Signed-off-by: Tim Smith <tsmith@chef.io>
| * Clarify that we're talking about the 1 and only SYSTEM user hereTim Smith2017-12-063-3/+3
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #6625 from chef/lcg/lwrp-test-speedupLamont Granquist2017-12-065-35/+56
|\ | | | | speedup lwrp tests
| * travis caught another speclcg/lwrp-test-speedupLamont Granquist2017-12-061-1/+1
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * speedup lwrp testsLamont Granquist2017-12-064-34/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | reset the global resource and provider handler maps before each and every test. note that mutation through the top-level keys will get preserved so this is not perfect, but deep-duping through every single test we run becomes expensive. this is sufficient to flush out the global state of all the test lwrp fixtures that we setup. it discovered several usage bugs where we either relied on this feature or where we were testing different fixtures from the ones we thought we were. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Merge pull request #6424 from chef/lcg/deep-merge-cache-rewriteLamont Granquist2017-12-069-159/+355
|\ \ | | | | | | Node attributes per-container lazy deep merge and deep merge performance improvements
| * | fixup some unit testslcg/deep-merge-cache-rewriteLamont Granquist2017-12-061-6/+7
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | remove more debugging and revert no-longer necessary fixLamont Granquist2017-12-061-4/+2
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | remove comments + debuggingLamont Granquist2017-12-062-3/+0
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | turboize arraysLamont Granquist2017-12-061-10/+36
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | turbo attribute mergingLamont Granquist2017-12-061-1/+16
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | Per-container deep merge cachingLamont Granquist2017-12-068-152/+311
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replaces the one-big top level deep merge cache with individual deep merge caches in every container. The Immutable container state becomes the deep merge cache. - Does not use a pure decorator style approach since that failed before because of ruby internals breaking things like `===` and `=~` on decorated objects, so we inherit (ultimately from Hash + Array). - The state being the container state is useful in ruby since APIs on Hash and Array poke around in internal state to make things fast. If we inherit from Hash/Array but don't have the correct internal state things go wonky. - Throwing away the internal state is equivalent to flushing the cache. - Since we throw away all linked objects when we do that, we flush at every level below the level being flushed (which is correct semantics). - If a user has a pointer to an old immutable object from a sub-level, that isn't mutated so the old object still contains the old view of the data (which I think is correct, although I have some doubts that its necessary, but it came along free for the ride). - When we reset the cache we do mutate the cache being reset, which might change data in held references. If this becomes an issue the fix would be to reset the cache at the level above by creating a new, "empty" ImmutableHash/ImmutableArray object and inserting it into the deep_merge_cache datastructure instead of clearing the internal state of the child object. I don't know practically how anyone would hit this, though, so would prefer to wait on doing that work until we see an actual bug report. - Because of the way ruby pokes around internally there's some weirdnesses like the pre-generation of the cache for all the values of a subarray when #each is called, which is due to the way that ruby walks through array-serialized hashes when called like: `Array#each { key, value| ... }` (which is an undocumented(?) thing in ruby). Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Fixup the functional testsTim Smith2017-12-061-2/+2
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 13.6.26 by Expeditorv13.6.26Chef Expeditor2017-12-065-12/+12
|
* Merge pull request #6617 from chef/improved_task_loggingThom May2017-12-061-23/+29
|\ | | | | Improved windows_task logging
| * Use converge_by to perform actual resource updatesTim Smith2017-12-051-19/+19
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Clarify why we're not doing anything hereTim Smith2017-12-051-2/+2
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add debug logging for the task existingTim Smith2017-12-051-0/+6
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Fix typos that prevented loggingTim Smith2017-12-051-2/+2
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 13.6.25 by Expeditorv13.6.25Chef Expeditor2017-12-055-10/+12
| |
* | Merge pull request #6616 from chef/new_ohaiTim Smith2017-12-052-5/+17
|\ \ | |/ |/| Update InSpec to 1.47 and Ohai to 13.7
| * Update InSpec to 1.47 and Ohai to 13.7Tim Smith2017-12-052-5/+17
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Update CHANGELOG.md with details from pull request #6614Chef Expeditor2017-12-051-4/+4
|
* Merge pull request #6614 from chef/opensuse_testingTim Smith2017-12-048-5/+41
|\ | | | | Add openSUSE testing in Travis & expand cookbooks we test
| * Make sure we install sudo on opensuseTim Smith2017-12-041-0/+2
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Fix the name of the opensuse kitchen boxTim Smith2017-12-041-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Chefstyle fixTim Smith2017-12-041-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove locale from the kitchen tests for nowTim Smith2017-12-043-5/+0
| | | | | | | | | | This cookbook needs some work Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add opensuse testing to TravisTim Smith2017-12-011-0/+18
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Exclude non-learnchef platforms from awesome chef cookbooksTim Smith2017-12-012-0/+4
| | | | | | | | | | | | There's no expectation that this will ever run on opensuse, amazon, or fedora Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add git and locale cookbooksTim Smith2017-12-014-1/+15
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add the cron cookbookTim Smith2017-12-012-0/+3
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Don't include the ubuntu cookbook on DebianTim Smith2017-12-011-4/+3
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add back the sysadmin group for sudoTim Smith2017-12-011-0/+1
| | | | | | | | | | | | We're adding the sysadmin group in the users cookbook. The update to the sudo cookbook took this out. Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove telnet for testing as it break opensuse for odd reasonsTim Smith2017-12-011-1/+1
|/ | | | | | | | | | | | | | | | It's not a chef thing. It's an SUSE thing. Reading installed packages... Resolving package dependencies... Problem: conflicting requests Solution 1: remove lock to allow installation of telnet-1.2-170.3.x86_64[openSUSE-Leap-42.3-Oss] Solution 2: remove lock to allow installation of telnet-1.2-170.3.x86_64[openSUSE-Leap-42.3-0] Solution 3: do not ask to install a solvable providing telnet.x86_64 = 1.2-170.3 Choose from above solutions by number or cancel [1/2/3/c] (c): Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 13.6.24 by Expeditorv13.6.24Chef Expeditor2017-12-015-10/+12
|
* Merge pull request #6590 from mal/knife-ssh-prefixLamont Granquist2017-12-013-80/+168
|\ | | | | Knife SSH prefix option
| * Add custom prefix attribute support to knife sshMal Graty2017-11-283-20/+131
| | | | | | | | Signed-off-by: Mal Graty <mal.graty@googlemail.com>
| * Refactor SSH attributeMal Graty2017-11-283-60/+37
| | | | | | | | Signed-off-by: Mal Graty <mal.graty@googlemail.com>
| * TyposMal Graty2017-11-282-2/+2
| | | | | | | | Signed-off-by: Mal Graty <mal.graty@googlemail.com>
* | Update CHANGELOG.md with details from pull request #6611Chef Expeditor2017-12-011-3/+4
| |
* | Merge pull request #6611 from chef/amazon_testingTim Smith2017-12-015-126/+267
|\ \ | | | | | | Add Amazon Linux testing to PRs in Travis
| * | Remove the old amazon travis defamazon_testingTim Smith2017-12-011-18/+0
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Use the new amazon compatible openssh cookbookTim Smith2017-11-301-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add Amazon Linux kitchen run to TravisTim Smith2017-11-301-0/+18
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Update the test depsTim Smith2017-11-301-93/+227
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
View Lorry Controller Status