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 !

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.