summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* 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>
* 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>
* Bump version to 16.7.21 by Chef Expeditorv16.7.21Chef Expeditor2020-11-183-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1567 from chef/aixTim Smith2020-11-172-22/+32
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update lib/ohai/plugins/aix/network.rb Tim Smith2020-11-171-1/+1
| | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io> Co-authored-by: pete higgins <pete@peterhiggins.org>
| * Further optimize the netstat -rn parsingTim Smith2020-11-171-9/+7
| | | | | | | | | | | | This is up to 2x faster than the previous implementation Signed-off-by: Tim Smith <tsmith@chef.io>
| * Fixes for test failuresTim Smith2020-11-172-6/+7
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Use .each_line vs. .lines.eachTim Smith2020-11-171-6/+6
| | | | | | | | | | | | This uses 2% less memory in my benchmarks Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update spec nameTim Smith2020-11-171-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Use pure ruby vs. shelling out to grep to parse netstat dataTim Smith2020-11-172-8/+20
| | | | | | | | | | | | | | | | | | | | | | Besides avoiding potentially pathing issues with grep this is faster: ``` Comparison: pure_ruby: 55.9 i/s grep: 41.6 i/s - 1.34x (± 0.00) slower ``` Signed-off-by: Tim Smith <tsmith@chef.io>
| * Don't set network[:interfaces] to an empty mash when we set it laterTim Smith2020-11-171-1/+0
|/ | | | | | Avoid setting this twice. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.20 by Chef Expeditorv16.7.20Chef Expeditor2020-11-163-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1565 from chef/perfTim Smith2020-11-161-1/+1
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update rubocop-performance to 1.9Tim Smith2020-11-161-1/+1
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.19 by Chef Expeditorv16.7.19Chef Expeditor2020-11-163-4/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1566 from chef/fix-specsTim Smith2020-11-161-1/+1
|\ | | | | Update mock logger to fix shellout specs.
| * Update mock logger to fix shellout specs.Pete Higgins2020-11-161-1/+1
|/ | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* Update CHANGELOG.md to reflect the promotion of 16.7.18Chef Expeditor2020-11-121-29/+26
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 16.7.18 by Chef Expeditorv16.7.18Chef Expeditor2020-11-073-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 16.7.17 by Chef Expeditorv16.7.17Chef Expeditor2020-11-073-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1561 from chef/lsbTim Smith2020-11-061-4/+4
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove the line anchor from the LSB plugin matcheslsbTim Smith2020-11-061-4/+4
| | | | | | | | | | | | These are 1 line at a time matches and we want everything until the end. There's no point in doing it this way. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #1562 from chef/remove_legacy_lsbTim Smith2020-11-062-49/+1
|\ \ | | | | | | Remove Linux LSB support for systems without lsb-release CLI
| * | Remove Linux LSB support for systems with lsb-release CLIremove_legacy_lsbTim Smith2020-11-062-49/+1
| |/ | | | | | | | | | | This was something we did for legacy Debian < 7. Seems like we're good to nuke this at this point since we don't ship on anything less than 9 now. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.7.16 by Chef Expeditorv16.7.16Chef Expeditor2020-11-073-5/+6
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1563 from chef/more_anchorsTim Smith2020-11-061-36/+36
|\ \ | |/ |/| Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove regex anchors in the Linux Memory pluginmore_anchorsTim Smith2020-11-061-36/+36
|/ | | | | | | We're trying to match all everything here so we don't care about the end of the line. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.7.15 by Chef Expeditorv16.7.15Chef Expeditor2020-11-073-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1560 from chef/fix_hostnamectlTim Smith2020-11-062-2/+4
|\ | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
View Lorry Controller Status