Skip to content

feat: add rustfs puppet module - #84

Merged
ashish1099 merged 4 commits into
masterfrom
feat/add-rustfs-puppet-module
Aug 25, 2026
Merged

feat: add rustfs puppet module#84
ashish1099 merged 4 commits into
masterfrom
feat/add-rustfs-puppet-module

Conversation

@VILJkid

@VILJkid VILJkid commented Aug 19, 2026

Copy link
Copy Markdown
Member

Signed-off-by: Sidharth Jawale sidharth@obmondo.com

@VILJkid
VILJkid requested a review from ashish1099 August 19, 2026 07:31
@VILJkid VILJkid self-assigned this Aug 19, 2026
@VILJkid
VILJkid force-pushed the feat/add-rustfs-puppet-module branch 10 times, most recently from d1e2241 to 4d2e6a8 Compare August 20, 2026 18:06
@VILJkid
VILJkid force-pushed the feat/add-rustfs-puppet-module branch from 4d2e6a8 to 50b1349 Compare August 20, 2026 18:33
configure => 'auto',
manual_config => undef,
domains => $domains,
version => '3.2.0',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it should be default, but thats fine for now

@VILJkid VILJkid Aug 21, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally yes, we shouldn't be needed to mention this, but this is done intentionally to still support haproxy 2.8 without them making any changes.
Haproxy 3.x will only be installed when provided version like this... Maybe we can refactor the code a bit to accommodate the latest version, but for now I think let's keep it as is...

@ashish1099
ashish1099 marked this pull request as ready for review August 25, 2026 07:01
@VILJkid
VILJkid marked this pull request as draft August 25, 2026 07:19
Signed-off-by: Sidharth Jawale <sidharth@obmondo.com>
@VILJkid
VILJkid force-pushed the feat/add-rustfs-puppet-module branch from 097a22e to d487131 Compare August 25, 2026 07:42
@ashish1099
ashish1099 marked this pull request as ready for review August 25, 2026 07:45
@ashish1099
ashish1099 merged commit 0da6267 into master Aug 25, 2026
1 check passed
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.

2 participants