Skip to content

chore: stop tracking Policyfile lock - #319

Merged
ramereth merged 2 commits into
mainfrom
codex/fix-policyfile-lock
Aug 16, 2026
Merged

chore: stop tracking Policyfile lock#319
ramereth merged 2 commits into
mainfrom
codex/fix-policyfile-lock

Conversation

@damacus

@damacus damacus commented Aug 15, 2026

Copy link
Copy Markdown
Member

Summary

  • stop tracking the generated Policyfile.lock.json file
  • rely on the existing .gitignore entry to keep locally generated locks out of commits
  • let Chef and Test Kitchen resolve the current policy when tests run

Verification

  • removed Policyfile.lock.json, then ran chef install Policyfile.rb
  • Chef resolved the policy and generated a fresh ignored lock successfully
  • cookstyle (33 files, 0 offenses)
  • chef exec rspec --format progress (50 examples, 0 failures)
  • KITCHEN_LOCAL_YAML=kitchen.dokken.yml kitchen test default-ubuntu-2404 --destroy=always (6 tests, 0 failures; idempotent second converge)

@damacus
damacus requested a review from a team as a code owner August 15, 2026 19:46
@damacus damacus changed the title fix: refresh Policyfile lock chore: stop tracking Policyfile lock Aug 15, 2026
@github-actions

Copy link
Copy Markdown

Slowest examples

Top 10 slowest examples (1.2 seconds, 33.02% of total time)
Example Description Time in seconds
spec/unit/library/helper_spec.rb:18 helper methods npm_dist should return a url based on the version 0.17588
spec/unit/resources/nodejs_install_spec.rb:33 nodejs_install package install on rocky linux is expected to disable dnf_module "nodejs" 0.14778
spec/unit/resources/nodejs_repository_spec.rb:17 nodejs_repository on ubuntu is expected to create remote_file "/etc/apt/keyrings/nodesource.asc" 0.14482
spec/unit/resources/nodejs_install_spec.rb:15 nodejs_install package install on ubuntu is expected to create nodejs_repository "nodesource" 0.13688
spec/unit/resources/nodejs_repository_spec.rb:18 nodejs_repository on ubuntu is expected to add apt_preference "nodesource" 0.12002
spec/unit/resources/nodejs_install_spec.rb:89 nodejs_install source install is expected to create link "/usr/local/bin/python" 0.09808
spec/unit/resources/nodejs_install_spec.rb:16 nodejs_install package install on ubuntu is expected to install package "nodejs" 0.09772
spec/unit/resources/nodejs_install_spec.rb:86 nodejs_install source install is expected to install build_essential "install build tools" 0.0958
spec/unit/resources/nodejs_install_spec.rb:111 nodejs_install source install on rocky linux is expected to install package "openssl-devel" 0.09314
spec/unit/resources/nodejs_repository_spec.rb:42 nodejs_repository remove on ubuntu is expected to delete file "/etc/apt/keyrings/nodesource.asc" 0.09308

@ramereth ramereth added the Skip: Announcements On Release this will not put a message in the #Announcements slack channel label Aug 16, 2026
@ramereth
ramereth merged commit 1f89455 into main Aug 16, 2026
77 checks passed
@ramereth
ramereth deleted the codex/fix-policyfile-lock branch August 16, 2026 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip: Announcements On Release this will not put a message in the #Announcements slack channel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants