Last 12 weeks · 1 commit
4 of 6 standards met
Summary The sample hook has referenced since it was added in 2017 (f4d6c28f), but is not a constant Ruby defines anywhere — the stdlib tempfile class is . As written, the hook crashes with every time is passed on the command line, because runs before the chef run starts. This fixes the typo ( → ) and adds the missing so the sample hook actually works out of the box. Verification appears nowhere else in the repository and is not a Ruby stdlib class; is the correct class (it is already used correctly in ). All methods used by the hook — , , , , — are part of the API. Reachable path: → CLI parsing sets → → raises . Test plan Static verification only; no Ruby runtime available locally. The change is a one-line constant rename plus a .
was meant for SysV startup serialization, and was never the right locationf or our lockfile. Moreover, recent systemd releases no longer keep the configuration that creates , so this means chefctl no longer works out of the box. Move the default to something more modern and sane. NOTE*: If Meta doesn't explicitly set a setting in their , they should before merging this. Signed-off-by: Phil Dibowitz
Repository: facebook/chef-utils. Description: Utilities related to Chef Stars: 287, Forks: 77. Primary language: Ruby. Languages: Ruby (94.2%), Erlang (4.5%), Shell (1.3%). License: Apache-2.0. Open PRs: 1, open issues: 4. Last activity: 3w ago. Community health: 75%. Top contributors: jaymzh, dafyddcrosby, odcinek, kcbraunschweig, gcweir, davide125, NaomiReeves, chantra, facebook-github-bot, fejese and others.