Skip to content

fix: update hokusai orb default remote docker version to be compatible with docker-cli 28#176

Merged
narikazu merged 1 commit intomainfrom
narikazu/docker-version
Mar 11, 2026
Merged

fix: update hokusai orb default remote docker version to be compatible with docker-cli 28#176
narikazu merged 1 commit intomainfrom
narikazu/docker-version

Conversation

@narikazu
Copy link
Copy Markdown
Member

@narikazu narikazu commented Mar 6, 2026

Changed setup-docker command default from docker23 to default (Docker 24) to fix incompatibility between hokusai's docker-cli 28 (API 1.52) and CircleCI's Docker 23 remote engine (max API 1.42).

This PR fixes the issue on Exchange's CI.

Server Engine Details:
  Version:          23.0.6
  API version:      1.42 (minimum version 1.12)
  Go version:       go1.19.9
  Git commit:       9dbdbd4
  Built:            2023-05-05T21:18:13.000000000+00:00
  OS/Arch:          linux/amd64
  Experimental:     false

Ticket

https://artsyproduct.atlassian.net/browse/PHIRE-2876

@narikazu narikazu self-assigned this Mar 6, 2026
@narikazu narikazu requested a review from artsyjian March 6, 2026 11:33
Comment thread src/hokusai/hokusai.yml
remote_docker_version:
type: string
default: docker23
default: default
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It seems jobs elsewhere (example) are still passing docker23 which may override this default set for setup-docker command. Curious if we need to update them as well.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good point! Updated docker23 to default in all places here.

…e with docker-cli 28

Changed setup-docker command default from docker23 to default (Docker 24)
to fix incompatibility between hokusai's docker-cli 28 (API 1.52) and
CircleCI's Docker 23 remote engine (max API 1.42).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@narikazu narikazu force-pushed the narikazu/docker-version branch from d264d1a to 40bfae2 Compare March 6, 2026 16:46
Copy link
Copy Markdown
Contributor

@artsyjian artsyjian left a comment

Choose a reason for hiding this comment

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

good find!

Copy link
Copy Markdown
Member

@starsirius starsirius left a comment

Choose a reason for hiding this comment

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

🎉 Thanks for unblocking Exchange builds!

@narikazu narikazu merged commit 775f916 into main Mar 11, 2026
3 checks passed
@narikazu narikazu deleted the narikazu/docker-version branch March 11, 2026 12:00
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.

3 participants