Skip to content

Conversation

houssain-barouni
Copy link
Collaborator

@houssain-barouni houssain-barouni commented Aug 29, 2025

Description

The test uses

  @MultiDbTestApplication(managedLifecycle = false)
  private static final TestCamundaApplication STANDALONE_CAMUNDA

so the application is not automatically stopped after the tests.
Also the used http and camunda clients are not closed.

Some logs found in the test logs of other tests (TasklistV1ApiFormPermissionsIT)!

Aug 27, 2025 10:33:34 PM io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference cleanQueue
SEVERE: *~*~*~ Previous channel ManagedChannelImpl{logId=303, target=0.0.0.0:1406} was garbage collected without being shut down! ~*~*~*
    Make sure to call shutdown()/shutdownNow()
java.lang.RuntimeException: ManagedChannel allocation site
	at io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference.<init>(ManagedChannelOrphanWrapper.java:102)
	at io.grpc.internal.ManagedChannelOrphanWrapper.<init>(ManagedChannelOrphanWrapper.java:60)
	at io.grpc.internal.ManagedChannelOrphanWrapper.<init>(ManagedChannelOrphanWrapper.java:51)
	at io.grpc.internal.ManagedChannelImplBuilder.build(ManagedChannelImplBuilder.java:731)
	at io.grpc.ForwardingChannelBuilder2.build(ForwardingChannelBuilder2.java:278)
	at io.camunda.client.impl.CamundaClientImpl.buildChannel(CamundaClientImpl.java:413)
	at io.camunda.client.impl.CamundaClientImpl.<init>(CamundaClientImpl.java:331)
	at io.camunda.client.impl.CamundaClientBuilderImpl.build(CamundaClientBuilderImpl.java:605)
	at io.camunda.it.orchestration.IncidentNotifierIT.shouldNotifyWebhookAboutIncident(IncidentNotifierIT.java:116)
Caused by: java.net.BindException: Address already in use
	at java.base/sun.nio.ch.Net.bind0(Native Method)
	at java.base/sun.nio.ch.Net.bind(Net.java:565)

Checklist

Related issues

closes #

@houssain-barouni houssain-barouni force-pushed the hb-incidentnotifierit-cleanup branch from 56d432d to 5da49e0 Compare August 29, 2025 19:52
@houssain-barouni houssain-barouni requested a review from a team August 29, 2025 20:26
@houssain-barouni houssain-barouni marked this pull request as ready for review August 29, 2025 20:26
Copy link
Contributor

@RomanJRW RomanJRW left a comment

Choose a reason for hiding this comment

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

Nice catch, LGTM 👍

@houssain-barouni houssain-barouni self-assigned this Aug 30, 2025
@houssain-barouni houssain-barouni added this pull request to the merge queue Aug 30, 2025
github-merge-queue bot pushed a commit that referenced this pull request Aug 30, 2025
## Description

<!-- Describe the goal and purpose of this PR. -->
The test uses
```java
  @MultiDbTestApplication(managedLifecycle = false)
  private static final TestCamundaApplication STANDALONE_CAMUNDA
```
so the application is not automatically stopped after the tests.
Also the used http and camunda clients are not closed.

Some logs found in the test logs of other tests
(TasklistV1ApiFormPermissionsIT)!
```
Aug 27, 2025 10:33:34 PM io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference cleanQueue
SEVERE: *~*~*~ Previous channel ManagedChannelImpl{logId=303, target=0.0.0.0:1406} was garbage collected without being shut down! ~*~*~*
    Make sure to call shutdown()/shutdownNow()
java.lang.RuntimeException: ManagedChannel allocation site
	at io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference.<init>(ManagedChannelOrphanWrapper.java:102)
	at io.grpc.internal.ManagedChannelOrphanWrapper.<init>(ManagedChannelOrphanWrapper.java:60)
	at io.grpc.internal.ManagedChannelOrphanWrapper.<init>(ManagedChannelOrphanWrapper.java:51)
	at io.grpc.internal.ManagedChannelImplBuilder.build(ManagedChannelImplBuilder.java:731)
	at io.grpc.ForwardingChannelBuilder2.build(ForwardingChannelBuilder2.java:278)
	at io.camunda.client.impl.CamundaClientImpl.buildChannel(CamundaClientImpl.java:413)
	at io.camunda.client.impl.CamundaClientImpl.<init>(CamundaClientImpl.java:331)
	at io.camunda.client.impl.CamundaClientBuilderImpl.build(CamundaClientBuilderImpl.java:605)
	at io.camunda.it.orchestration.IncidentNotifierIT.shouldNotifyWebhookAboutIncident(IncidentNotifierIT.java:116)
```
```
Caused by: java.net.BindException: Address already in use
	at java.base/sun.nio.ch.Net.bind0(Native Method)
	at java.base/sun.nio.ch.Net.bind(Net.java:565)
```

## Checklist

<!--- Please delete options that are not relevant. Boxes should be
checked by reviewer. -->
- [ ] Enable backports when necessary (fex. [for bug
fixes](https://github.com/camunda/camunda/blob/main/CONTRIBUTING.md#backporting-changes)
or [for CI
changes](https://github.com/camunda/camunda/wiki/CI-&-Automation#when-to-backport-ci-changes)).

## Related issues

closes #
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 30, 2025
@houssain-barouni houssain-barouni added this pull request to the merge queue Aug 30, 2025
Merged via the queue into main with commit 29d4dbe Aug 30, 2025
91 checks passed
@houssain-barouni houssain-barouni deleted the hb-incidentnotifierit-cleanup branch August 30, 2025 20:14
@backport-action
Copy link
Collaborator

Successfully created backport PR for release-8.8.0-alpha8:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants