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

Allow to toggle RPM repo configuration on openldap_install resource #141

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

Annih
Copy link
Contributor

@Annih Annih commented Jul 6, 2023

Description

On RHEL >= 8 the resource is configuring the OSUSOL repository to support openldap-server install.
However in some cases you may not need this specific repository.

Now by setting install_repository false on the resource, the repo won't be installed.
Also the openldap.package_install_repository attributes allow to control the value set to this property in the openldap::default recipe.

Default behaviour is preserved, and RHEL >= 8 servers will continue to install the repository.

Issues Resolved

None created.

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.

On RHEL >= 8 the resource is configuring the OSUSOL repository to
support openldap-server install.
However in some cases you may not need this specific repository.

Now by setting `install_repository false` on the resource, the repo
won't be installed.
Also the `openldap.package_install_repository` attributes allow to
control the value set to this property in the openldap::default recipe.

Default behaviour is preserved, and RHEL >= 8 servers will continue to
install the repository.

Signed-off-by: b.courtois <b.courtois@criteo.com>
@Annih Annih requested a review from a team as a code owner July 6, 2023 22:19
@xorima xorima added the Release: Minor Release to Chef Supermarket as a minor release when merged label Jul 7, 2023
@xorima xorima merged commit 03942e2 into sous-chefs:main Jul 7, 2023
7 of 8 checks passed
@kitchen-porter
Copy link
Contributor

Released as: 6.1.0

@Annih
Copy link
Contributor Author

Annih commented Jul 7, 2023

So efficient @xorima , thanks!

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

Successfully merging this pull request may close these issues.

None yet

3 participants