Skip to content
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

[Backport stable/8.0] ci(integration): split module and integration test jobs #12449

Merged
6 commits merged into from
Apr 21, 2023

Conversation

megglos
Copy link
Contributor

@megglos megglos commented Apr 17, 2023

Description

Backport of #12406 to stable/8.1.

relates to #12028

To reduces runtime as the module its and qa-integration tests cannot run in parallel.

(cherry picked from commit 68cd829)
@npepinpe
Copy link
Member

I'll rerun the test. It looks like we tried to delete the image? That doesn't make sense...I'll also ping the Testcontainers people, as this shouldn't happen.

Copy link
Member

@npepinpe npepinpe left a comment

Choose a reason for hiding this comment

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

bors merge

ghost pushed a commit that referenced this pull request Apr 17, 2023
12449: [Backport stable/8.1] ci(integration): split module and integration test jobs r=npepinpe a=megglos

# Description
Backport of #12406 to `stable/8.1`.

relates to #12028

Co-authored-by: Meggle (Sebastian Bathke) <sebastian.bathke@camunda.com>
@ghost
Copy link

ghost commented Apr 17, 2023

Build failed:

@npepinpe
Copy link
Member

I'll look into it tomorrow morning.

@npepinpe
Copy link
Member

OK so this fails very reliably, but why only on this branch??

@npepinpe
Copy link
Member

Fixed on Ole's branch, so we can merge them together now 👍

bors r+

@npepinpe
Copy link
Member

bors r-

@ghost
Copy link

ghost commented Apr 20, 2023

Canceled.

@npepinpe npepinpe changed the title [Backport stable/8.1] ci(integration): split module and integration test jobs [Backport stable/8.0] ci(integration): split module and integration test jobs Apr 20, 2023
@npepinpe
Copy link
Member

bors r+

ghost pushed a commit that referenced this pull request Apr 20, 2023
12449: [Backport stable/8.0] ci(integration): split module and integration test jobs r=npepinpe a=megglos

# Description
Backport of #12406 to `stable/8.1`.

relates to #12028

Co-authored-by: Meggle (Sebastian Bathke) <sebastian.bathke@camunda.com>
Co-authored-by: Nicolas Pepin-Perreault <nicolas.pepin-perreault@camunda.com>
@ghost
Copy link

ghost commented Apr 20, 2023

Build failed:

@npepinpe
Copy link
Member

bors r+

ghost pushed a commit that referenced this pull request Apr 20, 2023
12441: deps(maven): bump maven-dependency-analyzer from 1.13.0 to 1.13.1 r=npepinpe a=dependabot[bot]

Bumps [maven-dependency-analyzer](https://github.com/apache/maven-dependency-analyzer) from 1.13.0 to 1.13.1.
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/54a53397f1f286804cb9ee340ed0243df4d00eb0"><code>54a5339</code></a> [maven-release-plugin] prepare release maven-dependency-analyzer-1.13.1</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/9c05199709db540441a05578d4727d2a3b6326f8"><code>9c05199</code></a> [MSHARED-1149] Replace System.out by logger</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/93f9e364970d50e4afea06ec0280991070d81d24"><code>93f9e36</code></a> [MSHARED-1224] Prefer JDK classes to Plexus utils (<a href="https://redirect.github.com/apache/maven-dependency-analyzer/issues/81">#81</a>)</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/0cfe70fcbc00a57c10fbf82890ffdd925d284bad"><code>0cfe70f</code></a> [MSHARED-1205] Build on JDK 19, 20</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/9739f21104175ba0fd7c5b53d088cf21618ae8d9"><code>9739f21</code></a> [MSHARED-1219] Upgrade Parent to 39 - code reformat</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/0ce92bbcb8bd9b9a6efc4be21d1b4a8bee8ab2db"><code>0ce92bb</code></a> [MSHARED-1219] Upgrade Parent to 39</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/f72c12244669a8fb4a0a8c37d7adc9088506a737"><code>f72c122</code></a> [MSHARED-1220] Refresh download page</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/2ce51238d78ddf2093455cc763f81c5a7f66ea99"><code>2ce5123</code></a> Disable merge button, add jira autolink</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/5155be74246574895ede47aa10f8594ce3511075"><code>5155be7</code></a> [MSHARED-1218] Bump asm from 9.4 to 9.5 (<a href="https://redirect.github.com/apache/maven-dependency-analyzer/issues/83">#83</a>)</li>
<li><a href="https://github.com/apache/maven-dependency-analyzer/commit/03725618e1d03d7a9d0ac9b83dd1286123f8c0b3"><code>0372561</code></a> Bump asm from 9.3 to 9.4 (<a href="https://redirect.github.com/apache/maven-dependency-analyzer/issues/70">#70</a>)</li>
<li>Additional commits viewable in <a href="https://github.com/apache/maven-dependency-analyzer/compare/maven-dependency-analyzer-1.13.0...maven-dependency-analyzer-1.13.1">compare view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.maven.shared:maven-dependency-analyzer&package-manager=maven&previous-version=1.13.0&new-version=1.13.1)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting ``@dependabot` rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- ``@dependabot` rebase` will rebase this PR
- ``@dependabot` recreate` will recreate this PR, overwriting any edits that have been made to it
- ``@dependabot` merge` will merge this PR after your CI passes on it
- ``@dependabot` squash and merge` will squash and merge this PR after your CI passes on it
- ``@dependabot` cancel merge` will cancel a previously requested merge and block automerging
- ``@dependabot` reopen` will reopen this PR if it is closed
- ``@dependabot` close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- ``@dependabot` ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- ``@dependabot` ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- ``@dependabot` ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)


</details>

12449: [Backport stable/8.0] ci(integration): split module and integration test jobs r=npepinpe a=megglos

# Description
Backport of #12406 to `stable/8.1`.

relates to #12028

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Nicolas Pepin-Perreault <nicolas.pepin-perreault@camunda.com>
Co-authored-by: Meggle (Sebastian Bathke) <sebastian.bathke@camunda.com>
@ghost
Copy link

ghost commented Apr 20, 2023

Build failed (retrying...):

ghost pushed a commit that referenced this pull request Apr 20, 2023
12449: [Backport stable/8.0] ci(integration): split module and integration test jobs r=npepinpe a=megglos

# Description
Backport of #12406 to `stable/8.1`.

relates to #12028

Co-authored-by: Meggle (Sebastian Bathke) <sebastian.bathke@camunda.com>
Co-authored-by: Nicolas Pepin-Perreault <nicolas.pepin-perreault@camunda.com>
@ghost
Copy link

ghost commented Apr 20, 2023

Build failed:

10 seemed too unstable.
@megglos
Copy link
Contributor Author

megglos commented Apr 21, 2023

bors r+

ghost pushed a commit that referenced this pull request Apr 21, 2023
12449: [Backport stable/8.0] ci(integration): split module and integration test jobs r=megglos a=megglos

# Description
Backport of #12406 to `stable/8.1`.

relates to #12028

Co-authored-by: Meggle (Sebastian Bathke) <sebastian.bathke@camunda.com>
Co-authored-by: Nicolas Pepin-Perreault <nicolas.pepin-perreault@camunda.com>
@ghost
Copy link

ghost commented Apr 21, 2023

Build failed:

@megglos
Copy link
Contributor Author

megglos commented Apr 21, 2023

bors r+

@ghost
Copy link

ghost commented Apr 21, 2023

Build succeeded:

@ghost ghost merged commit 10cd745 into stable/8.0 Apr 21, 2023
@ghost ghost deleted the backport-12406-to-stable/8.0 branch April 21, 2023 12:15
This pull request was closed.
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