Set db-operator version on database CR during full reconcile

[robertbublik]

• Issue: Add an information about the version of db operator that was running the last reconciliation  db-operator#79

I had to add a namespace environment variable to allow db-operator to retrieve the namespace in which it runs in. Other than that I simply added a new field version to the Database CR.

[allanger]

One small thing

[allanger]

But also, CI has failed, so it would be nice to check what's wrong

[robertbublik]

But also, CI has failed, so it would be nice to check what's wrong

Well, it looks like it's working as expected. 😅

2024-04-25T18:41:01Z ERROR an error occurred during the reconciliation {"controller": "database", "controllerGroup": "kinda.rocks", "controllerKind": "Database", "Database": {"name":"db-operator-mysql","namespace":"db-operator-whz3ob85lz"}, "namespace": "db-operator-whz3ob85lz", "name": "db-operator-mysql", "reconcileID": "5f2862b2-729f-4b6f-805b-994981395cad", "error": "Database.kinda.rocks "db-operator-mysql" is invalid: operatorVersion: Required value"}

Can I somehow insert another image to the CI?

[allanger]

Ah, sure, we need to release the db-operator versions first, sorry

[allanger]

@bobertrublik
I guess we should add a note to the notes.txt, because it's kind of breaking for ones who don't install CRDs with the chart. Also, I would update the README. Could you please add the migration plan to both of them?

[robertbublik]

Hey, I added a migration plan to the README but I don't fully understand how the NOTES.txt works. At the top there is the following text

Breaking changes and migration paths:

{{ .Chart.Version }} is not breaking, you don't have to worry

but how does it know which version is breaking and which is not? 😅

[allanger]

but how does it know which version is breaking and which is not? 😅

Just replace

{{ .Chart.Version }} is not breaking, you don't have to worry

with something like

{{ .Chart.Version }} is not breaking if CRDs are managed by helm, otherwise please read

... Migration plan ...

[robertbublik]

I hope this looks fine for you. 🙂