Skip to content

chore: migrate logrotate to Policyfile - #221

Merged
damacus merged 3 commits into
mainfrom
chore/logrotate-policyfile
Jul 5, 2026
Merged

damacus merged 3 commits into
mainfrom
chore/logrotate-policyfile

Conversation

@damacus

@damacus damacus commented Jul 2, 2026

Copy link
Copy Markdown
Member

Summary

  • replace Berksfile dependency resolution with Policyfile.rb and Policyfile.lock.json
  • update ChefSpec and Kitchen to use Policyfile support
  • move lint-unit to sous-chefs/.github v9.0.0 and update setup/lint config for Policyfile

Validation

  • chef install Policyfile.rb
  • chef update Policyfile.rb
  • cookstyle
  • chef exec rspec --format documentation failed before examples due local ~/.chef/gem json native extension code-signing issue; reran with Workstation embedded gems
  • env GEM_HOME=/opt/chef-workstation/embedded/lib/ruby/gems/3.1.0 GEM_PATH=/opt/chef-workstation/embedded/lib/ruby/gems/3.1.0 /opt/chef-workstation/embedded/bin/rspec --format documentation
  • markdownlint-cli2 "**/*.md" "#CHANGELOG.md"
  • yamllint .
  • actionlint
  • git diff --check
  • env -u KITCHEN_LOCAL_YAML kitchen list
  • env -u KITCHEN_LOCAL_YAML kitchen diagnose default-ubuntu-2404
  • env KITCHEN_LOCAL_YAML=kitchen.dokken.yml kitchen list
  • env KITCHEN_LOCAL_YAML=kitchen.dokken.yml kitchen diagnose default-ubuntu-2404
  • env KITCHEN_LOCAL_YAML=kitchen.dokken.yml kitchen test default-ubuntu-2404 --destroy=always

@damacus
damacus requested a review from a team as a code owner July 2, 2026 08:18
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

Slowest examples

Top 7 slowest examples (0.33 seconds, 100% of total time)
Example Description Time in seconds
spec/unit/resources/app_spec.rb:23 logrotate_global tomcat-myapp-multi-path creates appropriate logrotate config 0.08544
spec/unit/resources/app_spec.rb:12 logrotate_global tomcat-myapp creates appropriate logrotate config 0.06426
spec/unit/resources/global_spec.rb:30 logrotate_global writes the configuration template 0.04099
spec/unit/resources/app_spec.rb:53 logrotate_global tomcat-myapp-sharedscripts creates appropriate logrotate config 0.03877
spec/unit/resources/package_spec.rb:13 logrotate_package installs the logrotate package creates a systemd unit file is expected to upgrade package "logrotate" 0.03701
spec/unit/resources/app_spec.rb:34 logrotate_global tomcat-myapp-no-enable does not create appropriate logrotate config 0.03602
spec/unit/resources/app_spec.rb:40 logrotate_global tomcat-myapp-custom-options creates appropriate logrotate config 0.03597

@damacus
damacus merged commit 62540f0 into main Jul 5, 2026
17 checks passed
@damacus
damacus deleted the chore/logrotate-policyfile branch July 5, 2026 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant