Bug 1947023

Summary:	Update API versions
Product:	Migration Toolkit for Virtualization	Reporter:	Fabien Dupont <fdupont>
Component:	Controller	Assignee:	Jeff Ortel <jortel>
Status:	CLOSED ERRATA	QA Contact:	Amos Mastbaum <amastbau>
Severity:	medium	Docs Contact:	Avital Pinnick <apinnick>
Priority:	high
Version:	2.0.0	CC:	istein, mguetta
Target Milestone:	---
Target Release:	2.0.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1948601 (view as bug list)		Environment:
Last Closed:	2021-06-10 17:11:46 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1948601

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