Skip to content

chore: upgrade atlas docker image to canary v1.1.7#2873

Merged
migmartri merged 2 commits intochainloop-dev:mainfrom
jiparis:jiparis/upgrade-atlas-latest
Mar 18, 2026
Merged

chore: upgrade atlas docker image to canary v1.1.7#2873
migmartri merged 2 commits intochainloop-dev:mainfrom
jiparis:jiparis/upgrade-atlas-latest

Conversation

@jiparis
Copy link
Member

@jiparis jiparis commented Mar 17, 2026

Upgrades the Atlas Docker image used for running database migrations from stable v1.1.6 to canary v1.1.7-0f00ade-canary (digest sha256:cbeb45f5342e8e614a60a7ffa355973e5c45f8a8529651c36b9a8ec7e9337d02).

Only Dockerfile.migrations is updated; common.mk and the CI test workflow retain v1.1.6 since the release is not yet done..

Signed-off-by: Jose I. Paris <jiparis@chainloop.dev>
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

2 issues found across 1 file

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="app/controlplane/Dockerfile.migrations">

<violation number="1" location="app/controlplane/Dockerfile.migrations:3">
P2: The `from:` annotation should record the concrete Atlas tag, not `latest`. Using a mutable tag here breaks the traceability of the pinned digest and can mislead future updates or manual verification.</violation>

<violation number="2" location="app/controlplane/Dockerfile.migrations:6">
P1: Keep the deployment Atlas version aligned with `common.mk` and CI. Right now migrations are validated with `v1.1.6` but deployed with a `v1.1.7` canary, so Atlas-specific behavior can fail only in production.

(Based on your team's feedback about cross-component and version compatibility.) [FEEDBACK_USED]</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Signed-off-by: Jose I. Paris <jiparis@chainloop.dev>
@jiparis jiparis requested review from javirln and migmartri March 17, 2026 22:39
@migmartri migmartri merged commit 56e1c4a into chainloop-dev:main Mar 18, 2026
15 checks passed
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