summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Make sure /proc/cmdline exists before reading itrubocop_Performance_RegexpMatchTim Smith2020-08-072-2/+3
| | | | | | Don't run match? against nil Signed-off-by: Tim Smith <tsmith@chef.io>
* Use match? instead of =~ when MatchData is not usedTim Smith2020-08-071-1/+1
| | | | | | We don't need this data. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.4.2 by Chef Expeditorv16.4.2Chef Expeditor2020-08-043-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1485 from cooperlees/route_is_valid_default_route_afTim Smith2020-08-032-1/+18
|\ | | | | Add Address Family Check to route_is_valid_default_route?
| * Add Address Family Check to route_is_valid_default_route?Cooper Lees2020-07-292-1/+18
| | | | | | | | | | | | | | | | | | | | | | - We can now have IPv6 next hops for IPv4 routes in Linux - We don't have enough information to validate the route as valid if they are differen address families Test: - Add an rspec test to show it returning false like we'd hoped - Have also tested on Facebook servers internally Signed-off-by: Cooper Lees <me@cooperlees.com>
* | Bump version to 16.4.1 by Chef Expeditorv16.4.1Chef Expeditor2020-08-043-5/+6
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1487 from chef/rubocop_Performance_DetectTim Smith2020-08-032-3/+3
|\ \ | | | | | | Use .find instead of .select.first
| * | Use .find instead of .select.firstrubocop_Performance_DetectTim Smith2020-08-032-3/+3
|/ / | | | | | | | | | | Avoid creating and object we don't need. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.4.0 by Chef Expeditorv16.4.0Chef Expeditor2020-08-033-4/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1486 from chef/lcg/faster-rubylang-pluginTim Smith2020-08-031-15/+4
|\ \ | |/ |/| Faster ruby language plugin
| * Faster ruby language pluginlcg/faster-rubylang-pluginLamont Granquist2020-08-031-15/+4
|/ | | | | | This drops two additional shell outs to run ruby. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Update CHANGELOG.md to reflect the promotion of 16.3.2Chef Expeditor2020-07-291-12/+9
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 16.3.2 by Chef Expeditorv16.3.2Chef Expeditor2020-07-283-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1484 from chef/spellingTim Smith2020-07-2811-34/+46
|\ | | | | Spelling fixes
| * Spelling fixesspellingTim Smith2020-07-2811-34/+46
|/ | | | | | Pull in the latest rake task, fix some variable names, fix a few legit typos. Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 16.3.1 by Chef Expeditorv16.3.1Chef Expeditor2020-07-243-4/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1483 from chef/encodingTim Smith2020-07-241-1/+0
|\ | | | | Remove redundant file encoding in spec
| * Remove redundant file encoding in specencodingTim Smith2020-07-231-1/+0
| | | | | | | | | | | | In Ruby 2+ this is the default. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Update CHANGELOG.md to reflect the promotion of 16.3.0Chef Expeditor2020-07-241-13/+10
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Bump version to 16.3.0 by Chef Expeditorv16.3.0Chef Expeditor2020-07-243-5/+6
|/ | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1482 from chef/readmeTim Smith2020-07-231-7/+7
|\ | | | | Readme updates
| * Readme updatesreadmeTim Smith2020-07-231-7/+7
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 16.2.5 by Chef Expeditorv16.2.5Chef Expeditor2020-07-223-5/+6
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1477 from cooperlees/handle_both_uintsTim Smith2020-07-221-1/+1
|\ \ | |/ |/| Handle IP to int conversion for inet + inet6
| * Move to @jaymzh .to_i suggestionCooper Lees2020-07-061-1/+1
| | | | | | | | Signed-off-by: Cooper Lees <me@cooperlees.com>
| * Fix bad copy paste + rerun rspec locallyCooper Lees2020-07-021-1/+1
| | | | | | | | Signed-off-by: Cooper Lees <me@cooperlees.com>
| * Go back to a one liner using .respond_to? rather than family == inetCooper Lees2020-07-021-8/+1
| | | | | | | | Signed-off-by: Cooper Lees <me@cooperlees.com>
| * Be more Rubonic and please the linterCooper Lees2020-07-011-6/+4
| | | | | | | | Signed-off-by: Cooper Lees <me@cooperlees.com>
| * Handle IP to int conversion for inet + inet6Cooper Lees2020-07-011-1/+10
| | | | | | | | | | | | | | | | | | - IPv4 can have IPv6 next hops, so we could use the wrong `.to_u*` method - Lets try both before throwing on int conversions for sorting purposes Addresses more for #1474 Signed-off-by: Cooper Lees <me@cooperlees.com>
* | Bump version to 16.2.4 by Chef Expeditorv16.2.4Chef Expeditor2020-07-073-4/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1478 from chef/perfTim Smith2020-07-0725-58/+58
|\ \ | |/ |/| Use .match? when we don't need the result
| * Use .match? when we don't need the resultTim Smith2020-07-0625-58/+58
|/ | | | | | Avoid creating a bunch of match objects when we're not going to use the match results. Signed-off-by: Tim Smith <tsmith@chef.io>
* Update CHANGELOG.md to reflect the promotion of 16.2.3Chef Expeditor2020-06-301-12/+9
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 16.2.3 by Chef Expeditorv16.2.3Chef Expeditor2020-06-303-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1475 from cooperlees/ipv4_v6_nhTim Smith2020-06-302-0/+20
|\ | | | | Linux network plugin: Handle IPv6 next hops for IPv4 routes
| * Add a Rubular PermalinkCooper Lees2020-06-301-0/+1
| | | | | | | | | | | | - Show the regex working ... Signed-off-by: Cooper Lees <me@cooperlees.com>
| * Remove white space to please linterCooper Lees2020-06-301-1/+1
| | | | | | Signed-off-by: Cooper Lees <me@cooperlees.com>
| * Move regex out of the loop Cooper Lees2020-06-301-4/+1
| | | | | | | | | | Save regex lookup each iteration. This is hot. Co-authored-by: pete higgins <pete@peterhiggins.org>
| * Change to double quotes for impactCooper Lees2020-06-241-1/+1
| | | | | | | | Signed-off-by: Cooper Lees <me@cooperlees.com>
| * Add spaces on rspec expected hashesCooper Lees2020-06-241-2/+2
| | | | | | | | Signed-off-by: Cooper Lees <me@cooperlees.com>
| * Linux network plugin: Handle IPv6 next hops for IPv4 routesCooper Lees2020-06-242-0/+22
| | | | | | | | | | | | | | | | | | - Add handling for `via` where the inet6 address family is specified - Add rspec test to show working Addresses part of #1474 Signed-off-by: Cooper Lees <me@cooperlees.com>
* | Bump version to 16.2.2 by Chef Expeditorv16.2.2Chef Expeditor2020-06-303-4/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #1476 from jaymzh/dockerTim Smith2020-06-301-2/+5
|\ \ | | | | | | Fix docker detection in ohai virtualization
| * | Fix docker detection in ohai virtualizationPhil Dibowitz2020-06-301-2/+5
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We ran into an issue where when running in docker containers on GH Actions (which is on azure which is on HyperV), we were getting detected as being run on a hyperv guest... which is indirectly true, but not really the most correct answer. It turns out we were being a bit too strict in this regex. I think. So here's the `/proc/self/cgroup`: ``` 12:blkio:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 11:devices:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 10:memory:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 9:freezer:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 8:cpuset:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 7:pids:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 6:rdma:/ 5:hugetlb:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 4:perf_event:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 3:cpu,cpuacct:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 2:net_cls,net_prio:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 1:name=systemd:/actions_job/4ab5ebdc794956d804d03db04557c8b79fa985cbf54bc25f51f2b0848e3f58bd 0::/system.slice/docker.service ``` As you can see the second field (or 1st field if you 0-index) is empty, and it's still docker. Signed-off-by: Phil Dibowitz <phil@ipom.com>
* | Fix a few typosTim Smith2020-06-233-6/+6
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Add cspell taskTim Smith2020-06-234-1/+77
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Update CHANGELOG.md to reflect the promotion of 16.2.1Chef Expeditor2020-06-231-11/+8
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 16.2.1 by Chef Expeditorv16.2.1Chef Expeditor2020-06-233-4/+12
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #1470 from chef/big_surTim Smith2020-06-223-30/+9
|\ | | | | Make sure Darwin hosts are always mac_os_x
| * Make sure Darwin hosts are always mac_os_xbig_surTim Smith2020-06-223-30/+9
|/ | | | | | | | | mac_os_x_server isn't a thing anymore and no third party darwin based operating systems ever took off. Fix this detectio and speed things up a tiny bit by assuming darwin == mac_os_x just like we already do with the platform_family Signed-off-by: Tim Smith <tsmith@chef.io>
View Lorry Controller Status