Split out of #1432 (item 6). Fixes 1–3 landed in #1439. This is a second warning site, deliberately kept out of that PR so the node up pin warning could be evaluated on its own.
Problem
An explicit --workspace-key (or RELAY_WORKSPACE_KEY) disables enrollment pickup entirely (packages/cli/src/cli/commands/node.ts:170-175). The enrollment store records workspace ids and the flag supplies a key, so the two cannot be reconciled and pickup is skipped rather than risk a token from the wrong workspace.
That is defensible, but it makes the intuitive fix — "just point it at the right workspace" — mint a duplicate node instead of serving the enrolled one, with no output saying so.
Expected
Warn when a stored enrollment exists and an explicit workspace key is present, naming the enrollment being skipped. Parallel to the pin warning added in #1439.
Refs #1432, #1439
Split out of #1432 (item 6). Fixes 1–3 landed in #1439. This is a second warning site, deliberately kept out of that PR so the
node uppin warning could be evaluated on its own.Problem
An explicit
--workspace-key(orRELAY_WORKSPACE_KEY) disables enrollment pickup entirely (packages/cli/src/cli/commands/node.ts:170-175). The enrollment store records workspace ids and the flag supplies a key, so the two cannot be reconciled and pickup is skipped rather than risk a token from the wrong workspace.That is defensible, but it makes the intuitive fix — "just point it at the right workspace" — mint a duplicate node instead of serving the enrolled one, with no output saying so.
Expected
Warn when a stored enrollment exists and an explicit workspace key is present, naming the enrollment being skipped. Parallel to the pin warning added in #1439.
Refs #1432, #1439