Skip to content

e2e wrapper does not forward IT_MONERO_VIEW_KEY, IT_TARI_VIEW_KEY or IT_TARI_SPEND_PUBLIC_KEY to the detached harness, so payout-confirm SKIPs on the bench #2675

Description

@pithead-developer

Line refs are against develop @ 71543ca6. Filed from p2pool-starter-stack/bench-ci#668.

Defect. tests/integration/e2e.sh launches the live harness on the bench over ssh (run_harness, the on_bench "... nohup setsid ./.e2e-run.sh ..." launch). The remote command sets only IT_RIG_TOKEN, IT_RIG_ROLLBACK_CHANGES, IT_RIG_POOLS_PROBE, RIG_LOCK_PARENT_* and RIG_LOCK_WAIT. Nothing forwards IT_MONERO_VIEW_KEY, IT_TARI_VIEW_KEY or IT_TARI_SPEND_PUBLIC_KEY. resolve_overrides in tests/integration/lib.sh (around line 194) reads exactly those three to run local-pruned-main-payout-confirm. So a value set in the wrapper's environment never reaches the harness, and the row SKIPs with needs IT_MONERO_VIEW_KEY whatever the caller supplies. The REMOTE_* inputs avoid this because they cross as --remote-* flags. The view keys have no such path. The wrapper's usage (ENV OVERRIDES:) does not list them either.

Evidence. bench-ci job 515@474e306ae3ae14d265604716cd525b739fb558bc (mode: matrix) logged SKIPPED scenario local-pruned-main-payout-confirm: needs IT_MONERO_VIEW_KEY. bench-ci#668 has the operator install the three values in the bench's tier4-e2e environment. Until the wrapper forwards them, the row stays skipped on the bench anyway.

Expected. The wrapper carries the three values to the detached harness the same way it carries IT_RIG_ROLLBACK_CHANGES and IT_RIG_POOLS_PROBE: on stdin, never on the remote command line, where ps on the bench would show them. It lists them under ENV OVERRIDES. A selftest or wrapper test proves that a value set for the wrapper arrives in the harness's environment, and that the row stays skipped when the values are unset.

Related: #2657 (its tier-4 proof is this row), #2497 item 3, #2454.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingneeds-triageFiled by a worker or the inventory for something out of scope; the next inventory ranks ittestingTests, CI, and test infrastructure

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions