Skip to content

Conversation

@petoa
Copy link
Contributor

@petoa petoa commented Nov 4, 2025

Bug / Requirement Description

Clearly and concisely describe the problem.

Solution description

Describe your code changes in detail for reviewers.

Checklist:

  • Test
  • Example (both test_plan.py and .rst)
  • Documentation (API)
  • News fragment present for release notes
  • MS info leakage check
  • For new driver: driver index page
  • For new assertion: ui/pdf/std renderers, documentation
  • For new cmdline arg: documentation

@petoa petoa requested a review from a team as a code owner November 4, 2025 07:55
@zhenyu-ms
Copy link
Contributor

can you add a functional testcase for this feature?

Copy link
Contributor

@Pyifan Pyifan left a comment

Choose a reason for hiding this comment

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

hi petoa, as we discussed offline, 1) let's add a new feature to avoid runpath conflict by adding and checking pid when regular testplan starts 2) the logic for remote testplan runpath shall probably in RemoteTestplan class rather than RemoteResouce because it is base class to many other classes 3) please add test and news frag

@petoa petoa changed the title Prevent parallel remote testplan executions on same runpath. Prevent parallel testplan executions on same runpath. Nov 12, 2025
@petoa petoa merged commit a76371b into morganstanley:main Nov 19, 2025
17 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.

3 participants