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
OCPBUGS-3614: [4.11] CARRY: client: don't construct transaction log string when it's not needed #1374
OCPBUGS-3614: [4.11] CARRY: client: don't construct transaction log string when it's not needed #1374
Conversation
@dcbw: No Bugzilla bug is referenced in the title of this pull request. In response to this:
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. |
@dcbw: No Bugzilla bug is referenced in the title of this pull request. In response to this:
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. |
@dcbw: This pull request references Jira Issue OCPBUGS-3614, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
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. |
@dcbw: No Bugzilla bug is referenced in the title of this pull request. In response to this:
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. |
@dcbw why not bump it it like in 4.12? |
/retest |
47f64b3
to
a80d1a9
Compare
@jcaamano did the revendor, PTAL thanks! |
/retest |
/lgtm |
/jira refresh |
@dcbw: This pull request references Jira Issue OCPBUGS-3614, which is valid. 6 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
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. |
Pulls in the following commits, for which we really just care about the client logging one for performance. 64a1543b160e3 client: don't construct transaction log string when it's not needed 14b9f5b67be30 Make table name a const above generated model b7a2d2d49c46d Fix register metrics once Signed-off-by: Dan Williams <dcbw@redhat.com>
a80d1a9
to
3e896a2
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dcbw, jcaamano 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 |
/retest-required |
/test 4.11-upgrade-from-stable-4.10-e2e-aws-ovn-upgrade |
/test e2e-aws-ovn-local-to-shared-gateway-mode-migration |
/label cherry-pick-approved |
/hold Revision 3e896a2 was retested 3 times: holding |
/hold cancel |
@dcbw: Overrode contexts on behalf of dcbw: ci/prow/e2e-metal-ipi-ovn-dualstack In response to this:
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. |
/override ci/prow/e2e-aws-ovn-upgrade not a network thing
|
@dcbw: Overrode contexts on behalf of dcbw: ci/prow/e2e-aws-ovn-upgrade In response to this:
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. |
@dcbw: The following tests failed, say
Full PR test history. Your PR dashboard. 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. |
@dcbw: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-3614 has been moved to the MODIFIED state. In response to this:
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. |
Backport of ovn-org/libovsdb#342 (see also 4.12 version #1348)
fmt.Sprintf() will be evaluated before the logging function is called, which means that even if the logging level is less than V(4) we'll still construct the whole string for the operation, just to discard it. That can be pretty expensive.