[OGUI-1844] Implementation of database service-controller structure and seeding creations by dwedr · Pull Request #3245 · AliceO2Group/WebUi
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR implements a comprehensive database and Vault infrastructure for the Tokenization service, including seeding, migrations, integration tests, and CI/CD updates.
Purpose: Establish database service-controller structure with automated seeding for both Vault and the database, along with integration tests to verify seeded data.
Key Changes:
- Database infrastructure with Sequelize ORM, migrations, and seeders
- Vault integration with certificate-based authentication and key management
- Comprehensive test suite including unit and integration tests
- Docker Compose orchestration for local development and testing
Reviewed changes
Copilot reviewed 68 out of 74 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
scripts/actions-certificates-creation.sh |
Generates test certificates for Vault and clients |
docker/vault/vault-setup.sh |
Initializes and configures Vault with policies and secrets |
docker/database/populate/*.sql |
Database and user creation scripts |
docker-compose*.yml |
Updated service orchestration with Vault and database dependencies |
backend/central-system/src/lib/database/* |
Database models, migrations, seeders, and Sequelize configuration |
backend/central-system/src/controllers/VaultController.ts |
Vault operations controller with event bus integration |
backend/central-system/src/services/Vault*.ts |
Vault service implementations (auth, sign, encrypt, credentials) |
backend/central-system/tests/**/*Integration.spec.ts |
Integration tests for Vault and database |
.github/workflows/tokenization.yml |
Updated CI pipeline with certificate generation and extended tests |
Files not reviewed (2)
- Tokenization/backend/central-system/package-lock.json: Language not supported
- Tokenization/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.