summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Remove rescue loading plist in the specs as this is a depmore_minor_specTim Smith2017-01-251-7/+0
| | | | | | There’s no need for this and we’re not doing this in the dozen other places we load deps. Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #948 from chef/minor_spec_cleanupThom May2017-01-25152-181/+247
|\ | | | | Minor spec file cleanup
| * Minor spec file cleanupminor_spec_cleanupTim Smith2017-01-24152-181/+247
|/ | | | | | | | | | | | | - Use require_relative in all the specs vs. expanding on the current file path. - Add missing license headers - Don't silently swallow ipaddr_extensions not being installed. Make it a development dep instead - Remove a rescue on ipaddress as we only do this in one place and it's a required gem - Make the Openstack plugin spec setup the same way as every other spec Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #946 from chef/fix_travisTim Smith2017-01-243-6/+8
|\ | | | | Update travis/appveyer preinstall to resolve rainbow failures
| * Update travis/appveyer preinstallsTim Smith2017-01-243-6/+8
|/ | | | | | Update Rubygems, fully uninstall bundler, install the latest bundler. Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #939 from chef/deprecate_old_configTim Smith2017-01-245-271/+2
|\ | | | | Remove deprecated config logic
| * Remove testing of old config logicdeprecate_old_configTim Smith2017-01-083-183/+0
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove deprecated config logicTim Smith2017-01-082-88/+2
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #933 from chef/commandTim Smith2017-01-242-386/+1
|\ \ | | | | | | Remove the deprecated run_command and popen4 methods
| * | Remove the deprecated run_command and popen4 methodscommandTim Smith2016-12-202-386/+1
| | | | | | | | | | | | | | | | | | This is very much legacy and has long since been replaced by shell_out Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #918 from ↵Tim Smith2017-01-242-2/+10
|\ \ \ | | | | | | | | | | | | | | | | chef/COOL-626/tduffield/use-cpu-name-for-model-name-windows Use name for Windows CPU model_name
| * | | Use name for Windows CPU model_nameCOOL-626/tduffield/use-cpu-name-for-model-name-windowsTom Duffield2016-12-022-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the "name" value for the model_name, and move description to the description field. Signed-off-by: Tom Duffield <tom@chef.io>
* | | | Merge pull request #930 from chef/sigarTim Smith2017-01-2413-675/+16
|\ \ \ \ | | | | | | | | | | Remove usage of the Sigar gem
| * | | | Remove usage of the Sigar gemsigarTim Smith2016-12-2013-675/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sigar was a really cool idea, but it’s a dead project now. The data is questionable at best and not getting any better as time goes on. We should figure out how to support these platforms vs. hoping that Sigar is doing the right thing. This does remove a lot of support for HP-UX, but I seriously doubt it worked to begin with. We should do that right sometime. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Merge pull request #938 from chef/ruby22Tim Smith2017-01-244-4/+2
|\ \ \ \ \ | | | | | | | | | | | | Require Ruby 2.2+
| * | | | | Require Ruby 2.2+ruby22Tim Smith2017-01-134-4/+2
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | chef/chef already requires Ruby 2.2. We've removed support for Ruby 2.1 in our other projects and community tooling. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Merge pull request #945 from chef/8.23v8.23.0ohai-8Tim Smith2017-01-243-2/+33
|\ \ \ \ \ | | | | | | | | | | | | Release 8.23.0
| * | | | | Release 8.23.0Tim Smith2017-01-243-2/+33
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Release 8.22.1Tom Duffield2017-01-231-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Tom Duffield <tom@chef.io>
* | | | | Merge pull request #944 from chef/pr/9418.22.1Tim Smith2017-01-233-310/+419
|\ \ \ \ \ | | | | | | | | | | | | C Plugin: Expand GCC data & only shellout to gcc if Xcode is installed on macOS
| * | | | | Only look for xlc on AIX and rename collect_cc to collect_sunproTim Smith2017-01-232-40/+52
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | Update release notes and chefstyle fixespr/941Tim Smith2017-01-232-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | Rework the specs to be platform specificTim Smith2017-01-202-263/+290
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add specs for checking that xcode exists on darwin Remove the odd double glibc mess Don't do ruby logic in the expects. That defeats the entire purpose of the specs if we're doing the same faulty logic. Just include the expected string Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | Fix gcc data collectionTim Smith2017-01-201-36/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | None of the values were being populated with the refactoring. Also gcc hadn’t been collecting anything for a while. This gives us real data for gcc: "c": { "gcc": { "configured_with": "--prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/usr/include/c++/4.2.1", "description": "Apple LLVM version 8.0.0 (clang-800.0.42.1)", "version": "8.0.0", "target": "x86_64-apple-darwin16.3.0", "thread_model": "posix" } },
| * | | | | Update GCC sample outputTim Smith2017-01-202-18/+23
| | | | | |
| * | | | | Don't check CC on DarwinTim Smith2017-01-201-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CC is clang on Darwin and the output is different so this just fails every time. We *should* add clang checks at some point though.
| * | | | | Remove extra windows check for glibcTim Smith2017-01-201-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We’re only collecting this data on Windows now so don’t bother checking if we’re on Windows again
| * | | | | Consistent loggingTim Smith2017-01-201-4/+3
| | | | | |
| * | | | | Further split out platforms in the C pluginTim Smith2017-01-201-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cl / devenv are windows only so avoid checking those on *nix HP UX things should only get checked on HPUX
| * | | | | add colons back to debug logErik Gomez2017-01-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Erik Gomez <e@eriknicolasgomez.com>
| * | | | | move functions to topErik Gomez2017-01-201-25/+27
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Erik Gomez <e@eriknicolasgomez.com>
| * | | | | fixed ChefstyleErik Gomez2017-01-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Erik Gomez <e@eriknicolasgomez.com>
| * | | | | refactor c.rb based on PR feedbackErik Gomez2017-01-201-60/+71
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Erik Gomez <e@eriknicolasgomez.com>
| * | | | | fix gcc and cc for darwin (no cli tools installed)Erik Gomez2017-01-201-21/+58
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If darwin is found: 1. shell out to /usr/bin/xcode-select -p 2a. If it returns an exit code other than 0, assume xcode command line tools are not installed and do not run these commands. 2b. If it returns an exit code of 0, run the commands. Signed-off-by: Erik Gomez <e@eriknicolasgomez.com>
* | | | | Merge branch 'tm/fix_app_logging'Tim Smith2017-01-203-5/+2
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Only explicitly initialize a logger when neededtm/fix_app_loggingThom May2017-01-203-5/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we're running Ohai as a standalone application, we need to fire up the logger with our desired location, but when we're run from inside Chef, we're passed the client's loggers. Signed-off-by: Thom May <thom@chef.io>
* | | | updating cloud plugin to populate azure private_ip as it's currently nilohai13Richard Shade2017-01-042-0/+7
| | | | | | | | | | | | | | | | Signed-off-by: Richard Shade <rshade@rightscale.com>
* | | | Merge pull request #934 from chef/travisLamont Granquist2016-12-221-2/+6
|\ \ \ \ | | | | | | | | | | Bump ruby versions and test on ruby head
| * | | | Bump ruby versions and test on ruby headTim Smith2016-12-211-2/+6
| | |_|/ | |/| | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Merge pull request #935 from chef/better_loggingTim Smith2016-12-2219-102/+127
|\ \ \ \ | |/ / / |/| | | Better debug logging
| * | | Better debug loggingTim Smith2016-12-2219-102/+127
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Gracefully fail in virtual box Include plugin name in more logs Be consistent in the log output format Help figure out how Virtualization systems were detected with logging Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Update RELEASE_NOTES.mdTim Smith2016-12-201-0/+14
|/ /
* | Merge pull request #921 from chef/cumulusTim Smith2016-12-203-61/+127
|\ \ | | | | | | Properly detect Cumulus Linux platform / version
| * | Fix comment typoTim Smith2016-12-201-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Better error handling if we can't determine Cumulus versioncumulusTim Smith2016-12-192-8/+14
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Reorder detection of platforms to speed things upTim Smith2016-12-151-22/+22
| | | | | | | | | | | | | | | | | | Check the more popular platforms first since those are most likely to match. Parallels have to come before RHEL though since it looks like RHEL. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add Cumulus Linux detection to the release notesTim Smith2016-12-151-17/+2
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Properly detect Cumulus Linux platform / versionTim Smith2016-12-152-22/+97
| | | | | | | | | | | | | | | | | | Detect cumulus Linux systems if they have /etc/cumulus directory. Use platform of ‘cumulus’ and platform_family debian since it’s Debian 8.5. Parse version from their config directory. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #925 from chef/VeertuThom May2016-12-204-0/+41
|\ \ \ | | | | | | | | Detect guests running on Veertu hypervisor
| * | | Detect guests running on Veertu hypervisorVeertuTim Smith2016-12-154-0/+41
|/ / / | | | | | | | | | | | | | | | | | | | | | Hypervisor for OS that runs Linux and WIndows https://veertu.com/ Signed-off-by: Tim Smith <tsmith@chef.io>