k8snetworkplumbingwg / multus-cni
63%
master: 50%

LAST BUILD BRANCH: CORENET-6353
DEFAULT BRANCH: master
Repo Added
Token ci5ZcGVrgFkswX3MaFriGX24zhmxxpDDa regen
Build 1009 Last
Files 26
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH failure-logs
branch: failure-logs
CHANGE BRANCH
x
Sync Branches

coverage: 62.778% (+0.03%) from 62.749%
8088657622

Pull #1236

github

igsilya 
server: More concise error messages.

On the CNI request failure, multus-cni prints out cmdArgs.  In all
cases, except for debug printing, this is done with %s and a special
printing function.  However, the handleCNIRequest is an exception for
some reason.  That leads to unintelligible error messages in case
of CNI request failures (severely abridged):

 CmdAdd (shim): CNI request failed with status 400:
 '&{ContainerID:<id> Netns:/var/run/netns/<uuid> IfName:eth0
    Args:<args> Path: StdinData:[125 121 111 117 114 32 97 100 118
    101 114 116 105 115 101 109 101 110 116 32 99 111 117 108 100
    32 98 101 32 104 101 114 101 125 ... another 650 numbers ]}
 ContainerID:"<id>" Netns:"/var/run/netns/<uuid>" IfName:"eth0"
 Args:"<args>" Path:"" ERRORED: error configuring pod ...

printCmdArgs() should be used for this case as well to avoid huge
hardly readable logs.

At the same time, the content of cniCmdArgs is always appended to
the error twice as seen in the example above.  The first time by the
HandleCNIRequest and another time by the handleCNIRequest.  Same for
the HandleDelegateRequest path.

Just removing the prefixing from the lower level handlers while
keeping higher level ones.  The 'ERRORED' part migrated to the higher
level handler functions to preserve the overall look of the error.

Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Pull Request #1236: server: More concise error messages.

1 of 4 new or added lines in 1 file covered. (25.0%)

3 existing lines in 1 file now uncovered.

2260 of 3600 relevant lines covered (62.78%)

16.39 hits per line

Relevant lines Covered
3600 RELEVANT LINES 2260 COVERED LINES
16.39 HITS PER LINE
Source Files on failure-logs

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
8088657622 failure-logs server: More concise error messages. On the CNI request failure, multus-cni prints out cmdArgs. In all cases, except for debug printing, this is done with %s and a special printing function. However, the handleCNIRequest is an exception for som... Pull #1236 igsilya github
62.78
See All Builds (504)

Badge your Repo: multus-cni

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh