Skip to content

Commit

Permalink
Update info.tcl test to revert client output limits sooner (#1462)
Browse files Browse the repository at this point in the history
We set the client output buffer limits to 10 bytes, and then execute
`info stats` which produces more than 10 bytes of output, which can
cause that command to throw an error.

I'm not sure why it wasn't consistently erroring before, might have been
some change related to the ubuntu upgrade though.

Issues related to ubuntu-tls are hopefully resolved now.

Signed-off-by: Madelyn Olson <[email protected]>
  • Loading branch information
madolson authored Dec 20, 2024
1 parent ffef236 commit b56f4f7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/unit/info.tcl
Original file line number Diff line number Diff line change
Expand Up @@ -406,10 +406,10 @@ start_server {tags {"info" "external:skip" "debug_defrag:skip"}} {
r config set client-output-buffer-limit "normal 10 0 0"
r set key [string repeat a 100000] ;# to trigger output buffer limit check this needs to be big
catch {r get key}
r config set client-output-buffer-limit $org_outbuf_limit
set info [r info stats]
assert_equal [getInfoProperty $info client_output_buffer_limit_disconnections] {1}
r config set client-output-buffer-limit $org_outbuf_limit
} {OK} {logreqres:skip} ;# same as obuf-limits.tcl, skip logreqres
} {} {logreqres:skip} ;# same as obuf-limits.tcl, skip logreqres

test {clients: pubsub clients} {
set info [r info clients]
Expand Down

0 comments on commit b56f4f7

Please sign in to comment.