summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Workaround SSL cert verification failure in test.fix-rubygems-ssl-test-failurePete Higgins2020-07-141-0/+4
| | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* Turn off ssl certificate verification in failing test.Pete Higgins2020-07-141-0/+9
| | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* Factor out common test setup.Pete Higgins2020-07-141-9/+4
| | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* Update CHANGELOG.md with details from pull request #10154Chef Expeditor2020-07-141-0/+2
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #10154 from chef/dont-allow-expectations-on-nilTim Smith2020-07-143-5/+5
|\ | | | | Don't allow setting expectations on nil.
| * Don't allow setting expectations on nil.dont-allow-expectations-on-nilPete Higgins2020-07-133-5/+5
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Update CHANGELOG.md with details from pull request #10155Chef Expeditor2020-07-141-3/+4
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10155 from chef/fix-windows-test-warningTim Smith2020-07-141-6/+2
|\ \ | | | | | | Use rspec constant stubbing.
| * | Use rspec constant stubbing.fix-windows-test-warningPete Higgins2020-07-131-6/+2
| |/ | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Bump version to 16.3.7 by Chef Expeditorv16.3.7Chef Expeditor2020-07-147-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10140 from chef/lcg/server-api-version-to-2Lamont Granquist2020-07-131-1/+1
|\ \ | | | | | | Bump the server api version to 2
| * | Bump the server api version to 2lcg/server-api-version-to-2Lamont Granquist2020-07-091-1/+1
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | Bump version to 16.3.6 by Chef Expeditorv16.3.6Chef Expeditor2020-07-147-19/+20
| | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | Merge pull request #10146 from ↵Tim Smith2020-07-132-4/+31
|\ \ \ | | | | | | | | | | | | | | | | MsysTechnologiesllc/antima/203_windows_path_cookbook_upload_fixes Fixed `knife cookbook upload -o` windows path issue
| * | | PR review comment changesantima-gupta2020-07-142-11/+10
| | | | | | | | | | | | | | | | Signed-off-by: antima-gupta <agupta@msystechnologies.com>
| * | | Fixed windows cookbooks upload issue.antima-gupta2020-07-132-4/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated cookbook_path option used File::PATH_SEPARATOR to split path. Added rspec for upload multiple paths cookbooks. Signed-off-by: antima-gupta <agupta@msystechnologies.com>
* | | | Bump version to 16.3.5 by Chef Expeditorv16.3.5Chef Expeditor2020-07-147-19/+20
| | | | | | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | | Merge pull request #10157 from chef/fix-execute-resource-with-integer-userTim Smith2020-07-132-2/+12
|\ \ \ \ | | | | | | | | | | Fix execute resource with integer user parameter.
| * | | | Fix execute resource with integer user parameter.Pete Higgins2020-07-132-2/+12
| | |_|/ | |/| | | | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | | | Bump version to 16.3.4 by Chef Expeditorv16.3.4Chef Expeditor2020-07-147-20/+20
| | | | | | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | | Merge pull request #10156 from chef/csnapp/fix_win_hab_testTim Smith2020-07-131-1/+1
|\ \ \ \ | |/ / / |/| | | Fix extra quote in habitat test pipeline config
| * | | Fix extra quote in habitat test pipeline configChristopher A. Snapp2020-07-131-1/+1
|/ / / | | | | | | | | | Signed-off-by: Christopher A. Snapp <csnapp@chef.io>
* | | Bump version to 16.3.3 by Chef Expeditorv16.3.3Chef Expeditor2020-07-137-18/+20
| | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | Merge pull request #10102 from chef/csnapp/9282Tim Smith2020-07-1310-107/+232
|\ \ \ | | | | | | | | Test and Promote Habitat builds on Linux
| * | | add a label to the shell specscsnapp/9282Christopher A. Snapp2020-07-131-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These fail in the Habitat plan verification tests because bundle isn't on the PATH in the same way as previously discovered in the version specs in commit 3f2d235ac75. Signed-off-by: Christopher A. Snapp <csnapp@chef.io>
| * | | Modify habitat tests to call rspec the same as omnibusChristopher A. Snapp2020-07-131-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Christopher A. Snapp <csnapp@chef.io>
| * | | Enable promotion of habitat artifacts to stableChristopher A. Snapp2020-07-131-0/+1
| | | | | | | | | | | | | | | | Signed-off-by: Christopher A. Snapp <csnapp@chef.io>
| * | | Add Linux support to habitat test pipelineChristopher A. Snapp2020-07-132-4/+33
| | | | | | | | | | | | | | | | Signed-off-by: Christopher A. Snapp <csnapp@chef.io>
| * | | Add windows + linux hab tests to verify pipelineChristopher A. Snapp2020-07-136-28/+124
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes the verify/habitat pipeline by bringing windows and linux verification tests into the default verify pipeline. Signed-off-by: Christopher A. Snapp <csnapp@chef.io>
| * | | refactor Linux hab plan based on lessons from Windows planRobb Kidd2020-07-131-69/+68
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lots of things changing in here. Let's take them in a possibly weird order. * do_prepare() Move all the settng of build environment ..uh.. settings into the prepare function. Notable things: - GEM_HOME is set to a vendor directory under pkg_prefix so that all gem installs (regardless of bundle install or gem install) will be installed _there_. Expect to see all gems shipped with chef-client under this directory. - Multiple bundle config --local commands. These are executed in a subshell after changing directory to the CACHE_PATH so that the bundler configuration will be written out to a `.bundle/config` file there. bundle commands will be run from this location later, so these settings become effective for the duration of the build and obviate the need for a _bundle helper function. * pkg_bin_dirs=() Add the `vendor/bin` directory (remember, vendor/ is where gems get installed, so the binstubs created by the installation will appear here in vendor/bin/) to the PATH of this package's runtime environment because there are commands in there people sometimes want to run. They won't be fast and they might end up with conflicts with gems installed by the user or by downstream pacakges because they haven't been appbundled. * do_setup_environment() These are environment variables we'll want set to these defaults at runtime. - GEM_PATH gets this package's gem installation directory pushed onto it making the gems available at runtime regardless of whether bundle exec is used. - APPBUNDLER_ALLOW_RVM is a flag looked for by the binstubs generated by appbundler. Setting this to "true" has nothing to do with RVM and everything to do with asking the appbundled binstub to not wipe out the carefully constructed `GEM_PATH` from this package and any possible others that "publish" to the runtime environment that gems are available within. - SSL_CERT_FILE - trust the CA cert package from core, by default, but downstream packages can override this if they like with something custom. * do_build() Life is simpler in the build function now. Build environment is all setup, so here we: - bundle install to retrieve dependencies based on the Gemfile+.lock - run the rake install for the project which builds and installs the gems whose source are in this repo (chef, chef-bin, chef-config) - for any gem installed via a git reference, we change to the source directory and rake install it, too, to install it like any other gem This results in a clean collection of all the required gems installed like normal gems under "vendor/gems." With that directory on the GEM_PATH, we don't need to care about whether the gems were installed by path reference or via git. * do_install() OK. I fibbed about not needing to tell bundler anything else. For install, we're going to generate the appbundled binstubs that lockdown the versions of gems required by particular commands. appbundler uses the project's Gemfile+lock to determine those version, so for the install actions, we tell bundler via an environment variable that the Gemfile to work with is over in CACHE_PATH. Then we iterate over the names of gems we want appbundled binstubs for in this package. Those will land in pkg_prefix/bin/. Note: there's nothing to copy here during install because all the gem install actions were targeting pkg_prefix/vendor so what we would copy is here already. * do_after() Stuff to make the package smaller. - We don't need the cached .gem artifacts. - We don't need the cache of git repos retrieved by bundler (those gems are installed at this point like any other gem.) - We don't need to ship the gem API documentation. (I imagine we could change our minds about this later.) - We don't need the spec tests for all the gems ... except for our own chef gem which, for Reasons, we keep the spec suite around for post-build functional testing. * pkg_deps() Ruby comes with a version of bundler that works at the moment. We don't have to risk conflicts between the version of Ruby this package depends on and the version of Ruby core/bundler was built against. Signed-off-by: Robb Kidd <robb@thekidds.org> Signed-off-by: Christopher A. Snapp <csnapp@chef.io>
* | | Update CHANGELOG.md with details from pull request #10150Chef Expeditor2020-07-131-3/+4
| | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | Merge pull request #10150 from chef/fix-windows-test-warningTim Smith2020-07-131-3/+4
|\ \ \ | | | | | | | | Fix two warnings in tests.
| * | | Fix two warnings in tests.Pete Higgins2020-07-131-3/+4
| |/ / | | | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | | Bump version to 16.3.2 by Chef Expeditorv16.3.2Chef Expeditor2020-07-137-19/+20
| | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | Merge pull request #10149 from chef/dont-run-dnf-test-on-windowsLamont Granquist2020-07-131-1/+1
|\ \ \ | |/ / |/| | Don't run the dnf test that Windows doesn't like on Windows.
| * | Don't run the dnf test that Windows doesn't like on Windows.Pete Higgins2020-07-131-1/+1
|/ / | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Bump version to 16.3.1 by Chef Expeditorv16.3.1Chef Expeditor2020-07-127-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10144 from chef/fix-windows-unit-testsTim Smith2020-07-122-14/+18
|\ \ | | | | | | Fix some windows unit tests
| * | Fix windows package tests.fix-windows-unit-testssnehaldwivedi2020-07-101-8/+13
| | | | | | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
| * | Fix knife cookbook upload tests.snehaldwivedi2020-07-101-6/+5
| | | | | | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | | Bump version to 16.3.0 by Chef Expeditorv16.3.0Chef Expeditor2020-07-117-19/+20
| | | | | | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | | Merge pull request #10145 from chef/chefstyleTim Smith2020-07-101-4/+4
|\ \ \ | |/ / |/| | Minor chefstyle fixes in the spellcheck task
| * | Minor chefstyle fixes in the spellcheck taskTim Smith2020-07-101-4/+4
|/ / | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.2.98 by Chef Expeditorv16.2.98Chef Expeditor2020-07-107-19/+20
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #10142 from chef/fix-spellcheck-taskTim Smith2020-07-103-6/+23
|\ \ | | | | | | Fix spellcheck CI task
| * | Add spellcheck exceptions.fix-spellcheck-taskPete Higgins2020-07-101-1/+5
| | | | | | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
| * | Refer to specific rake file to avoid needing bundler to use rake in CI.Pete Higgins2020-07-102-5/+18
| |/ | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Bump version to 16.2.97 by Chef Expeditorv16.2.97Chef Expeditor2020-07-107-20/+21
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #9998 from plexinc/fix_git_resourceLamont Granquist2020-07-102-4/+34
|\ \ | |/ |/|
| * Don't try a git branch -f if already on the same branchMatthieu Serrepuy2020-07-102-4/+34
| | | | | | | | Signed-off-by: Matthieu Serrepuy <lotooo@gmail.com>
View Lorry Controller Status