Skip to content

Conversation

@dthain
Copy link
Member

@dthain dthain commented Nov 11, 2025

Fixed vine tests to all use any available port (port=0) and skip the use of manager reporting, so that we don't have vine_status polluted with a lot of test jobs.

Proposed Changes

Give an overall description of the changes, along with the context and motivation.
Mention relevant issues and pull requests as needed.

  • Remove manager name reporting from automated tests.

Merge Checklist

The following items must be completed before PRs can be merged.
Check these off to verify you have completed all steps.

  • make test Run local tests prior to pushing.
  • make format Format source code to comply with lint policies. Note that some lint errors can only be resolved manually (e.g., Python)
  • make lint Run lint on source code prior to pushing.
  • Manual Update: Update the manual to reflect user-visible changes.
  • Type Labels: Select a github label for the type: bugfix, enhancement, etc.
  • Product Labels: Select a github label for the product: TaskVine, Makeflow, etc.
  • PR RTM: Mark your PR as ready to merge.

- Fix tests to all use any available port.  (port=0)
@dthain dthain added build For changes to the build and test infrastructure. TaskVine labels Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build For changes to the build and test infrastructure. TaskVine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant