Skip to content

chore(deps): update marshmallow 3.22.0 → 3.26.2 [renovate-agent]#2819

Draft
notify-pr-bot[bot] wants to merge 16 commits intomainfrom
renovate-agent/marshmallow-3.26.2-20260327
Draft

chore(deps): update marshmallow 3.22.0 → 3.26.2 [renovate-agent]#2819
notify-pr-bot[bot] wants to merge 16 commits intomainfrom
renovate-agent/marshmallow-3.26.2-20260327

Conversation

@notify-pr-bot
Copy link
Copy Markdown

@notify-pr-bot notify-pr-bot bot commented Mar 27, 2026 •
edited by Copilot AI
Loading

Summary | Résumé

Updates marshmallow from 3.22.0 to 3.26.2 (security update). Also restores the Redis fixture setup in tests/app/celery/test_int_annual_limit_seeding.py that was accidentally removed by a revert commit merged from main, which was causing the CI Python tests to fail with a Redis connection error.

Related Issues | Cartes liées

Test instructions | Instructions pour tester la modification

  1. Verify CI Python tests pass — specifically tests/app/celery/test_int_annual_limit_seeding.py::test_int_annual_limit_seeding_and_incrementation_flows_in_celery.
  2. Confirm no regressions in other test suites.

Release Instructions | Instructions pour le déploiement

None.

Reviewer checklist | Liste de vérification du réviseur

  • This PR does not break existing functionality.
  • This PR does not violate GCNotify's privacy policies.
  • This PR does not raise new security concerns. Refer to our GC Notify Risk Register document on our Google drive.
  • This PR does not significantly alter performance.
  • Additional required documentation resulting of these changes is covered (such as the README, setup instructions, a related ADR or the technical documentation).

jzbahrai and others added 14 commits March 27, 2026 17:40
@jzbahrai
fixes
776f97b
@jzbahrai
push
0edc2b7
@jzbahrai
chore: trigger renovate-agent workflow on task/fixes push
eb726ea
@jzbahrai
fix: checkout triggering ref instead of hardcoded main
2fbbb4a
@jzbahrai
feat: auto-fix tests via Copilot agentic workflow when renovate-agent…
8bea622 
… tests fail

- Add --create-draft-pr mode to renovate_agent.py: creates a draft PR
  with 'renovate-fix-needed' label when tests fail on a renovate branch
- Update renovate-agent.yml: swap --cleanup for --create-draft-pr on
  test failure so the branch and PR are preserved for Copilot
- Add fix-renovate-tests.md: gh-aw agentic workflow that fires when
  'renovate-fix-needed' label is applied; Copilot reads CI logs, edits
  code, pushes fix, removes label, and triggers CI again
- Add promote-renovate-pr.yml: converts draft PR to ready-for-review
  automatically once CI turns green on a renovate-agent/** branch

NOTE: fix-renovate-tests.lock.yml still needs to be generated by
running 'gh aw compile .github/workflows/fix-renovate-tests.md'
on a non-QEMU (native x86_64) machine.
@jzbahrai
fix: remove disallowed contents:write permission from gh-aw workflow
9cdab55
@jzbahrai
fix: remove disallowed pull-requests:write permission from gh-aw work…
0d43ab5 
…flow
@jzbahrai
chore: add compiled gh-aw lock file for fix-renovate-tests workflow
4bfba86
@jzbahrai
fix: remove GH_AW_CI_TRIGGER_TOKEN; re-trigger CI via PR_BOT instead
89afd1c 
- Remove github-token-for-extra-empty-commit from fix-renovate-tests.md
  (avoids needing a new long-lived PAT in the org)
- Add retrigger-ci-after-copilot-fix.yml: fires on pull_request:unlabeled
  for 'renovate-fix-needed'; uses existing PR_BOT_GITHUB_TOKEN app to
  push an empty commit that triggers CI on the renovate-agent branch
- Fix promote-renovate-pr.yml: workflow name was 'Continuous Integration
  Testing' but actual name in test.yaml is 'Python tests'
@jzbahrai
chore: add workflow_dispatch workflow to compile gh-aw lock file on n…
b45bbb7 
…ative Linux
@jzbahrai
fix: trigger compile-gh-aw-lock on push to fix-renovate-tests.md (not…
55e7c9b 
… just workflow_dispatch)
@jzbahrai
chore: trigger gh-aw lock file compilation on CI
9483c7b
@github-actions
chore: recompile gh-aw lock file [skip changelog]
fab324a
@notify-pr-bot
chore(deps): update marshmallow [renovate-agent]
a96d8aa
@github-actions github-actions bot mentioned this pull request Mar 30, 2026
Open
@jzbahrai
Copy link
Copy Markdown
Collaborator

jzbahrai commented Mar 30, 2026

@copilot fix the failing CI tests. The PR updates marshmallow from 3.22.0 to 3.26.2. Read the CI logs, identify what broke, and make the minimum code changes. Do NOT modify pyproject.toml, poetry.lock, or .github/ files.

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 30, 2026 •
edited
Loading

@copilot fix the failing CI tests. The PR updates marshmallow from 3.22.0 to 3.26.2. Read the CI logs, identify what broke, and make the minimum code changes. Do NOT modify pyproject.toml, poetry.lock...

The CI failure was caused by a missing Redis fixture in tests/app/celery/test_int_annual_limit_seeding.py. The febd8f9 revert commit (merged from main) accidentally removed the pmr_redis_config fixture and redis = create_redis_fixture() setup that ensures a Docker Redis container runs on port 6380 during the test. Without it, the test tried to connect to localhost:6380 and got ConnectionError.

Fixed in 7fc7235 by restoring the fixture setup (imports + pmr_redis_config + create_redis_fixture + redis parameter on the test function), identical to what was in the pre-revert state.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • marshmallow.readthedocs.io
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI requested a review from jzbahrai March 30, 2026 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jzbahrai jzbahrai Awaiting requested review from jzbahrai

@jimleroyer jimleroyer Awaiting requested review from jimleroyer jimleroyer will be requested when the pull request is marked ready for review jimleroyer is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

renovate-agent renovate-fix-needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jzbahrai