Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Disable one test which requires changes to require_plugin() which will be ↵OC-9924 | sersut | 2013-12-13 | 3 | -32/+17 |
| | | | | done as a separate PR in order to keep this PR small. | ||||
* | Refactor hints to its own class. | sersut | 2013-12-13 | 6 | -73/+199 |
| | |||||
* | This PR includes a few refactorings. Since they are intertwined they are ↵ | sersut | 2013-12-13 | 13 | -306/+275 |
| | | | | | | | | | | done in a single PR :( 1-) VersionVI and VersionVII classes are moved to their own classes. 2-) VersionVII class only gets the data instead of full System object. 3-) A new integration testing infrastructure is started (similar to Chef integration tests). 4-) Loader code is refactored. 5-) New integration tests for Loader are added. | ||||
* | Move VersionVI and VersionVII plugin classes into their own files. | sersut | 2013-12-12 | 8 | -119/+181 |
| | |||||
* | Merge pull request #236 from opscode/fix-windows-tests | Seth Chisamore | 2013-12-05 | 1 | -4/+11 |
|\ | | | | | Clean up `jenkins_run_tests.bat` | ||||
| * | Clean up `jenkins_run_tests.bat` | Seth Chisamore | 2013-12-05 | 1 | -4/+11 |
|/ | | | | This change ensures the bat file is in sync with what we have in opscode/chef. | ||||
* | Merge branch 'plugin-provider-cleanup' | danielsdeleo | 2013-12-04 | 12 | -220/+295 |
|\ | | | | | | | Extract a class to maintain the provided attributes to providing plugins mapping | ||||
| * | Remove no-longer-relevant "TODO" | danielsdeleo | 2013-12-04 | 1 | -1/+0 |
| | | |||||
| * | Rename variables/methods for ProvidesMap to provides_map | danielsdeleo | 2013-12-04 | 9 | -40/+41 |
| | | | | | | | | | | | | | | | | | | The name "attributes" to refer to the map of attributes<->plugins providing attributes was confusing because it was inaccurate and often used near a local variable named "attributes" containing a very different kind of data (such as an Array of strings referring to attributes another plugin depended on). Renaming to "provides_map" clears up the confusion. | ||||
| * | Remove access to ProvidesMap internal data structure | danielsdeleo | 2013-12-04 | 4 | -28/+13 |
| | | | | | | | | | | Only test code was accessing this, modified to use the public API of the class. | ||||
| * | Extract ProvidesMap and tests to their own files | danielsdeleo | 2013-12-04 | 6 | -211/+245 |
| | | | | | | | | | | | | Also eliminates redundant tests in other classes that primarily test ProvidesMap behavior. Higher-level tests have been left in place to verify the interaction of the components. | ||||
| * | Extract plugin<->provides mapping to custom class | danielsdeleo | 2013-12-04 | 5 | -72/+125 |
| | | | | | | | | | | | | | | | | Code to map a plugin to the attributes it provides or look up a plugin by provided attributes was located in different parts of the code base where it operated directly on the underlying data structure. Extracting to a class lets us access the information we want using domain specific terminology so things are easier to understand. | ||||
| * | Add support for :focus tests to spec helper | danielsdeleo | 2013-12-04 | 1 | -0/+4 |
| | | |||||
| * | Fix warnings in DSL::Plugin spec | danielsdeleo | 2013-12-04 | 1 | -12/+11 |
|/ | |||||
* | OHAI-529 ensure virtual IPs are never chosen for privateaddress | Eric Saxby | 2013-11-26 | 2 | -13/+25 |
| | | | | | - Refactor privateaddress logic into smaller, more clear methods based on comments from @btm - Make rspec descriptions a little more readable | ||||
* | OHAI-529 Remove ipaddr_extension from Gemfile | Eric Saxby | 2013-11-26 | 2 | -48/+56 |
| | | | | | | Specs for the ip_scopes plugin can be run in two ways: * Run rspec in the context of `bundle exec` * Install ipaddr_extensions, then run rspec outside of `bundle exec` | ||||
* | OHAI-529 sort interfaces in ip_scopes before assigning privateaddress | Eric Saxby | 2013-11-26 | 1 | -1/+1 |
| | | | | | | Hash keys are ordered in Ruby > 1.9, so the plugin behaved differently between 1.8 and newer versions. In order to make the code (and tests) more deterministic, let's sort network interfaces before doing work. | ||||
* | OHAI-529: ip_scopes prefers ethernet to ppp | Eric Saxby | 2013-11-26 | 3 | -2/+91 |
| | | | | | | | | | On hosts with virtual interfaces on RFC1918 networks, the ip_scopes plugin may set `node.privateaddress` to a virtual address. This can cause problems with automation that relies on the `privateaddress` attribute. This patch prefers non-ppp interfaces when setting `privateaddress`, though it will use ppp interfaces if they are the only ones. | ||||
* | Merge branch 'OHAI-358' | Bryan McLellan | 2013-11-25 | 2 | -3/+10 |
|\ | |||||
| * | update test with current master | Elan Ruusamäe | 2013-11-25 | 1 | -3/+1 |
| | | | | | | | | | | | | | | | | test passes: $ ruby -S rspec spec/unit/plugins/php_spec.rb Conflicts: spec/unit/plugins/php_spec.rb | ||||
| * | [OHAI-358] add test for php version parse | Elan Ruusamäe | 2013-11-25 | 1 | -0/+9 |
| | | |||||
| * | [OHAI-509] fix build date parsing from php if php patched with suhosin: | Elan Ruusamäe | 2013-11-25 | 1 | -3/+3 |
|/ | | | | | | | | | | $ php -v PHP 5.3.27 with Suhosin-Patch (cli) (built: Aug 30 2013 04:30:30) Copyright (c) 1997-2013 The PHP Group Zend Engine v2.3.0, Copyright (c) 1998-2013 Zend Technologies Conflicts: lib/ohai/plugins/php.rb | ||||
* | OHAI-525 : can now detect qemu/kvm according the cpu type | Thibaut Notteboom | 2013-11-25 | 2 | -1/+17 |
| | |||||
* | Merge branch 'OHAI-527' | Bryan McLellan | 2013-11-25 | 1 | -0/+19 |
|\ | |||||
| * | added comment to help someone | caryp | 2013-11-13 | 1 | -0/+19 |
|/ | |||||
* | Merge branch 'OHAI-514' | danielsdeleo | 2013-11-11 | 2 | -93/+171 |
|\ | | | | | | | Fixes OHAI-514, https://tickets.opscode.com/browse/OHAI-514 | ||||
| * | Fix Mac OS X stub java workaroundOHAI-514 | danielsdeleo | 2013-11-07 | 2 | -93/+171 |
|/ | |||||
* | Remove self-dependency in libvirt plugin | danielsdeleo | 2013-11-07 | 1 | -2/+0 |
| | |||||
* | Merge pull request #221 from tas50/OHAI-522 | Bryan McLellan | 2013-11-06 | 1 | -1/+1 |
|\ | | | | | Correct help output to reference ohai not chef | ||||
| * | Correct help output to reference ohai not chef | Tim Smith | 2013-10-31 | 1 | -1/+1 |
| | | |||||
* | | Merge branch 'OHAI-495' | Bryan McLellan | 2013-11-04 | 2 | -11/+25 |
|\ \ | |||||
| * | | Update Virtualbox guest test for the dmidecode detection | Tim Smith | 2013-11-04 | 1 | -8/+20 |
| | | | |||||
| * | | Detect virtualbox guests via dmidecode so guest additions aren't | Tim Smith | 2013-11-04 | 1 | -3/+5 |
|/ / | | | | | | | required | ||||
* | | Merge pull request #217 from opscode/OC-10247 | Claire McQuin | 2013-10-30 | 12 | -33/+55 |
|\ \ | |/ |/| | Fail Ohai when > 1 collect_data blocks defined per platform | ||||
| * | fix mime-types dependency version | Claire McQuin | 2013-10-29 | 2 | -2/+4 |
| | | |||||
| * | check for detailed warning message | Claire McQuin | 2013-10-29 | 1 | -1/+1 |
| | | |||||
| * | add exceptions to ohai/exception.rb | Claire McQuin | 2013-10-29 | 10 | -36/+26 |
| | | |||||
| * | add PluginDefinitionError to Ohai, rescue in loader.rb | Claire McQuin | 2013-10-29 | 5 | -4/+28 |
| | | |||||
| * | fail when more than one collect_data blocks defined per platform | Claire McQuin | 2013-10-29 | 3 | -14/+20 |
|/ | |||||
* | Add a CONTRIBUTING document | Bryan McLellan | 2013-10-24 | 1 | -0/+153 |
| | | | | Copied from Chef, updated a bit for Ohai and recent changes | ||||
* | Merge pull request #214 from opscode/OC-10249 | Claire McQuin | 2013-10-17 | 8 | -112/+135 |
|\ | | | | | Rename providers -> plugins && seperate v6 methods from v7 methods in system.rb | ||||
| * | collect v7 methods at top and collect v6 methods at bottom of system.rb | Claire McQuin | 2013-10-17 | 1 | -33/+43 |
| | | |||||
| * | change _providers to _plugins | Claire McQuin | 2013-10-17 | 8 | -80/+93 |
| | | |||||
* | | Merge pull request #208 from opscode/OC-9409 | Claire McQuin | 2013-10-17 | 185 | -2258/+1561 |
|\ \ | |/ | | | Add platform option to collect_data | ||||
| * | update new solaris2 plugins and specs | Claire McQuin | 2013-10-17 | 4 | -6/+6 |
| | | |||||
| * | update spec_helper, system_spec to Ohai::Mixin::OS | Claire McQuin | 2013-10-17 | 2 | -3/+3 |
| | | |||||
| * | actually make os a mixin and update corresponding files | Claire McQuin | 2013-10-17 | 4 | -33/+35 |
| | | |||||
| * | change :providers to :_providers to avoid attribute conflicts | Claire McQuin | 2013-10-17 | 8 | -32/+32 |
| | | |||||
| * | insert plugin documentation URL to v6 deprecation message | Claire McQuin | 2013-10-17 | 1 | -1/+1 |
| | | |||||
| * | remove @todo that was done | Claire McQuin | 2013-10-17 | 1 | -1/+0 |
| | |