5
95%
main: 95%

Ran 08 Nov 2023 06:24PM UTC

Files 41

Run time 2s

Badge

Embed ▾

Committed 08 Nov 2023 05:42PM UTC coverage: 90.235% (+0.09%) from 90.146%

Job # 6802008362.5

Build Type

push

github

Committed by

thibaultcha

Commit Message

refactor(proxy-wasm) improve pwexec resurrection and instance lifecycle

The main goal of this overhaul is to simplify `on_context_create`, make
it fully re-entrant *and* properly handle instance recycling at the same
time.

The way to do so, in my opinion, was to move `pwexec` creation where
`rexec` already was. In other words, always lookup the context id in the
instance rbtree, and if not found, create it. This means that
surrounding code also needed big overhauls. It also removes the
reference counting poor man's GC of the older implementation. The code
became really ugly by then so I took the time to also review this
module's code structure instead of making a *very* ugly commit.

This new ngx_proxy_wasm.c file should be much easier to read and follow
now.

One change I do not fully like is moving the `next_id` to a global
counter, but we do not have a "global proxy-wasm conf" object yet. I
also started thinking about pre-allocating a number of `pwexecs` (like
`worker_connections`) and use free/busy queue that all filter chains can
dip into to get a context id + context memory zone. Perhaps for a later
time.

Run Details

6579 of 7291 relevant lines covered (90.23%)

2472.41 hits per line

Kong / ngx_wasm_module / 6802008362 / 5
95%
main: 95%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job unit-ngx_1.25.3-wasmtime-ssl-debug-hup-static - 6802008362.5

Kong / ngx_wasm_module / 6802008362 / 5 95% main: 95%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job unit-ngx_1.25.3-wasmtime-ssl-debug-hup-static - 6802008362.5

Kong / ngx_wasm_module / 6802008362 / 5
95%
main: 95%

README BADGES
x