Skip to content

Conversation

mariadb-mmontes
Copy link
Member

⚠️ Work in progress ⚠️, please don't merge!

@dbart dbart self-assigned this Aug 19, 2025
@mariadb-mmontes mariadb-mmontes force-pushed the mariadb-enterprise-operator-25.08 branch from 3b5a8ec to 19bc421 Compare August 21, 2025 17:12
@mariadb-mmontes mariadb-mmontes force-pushed the mariadb-enterprise-operator-25.08 branch from 7d1ea54 to 03deabe Compare August 22, 2025 09:13
Copy link

@mariadb-stefangenov mariadb-stefangenov left a comment

Choose a reason for hiding this comment

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

Some suggestions and a small typo

```bash
helm uninstall mariadb-enterprise-operator
```
Alternatively, you may only downscale and delete the webhook configurations:

Choose a reason for hiding this comment

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

I wonder if it will be easier to provide a single main migration path and provide "Alternatives" in a separate secition? This should make it more readable

Copy link
Member Author

Choose a reason for hiding this comment

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

I think jumping to an Alternatives section would break the flow. The idea is that we have a no brainer guide for migrating, yet, giving some options to not fully uninstall de operator.


This guide illustrates, step by step, how to migrate to `25.8.0` from previous versions.

- Uninstall you current `mariadb-enterprise-operator` for preventing conflicts:

Choose a reason for hiding this comment

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

I wonder if we can set these as Headers ## to make clear distinction if each section?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think I've tried this approach some time ago in the community operator, but, I was feeling that the headers were kindof redundant, the comands are mostly self-explanatory, but I could be wrong. Thanks for the feedback

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.

3 participants