NVIDIA / nodewright / 31846917330

79%

Build Type

push

github

Committed by web-flow

Commit Message

fix(operator): rename skyhook -> nodewright in operator error and log text (#476)

* fix(operator): rename skyhook -> nodewright in fmt.Errorf text

Part 2/3 of #403 -- error text surfaced to callers via wrapped errors. Event/condition text landed in PR1, log messages/keys are PR3.

Refs #403

Signed-off-by: Mohit Yadav <ymohit799057@gmail.com>

* fix(operator): rename skyhook -> nodewright in operator log text

Third and last part of #403, after the event text (#410) and the
fmt.Errorf text in the preceding commit. Covers the log messages, the
"skyhook" structured-log key, the ready-condition logger name, and the
event source component.

Two of these are observability breaks rather than cosmetics, so they are
called out rather than swept in silently: a log query matching
skyhook=<name> now needs nodewright=<name>, and
--field-selector source=skyhook-controller now needs
source=nodewright-controller.

Deliberately unchanged: Go identifiers, the agent contract surfaces
(SKYHOOK_* env vars, /skyhook-package mounts), the legacy migration
shims and the legacy Skyhook webhook, whose text genuinely refers to
skyhook.nvidia.com resources, the deprecated skyhook_* metrics, and the
CLI.

Closes #403

Signed-off-by: Brian Lockwood <lockwobr@gmail.com>

---------

Signed-off-by: Mohit Yadav <ymohit799057@gmail.com>
Signed-off-by: Brian Lockwood <lockwobr@gmail.com>
Co-authored-by: Mohit Yadav <ymohit799057@gmail.com>

Coverage Stats

9 of 30 new or added lines in 5 files covered. (30.0%)

8 existing lines in 1 file now uncovered.

10321 of 13077 relevant lines covered (78.92%)

7.84 hits per line