Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BACKPORT] Add launcher pods logs #24

Merged
merged 8 commits into from Nov 10, 2021

Conversation

aufi
Copy link
Collaborator

@aufi aufi commented Nov 10, 2021

2.2 backport of #16

Extending both full and targeted must-gather log gathering with logs of VM's virt-launcher pods.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1995075

aufi and others added 8 commits October 27, 2021 14:09
Extending full must-gather with gathering logs of VM Launcher pods.

Related to https://bugzilla.redhat.com/show_bug.cgi?id=1995075
Update VM parsing to use Plan instead of VMImport
Adding information about empty files filtering of targeted must-gather
to README.
Removing golang image from docker registry which was used for pprof
gathering which is not part of forklift-must-gather.
Forklift 2.2 replaced VMIO functions, adapting the gathering code to
correctly parsing VM-related objects (VM itself, DVs). As a sideeffect, VM name (instead
of within-plan ID) is used be passed in parameters (as Forklift UI
already does).

Fixes BZs:
https://bugzilla.redhat.com/show_bug.cgi?id=2019307
https://bugzilla.redhat.com/show_bug.cgi?id=1995197
https://bugzilla.redhat.com/show_bug.cgi?id=2020014
Adding virt-launcher pod logs to be captured for all matching VMs. Note,
the launcher pod logs are dedicated to their VMs, so there is no
additional filtering applied to the given launcher pod logs.

Related to https://bugzilla.redhat.com/show_bug.cgi?id=1995075
@aufi
Copy link
Collaborator Author

aufi commented Nov 10, 2021

Sorry for messy commits in the PR, code changes are OK.

@aufi aufi merged commit 172ca7d into kubev2v:release-v2.2.0 Nov 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant