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.

Sign up for GitHub

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

Jump to bottom

[release-1.1] launcher, source: Log migration info by default #10820

Merged
merged 1 commit into from
Dec 3, 2023
Merged

[release-1.1] launcher, source: Log migration info by default #10820

merged 1 commit into from
Dec 3, 2023

Conversation

kubevirt-bot
Copy link
Contributor

This is an automated cherry-pick of #10809

/assign orelmisan

Source virt-launcher: Log migration info by default

@orelmisan
launcher, source: Log migration info by default
5177506 
Currently, the migration info of the source pod
is logged from log level 4.
This information is helpful when debugging problematic
migrations, and is not logged by default.

Log this information with log level to 2, in order for it
to be printed by default.

The log is printed every four seconds.

Signed-off-by: Orel Misan <omisan@redhat.com>
@kubevirt-bot kubevirt-bot mentioned this pull request Nov 30, 2023
Merged
8 tasks
@kubevirt-bot kubevirt-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XS labels Nov 30, 2023
@orelmisan
Copy link
Member

/cc @acardace

@orelmisan
Copy link
Member

/retest-required

@kubevirt-bot
Copy link
Contributor Author

kubevirt-bot commented Dec 1, 2023 •
edited

@kubevirt-bot: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-kubevirt-e2e-arm64 5177506 link false /test pull-kubevirt-e2e-arm64-1.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

RamLavi
RamLavi approved these changes Dec 3, 2023
View reviewed changes
Copy link
Contributor

@RamLavi RamLavi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@kubevirt-bot kubevirt-bot added the lgtm Indicates that a PR is ready to be merged. label Dec 3, 2023
@enp0s3
Copy link
Contributor

enp0s3 commented Dec 3, 2023

/approve

@kubevirt-bot
Copy link
Contributor Author

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: enp0s3

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubevirt-bot kubevirt-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 3, 2023
@orelmisan
Copy link
Member

/retest-required

@kubevirt-commenter-bot
Copy link

/retest-required
This bot automatically retries required jobs that failed/flaked on approved PRs.
Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

@kubevirt-bot kubevirt-bot merged commit 20cf383 into kubevirt:release-1.1 Dec 3, 2023
35 of 36 checks passed
@kubevirt-bot kubevirt-bot mentioned this pull request Jan 2, 2024
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RamLavi RamLavi RamLavi approved these changes

@iholder101 iholder101 Awaiting requested review from iholder101

@vasiliy-ul vasiliy-ul Awaiting requested review from vasiliy-ul

@acardace acardace Awaiting requested review from acardace

Assignees

@orelmisan orelmisan

@RamLavi RamLavi

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@kubevirt-bot @orelmisan @enp0s3 @kubevirt-commenter-bot @RamLavi