fix: remove redundant find_kraken_node() call that adds ~7s to startup (#1566) (#1567)

The call on line 274 fetches every pod across all namespaces via
list_pod_for_all_namespaces() to look for a kraken-deployment pod,
but the return value is discarded and the method has no side effects.
list_killable_nodes() in krkn-lib already calls find_kraken_node()
internally when node scenarios actually run, making this startup
call purely redundant.

Signed-off-by: ddjain <darjain@redhat.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Paige Patton <64206430+paigerube14@users.noreply.github.com>
This commit is contained in:
Darshan Jain
2026-08-14 09:47:05 -04:00
committed by GitHub
co-authored by Cursor Paige Patton
parent 51f41e3595
commit 82ad259fee
-3
View File
@@ -270,9 +270,6 @@ def main(options, command: Optional[str], out: Optional[dict] = None) -> int:
distribution = "openshift"
logging.info("Detected cluster platform: %s" % (distribution))
# find node kraken might be running on
kubecli.find_kraken_node()
# Set up kraken url to track signal
if not 0 <= int(port) <= 65535:
logging.error("%s isn't a valid port number, please check" % (port))