Skip to content

Conversation

devcrocod
Copy link
Contributor

@devcrocod devcrocod commented Sep 15, 2025

Ignore flaky testMultipleClientParallel

@devcrocod devcrocod requested a review from kpavlov September 15, 2025 14:52
@devcrocod devcrocod marked this pull request as ready for review September 15, 2025 14:54
@Copilot Copilot AI review requested due to automatic review settings September 15, 2025 14:54
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR marks the testMultipleClientParallel test as ignored due to flakiness issues, referencing GitHub issue #262 for context.

  • Adds @Ignore annotation to disable a flaky test
  • Includes an import for the kotlin.test.Ignore annotation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@kpavlov kpavlov left a comment

Choose a reason for hiding this comment

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

It's not good. Let's address the root cause separately

@devcrocod devcrocod force-pushed the devcrocod/ignore-flaky-test branch from db02bc3 to 86882fd Compare September 15, 2025 15:27
@devcrocod devcrocod force-pushed the devcrocod/ignore-flaky-test branch from 86882fd to 303303f Compare September 15, 2025 15:45
@devcrocod devcrocod merged commit ba63122 into main Sep 15, 2025
4 checks passed
@devcrocod devcrocod deleted the devcrocod/ignore-flaky-test branch September 15, 2025 16:56
svc-squareup-copybara pushed a commit to cashapp/misk that referenced this pull request Sep 16, 2025
| Package | Type | Package file | Manager | Update | Change |
|---|---|---|---|---|---|
|
[io.modelcontextprotocol:kotlin-sdk-server](https://github.com/modelcontextprotocol/kotlin-sdk)
| dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`0.7.0` -> `0.7.1` |
|
[io.modelcontextprotocol:kotlin-sdk-core](https://github.com/modelcontextprotocol/kotlin-sdk)
| dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`0.7.0` -> `0.7.1` |
|
[com.autonomousapps.dependency-analysis](https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin)
| plugin | misk/gradle/libs.versions.toml | gradle | patch | `3.0.2` ->
`3.0.3` |
| [software.amazon.awssdk:sdk-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |
| [software.amazon.awssdk:sqs](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |
| [software.amazon.awssdk:s3](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |
| [software.amazon.awssdk:regions](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |
|
[software.amazon.awssdk:dynamodb-enhanced](https://aws.amazon.com/sdkforjava)
| dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |
| [software.amazon.awssdk:dynamodb](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |
| [software.amazon.awssdk:aws-core](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |
| [software.amazon.awssdk:bom](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |
| [software.amazon.awssdk:auth](https://aws.amazon.com/sdkforjava) |
dependencies | misk/gradle/libs.versions.toml | gradle | patch |
`2.33.9` -> `2.33.10` |

---

### Release Notes

<details>
<summary>modelcontextprotocol/kotlin-sdk
(io.modelcontextprotocol:kotlin-sdk-server)</summary>

###
[`v0.7.1`](https://github.com/modelcontextprotocol/kotlin-sdk/releases/tag/0.7.1)

[Compare
Source](modelcontextprotocol/kotlin-sdk@0.7.0...0.7.1)

##### What's Changed

- Bump io.github.oshai:kotlin-logging from 7.0.7 to 7.0.11 by
[@&#8203;dependabot](https://github.com/dependabot)\[bot] in
[#&#8203;211](modelcontextprotocol/kotlin-sdk#211)
- Bump org.jlleitschuh.gradle.ktlint from 13.0.0 to 13.1.0 by
[@&#8203;dependabot](https://github.com/dependabot)\[bot] in
[#&#8203;241](modelcontextprotocol/kotlin-sdk#241)
- Bump com.anthropic:anthropic-java from 0.8.0 to 2.6.0 by
[@&#8203;dependabot](https://github.com/dependabot)\[bot] in
[#&#8203;246](modelcontextprotocol/kotlin-sdk#246)
- Fix upload reports in GitHub actions by
[@&#8203;skarpovdev](https://github.com/skarpovdev) in
[#&#8203;264](modelcontextprotocol/kotlin-sdk#264)
- Bump ktor from 3.2.3 to 3.3.0 by
[@&#8203;dependabot](https://github.com/dependabot)\[bot] in
[#&#8203;265](modelcontextprotocol/kotlin-sdk#265)
- fix hang caused by StreamableHttpClientTransport
([#&#8203;226](modelcontextprotocol/kotlin-sdk#226))
by [@&#8203;AdrianMiska](https://github.com/AdrianMiska) in
[#&#8203;227](modelcontextprotocol/kotlin-sdk#227)
- Bump kotlin from 2.2.0 to 2.2.10 by
[@&#8203;dependabot](https://github.com/dependabot)\[bot] in
[#&#8203;230](modelcontextprotocol/kotlin-sdk#230)
- Add module documentation and enhance Dokka configuration by
[@&#8203;kpavlov](https://github.com/kpavlov) in
[#&#8203;261](modelcontextprotocol/kotlin-sdk#261)
- Mark `testMultipleClientParallel` as ignored due to flakiness by
[@&#8203;devcrocod](https://github.com/devcrocod) in
[#&#8203;267](modelcontextprotocol/kotlin-sdk#267)

##### New Contributors

- [@&#8203;AdrianMiska](https://github.com/AdrianMiska) made their first
contribution in
[#&#8203;227](modelcontextprotocol/kotlin-sdk#227)

**Full Changelog**:
<modelcontextprotocol/kotlin-sdk@0.7.0...0.7.1>

</details>

<details>
<summary>autonomousapps/dependency-analysis-android-gradle-plugin
(com.autonomousapps.dependency-analysis)</summary>

###
[`v3.0.3`](https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/blob/HEAD/CHANGELOG.md#Version-303)

- \[Fix]: compare versions via `VersionNumber`, not raw strings.
- \[Fix] Forcibly downgrade kotlin-stdlib to 2.0.21 for Gradle 8.11
support.

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am
every weekday" in timezone Australia/Melbourne, Automerge - At any time
(no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Never, or you tick the rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://github.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

GitOrigin-RevId: 5f728b0acc73a9742f56946b97f85ad1e3355442
skarpovdev pushed a commit to skarpovdev/kotlin-sdk that referenced this pull request Sep 16, 2025
skarpovdev pushed a commit to skarpovdev/kotlin-sdk that referenced this pull request Sep 16, 2025
kpavlov pushed a commit to skarpovdev/kotlin-sdk that referenced this pull request Sep 16, 2025
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