Azure / ARO-RP / 28573225742

39%

Build Type

push

github

Committed by web-flow

Commit Message

ARO-27777 - refactor(monitor): wire arocli into shared http client pool (#4912)

* feat(operator/clientset): add NewForConfigAndClient helpers

Co-authored-by: Cursor <cursoragent@cursor.com>

* refactor(monitor): share arocli http client with monitor pool

Co-authored-by: Cursor <cursoragent@cursor.com>

* chore(makefile): fix operator clientset generation paths

Use module import paths, temporary output-base sync, and run repository fmt pipeline for generate-operator-apiclient.

Co-authored-by: Cursor <cursoragent@cursor.com>

* refactor(operator/clientset): remove temporary companion constructors

Delete manual config-and-client companion files now that the generated clientset path is fixed.

Co-authored-by: Cursor <cursoragent@cursor.com>

* chore(codegen): regenerate operator clientset with v0.25.16

Regenerate operator clientset outputs after fixing generation paths so generated clients expose NewForConfigAndClient directly.

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(makefile): fail fast in operator clientset generation

Ensure generate-operator-apiclient exits on client-gen/rsync errors and always cleans up the temporary output directory via trap.

Co-authored-by: Cursor <cursoragent@cursor.com>

* test(monitor): remove dead close fake clientset test

Drop the obsolete fake ARO clientset close test now that Monitor.Close() only closes the shared HTTP client. This keeps the PR scoped to behavior that still exists.

---------

Co-authored-by: Cursor <cursoragent@cursor.com>

Coverage Stats

0 of 43 new or added lines in 6 files covered. (0.0%)

3 existing lines in 1 file now uncovered.

30804 of 78373 relevant lines covered (39.3%)

0.45 hits per line