Skip to content

Conversation

korthout
Copy link
Member

@korthout korthout commented Aug 25, 2025

Description

This reverts commit 6a00dee.

This was necessary as it was breaking the release dry-run.

Note that his means that ArchUnit tests are no longer executed in the General Unit Tests section, as this is run with -D skipQaBuild=true. We'll fix that in a later PR.

Checklist

Related issues

relates to https://camunda.slack.com/archives/C06UWQNCU7M/p1756114363166019?thread_ts=1755827877.666109&cid=C06UWQNCU7M

This reverts commit 6a00dee.

That's necessary as this was breaking the release dry-run.
@korthout korthout requested a review from cmur2 August 25, 2025 10:34
@github-actions github-actions bot added component/zeebe Related to the Zeebe component/team component/operate Related to the Operate component/team component/tasklist Related to the Tasklist component/team component/optimize Related to Optimize component/team labels Aug 25, 2025
@korthout korthout marked this pull request as ready for review August 25, 2025 10:34
@cmur2 cmur2 enabled auto-merge August 25, 2025 11:00
@cmur2 cmur2 added the component/release Related to the monorepo release process label Aug 25, 2025
@cmur2
Copy link
Member

cmur2 commented Aug 25, 2025

CI fails consistently due to trying to pull an unreleased Docker tag of camunda/zeebe (we're doing a release of that version in parallel just now):

Error: Run 1: RollingUpdateTest.shouldBeAbleToRestartContainerWithNewVersion(String, String)[2] ? Completion org.testcontainers.containers.ContainerFetchException: Can't get Docker image: RemoteDockerImage(imageName=camunda/zeebe:8.6.25, imagePullPolicy=DefaultPullPolicy(), imageNameSubstitutor=org.testcontainers.utility.ImageNameSubstitutor$LogWrappedImageNameSubstitutor@5b6f0bac)

@cmur2
Copy link
Member

cmur2 commented Aug 25, 2025

I can only assume that the codes checks the GitHub tags and tries to infer that the Dockerhub tag should also exist - reaonable assumption but it has edge cases....

@cmur2
Copy link
Member

cmur2 commented Aug 25, 2025

Edge case 😅

image

@cmur2 cmur2 added this pull request to the merge queue Aug 25, 2025
@korthout
Copy link
Member Author

@cmur2 I don't see how your comments relate to this PR. What do you mean?

Merged via the queue into stable/8.7 with commit 128b1d7 Aug 25, 2025
143 of 147 checks passed
@cmur2 cmur2 deleted the korthout-fix-release-dry-run-qa-module-on-8-7 branch August 25, 2025 11:47
@cmur2
Copy link
Member

cmur2 commented Aug 25, 2025

@cmur2 I don't see how your comments relate to this PR. What do you mean?

Sorry, I pasted the log of a failing CI run on this PR without the link: here you see the error before my retries (and before 8.6.25 was fully released): https://github.com/camunda/camunda/actions/runs/17206355916/job/48809373302

It relates insofar as that the timing of this PR exposed a flaw in the Zeebe QA - Update tests on the base branch.

@korthout korthout mentioned this pull request Aug 25, 2025
1 task
@korthout
Copy link
Member Author

@cmur2 Ah thanks for sharing the link. No I didn't see the failures before the retries. Sadly, GH doesn't show retries in a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/operate Related to the Operate component/team component/optimize Related to Optimize component/team component/release Related to the monorepo release process component/tasklist Related to the Tasklist component/team component/zeebe Related to the Zeebe component/team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants