Bug 1947023 - Update API versions
Summary: Update API versions
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Migration Toolkit for Virtualization
Classification: Red Hat
Component: Controller
Version: 2.0.0
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ---
: 2.0.0
Assignee: Jeff Ortel
QA Contact: Amos Mastbaum
Avital Pinnick
URL:
Whiteboard:
Depends On:
Blocks: 1948601
TreeView+ depends on / blocked
 
Reported: 2021-04-07 14:05 UTC by Fabien Dupont
Modified: 2021-06-24 21:03 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1948601 (view as bug list)
Environment:
Last Closed: 2021-06-10 17:11:46 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2021:2381 0 None None None 2021-06-10 17:11:56 UTC

Description Fabien Dupont 2021-04-07 14:05:10 UTC 
Description of problem:

Kubernetes API version for CustomResourceDefinition is now v1. So, Forklift/MTV should use this API version when declaring its CRDs.

MTV 2.0.0 is our first stable version. MTV API version should be v1beta1.
Comment 1 Fabien Dupont 2021-04-07 14:07:01 UTC 
From a QE point of view, it means that the API calls to create the MTV CRs should reference forklift.konveyor.io/v1beta1 API version.
Comment 2 Jeff Ortel 2021-04-12 12:53:41 UTC 
https://github.com/konveyor/forklift-controller/pull/206
Comment 3 Fabien Dupont 2021-04-16 07:32:51 UTC 
(In reply to Fabien Dupont from comment #1)
> From a QE point of view, it means that the API calls to create the MTV CRs
> should reference forklift.konveyor.io/v1beta1 API version.

Ok, so that's incorrect, since this PR only changes the CRD definition. The test for this BZ is only regression test.
Comment 4 Fabien Dupont 2021-04-16 07:33:05 UTC 
The fix should be part of build 2.0.0-17 / iib:66911.
Comment 5 Amos Mastbaum 2021-04-25 15:49:56 UTC 
@fdupont 
I am not sure what to test here

in 2.0.0-20:


"...no matches for kind "Plan" in version "forklift.konveyor.io/v1beta1"



are we missing something?
Comment 6 Fabien Dupont 2021-04-27 19:37:23 UTC 
Yes, I wrote in comment #3 that this should only be regression tests.
Comment 7 Ilanit Stein 2021-05-05 14:38:27 UTC 
Tested the move to apiVersion v1beta1 on mtv-operator-bundle-container-2.0.0-4 
on MTV installation and in plan/migration CRs
Comment 10 errata-xmlrpc 2021-06-10 17:11:46 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (MTV 2.0.0 images), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2021:2381
Note You need to log in before you can comment on or make changes to this bug.