Skip to content

Skip PgTerminateBackendTest.ConflictingSubTxn on macOS#31034

Merged
rahulb-yb merged 1 commit intoyugabyte:masterfrom
Shopify:skip-strace-test-on-macos
Apr 14, 2026
Merged

Skip PgTerminateBackendTest.ConflictingSubTxn on macOS#31034
rahulb-yb merged 1 commit intoyugabyte:masterfrom
Shopify:skip-strace-test-on-macos

Conversation

@ellabaron-code
Copy link
Copy Markdown
Collaborator

@ellabaron-code ellabaron-code commented Apr 9, 2026

The test uses strace to monitor process exit, which is Linux-only.
macOS has no practical equivalent (dtruss requires root and is
restricted by SIP).


Phorge: D51951

  The test uses strace to monitor process exit, which is Linux-only.
  macOS has no practical equivalent (dtruss requires root and is
  restricted by SIP).
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request modifies src/yb/yql/pgwrapper/pg_terminate_backend-test.cc to disable the ConflictingSubTxn test on macOS. There is no feedback to provide.

@iSignal
Copy link
Copy Markdown
Contributor

iSignal commented Apr 9, 2026

trigger jenkins

@hari90
Copy link
Copy Markdown
Contributor

hari90 commented Apr 9, 2026

Phorge diff synced: D51951
Commit: a812da2c

Jenkins build has been triggered. Results will be posted here once it completes.


JenkinsBot

@iSignal iSignal requested a review from basavaraj29 April 9, 2026 18:02
@hari90
Copy link
Copy Markdown
Contributor

hari90 commented Apr 11, 2026

Jenkins build for commit a812da2c: Pass

Passed: 9


🔨 DB Build/Test Job Summary

Build Total Passed Failed Failed After Retries
D51951-alma8-clang21-tsan 3 3 0 0
D51951-mac14-clang-release 2 2 0 0
D51951-ubuntu22.04-clang21-debug 2 2 0 0
D51951-arm-mac14-clang-release 2 2 0 0
D51951-alma8-gcc12-fastdebug 3 3 0 0
D51951-arm-alma8-clang21-release 3 3 0 0
D51951-alma8-clang21-release 3 3 0 0
D51951-alma9-clang21-asan 3 3 0 0

Full status


JenkinsBot

@rahulb-yb rahulb-yb merged commit c3f66a6 into yugabyte:master Apr 14, 2026
5 checks passed
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.

5 participants