stacklok / toolhive / 18714029782

58%

Build Type

push

github

Committed by web-flow

Commit Message

Add a --network option (#1867)

* Add a --network option

Signed-off-by: nigel brown <nigel@stacklok.com>

* Fix test: verify actual target port instead of hardcoded value

The test was failing because FindOrUsePort(9000) may return a different
port if 9000 is busy. Changed the assertion to verify that MCP_PORT
matches the actual target port that was selected, rather than assuming
it will always be 9000.

* Fix linting and documentation

- Fix import formatting in client.go (move slices to stdlib section)
- Regenerate CLI docs to include new --network flag
- Remove trailing whitespace

* Update the docs

* Remove debug logging statements

- Remove temporary DEBUG log statements from Docker client
- Clean up redundant logging in network mode configuration
- Simplify logging in createMcpContainer to be more concise

* Removed NetworkMode from RunConfig
and added Mode field to NetworkPermissions

* Regenerate swagger docs after adding Mode to NetworkPermissions

* Update CRD documentation for NetworkPermissions Mode field

---------

Signed-off-by: nigel brown <nigel@stacklok.com>

Run Details

19 of 41 new or added lines in 3 files covered. (46.34%)

22 existing lines in 5 files now uncovered.

18676 of 39112 relevant lines covered (47.75%)

15.92 hits per line