Skip to content

Fixes for build contract when not running in our docker:dind image#69

Open
solsson wants to merge 2 commits intomasterfrom
fixes-for-build-contract-not-in-docker
Open

Fixes for build contract when not running in our docker:dind image#69
solsson wants to merge 2 commits intomasterfrom
fixes-for-build-contract-not-in-docker

Conversation

@solsson
Copy link
Contributor

@solsson solsson commented Mar 18, 2026

I had accidentally accepted an npm lib upgrade through #63 which breaks our API use. Also new docker CE installs (on Ubuntu for example) don't have the docker-compose CLI.

Yolean k8s-qa and others added 2 commits March 18, 2026 10:00
Dependabot bumped npm from 6.14.18 to 11.6.4 (e593e9f) as part of a
form-data security fix, but npm removed its programmatic API in v8.
build-contract-predockerbuild uses require('npm') which breaks with v8+.

Pin to ~6.14.18 so dependabot can only bump within 6.14.x.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
docker-compose (standalone binary) is not installed on modern systems.
Use docker compose (v2 plugin) which is the standard since Docker 23.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@solsson solsson requested a review from atamon March 18, 2026 13:45
@solsson solsson changed the title Fixes for build contract not in docker Fixes for build contract when not running in our docker:dind-based image Mar 18, 2026
@solsson solsson changed the title Fixes for build contract when not running in our docker:dind-based image Fixes for build contract when not running in our docker:dind image Mar 18, 2026
This was referenced Mar 18, 2026
Copy link
Contributor

@atamon atamon left a comment

Choose a reason for hiding this comment

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

Looks sensible

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.

2 participants