feat(Drone): Add more configuration and split it #22

Merged
kosssi merged 1 commits from drone into main 2022-01-26 10:54:07 +01:00
Owner

Détails

  • intégration d'option pour l'utilisation de Drone
  • séparation des docker-compose en plusieurs fichiers

Pourquoi

  • Pour permettre une configuration plus avancé de Drone
  • Pour nous permettre d'intégrer facilement Drone avec les options que l'on veut
## Détails - intégration d'option pour l'utilisation de Drone - séparation des docker-compose en plusieurs fichiers ## Pourquoi - Pour permettre une configuration plus avancé de Drone - Pour nous permettre d'intégrer facilement Drone avec les options que l'on veut
kosssi added 1 commit 2022-01-21 19:26:06 +01:00
kosssi force-pushed drone from 5127e72dca to a841b7f5bd 2022-01-26 08:54:15 +01:00 Compare
kosssi force-pushed drone from a841b7f5bd to 6f9a7aaba4 2022-01-26 08:57:22 +01:00 Compare
kosssi requested review from killian 2022-01-26 08:57:39 +01:00
killian reviewed 2022-01-26 09:24:46 +01:00
@ -0,0 +4,4 @@
## Installation
L'installation de la partie _runner_ a été coupé en plusieurs fichiers dont les noms sont assez explicite. De nombreux liens vers la documentation officiel ont été mis dans les fichiers _Docker Compose_.

Pour les fautes d'orthographe :
a été coupée
assez explicites
documentation officielle

Pour les fautes d'orthographe : `a été coupée` `assez explicites` `documentation officielle`
kosssi marked this conversation as resolved
killian reviewed 2022-01-26 09:25:11 +01:00
@ -0,0 +6,4 @@
L'installation de la partie _runner_ a été coupé en plusieurs fichiers dont les noms sont assez explicite. De nombreux liens vers la documentation officiel ont été mis dans les fichiers _Docker Compose_.
Il est possible de mettre en place une interface utilisateur pour visualiser les logs, les tâches exécutés. C'est pratique pour débugger.

Pour les fautes d'orthographe :
tâches exécutées.

Pour les fautes d'orthographe : `tâches exécutées.`
kosssi marked this conversation as resolved
killian reviewed 2022-01-26 09:25:43 +01:00
@ -0,0 +8,4 @@
Il est possible de mettre en place une interface utilisateur pour visualiser les logs, les tâches exécutés. C'est pratique pour débugger.
Le coté multiplatforme permet d'avoir plusieurs _runner_ sur des architectures différentes selon là ou on les déploie.

Pour les fautes d'orthographe :
là où

Pour les fautes d'orthographe : `là où`
kosssi marked this conversation as resolved
killian reviewed 2022-01-26 09:30:52 +01:00
@ -0,0 +2,4 @@
## Installation
L'installation de la partie serveur a été coupé en plusieurs fichiers dont les noms sont assez explicite. De nombreux liens vers la documentation officiel ont été mis dans les fichiers _Docker Compose_.

Pour les fautes d'orthographe :
a été coupée
assez explicites
documentation officielle

Pour les fautes d'orthographe : `a été coupée` `assez explicites` `documentation officielle`
kosssi marked this conversation as resolved
killian reviewed 2022-01-26 09:31:18 +01:00
@ -0,0 +36,4 @@
drone user add prometheus --machine --token=${PROMETHEUS_TOKEN}
```
En oubliant pas au moment de l'installation d'identifier précisément les utilisateurs ayant le droit d'executer Drone avec la variable `DRONE_USER_FILTER=kosssi,killian,prometheus,${DRONE_ADMIN_USER}`

Pour les fautes d'orthographe :
En n'oubliant pas
d'exécuter

Pour les fautes d'orthographe : `En n'oubliant pas` `d'exécuter`
kosssi marked this conversation as resolved
killian reviewed 2022-01-26 09:31:30 +01:00
@ -0,0 +38,4 @@
En oubliant pas au moment de l'installation d'identifier précisément les utilisateurs ayant le droit d'executer Drone avec la variable `DRONE_USER_FILTER=kosssi,killian,prometheus,${DRONE_ADMIN_USER}`
[Documentation officiel](https://docs.drone.io/cli/user/drone-user-add/)

Pour les fautes d'orthographe :
Documentation officielle

Pour les fautes d'orthographe : `Documentation officielle`
kosssi marked this conversation as resolved
killian reviewed 2022-01-26 09:35:00 +01:00
@ -0,0 +34,4 @@
#DRONE_GIT_ALWAYS_AUTH=
DRONE_GITEA_SERVER=gitea.cool.life
DRONE_GITEA_CLIENT_ID=UI76T78G-HDZ8-7CSD-6SDZ-YUIDG8Z7DSQ8
DRONE_GITEA_CLIENT_SECRET=y9ruXnEqluXjKUcfs5yIFlH83yb1OpP32NCf0h5YJwg=

Même si ce ne sont pas des vrais secrets ça me fait toujours transpirer de voir ces chaînes de caractères en clair. Tu peux changer les exemples dans ce fichier par des chaines plus génériques du style secret-a-changer afin que l'on soit sûrs qu'il ne s'agisse pas de vrais secrets ?

Même si ce ne sont pas des vrais secrets ça me fait toujours transpirer de voir ces chaînes de caractères en clair. Tu peux changer les exemples dans ce fichier par des chaines plus génériques du style `secret-a-changer` afin que l'on soit sûrs qu'il ne s'agisse pas de vrais secrets ?
kosssi marked this conversation as resolved
killian approved these changes 2022-01-26 09:36:01 +01:00
killian left a comment
Owner

Mis à part les fautes d'orthographe et les exemples de secret à remplacer, ça me semble ok même si ça fait beaucoup de fichiers d'un coup et que je peux rater la compréhension de la PR.

Mis à part les fautes d'orthographe et les exemples de secret à remplacer, ça me semble ok même si ça fait beaucoup de fichiers d'un coup et que je peux rater la compréhension de la PR.
kosssi force-pushed drone from 6f9a7aaba4 to 567349390c 2022-01-26 10:53:17 +01:00 Compare
kosssi merged commit cb5d8cf5ff into main 2022-01-26 10:54:07 +01:00
kosssi deleted branch drone 2022-01-26 10:54:07 +01:00
This repo is archived. You cannot comment on pull requests.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: resilien/services#22
No description provided.