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

Provider cleanup #723

Merged
merged 3 commits into from
Apr 4, 2024
Merged

Provider cleanup #723

merged 3 commits into from
Apr 4, 2024

Conversation

stevenpost
Copy link
Contributor

@stevenpost stevenpost commented Mar 29, 2024

This PR cleans up the provider.

A detailed list of changes:

  • Remove deprecated SSL options (use TLS instead)
  • Support pure yaml config in provider
  • Refactor the replicaset provider to improve maintainability

witjoh
witjoh previously approved these changes Mar 29, 2024
@stevenpost stevenpost force-pushed the provider_cleanup branch 3 times, most recently from 09a0517 to ef7b2f2 Compare April 2, 2024 07:25
@stevenpost stevenpost marked this pull request as draft April 2, 2024 13:59
@stevenpost
Copy link
Contributor Author

stevenpost commented Apr 2, 2024

I decided to split this into 2 parts. A 'safe' part, where it is just cleanup (#726), but no real changes in behaviour and another 'unsafe' part with changes in behaviour. The second may provoke a bit more discussion and is this PR.

@witjoh witjoh dismissed their stale review April 2, 2024 14:18

This PR will be split by @stevenpost

@stevenpost stevenpost force-pushed the provider_cleanup branch 4 times, most recently from 8cb718d to 1250922 Compare April 3, 2024 09:33
@stevenpost stevenpost marked this pull request as ready for review April 3, 2024 09:58
@stevenpost
Copy link
Contributor Author

I did some refactoring of the replicaset provider to improve maintainability.

My current test setup and requirements:

  • a 3 node cluster
  • PuppetDB
  • Puppet needs to run only once to fully set up a node (and extend the replicaset)

Test procedure:

  • Spin up 3 empty VMs
  • Run Puppet on all three nodes sequentially

The node will only have itself in the replicaset.
The second and third nodes will extend the existing replicaset.

@stevenpost stevenpost requested a review from witjoh April 3, 2024 10:18
@h-haaks
Copy link
Contributor

h-haaks commented Apr 3, 2024

Is it possible for you to add your setup in a beaker hosts file under spec/acceptance/nodesets?

@stevenpost
Copy link
Contributor Author

Is it possible for you to add your setup in a beaker hosts file under spec/acceptance/nodesets?

Not sure how to go about that, but I'll give it a shot.

@h-haaks
Copy link
Contributor

h-haaks commented Apr 3, 2024

Hmm, this a994a6c was not what I had in mind :)
I was talking about creating a beaker hosts yaml file in the spec/acceptanse/nodesets folder with 3 host.
This file could then be used with bundle exec rake beaker

I suppose its better to do this in a separate PR :)

@stevenpost
Copy link
Contributor Author

Hmm, this a994a6c was not what I had in mind :) I was talking about creating a beaker hosts yaml file in the spec/acceptanse/nodesets folder with 3 host. This file could then be used with bundle exec rake beaker

I suppose its better to do this in a separate PR :)

I moved that commit into a separate PR (#729). I think I understand what you mean, no clue on how to do that, though. Also, I don't see what will be actually tested. Can you expand on that (perhaps in the new PR)?

@h-haaks h-haaks merged commit 04e39f3 into voxpupuli:master Apr 4, 2024
48 checks passed
@stevenpost stevenpost deleted the provider_cleanup branch April 4, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants