feat(status): expose mode on /version and GET /authz - #1899
Conversation
Signed-off-by: Neil <neiljar888@gmail.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Codecov Report❌ Patch coverage is
❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
What type of PR is this?
/kind enhancement
What this PR does / why we need it:
Expose daemon dataplane
modeonGET /versionand addGET /authzreturning{"enabled": bool}.Today dump/BPF callers must guess
kernel-nativevsdual-engine, andkmeshctl authz statusGETs/authzbut the status server only allowed POST. This is groundwork for the MCP server (#1800) so tools likeget_version,config_dump, andget_authz_statuscan use structured status APIs.Which issue(s) this PR fixes:
Related to #1800
Special notes for your reviewer:
modeis filled fromBpfConfig.Mode(defaultdual-engine); omitted for client-only version info viaomitempty/authz?enable=behavior unchangedDoes this PR introduce a user-facing change?: