Skip to content

Add new test for oneways combined with -ORBTransportIdleTimeout (backport #2569) - #2571

Open
mergify[bot] wants to merge 28 commits into
ace6tao2from
mergify/bp/ace6tao2/pr-2569
Open

Add new test for oneways combined with -ORBTransportIdleTimeout (backport #2569)#2571
mergify[bot] wants to merge 28 commits into
ace6tao2from
mergify/bp/ace6tao2/pr-2569

Conversation

@mergify

@mergify mergify Bot commented Sep 13, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • New Features
    • Added coverage for one-way request transport idle timeouts.
    • Verifies that idle transports are released correctly after one-way processing on both client and server sides.
    • Added build and execution support, including optional debug logging.
  • Documentation
    • Added usage instructions, test flow details, and expected transport cache behavior.

This is an automatic backport of pull request #2569 done by [Mergify](https://mergify.com).

(cherry picked from commit e91dbdc)
(cherry picked from commit 2c992d5)
(cherry picked from commit f763d7b)
(cherry picked from commit c5821e8)
(cherry picked from commit bfb3af4)
    * TAO/tests/Transport_Idle_Timeout_Oneway/run_test.pl:

(cherry picked from commit 8d36241)
(cherry picked from commit ceefb87)
    * TAO/bin/tao_orb_tests.lst:

(cherry picked from commit a44d804)

# Conflicts:
#	TAO/bin/tao_orb_tests.lst
@mergify

mergify Bot commented Sep 13, 2026

Copy link
Copy Markdown
Author

Cherry-pick of a44d804 has failed:

On branch mergify/bp/ace6tao2/pr-2569
Your branch is ahead of 'origin/ace6tao2' by 27 commits.
  (use "git push" to publish your local commits)

You are currently cherry-picking commit a44d804.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   TAO/bin/tao_orb_tests.lst

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify Bot added the conflicts label Sep 13, 2026
@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: b506be67-bc29-45cd-987e-4801b59ac169

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mergify

mergify Bot commented Sep 13, 2026

Copy link
Copy Markdown
Author

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 2 high · 7 minor

Alerts:
⚠ 9 issues (≤ 0 issues of at least minor severity)

Results:
9 new issues

Category Results
BestPractice 1 minor
ErrorProne 2 high
CodeStyle 6 minor

View in Codacy

🟢 Metrics 26 complexity

Metric Results
Complexity 26

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

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

Development

Successfully merging this pull request may close these issues.

1 participant