Merge pull request #17 from Aignosi/release/SIENTIAPDE-1352
SIENTIAPDE-1352: Update deploy workflow to use reusable workflow and add permissions, update README with CI/CD details.
This commit is contained in:
15
README.md
15
README.md
@@ -821,10 +821,21 @@ All tools are configured in the `pyproject.toml` file:
|
||||
|
||||
### CI/CD Integration
|
||||
|
||||
The `.github/workflows/quality-gate.yml` workflow automatically runs all validations on each push/PR:
|
||||
The project uses **reusable workflows** from `Aignosi/github_workflow_templates` for CI/CD:
|
||||
|
||||
#### Quality Gate (`.github/workflows/quality-gate.yml`)
|
||||
Runs automatically on each Pull Request to `main`:
|
||||
- ✅ Formatting and linting block merge if they fail
|
||||
- ⚠️ Type checking and security generate warnings but don't block
|
||||
- ✅ Tests must pass with minimum 80% coverage
|
||||
- ✅ SonarQube analysis for code quality metrics
|
||||
- ✅ Automatic version calculation based on branch pattern
|
||||
|
||||
#### Deploy (`.github/workflows/deploy.yml`)
|
||||
Runs automatically when a PR is merged to `main`:
|
||||
- ✅ Builds and pushes Docker image to Azure Container Registry
|
||||
- ✅ Creates GitHub release with calculated version
|
||||
- ✅ Deploys to Kubernetes using Helm
|
||||
|
||||
### Best Practices
|
||||
|
||||
@@ -1367,7 +1378,7 @@ $ docker logs -f sientia-dataops-model-manager
|
||||
### Login using access token
|
||||
|
||||
```bash
|
||||
$ docker login -u bruno-aignosi -p LD/IyZ4vtDI7khRYnH4HzfdTx3toorg6hlCetJM54n+ACRDim3xO aignosi.azurecr.io
|
||||
$ docker login -u <username> -p <access-token> aignosi.azurecr.io
|
||||
```
|
||||
|
||||
### Push image to repository
|
||||
|
||||
Reference in New Issue
Block a user