seantronsen/replace minio #70

seantronsen · 2026-01-21T19:09:21Z

Pull Request Template

Thank you for your contribution! Please ensure the following before submitting:

Checklist

My code follows the style guidelines of this project
I have added/updated comments where needed
I have added tests that prove my fix is effective or my feature works
I have run make test (or equivalent) locally and all tests pass
DCO Sign-off: All commits are signed off (git commit -s) with my real name and email
REUSE Compliance:
- Each new/modified source file has SPDX copyright and license headers
- Any non-commentable files include a <filename>.license sidecar
- All referenced licenses are present in the LICENSES/ directory

Description

This is a port of PR OpenCHAMI/tutorial-2025#12 which targeted: OpenCHAMI/tutorial-2025#11

Below is a modified variant of the description listed in the original PR.

The primary purpose of this PR is to switch the tutorial from minio to versitygw as the former is terminating support for open-source. To this end, the existing openchami quadlet for versitygw was introduced. Some additional updates were introduced and are listed below:

adds instructions for coredns Corefile to address port conflicts with dnsmasq and aardvark
adds aws CLI to address s3cmd XML schema incompatibilities for setacl
ensures all relevant shell output blocks are up to date w.r.t the latest openchami releases at the time of writing

Type of Change

Bug fix
New feature
Breaking change
Documentation update

For more info, see Contributing Guidelines.

davidallendj · 2026-01-21T19:20:27Z

Looks good from a glance. I want to give it a try in a bit for next week.

synackd

Initial review before testing.

There seems to be a bug with the GitHub UI that prevents me from commenting on multiple lines under certain conditions so I've commented on one line for some of these.

content/docs/tutorial.md

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

…more info Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

davidallendj

I've tested this and was able to boot a node using Versity S3 instead of MinIO.

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

content/docs/tutorial.md

synackd

Review for DNS port conflict warning.

content/docs/tutorial.md

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

synackd · 2026-01-27T18:15:17Z

All of the threads have been resolved, but I would like to test this again on both the cloud and VM setups before approving.

Also, I'd like to get @davidallendj's blessing as well since he will be presenting the tutorial soon.

synackd

Ran through the tutorial once more and figured we might as well update the kernel versions while we have actual output. Probably most of the comments are that. 🙂

content/docs/tutorial.md

davidallendj · 2026-01-28T00:29:09Z

All of the threads have been resolved, but I would like to test this again on both the cloud and VM setups before approving.

Also, I'd like to get @davidallendj's blessing as well since he will be presenting the tutorial soon.

I'm going to run it again after we present tomorrow on Thursday. When I ran through it before though, it looked good, but I'll look at the changes and see if there's anything that stands out.

synackd · 2026-01-28T00:32:09Z

Cool. I ran through this today both on Jetstream2 and Libvirt VMs and, with my changes, everything worked.

(There was one thing with the Kickstart stuff for the head node VM where some packages didn't get installed, but that's a problem for a different PR.)

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> update with rocky 9.7 - squash me Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> update with rocky 9.7 - squash me Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> update with rocky 9.7 - squash me Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> update with rocky 9.7 - squash me Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> update with rocky 9.7 - squash me Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

seantronsen · 2026-01-28T01:42:56Z

Cool. I ran through this today both on Jetstream2 and Libvirt VMs and, with my changes, everything worked.

(There was one thing with the Kickstart stuff for the head node VM where some packages didn't get installed, but that's a problem for a different PR.)

Noted, everything should be all good to go now, but I'll double check it once more before I call it a night.

seantronsen · 2026-01-28T04:30:52Z

Testing completed: Instructions for the bare-metal and cloud-instance paths function without any issues.

There are some discrepancies in CLI output where sometimes rocky9.6 is referenced despite the recent efforts to update all such instances to 9.7. It's nothing that will affect the tutorial though.

Still, I will take a moment to look for all such instances and make a quick swap since the effort to change all outputs to 9.7 is already underway.

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

seantronsen · 2026-01-28T04:46:26Z

Testing completed: Instructions for the bare-metal and cloud-instance paths function without any issues.

There are some discrepancies in CLI output where sometimes rocky9.6 is referenced despite the recent efforts to update all such instances to 9.7. It's nothing that will affect the tutorial though.

Still, I will take a moment to look for all such instances and make a quick swap since the effort to change all outputs to 9.7 is already underway.

Complete as of: 1fca7f1

synackd

Caught one of my own errors...

content/docs/tutorial.md

Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>

