Skip to content

PR-Review #780: waitForCondition-Lint lehnt templatisierte Ports ab, gMSA-Freigabe für scheduledTask passt nicht zur Engine #790

Closed

Description

@Sev7eNup

PR: #780 · Autor: @Sev7eNup · Status: merged

  • mittel – src/nodepilot-ui/src/lib/activityConfigFacts.ts:217: Im Modus portOpen verlangt die neue Prüfung typeof config.port === 'number'. Die Engine akzeptiert aber ausdrücklich auch Templates und numerische Strings (WaitForConditionActivity.cs:251/387, TryGetIntOrNumericString). Die Repo-eigenen Beispiele scripts/test-suite/positive/29-waitForCondition.json:173 ("port": "{{disc.param.selfPort}}") und scripts/muster-einzeltests.json bekommen deshalb einen missing-required-config-Lint-Fehler, und Lint-Fehler blockieren die Pre-Publish-Checkliste.
    Vorschlag: Templates und numerische Strings zulassen und nur echte Zahlen auf den Bereich 1..65535 prüfen. Dazu einen Test mit templatisiertem Port.
  • mittel (im Lab verifizieren) – activityConfigFacts.ts:129-134 und die Doku in activities-reference.md (DE/EN): Der PR erlaubt neu Managed Service Accounts (…$) als runAsUser. ScheduledTaskActivity.cs:239-241 registriert den Principal aber weiterhin mit -LogonType ServiceAccount, und der Doc-Kommentar der Engine (:46) nennt nur die NT-AUTHORITY-Built-ins. Für gMSA ist der dokumentierte Weg -LogonType Password. DOMAIN\gmsa$ besteht damit voraussichtlich die Designer-Validierung und scheitert erst bei der Registrierung auf dem Ziel.
    Vorschlag: Auf einem Lab-Host prüfen. Danach entweder die Engine für $-Konten auf -LogonType Password umstellen (mit Test) oder das gMSA-Versprechen aus Hint, Placeholder, Validierung und Doku entfernen.

Automatischer Review-Fund des nächtlichen PR-Review-Laufs – braucht Triage.

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

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions