Bug 2023680 - "Importer" and "Conversion" pod logs in namespaces other than "openshift-mtv" not included in logs downloaded by using the web console.
Summary: "Importer" and "Conversion" pod logs in namespaces other than "openshift-mtv"...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Migration Toolkit for Virtualization
Classification: Red Hat
Component: Must-Gather
Version: 2.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: 2.2.0
Assignee: Marek Aufart
QA Contact: Maayan Hadasi
Avital Pinnick
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-11-16 10:43 UTC by Maayan Hadasi
Modified: 2021-12-09 19:21 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-12-09 19:21:12 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2021:5066 0 None None None 2021-12-09 19:21:20 UTC

Description Maayan Hadasi 2021-11-16 10:43:02 UTC
Description of problem:
When the target namespace is set to another namespace than openshift-mtv in the migration plan, the importer and conversion pods are not included in the logs file downloaded from UI (Plans/VMs page)


Version-Release number of selected component (if applicable):
MTV 2.2.0-84 / iib:133381


How reproducible:
100%


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Marek Aufart 2021-11-18 11:57:03 UTC
Fix for virt-launcher was merged and backported https://github.com/konveyor/forklift-must-gather/pull/26

CDI-imported are subject os BZ/RFE https://bugzilla.redhat.com/show_bug.cgi?id=2024506

Comment 2 Marek Aufart 2021-11-19 15:01:56 UTC
If the virt-launcher pod logs are missing, please check first if the pod is even present in the cluster. If the virt-launcher pod is expected, but not found in given namespace, must-gather command prints warning bellow:

...
[must-gather-vrcm4] POD 2021-11-19T09:23:39.899903093Z Virt-launcher Pod for name-space,v2v-migration-warm-rhel8-2disks2nics doesn't exist, skipping.
...

Comment 3 Fabien Dupont 2021-11-20 07:36:43 UTC
Please verify with the mtv-operator-bundle-container-v2.2.0-90 / iib:136635, or later.

Comment 4 Maayan Hadasi 2021-11-22 11:49:48 UTC
The issue was tested with MTV build 2.2.0-90 / iib:136635 and looks like it was partially fixed.
When the target namespace is other than openshift-mtv, CDI-importer logs are collected but there are no virt-v2v conversion logs

Comment 5 Marek Aufart 2021-11-22 16:22:07 UTC
Sorry for misunderstanding about the log from virt-v2v conversion pods vs. virt-launcher/CDI-import pods.

Fix PR is up https://github.com/konveyor/forklift-must-gather/pull/32

Comment 6 Marek Aufart 2021-11-23 09:14:35 UTC
Backported in https://github.com/konveyor/forklift-must-gather/pull/33

Comment 7 Maayan Hadasi 2021-11-24 14:36:19 UTC
The issue is reproducible using MTV 2.2.0-96 / iib:137780
virt-v2v conversion logs are not included in the logs file downloaded from UI

Moving back to ASSIGNED

Comment 8 Marek Aufart 2021-11-24 14:39:51 UTC
Thanks for reporting it, there are outdated (not latest) must-gather images in the deployment.

Comment 9 Fabien Dupont 2021-11-24 22:55:51 UTC
Seems that we had a race condition between two builds and the wrong commit was used to build 2.2.0-96.

Please verify with the mtv-operator-bundle-container-v2.2.0-97 / iib:138532, or later.

Comment 10 Maayan Hadasi 2021-11-25 10:45:59 UTC
Verification is blocked due to Bug 2026620 - [Targeted must-gather][UI] Cannot complete must-gather for migration plan/VM

Comment 11 Fabien Dupont 2021-11-26 15:36:04 UTC
Please verify with mtv-operator-bundle-container-2.2.0-100 / ii:138977, or later.

Comment 12 Maayan Hadasi 2021-11-28 09:34:55 UTC
@maufart Hi,
Using MTV 2.2.0-100, I can see that for a migration plan with 2 VMs, one of the available virt-v2v conversion logs is missing (mig-plan-cold-vm-8325-ksqld):

$ tree
.
└── must-gather
    └── namespaces
        ├── mguetta
        │   ├── crs
        │   │   ├── datavolume
        │   │   │   ├── mig-plan-cold-vm-7958-gbx4x.yaml
        │   │   │   ├── mig-plan-cold-vm-7958-n2vd5.yaml
        │   │   │   └── mig-plan-cold-vm-8325-jhx7d.yaml
        │   │   └── virtualmachine
        │   │       ├── mguetta-2disks-for-warm.yaml
        │   │       └── mguetta-win2019.yaml
        │   └── logs
        │       ├── importer-mig-plan-cold-vm-7958-gbx4x
        │       │   └── current.log
        │       ├── importer-mig-plan-cold-vm-7958-n2vd5
        │       │   └── current.log
        │       ├── importer-mig-plan-cold-vm-8325-jhx7d
        │       │   └── current.log
        │       ├── mig-plan-cold-vm-7958-gd7m4
        │       │   └── current.log
        │       └── virt-launcher-mguetta-2disks-for-warm-59r5m
        │           └── current.log
        └── openshift-mtv
            ├── crs
            │   └── plan
            │       └── mig-plan-cold.yaml
            └── logs
                └── forklift-controller-547db9b7f9-txfv8
                    └── current.log

Comment 13 Marek Aufart 2021-11-29 09:40:54 UTC
THanks for report and proving env for checking, you're right, I'm working on fix.

Comment 14 Maayan Hadasi 2021-11-29 09:54:31 UTC
(In reply to Marek Aufart from comment #13)
> THanks for report and proving env for checking, you're right, I'm working on
> fix.

Thanks for the confirmation. Moving to ASSIGNED

Comment 15 Marek Aufart 2021-11-29 12:24:23 UTC
Fix https://github.com/konveyor/forklift-must-gather/pull/34

Comment 16 Marek Aufart 2021-11-29 14:34:29 UTC
Backported in https://github.com/konveyor/forklift-must-gather/pull/35

Comment 17 Maayan Hadasi 2021-11-30 14:46:55 UTC
Verified as fixed.

Versions:
MTV 2.2.0-102 / iib:139741
CNV 4.9.1-23 / iib:133012

Comment 20 errata-xmlrpc 2021-12-09 19:21:12 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.2.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:5066


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