Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused lock_dir method #164

Merged
merged 1 commit into from
Jul 25, 2024
Merged

Remove unused lock_dir method #164

merged 1 commit into from
Jul 25, 2024

Conversation

ramereth
Copy link
Contributor

This conflicts with a method with the same name in the apache2 cookbook which
has been causing some idempotency issues when used together. It seems this
method is no longer being used so it's safe to remove.

In addition, update test CI platforms.

Signed-off-by: Lance Albertson lance@osuosl.org

Description

Describe what this change achieves

Issues Resolved

List any existing issues this PR resolves

Check List

  • A summary of changes made is included in the CHANGELOG under ## Unreleased
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@ramereth ramereth requested a review from a team as a code owner July 25, 2024 15:57
@ramereth ramereth added the Release: Patch Release to Chef Supermarket as a version patch when merged label Jul 25, 2024
This conflicts with a method with the same name in the apache2 cookbook which
has been causing some idempotency issues when used together. It seems this
method is no longer being used so it's safe to remove.

In addition, update test CI platforms.

Signed-off-by: Lance Albertson <lance@osuosl.org>
@ramereth ramereth merged commit 9400331 into main Jul 25, 2024
30 checks passed
@ramereth ramereth deleted the rm-lock_dir-method branch July 25, 2024 16:03
@kitchen-porter
Copy link
Contributor

Released as: 7.0.26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release: Patch Release to Chef Supermarket as a version patch when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants