New crVersion key in ``deploy/cr.yaml`` to indicate the API version that the Custom Resource corresponds to

Description

I believe I have discovered a bug that breaks all PXC deployments submitted via the operator API. Any users who rely on the API to create PXCs will have a very bad time when attempting to upgrade to v1.5.0, as the bug is quite deep and no obvious workarounds are apparent to me. Hence the critical designation.

The bug's reproduction and my notes can be found here: https://github.com/MichaelSaah/operator-api-bug-repro

Start with "reproduction-steps.md" and follow with "notes.md".

Environment

None

Smart Checklist

Activity

Jira Bot September 29, 2020 at 8:56 PM

To:
CC:

Hi, I'm jira-bot, Percona's Jira automation tool. I've detected that someone from
Percona has made an edit to the Summary field of an issue that you reported.

I'm not sentient (yet) so I'm not sure whether the person fixed a typo, changed
a few words, or completely rewrote the text. In any case, it is Percona Engineering's
intention to make the Summary and Description of an issue as accurate as possible
so that we're fixing the actual problem you're encountering, and to avoid
misunderstandings about symptoms and causes.

If the current Summary does not accurately reflect the problem you are reporting,
or if you feel the change was otherwise inappropriate in some way, please add a
new comment explaining things and we'll address it as soon as we can.

This message will be added only once per issue, regardless of how many times
the Summary is edited.

message-code:summary-edited

Davi S Evangelista August 7, 2020 at 1:53 PM

I can confirm the workaround works... I also agree it does not resemble like proper solution

Michael Saah August 6, 2020 at 1:11 PM

While we can certainly do what you suggest as a workaround, it is just a workaround. Is there any intention to address the root cause?

Mykola Marzhan August 6, 2020 at 7:32 AM

Hi ,

 

CR object should have API version inside the annotation, it is set by kubectl by default, but if we use direct API calls it should be included manually.

you can create object from "kubectl apply" command and get object content with the next command

or please find the same field content in the upgrade doc

Done

Details

Assignee

Reporter

Time tracking

1w 4d 2h 55m logged

Fix versions

Affects versions

Priority

Smart Checklist

Created August 1, 2020 at 12:34 AM
Updated March 5, 2024 at 6:10 PM
Resolved September 3, 2020 at 8:15 AM