synackd

Approving since all of my comments were addressed and I have tested this myself. Refraining from merging until @davidallendj reviews.

seantronsen changed the title ~~Seantronsen/replace minio~~ seantronsen/replace minio Jan 21, 2026

synackd requested review from davidallendj and synackd January 21, 2026 20:17

synackd requested changes Jan 21, 2026

View reviewed changes

davidallendj reviewed Jan 21, 2026

View reviewed changes

content/docs/tutorial.md Outdated Show resolved Hide resolved

seantronsen added 12 commits January 22, 2026 08:30

add awscli as required dependency in part 0

e4d5919

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

replace mentions of minio with vgw in part 01

79766eb

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

update output for ochami version

a285d05

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

add port reassignment for coredns and explanation

8c00f4f

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

update configuration instructions for s3 tooling

389cc7f

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

update instructions for S3 bucket creation

15d81fd

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

update s3cmd ls output for accuracy and add instructions for getting …

25bf5ea

…more info Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

replace minio port with versitygw server port

4f14e27

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

update image build commands to include vgw auth

d4d6231

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

update smd component get output

2afba79

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

replace remaining mentions of minio

7afa282

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

add clarification comment to vgw gensecrets

7a57dcb

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

seantronsen force-pushed the seantronsen/replace-minio branch from 7f86de9 to 7a57dcb Compare January 22, 2026 15:32

seantronsen added 2 commits January 22, 2026 09:21

update language about coredns port change

09e5236

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

convert credential export reminder to code block

5f1ca68

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

davidallendj reviewed Jan 22, 2026

View reviewed changes

seantronsen added 4 commits January 22, 2026 11:02

use named endpoints when possible

22a4195

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

disable dnsmasq on virtual net and remove port change mandate

ab5c907

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

adjust language for port conflicts below corefile config

6731110

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

remove sudo requirements from image builder alias

290824f

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

synackd reviewed Jan 26, 2026

View reviewed changes

content/docs/tutorial.md Outdated Show resolved Hide resolved

synackd requested changes Jan 26, 2026

View reviewed changes

content/docs/tutorial.md Outdated Show resolved Hide resolved

content/docs/tutorial.md Outdated Show resolved Hide resolved

content/docs/tutorial.md Outdated Show resolved Hide resolved

seantronsen added 2 commits January 26, 2026 17:52

apply suggested language addition from @synackd

a5c60aa

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

format text block

11ed160

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

change port conflict note callout style to warning

1130253

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

synackd requested changes Jan 27, 2026

View reviewed changes

seantronsen and others added 7 commits January 27, 2026 18:35

dns port conflict warning language adjustment

1d81beb

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

format text block

513c7f5

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

swap template with real path for s3 credentials error msg

0f5b121

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

fix comma nitpick

249a9a5

Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

link to coredns docs and improve readability

3ce45c7

Co-authored-by: Devon Bautista <17506592+synackd@users.noreply.github.com> Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

adjust curl cli to auto pull latest vgw for host os

932182b

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

seantronsen force-pushed the seantronsen/replace-minio branch from b8883bd to 932182b Compare January 28, 2026 01:35

swap all remaining references to rocky 9.6 to 9.7

1fca7f1

Signed-off-by: Sean Tronsen <sean.tronsen@gmail.com>

synackd requested changes Jan 28, 2026

View reviewed changes

content/docs/tutorial.md Outdated Show resolved Hide resolved

remove extra quotes, correct paths in kernel/initrd uris

784ca84

Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>

synackd force-pushed the seantronsen/replace-minio branch from 22dc140 to 784ca84 Compare January 28, 2026 04:52

synackd approved these changes Jan 28, 2026

View reviewed changes

seantronsen/replace minio #70

Are you sure you want to change the base?

seantronsen/replace minio #70

Conversation

seantronsen commented Jan 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request Template

Checklist

Description

Type of Change

Uh oh!

davidallendj commented Jan 21, 2026

Uh oh!

synackd left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

davidallendj left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

synackd left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

synackd commented Jan 27, 2026

Uh oh!

synackd left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

davidallendj commented Jan 28, 2026

Uh oh!

synackd commented Jan 28, 2026

Uh oh!

seantronsen commented Jan 28, 2026

Uh oh!

seantronsen commented Jan 28, 2026

Uh oh!

seantronsen commented Jan 28, 2026

Uh oh!

synackd left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

synackd left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

seantronsen commented Jan 21, 2026 •

edited

Loading