• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

NVIDIA / holodeck / 23049306583
49%
main: 48%

Build:
Build:
LAST BUILD BRANCH: pull-request/773
DEFAULT BRANCH: main
Ran 13 Mar 2026 11:46AM UTC
Jobs 1
Files 38
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

13 Mar 2026 11:43AM UTC coverage: 49.153% (-0.2%) from 49.313%
23049306583

push

github

ArangoGutierrez
fix: wire Transport into SSH connection path and add cleanup

Address Distinguished Engineer review findings for PR #724:

B1 (BLOCKING): connectOrDie() now uses transport.Dial() + ssh.NewClientConn()
when a Transport is provided, instead of ignoring the transport and calling
ssh.Dial("tcp", ...) directly. All reconnection paths (waitForNodeReboot,
provision) pass p.transport through.

B2 (BLOCKING): installK8sPrereqs() and GetClusterHealth() now use
transportOptsForNode() to pass transport options to New().

R1: SSMTransport.Dial() is now idempotent — closes any existing session
before starting a new one.

R2: Transport interface includes Close() method. DirectTransport has a
no-op Close(). SSMTransport.Close() terminates the SSM process.

R3: SSM stderr is captured in a bytes.Buffer and included in error
messages for diagnostics.

R4: GetClusterHealth() builds transport options from node info in
environment status.

Signed-off-by: Eduardo Arango <earango@nvidia.com>
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>

5 of 69 new or added lines in 3 files covered. (7.25%)

2959 of 6020 relevant lines covered (49.15%)

0.55 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
12
40.23
2.39% pkg/provisioner/transport.go
24
4.58
2.17% pkg/provisioner/provisioner.go
28
12.92
-0.5% pkg/provisioner/cluster.go
Jobs
ID Job ID Ran Files Coverage
1 23049306583.1 13 Mar 2026 11:46AM UTC 38
49.15
GitHub Action Run
Source Files on build 23049306583
  • Tree
  • List 38
  • Changed 3
  • Source Changed 3
  • Coverage Changed 3
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #23049306583
  • 362df8eb on github
  • Prev Build on pull-request/722 (#23047413512)
  • Next Build on pull-request/724 (#23051946609)
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc