Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 9, 2025

Issue #371 questions whether the Arch Linux bug #47052 workaround is still needed. The bug was filed in 2015 and has been resolved in modern Arch Linux/systemd.

Changes

  • Removed comment referencing https://bugs.archlinux.org/task/47052
  • Removed RUN echo >/etc/security/limits.conf from arch_platform method in lib/kitchen/docker/helpers/dockerfile_helper.rb

The workaround blanked out /etc/security/limits.conf to avoid systemd issues. Preserving this file is now preferable as it may contain application-required resource limits.

Fixes #371

Original prompt

This section details on the original issue you should resolve

<issue_title>This seems to be fixed, does it still need to be here?</issue_title>
<issue_description>

# See https://bugs.archlinux.org/task/47052 for why we
</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

The workaround was blanking out /etc/security/limits.conf to work around
an Arch Linux bug from 2015. Modern Arch Linux and systemd have resolved
this issue, so the workaround is no longer necessary.

Fixes #371

Co-authored-by: damacus <40786+damacus@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove reference to resolved issue in docker driver Remove obsolete Arch Linux limits.conf workaround Nov 9, 2025
Copilot AI requested a review from damacus November 9, 2025 00:13
@damacus damacus marked this pull request as ready for review November 9, 2025 00:17
@damacus
Copy link
Contributor

damacus commented Nov 9, 2025

@copilot fix chefstyle errors

Copy link
Contributor Author

Copilot AI commented Nov 9, 2025

@copilot fix chefstyle errors

The code already passes all chefstyle checks. Running chefstyle and chefstyle -a both report "21 files inspected, no offenses detected". No fixes needed.

@damacus damacus merged commit ed14bc6 into main Nov 9, 2025
13 of 15 checks passed
@damacus damacus deleted the copilot/remove-fixed-issue-reference branch November 9, 2025 01:55
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.

This seems to be fixed, does it still need to be here?

2 participants