Bug 1644689 - kubevirt-apb doesn't use cdi manifests from kubevirt-cdi-manifests rpm to deploy CDI
Summary: kubevirt-apb doesn't use cdi manifests from kubevirt-cdi-manifests rpm to dep...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Installation
Version: 1.3
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 1.3
Assignee: Ryan Hallisey
QA Contact: Lukas Bednar
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-10-31 12:15 UTC by Lukas Bednar
Modified: 2018-12-05 18:57 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-12-05 18:56:56 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2018:3776 0 None None None 2018-12-05 18:57:10 UTC

Description Lukas Bednar 2018-10-31 12:15:51 UTC
Description of problem:

kubevirt-apb doesn't use cdi manifests from kubevirt-cdi-manifests rpm to deploy CDI.
Instead we use manifests from kubevirt-ansible/roles/cdi/templates which is not what was shipped.

Version-Release number of selected component (if applicable):
kubevirt-apb-v1.3.0-3


How reproducible: 100%


Steps to Reproduce:
1. Deploy KubeVirt using APB

Actual results:
kubevirt-apb is using manifests from kubevirt-ansible to deploy CDI, instead of using manifests from kubevirt-cdi-manifests rpm.

/usr/share/kubevirt-cdi/manifests/cdi-controller.yaml

Expected results:

kubevirt-apb should use manifests from kubevirt-cdi-manifests to deploy CDI, not from kubevirt-ansible.

/usr/share/kubevirt-cdi/manifests/cdi-controller.yaml

Additional info:

Comment 1 Nelly Credi 2018-11-06 14:03:06 UTC
@Lukas, is there a diff between the tempaltes which is causing functional issue?

Comment 2 Lukas Bednar 2018-11-06 14:22:38 UTC
@Nelly I am not aware of that, but I am afraid that it does not matter.
Templates which are used by APB are not the ones which were shipped in manifest rpm.

In addition I can see that manifest rpm is being installed into APB container, and then it is not used.

It doesn't feel right ... we should neither 1) remove that rpm from apb and use template from kubevirt-ansible. 2) OR add template to rpm and use it from apb .

I don't mind what why we will go, but we should make it straight.

Comment 3 Ryan Hallisey 2018-11-06 17:09:34 UTC
I changed it to use kubevirt-cdi-manifests.  Build is blocked by https://projects.engineering.redhat.com/browse/FACTORY-3482

Comment 4 Ryan Hallisey 2018-11-08 15:34:27 UTC
brew-pulp-docker01.web.prod.ext.phx2.redhat.com:8888/cnv-tech-preview/kubevirt-apb:v1.3.0-5

Comment 5 Lukas Bednar 2018-11-13 12:07:06 UTC
Working with kubevirt-apb-v3.11-4 it looks good there.

Comment 7 errata-xmlrpc 2018-12-05 18:56:56 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, 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-2018:3776


Note You need to log in before you can comment on or make changes to this bug.