potentially move production `cot_restricted_scopes` etc to the scriptworker-scripts repo
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
Research direction
Start by reading scriptworker.cot.config.DEFAULT_CONFIG and the existing production CoT tests, then compare how scriptworker-scripts packages configuration into its Docker images. Define where each production config belongs and how scriptworker tests obtain it. Done means the configs are maintained in scriptworker-scripts, available in the appropriate images, and production tests run against them in both repositories when relevant changes land.
Written by the indexing model from the issue text.
Description
This may include:
- zeroing out, or minimizing, the production CoT configs in
scriptworker.cot.config.DEFAULT_CONFIG- e.g.
cot_restricted_scopes,trusted_vcs_rules.valid_tasks_for,cot_restricted_trees, etc.
- e.g.
- adding a single config file in scriptworker-scripts with the production CoT configs (or a config file per
cot_product).- this file should be baked into the appropriate docker images.
- download and use the production scriptworker-scripts production CoT config file during scriptworker production tests
- we should also run these tests in scriptworker-scripts, on CoT config file change.
Details:
tom.prince:matrix: 10:36
@aki I was wondering if it would make sense to move cot_restricted_scopes to the -scripts repo, now that there is a single script repo (possibly also some of the other default config as well).aki 10:37
@tom.prince moving it makes some sense overall. however, we do use those for testing production cot in the scriptworker repo.
10:37
we may want to move those tests if the production config is elsewhere
10:38
and then it's harder to see if you broke something when you refactor cotaki 10:51
we solved for that in puppet by allowing for it to be set in both places, with eventual consistency (landing a puppet change for emergency changes, updating scriptworker with less urgency) and removing the puppet copy once they were in sync
10:52
could also download a config file from scriptworker-scripts during scriptworker teststom.prince:matrix: 12:26
Yeah, the later seems like a better solution.
- Dominant language
- Python
- Stars
- 5
- Forks
- 40
- Avg merge
- 22h 14m
- Merged PRs (30d)
- 10
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from mozilla-releng/scriptworker
-
Dependency DashboardOpen
Difficulty 2/5 1-3 hours Newbie friendliness 25/100
mozilla-releng/scriptworker#748 ·
Maintainers usually reply within 1 day
-
verify_cot fails to validate pull requests on repos that are forksMay be free again @ahal claimed this 889 days ago, and no pull request is open. Open
mozilla-releng/scriptworker#645 · 4 comments · 1 assignee ·
Maintainers usually reply within 1 day
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
mozilla-releng/scriptworker#584 ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
mozilla-releng/scriptworker#580 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
mozilla-releng/scriptworker#570 ·
Maintainers usually reply within 1 day
All issues in mozilla-releng/scriptworker
Similar issues
-
tool-calling
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
vllm-project/vllm#59838 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 92/100
raullenchai/Rapid-MLX#4042 ·
Maintainers usually reply within 1 day
-
documentation
Difficulty 1/5 Under an hour Newbie friendliness 92/100
transitmatters/mbta-slow-zone-bot#70 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
litestar-org/advanced-alchemy#811 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day