| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Signed-off-by: Jaymala Sinha <jsinha@chef.io>
|
|
|
|
| |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\
| |
| |
| |
| | |
MsysTechnologiesllc/VSingh/MSYS-1141_knife_config_get_conf_d_fixes
[knife config get] Fix TypeError: no implicit conversion of nil into String
|
|/
|
|
|
|
| |
- Now showing relevant logs for config.d configuration files.
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
|
|
|
| |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\
| |
| | |
Bump omnibus to 6.1.10
|
|/
|
|
|
|
| |
Pull in fix https://github.com/chef/omnibus/pull/910
Signed-off-by: Bryan McLellan <btm@chef.io>
|
|\
| |
| | |
[ci] include the platform target in the hab package promotion command
|
|/
|
|
|
|
|
| |
Or else it won't work when there are multiple platform targets for a
package.
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
|
|
|
| |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\
| |
| | |
a Windows Habitat plan
|
| |
| |
| |
| | |
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Removed the promote_hab_package actions from the omnibus release
workloads.
Defined a new workload subscribed to BuildKite successfully publishing
to unstable packages for all platforms listed in .bldr.toml. The
habitat/test pipeline is now responsible for testing the unstable
packages just built and promoting them if they all pass. Currently
scoped only to the Windows package because the Linux packages don't test
well yet.
Co-authored-by: Jeremiah Snapp <jeremiah.snapp@gmail.com>
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Expeditor uses this to know for which platforms to crank out packages.
https://expeditor.chef.io/docs/patterns/habitat-packages/#bldr-toml
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
| |
| |
| |
| |
| |
| |
| |
| | |
We can disable the :choco_installed specs directly in the execution of
RSpec instead of deleting the choco.exe binary from the test system to
fool the "filter_run_excluding choco_installed" config in spec_helper.
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Adds a test PowersShell script to habitat/tests/ to run some simple
tests on executable version output and then run the functional specs
suite like the omnibus_test script does.
scripts/ci/verify-plan.ps1 will perform a throwaway build of
the plan under a "ci" origin and then run the test script upon the built
package.
The habitat/verify pipeline was updated to run the verify-plan.ps1 script.
Signed-off-by: Robb Kidd <robb@thekidds.org>
add Windows plan verification to verify-hab pipeline
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
These fail in the Habitat plan verification tests because bundle isn't
on the PATH for Windows builds (bundle.cmd is). Adding a label to the
spec lets testing chose whether to run it or not.
Default is that the specs are included in a test run.
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Tell ffi-yajl to use the C-extensions we know are there because we
depend on MRI. C is fast, y'know, and we don't need the FFI portability
for other Ruby runtimes.
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
| |
| |
| |
| | |
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Based on Stuart Preston's work. Currently depends on the very large
ruby-plus-devkit package[1] that is a repackaging of
RubyInstaller+DevKit[2]. The DevKit is essentially MSYS2, which is its
own software distribution. It can get large. It is depended on currently
because it RubyInstaller+DevKit has been the most common Ruby on Windows
solution for build environment and library ecosystem.
Whatever Ruby this plan ultimately depends on is expected to have its
executables on the PATH and any gems it ships with (say, bundler, rake,
etc) added to GEM_PATH which is totally a thing a package can do in
SetupEnvironment and pushing directories onto GEM_HOME.
[1] https://bldr.habitat.sh/#/pkgs/chef/ruby-plus-devkit
https://github.com/chef/chef-plans/tree/master/ruby-plus-devkit
[2] https://rubyinstaller.org/about/comparison/
* bundle install'ing
With core/git not being fully capable of supporting a `bundle install`
because of semi-hardcoded paths to binaries, git is baked into the
"devkit" portion of Chef's chef/ruby-plus-devkit.
That means `bundle install` works, so this build uses a pattern
established in the chef omnibus software definition:
1. bundle install the Ruby dependencies
2. gem install the local gems
3. install the git-ref'd gems
At the end of these steps, all the gems are "installed" in the standard
directories under GEM_HOME/gems and binstubs are in GEM_HOME/bin.
* add the other Ruby binstubs to the PATH
Gems got their bundle/gem installed binstubs placed in GEM_HOME/bin
a.k.a vendor/bin. That path has been added to the package's bin paths
for use in the build and later in testing.
Appbundler is executed in Invoke-Install to create our precisely-pinned
binstubs for only the gems that supply executables we expect users of
chef-client to use. Those appbundled binstubs are placed into the
package's bin directory under $pkg_prefix/bin.
While the binstubs under vendor/bin aren't locked down with appbundler,
they're worth having on the PATH, particularly for testing the build.
* Trimming the Fat
Remove many of the byproducts of gems that appear in $pkg_prefix. Some
C-extension build logs. Other gems' spec directories. We keep the chef
gem's spec directory because we will be testing builds with the
functional portion of the test suite.
* Stripping the FS_ROOT
The studio's FS_ROOT appears in the Gemfiles generated by appbundler.
Added a function to remove the build environment's FS_ROOT from a given
file. Currently only using it on the chef gem's Gemfile so that it
can be used with `bundle install` and `bundle exec` to test the build.
* build in "${HAB_CACHE_SRC_PATH}/${pkg_dirname}"
"${HAB_CACHE_SRC_PATH}/${pkg_dirname}" is $CACHE_PATH, but we can't use
CACHE_PATH directly because it seems the Windows plan build script
doesn't actually set that variable unless pkg_version is computed and
updated; probably a bug, need to investigate.
Mainly opted for this to take advantage of the clean-room features of
builds done in the CACHE directory.
Use git to archive up only the version-controlled files and place that
archive where Hab would normally download a source artifact. Set the
$pkg_filename so the plan build knows what to archive and unpack.
No-Op the Verify function because the archive "downloaded" was just
created.
For Install, set the location of the Gemfile for bundler via an
environment variable instead of a `bundle config --local` command so
that there is no need to clean up the generated `.bundle/config`.
* have the plan check for a minimum hab version
The Windows plan must be built with Hab version 0.85.0 or greater
because of its use of the -IsPath option when setting/pushing paths in
SetupEnvironment. This change adds a version check to the Begin build
callback to bail with an informative error message when the required
minimum version isn't present. Otherwise, the error that appears is
the slightly arcane:
```
An error occured in the build!
You cannot call a method on a null-valued expression.
At C:\hab\pkgs\core\hab-studio\0.83.0\20190712234514\bin\hab-plan-build.ps1:1447 char:5
+ $pathArray = $path.Split($separator)
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo : InvalidOperation: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : InvokeMethodOnNull
```
Co-authored-by: Stuart Preston <stuart@chef.io>
Co-authored-by: John Snow <thelunaticscripter@outlook.com>
Signed-off-by: Robb Kidd <robb@thekidds.org>
|
|
|
|
| |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\
| |
| |
| |
| | |
MsysTechnologiesllc/VSingh/MSYS-1109_chefignore_for_respective_cookbook_dir
Fix multiple chefignore file issues
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Fix cookbook syntax_check dir path error if cookbook_path is an array
Fix invalid error if cookbook_path set to nil or empty Array and TypeError: no implicit conversion of nil into String
Chefignore for each cookbooks/:cookbook
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| |
| |
| |
| | |
- Minor update in #readable_file_or_symlink? method to use arg path params instead of instance var.
- Change log level trace to debug for missing lookup path.
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\ \
| | |
| | | |
Allow for the mixlib-authentication 3.x gem
|
|/ /
| |
| |
| |
| |
| |
| | |
This gem is 70k smaller than the 2.x gem since it doesn't ship extra
files
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\ \
| | |
| | | |
Bump all deps to the latest versions
|
|/ /
| |
| |
| | |
Signed-off-by: Tim Smith <tsmith@chef.io>
|
| |
| |
| |
| | |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\ \
| | |
| | | |
Turn on unified_mode for 35 core resources
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* windows_script inherits from script and is obviously safe
* all of the sub-providers of the group provider i have not
audited.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| | |
These are the ones that off of the top of my head it should be perfectly
safe to turn on `unifed_mode` with minimal auditing.
Most of these are so old that they do not use any sub-resources at all.
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
|
| |
| |
| |
| | |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\ \
| | |
| | |
| | |
| | | |
MsysTechnologiesllc/VSingh/fix-knife-cookbook-metadata-from-file-name-args
Fix knife cookbook metadata from file name args
|
|/ /
| |
| |
| | |
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|
| |
| |
| |
| | |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\ \
| | |
| | | |
Fix license acceptance in `omnibus/kitchen.yml`
|
|/ /
| |
| |
| | |
Signed-off-by: Christopher A. Snapp <csnapp@chef.io>
|
| |
| |
| |
| | |
Obvious fix; these changes are the result of automation not creative thinking.
|
|\ \
| |/
|/| |
[Data-collector] Add cookbooks attribute to run_end_message
|
| |
| |
| |
| |
| |
| | |
- Rename "expanded_cookbooks" to "cookbooks".
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
|