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

Release v7.0.0 #2367

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Release v7.0.0 #2367

wants to merge 5 commits into from

Conversation

pjuarezd
Copy link
Member

@pjuarezd pjuarezd commented Dec 12, 2024

This release will require also a sidecar release with image name quay.io/minio/operator-sidecar:v7.0.0

cesnietor
cesnietor previously approved these changes Dec 12, 2024
ramondeklein
ramondeklein previously approved these changes Dec 13, 2024
docs/notes/v7.0.0.md Outdated Show resolved Hide resolved

Some breaking changes motivates a mayor release, here is the list of breaking changes:

* remove minioJob
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the migration/upgrade path here?

What needs to change on an existing tenant for the update to work?

Or are these not expected to break workflows if you have a tenant using them?

Copy link
Member Author

Choose a reason for hiding this comment

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

For the Tenant CRD is as simple as update the Operator version, it is not going to be disruptive at all.
For the MinIO Job there is no replacement in the Operator, AIStor have a similar functionality for enterprise, the AdminJob, I would say the recommendation if MinIOJob is needed, to move to the AIStor operator

Copy link
Contributor

Choose a reason for hiding this comment

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

So if someone has an existing MinioJob YAML its just going to stop working with 7.0.

FOr PDB same - do they have to scrub PDB from tenant first?

Copy link
Member Author

Choose a reason for hiding this comment

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

right, MinIOJob will stop working on 7.0.0

Copy link
Contributor

Choose a reason for hiding this comment

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

MinIOJob is a one-time operation.

@pjuarezd Does Helm remove the CRD during upgrade? If not then we may want to add code to deal with this.

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, the Helm upgrade will remove the CRD @ramondeklein

Co-authored-by: Ravind Kumar <ravindk89@gmail.com>
@pjuarezd pjuarezd dismissed stale reviews from ramondeklein and cesnietor via 976d095 December 13, 2024 20:04
@donatello
Copy link
Member

  1. Is the sidecar here a new component?
  2. What is the release steps to build and publish it?
  3. Is minio/operator to be released with goreleaser in the usual way?

@pjuarezd
Copy link
Member Author

  1. Is the sidecar here a new component?
  2. What is the release steps to build and publish it?
  3. Is minio/operator to be released with goreleaser in the usual way?

I think we can do the release the old way, using goreleaser, sidecar/ subdir has a .goreleaser file as well.

operator sidecar is already on quay https://quay.io/repository/minio/operator-sidecar?tab=tags

@sathieu sathieu mentioned this pull request Dec 17, 2024
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.

6 participants