Skip to content

Add transform for Python requirements installation - #1447

Open
rogerinhoam wants to merge 4 commits into
mozilla:mainfrom
rogerinhoam:refactor/python-requirements-transform
Open

rogerinhoam wants to merge 4 commits into
mozilla:mainfrom
rogerinhoam:refactor/python-requirements-transform

Conversation

@rogerinhoam

Copy link
Copy Markdown

Summary

  • add a python-requirements taskgraph transform for one or more requirements files
  • migrate evaluate-teacher-ensemble away from an inline requirements install
  • validate the generated command and error when the expected virtualenv activation is missing

Testing

  • python -m unittest taskcluster.test.test_python_requirements -v
  • taskgraph full --json --parameters taskcluster/test/params/small-ru-en.yml (153 tasks, 457 dependencies)
  • compared the generated graph against main; both files have the same SHA-256 hash

Addresses #588.

@rogerinhoam
rogerinhoam requested review from a team as code owners July 15, 2026 21:52
@rogerinhoam
rogerinhoam requested a review from jcristau July 15, 2026 21:52

@evgenyrp evgenyrp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM if it works. To test it in CI, we'll need to land first because it's an untrusted fork.

@bhearsum

Copy link
Copy Markdown
Collaborator

LGTM if it works. To test it in CI, we'll need to land first because it's an untrusted fork.

For future reference, you can mark any tasks that don't require scopes for things that shouldn't be available to non-collaborators as being able to run on untrusted pull requests. (See taskcluster documentation and an example from another repo.)

@evgenyrp

Copy link
Copy Markdown
Collaborator

LGTM if it works. To test it in CI, we'll need to land first because it's an untrusted fork.

For future reference, you can mark any tasks that don't require scopes for things that shouldn't be available to non-collaborators as being able to run on untrusted pull requests. (See taskcluster documentation and an example from another repo.)

@bhearsum I added it for tests and linter: #1448

@evgenyrp evgenyrp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Linting failed; please run task lint-fix. See https://mozilla.github.io/translations/docs/contributing/development/

This branch has not been deployed

No deployments
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