Skip to content

ci: pin glamsterdam ethereum-package to 6.1.0 #20593

Merged
AskAlexSharov merged 6 commits into
mainfrom
asharov/checking-main-ci
Apr 16, 2026
Merged

ci: pin glamsterdam ethereum-package to 6.1.0 #20593
AskAlexSharov merged 6 commits into
mainfrom
asharov/checking-main-ci

Conversation

@AskAlexSharov

Copy link
Copy Markdown
Collaborator

No description provided.

Commit 835dd9b in ethpandaops/ethereum-package introduced GpuConfig,
a Starlark built-in that requires Kurtosis CLI >=1.18.1. CI installs
an older version so Starlark eval fails. Pin to e07503d (the commit
just before the regression) until CI is upgraded to Kurtosis 1.18.1.
Commit 835dd9b on main introduced GpuConfig, a Starlark built-in that
requires Kurtosis CLI >=1.18.1. CI installs an older version so Starlark
eval fails at startup. Pin to the 6.1.0 release tag which has GLOAS
support but not the breaking GpuConfig. Unpin once CI is on 1.18.1.
@AskAlexSharov AskAlexSharov enabled auto-merge April 16, 2026 03:41

@erigon-copilot erigon-copilot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[SharovBot]

LGTM.

The pin is well justified and the comment captures the dependency clearly:

  • picked up commit introducing
  • that requires Kurtosis CLI
  • current CI image/tooling is older, so Starlark evaluation breaks before the actual test logic runs

Pinning to is the right short-term stabilization until CI is upgraded, and the inline comment makes the rollback condition explicit.

@AskAlexSharov AskAlexSharov added this pull request to the merge queue Apr 16, 2026
@AskAlexSharov AskAlexSharov changed the title [wip] checking main ci [wip] ci: pin glamsterdam ethereum-package to 6.1.0 Apr 16, 2026
@AskAlexSharov AskAlexSharov removed this pull request from the merge queue due to a manual request Apr 16, 2026
@AskAlexSharov AskAlexSharov enabled auto-merge April 16, 2026 04:58
@AskAlexSharov AskAlexSharov added this pull request to the merge queue Apr 16, 2026
@AskAlexSharov AskAlexSharov changed the title [wip] ci: pin glamsterdam ethereum-package to 6.1.0 ci: pin glamsterdam ethereum-package to 6.1.0 Apr 16, 2026
Merged via the queue into main with commit ee703e2 Apr 16, 2026
36 checks passed
@AskAlexSharov AskAlexSharov deleted the asharov/checking-main-ci branch April 16, 2026 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant