Skip to content

Raid v2#1008

Open
maximeroucher wants to merge 26 commits into
mainfrom
Raid-V2
Open

Raid v2#1008
maximeroucher wants to merge 26 commits into
mainfrom
Raid-V2

Conversation

@maximeroucher

@maximeroucher maximeroucher commented May 17, 2026

Copy link
Copy Markdown
Member

Description

Summary

Restructuring some of raid registering to match challenger idea :

  • Edition system
  • Participant derived from coreuser instead of separate entity
  • Volunteer registration and handling

Classification

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 🔨 Refactor (non-breaking change that neither fixes a bug nor adds a feature)
  • 🔧 Infra CI/CD (changes to configs of workflows)
  • 💥 BREAKING CHANGE (fix or feature that require a new minimal version of the front-end)
  • 😶‍🌫️ No impact for the end-users

Impact & Scope

  • Core functionality changes
  • Single module changes
  • Multiple modules changes
  • Database migrations required
  • Other: ...

Testing

  • 1. Tested this locally
  • 2. Added/modified tests that pass the CI (or tested in a downstream fork)
  • 3. Tested in a deployed pre-prod
  • 0. Untestable (exceptionally), will be tested in prod directly

Documentation

  • Updated the docs accordingly :
  • " Docstrings
  • # Inline comments
  • No documentation needed

Comment thread app/core/core_endpoints/cruds_core.py Outdated
Comment thread app/modules/raid/utils/validation_checker.py Outdated
Comment thread tests/config.test.yaml Outdated
Comment thread tests/conftest.py Outdated
Comment thread tests/test_migrations.py Outdated
Comment thread app/modules/raid/cruds_raid.py
@PabloBellissant

Copy link
Copy Markdown

The CI checks are currently failing:
Lint and format / lintandformat (pull_request)

Could you please check the linting and formatting issues please ?

@maximeroucher

Copy link
Copy Markdown
Member Author

Everything fixed, comment taken into accounts, if you want to check the changes.
Nice review, btw, raised only legitimate and fair points, GG for the level up !
Keep it up, the project is on really good hands ❤️

@maximeroucher
maximeroucher requested a review from warix8 June 5, 2026 22:26
@codecov

codecov Bot commented Jun 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.65459% with 165 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.80%. Comparing base (2afec59) to head (cda594d).

Files with missing lines Patch % Lines
app/modules/raid/endpoints_raid.py 62.50% 84 Missing ⚠️
app/modules/raid/cruds_raid.py 65.21% 40 Missing ⚠️
app/modules/raid/utils/utils_raid.py 34.37% 21 Missing ⚠️
app/modules/raid/utils/validation_checker.py 91.58% 9 Missing ⚠️
app/modules/raid/schemas_raid.py 95.85% 7 Missing ⚠️
app/modules/raid/dependencies_raid.py 93.10% 2 Missing ⚠️
app/modules/raid/factory_raid.py 95.65% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1008      +/-   ##
==========================================
- Coverage   84.61%   83.80%   -0.82%     
==========================================
  Files         214      217       +3     
  Lines       15523    16012     +489     
==========================================
+ Hits        13135    13419     +284     
- Misses       2388     2593     +205     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

maximeroucher and others added 13 commits July 22, 2026 18:55
…hine

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…machine

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…olunteers

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…teers

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…lunteer lookup

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… completeness

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…dition scoping

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…d volunteer flow

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
maximeroucher and others added 2 commits July 22, 2026 21:32
Adds GET /raid/participants (manage_raid-gated), mirroring GET /raid/volunteers,
built on the existing get_all_participants CRUD with an optional status filter.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
maximeroucher and others added 2 commits July 22, 2026 22:06
Documenso PR #1023 landed on main with 60-documents.py (parent
7dbe3290e145), colliding with our raid_editions_and_state which
targeted the same parent. Re-parent raid_editions_and_state onto
84ee3296cc58 so alembic sees a single head again.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
The 8 xdist workers each drop and re-create ~126 tables plus indexes
and FKs in a single transaction, which brushes past the default
max_locks_per_transaction=64 * max_connections=100 lock-table size and
sporadically fails with "out of shared memory / You might need to
increase max_locks_per_transaction". Append the tuned values to
postgresql.conf, restart the service container, and wait for readiness
before running tests.

GHA services can't set command args directly, so we name the container
and docker exec into it after startup.
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