summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix XLC compiler detection to support 3 part version numbersxlcTim Smith2020-11-232-3/+15
| | | | | | Add a spec with the newer XLC output from our build systems. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.34 by Chef Expeditorv16.7.34Chef Expeditor2020-11-233-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1582 from chef/style_cleanupTim Smith2020-11-2331-117/+108
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Misc RuboCop cleanup of the codebaseTim Smith2020-11-2331-117/+108
|/ | | | | | A few misc things Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.33 by Chef Expeditorv16.7.33Chef Expeditor2020-11-193-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1580 from chef/lpar_detectionTim Smith2020-11-182-5/+3
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Fix lpar name detectionlpar_detectionTim Smith2020-11-182-5/+3
| | | | | | | | | | | | LPAR partition names can have spaces in them, which is problematic with our previous split method which would throw away anything past the space. On our internal AIX cluster "virtlpar03 - 7.1 testers" became just "virtlpar03". Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.7.32 by Chef Expeditorv16.7.32Chef Expeditor2020-11-193-5/+6
|/ | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1578 from chef/simplify_stripsTim Smith2020-11-1814-43/+41
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * 1 line from_cmd in vmware as wellTim Smith2020-11-181-2/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Use strip instead of split($/) to cleanup shelloutTim Smith2020-11-1814-42/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``` irb(main):008:0> "something\n ".split($/)[0] => "something" ``` ``` irb(main):009:0> "something\n ".strip => "something" ``` Calculating ------------------------------------- split_stuff 120.000 memsize ( 0.000 retained) 3.000 objects ( 0.000 retained) 2.000 strings ( 0.000 retained) strip_stuff 80.000 memsize ( 0.000 retained) 2.000 objects ( 0.000 retained) 2.000 strings ( 0.000 retained) Comparison: strip_stuff: 80 allocated split_stuff: 120 allocated - 1.50x more Warming up -------------------------------------- split_stuff 328.234k i/100ms strip_stuff 604.612k i/100ms Calculating ------------------------------------- split_stuff 3.648M (± 3.2%) i/s - 18.381M in 5.044570s strip_stuff 5.993M (± 2.8%) i/s - 30.231M in 5.048664s Comparison: strip_stuff: 5992617.3 i/s split_stuff: 3647589.6 i/s - 1.64x (± 0.00) slower Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.7.31 by Chef Expeditorv16.7.31Chef Expeditor2020-11-193-5/+6
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1577 from chef/simplify_kernelTim Smith2020-11-182-6/+6
|\ \ | |/ |/| Signed-off-by: Tim Smith <tsmith@chef.io>
| * Simplify the aix kernel pluginTim Smith2020-11-182-6/+6
| | | | | | | | | | | | | | There's no point in querying that we're on aix when we only execute this collect on aix. Just hard code it to aix for historic reasons. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.7.30 by Chef Expeditorv16.7.30Chef Expeditor2020-11-183-5/+6
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1576 from chef/aix_virtualizationTim Smith2020-11-182-4/+11
|\ \ | |/ |/| Signed-off-by: Tim Smith <tsmith@chef.io>
| * Gather wpar stateTim Smith2020-11-182-2/+10
| | | | | | | | | | | | This is an important thing to know about the wpars Signed-off-by: Tim Smith <tsmith@chef.io>
| * Simplify shellout and chompsTim Smith2020-11-181-3/+2
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.29 by Chef Expeditorv16.7.29Chef Expeditor2020-11-183-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1575 from chef/aix_kernelTim Smith2020-11-182-4/+3
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Just set the aix platform / platform_family directlyTim Smith2020-11-182-4/+3
| | | | | | | | | | | | If we're in a collect block for aix then we're on aix. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.7.28 by Chef Expeditorv16.7.28Chef Expeditor2020-11-183-5/+6
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1574 from chef/aix_kernelTim Smith2020-11-182-10/+9
|\ \ | |/ | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Avoid calling uname 4 times on aixTim Smith2020-11-182-10/+9
| | | | | | | | | | | | Just get all the data we need and then split it. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.7.27 by Chef Expeditorv16.7.27Chef Expeditor2020-11-183-5/+6
|/ | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1573 from chef/more_aix_networkTim Smith2020-11-182-12/+124
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Avoid running the regex for each lineTim Smith2020-11-181-6/+4
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Convert shellouts to grep to use regexes in Ruby insteadTim Smith2020-11-182-6/+120
| | | | | | | | | | | | This is significantly faster and having all the data in the specs makes it easier to refactor this later without having access to an AIX system. Signed-off-by: Tim Smith <tsmith@chef.io>
| * Don't try to create the arp mash once each lineTim Smith2020-11-181-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.7.26 by Chef Expeditorv16.7.26Chef Expeditor2020-11-183-5/+6
|/ | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1572 from chef/memoryTim Smith2020-11-182-8/+8
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Simplify how we gather memory on AIXTim Smith2020-11-182-8/+8
|/ | | | | | The grep needs to get refactored away, but at least this way we avoid doing math on all the values we gather. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.25 by Chef Expeditorv16.7.25Chef Expeditor2020-11-183-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1570 from chef/more_aixTim Smith2020-11-181-40/+35
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Use each_slice to further simplify key/value parsingTim Smith2020-11-181-3/+2
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * lin -> line variableTim Smith2020-11-181-8/+8
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Collapse down some shellout uses to avoid setting variablesTim Smith2020-11-181-4/+2
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update the provides for this plugin to match other network pluginsTim Smith2020-11-181-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update lib/ohai/plugins/aix/network.rb Tim Smith2020-11-181-1/+1
| | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Avoid setting the addresses mash twiceTim Smith2020-11-181-2/+1
| | | | | | | | | | | | Just do it at the top of things Signed-off-by: Tim Smith <tsmith@chef.io>
| * Refactor how we parse ifconfig in AIXTim Smith2020-11-181-28/+27
|/ | | | | | | | | | | There's a few different things going on here. - more descriptive variable names. Variables like splat were not very useful and others like iface actually held more than 1 value - replace a gsub and split with just a split on a regex from @phiggins - avoid the whole weird splat thing and ranges and instead just split each interface into the name and all the data. This makes the variables more clear when we use them throughout - avoid some positional math and just use shift to simply pop off the leading values to build the mash Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.24 by Chef Expeditorv16.7.24Chef Expeditor2020-11-183-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1571 from chef/turn_off_cachingTim Smith2020-11-182-6/+0
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Disable gem caching / rubygems updates in BuildkiteTim Smith2020-11-182-6/+0
|/ | | | | | | | Since we're pulling chef/chef from git now we're basically updating the cache each and every time this runs, which is very slow. It's possibly faster to just do it from scratch each time. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.23 by Chef Expeditorv16.7.23Chef Expeditor2020-11-183-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1569 from chef/joyentTim Smith2020-11-172-178/+0
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove the :Joyent plugin as Joyent cloud went EOL 11/2019joyentTim Smith2020-11-172-178/+0
|/ | | | | | | | There's no reason to execute this plugin on every Ohai run. The Cloud is EOL. https://docs.joyent.com/joyent-public-cloud-eol/faq Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.22 by Chef Expeditorv16.7.22Chef Expeditor2020-11-183-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1568 from chef/splitTim Smith2020-11-176-10/+10
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Simplify all our splitsTim Smith2020-11-176-10/+10
|/ | | | | | | As Pete pointed out there's no need to split on whitespace when that's the default Signed-off-by: Tim Smith <tsmith@chef.io>