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

Clean server install resource #363

Conversation

fernandoaleman
Copy link
Contributor

@fernandoaleman fernandoaleman commented Apr 10, 2023

Description

Remove unused properties from server_install resource

Removed the following properties:

  • mycnf_file
  • extconf_directory
  • data_directory
  • external_pid_file
  • port
  • initdb_locale

Updated documentation to reflect the changes.

Fix default pid file for rhel, fedora and amazon

Updated default mariadb version to 10.11 in all tests

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.

Removed the following properties:

- mycnf_file
- extconf_directory
- data_directory
- external_pid_file
- port
- initdb_locale

Updated documentation to reflect the changes.
@fernandoaleman fernandoaleman marked this pull request as ready for review April 10, 2023 21:01
@fernandoaleman fernandoaleman requested a review from a team as a code owner April 10, 2023 21:01
@damacus
Copy link
Member

damacus commented Apr 11, 2023

Looks good!

can you remove amalon-linux2 from the Ci matrix too, please?

@damacus
Copy link
Member

damacus commented Apr 12, 2023

@fernandoaleman I've rebased. Can you double check that makes sense please?

@damacus damacus added the Release: Patch Release to Chef Supermarket as a version patch when merged label Apr 12, 2023
- Update documentation
- Set tests to mariadb version 10.11
- Remove tests for mariadb versions 10.3 and 10.4
@fernandoaleman
Copy link
Contributor Author

@damacus I added another commit defaulting mariadb to version 10.11 in all the testing and removing tests on version 10.3 and 10.4. However, I noticed that client_install tests are failing on Debian and Ubuntu. I'll take a look at those tomorrow and submit a fix when ready.

@fernandoaleman
Copy link
Contributor Author

@damacus This PR keeps failing CI because of the check-metadata job, I'm guessing it's because it's from a fork. Would you like me to close this PR and re-submit from a sous-chefs branch instead of a fork?

@damacus
Copy link
Member

damacus commented Apr 13, 2023

If you could for now until I fix the logic I'm not sure which bit of the permissions is failing right now.

@fernandoaleman
Copy link
Contributor Author

@damacus Closing this PR and re-opening PR #365 in order to get past the CI check-metadata failures.

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.

None yet

2 participants