wip -- core.deploy -- Scripts to deploy {www,dev}.cof #645

Open
delobell wants to merge 1 commit from aureplop/core-deploy_initial into master
delobell commented 2019-01-07 22:37:24 +01:00 (Migrated from git.eleves.ens.fr)
No description provided.
delobell commented 2019-01-07 22:37:54 +01:00 (Migrated from git.eleves.ens.fr)

@mpepin je me demande si je suis pas allé trop loin avec l'affaire des services 🤔

@mpepin je me demande si je suis pas allé trop loin avec l'affaire des services :thinking:
mpepin commented 2019-01-13 12:36:50 +01:00 (Migrated from git.eleves.ens.fr)

Je n'ai pas bien compris ça (les dossiers provisioning/XXX.cof/)

Je n'ai pas bien compris ça (les dossiers `provisioning/XXX.cof/`)
mpepin commented 2019-01-13 12:41:12 +01:00 (Migrated from git.eleves.ens.fr)
  • Ce n'est pas le bon intepréteur python, il faudrait utiliser celui du virtualenv
  • Il faudrait aussi définir la bonne variable d'environnement DJANGO_SETTINGS_MODULE
- Ce n'est pas le bon intepréteur python, il faudrait utiliser celui du virtualenv - Il faudrait aussi définir la bonne variable d'environnement `DJANGO_SETTINGS_MODULE`
mpepin commented 2019-01-13 12:48:45 +01:00 (Migrated from git.eleves.ens.fr)

ou plus succintement (sans if) :

git_check_clean_workingtree() {
    [ -z "$(git status --porcelain)" ]
}

ou plus succintement (sans if) : ```sh git_check_clean_workingtree() { [ -z "$(git status --porcelain)" ] } ```
mpepin commented 2019-01-13 13:00:37 +01:00 (Migrated from git.eleves.ens.fr)

Certaines commandes doivent être exécutées en tant que root (systemctl principalement) et d'autres doivent être exécutées en tant que l'utilisateur gestioCOF (migrate et collectstatic par exemple). L'utilisateur gestioCOF ne peut pas sudo.

Une solution peut-être serait d'exécuter tout le script en tant que root et de lancer les commandes python avec sudo -u gestioCOF. Il faudra penser à définir DJANGO_SETTINGS_MODULE.

Certaines commandes doivent être exécutées en tant que root (`systemctl` principalement) et d'autres doivent être exécutées en tant que l'utilisateur `gestioCOF` (`migrate` et `collectstatic` par exemple). L'utilisateur `gestioCOF` ne peut pas `sudo`. Une solution peut-être serait d'exécuter tout le script en tant que root et de lancer les commandes python avec `sudo -u gestioCOF`. Il faudra penser à définir `DJANGO_SETTINGS_MODULE`.
delobell commented 2019-01-13 14:06:52 +01:00 (Migrated from git.eleves.ens.fr)

Je comptais les versionner ici.

Y'a un inconvénient à souligner et dont il faudrait discuter avec @tobast.
Est-ce qu'on accepte qu'un changement dans la manière de faire tourner GC doivent être fait ici ou est-ce trop contraignant pour les sysadmins qui seraient amené à modifier ça ?

Après c'est dans l'optique de ne le faire tourner que pour dev. (en tout cas c'est mon avis).
Avoir les deux configs ici permettent en revanche aux dévs de GC de pouvoir tester plus simplement des "grands" changements et de comprendre comment les choses marchent en ayant une Vagrant "à jour". Pour le debug c'est utile aussi (ping @mpepin :P).

Au pire, si un changement doit être fait et que le script n'est pas mis à jour… et bien soit, on attendra qu'une bonne volonté répare ça ici ;)

Je comptais les versionner ici. Y'a un inconvénient à souligner et dont il faudrait discuter avec @tobast. Est-ce qu'on accepte qu'un changement dans la manière de faire tourner GC doivent être fait ici ou est-ce trop contraignant pour les sysadmins qui seraient amené à modifier ça ? Après c'est dans l'optique de ne le faire tourner que pour `dev.` (en tout cas c'est mon avis). Avoir les deux configs ici permettent en revanche aux dévs de GC de pouvoir tester plus simplement des "grands" changements et de comprendre comment les choses marchent en ayant une Vagrant "à jour". Pour le debug c'est utile aussi (ping @mpepin :P). Au pire, si un changement doit être fait et que le script n'est pas mis à jour… et bien soit, on attendra qu'une bonne volonté répare ça ici ;)
delobell commented 2019-01-13 14:07:52 +01:00 (Migrated from git.eleves.ens.fr)

bien vu !

bien vu !
delobell commented 2019-01-13 14:31:23 +01:00 (Migrated from git.eleves.ens.fr)
Voir https://merle.eleves.ens.fr/cof-geek/pl/ipef6uhe738yxez5x6de4oowcy et suivants
delobell commented 2019-01-13 14:40:48 +01:00 (Migrated from git.eleves.ens.fr)

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/341/diffs?diff_id=1075&start_sha=7cd4c711b6eea053a5401ff3514e7321aa46083a#34f14453eecbd8bf4b9e0d954d1e6a9a25436299_88_88)
delobell commented 2019-01-13 14:40:49 +01:00 (Migrated from git.eleves.ens.fr)

added 3 commits

  • 7cd4c711...670fda3c - 2 commits from branch master
  • a99ade48 - wip -- core.deploy -- Scripts to deploy {www,dev}.cof

Compare with previous version

added 3 commits <ul><li>7cd4c711...670fda3c - 2 commits from branch <code>master</code></li><li>a99ade48 - wip -- core.deploy -- Scripts to deploy {www,dev}.cof</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/341/diffs?diff_id=1075&start_sha=7cd4c711b6eea053a5401ff3514e7321aa46083a)
delobell commented 2019-01-13 14:45:28 +01:00 (Migrated from git.eleves.ens.fr)

added 21 commits

  • a99ade48...eb83c58f - 20 commits from branch master
  • e6d47c2b - wip -- core.deploy -- Scripts to deploy {www,dev}.cof

Compare with previous version

added 21 commits <ul><li>a99ade48...eb83c58f - 20 commits from branch <code>master</code></li><li>e6d47c2b - wip -- core.deploy -- Scripts to deploy {www,dev}.cof</li></ul> [Compare with previous version](https://git.eleves.ens.fr/klub-dev-ens/gestioCOF/merge_requests/341/diffs?diff_id=1076&start_sha=a99ade48610fbc92836ce82a0b0083d8a7f0c1bc)
This pull request has changes conflicting with the target branch.
  • provisioning/bootstrap.sh
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin aureplop/core-deploy_initial:aureplop/core-deploy_initial
git checkout aureplop/core-deploy_initial

Merge

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff aureplop/core-deploy_initial
git checkout master
git merge --ff-only aureplop/core-deploy_initial
git checkout aureplop/core-deploy_initial
git rebase master
git checkout master
git merge --no-ff aureplop/core-deploy_initial
git checkout master
git merge --squash aureplop/core-deploy_initial
git checkout master
git merge --ff-only aureplop/core-deploy_initial
git checkout master
git merge aureplop/core-deploy_initial
git push origin master
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
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: DGNum/gestioCOF#645
No description provided.