Skip to content

chipingress: add RegisterSchemas method and noop client#1646

Merged
pkcll merged 2 commits intoINFOPLAT-2731-common-beholder-expose-chip-clientfrom
INFOPLAT-2731-common-chipingress-refactor
Oct 28, 2025
Merged

chipingress: add RegisterSchemas method and noop client#1646
pkcll merged 2 commits intoINFOPLAT-2731-common-beholder-expose-chip-clientfrom
INFOPLAT-2731-common-chipingress-refactor

Conversation

@pkcll
Copy link
Copy Markdown
Contributor

@pkcll pkcll commented Oct 24, 2025

What

  • add RegisterSchemas method
  • add noop client implementation

Related PR's

Why

Expose chipingress client in beholder
This is prep work for OTI to chip-ingress migration

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 24, 2025

✅ API Diff Results - No breaking changes


📄 View full apidiff report

@pkcll pkcll changed the title chipingress: add RegisterSchemas method; add noop client implementation chipingress: add RegisterSchemas method and noop client Oct 24, 2025
@pkcll pkcll marked this pull request as ready for review October 27, 2025 13:46
@pkcll pkcll requested a review from a team as a code owner October 27, 2025 13:46
@pkcll pkcll requested review from hendoxc and jmank88 October 27, 2025 13:46
@pkcll pkcll changed the base branch from main to INFOPLAT-2731-common-beholder-expose-chip-client October 28, 2025 14:17
@pkcll pkcll merged commit a6e6622 into INFOPLAT-2731-common-beholder-expose-chip-client Oct 28, 2025
18 of 22 checks passed
@pkcll pkcll deleted the INFOPLAT-2731-common-chipingress-refactor branch October 28, 2025 14:38
jmank88 pushed a commit that referenced this pull request Oct 28, 2025
* beholder: use chipingress.noop for beholder noop client

* beholder: remove chipingress wrapper; expose chipingress client directly

* beholder: use chipingress noop client struct

* chipingress: add RegisterSchemas method and noop client (#1646)

* chipingress: add RegisterSchemas method; add noop client implementation

* export chipingress noop client

* bump chipingress
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants