-
Notifications
You must be signed in to change notification settings - Fork 694
[stable/8.7] revert: "fix: add qa module to root" #37172
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[stable/8.7] revert: "fix: add qa module to root" #37172
Conversation
This reverts commit 6a00dee. That's necessary as this was breaking the release dry-run.
CI fails consistently due to trying to pull an unreleased Docker tag of
|
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 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. |
@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. |
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