HackENS/hackens-org-configurations
5
0
Fork 0
Code Issues 2 Pull requests 1 Projects Releases Packages Wiki Activity

Sites statique #5

Open
sinavir wants to merge 12 commits from static_website into master
pull from: static_website
merge into: HackENS:master
Conversation 3 Commits 12 Files changed 12 +228 -28
sinavir commented 2022-03-31 12:06:10 +02:00 (Migrated from git.rz.ens.wtf)
Copy link

Testé sans ACME.
ACME est en staging.

Testé sans ACME. ACME est en staging.
sinavir commented 2022-04-02 18:08:39 +02:00 (Migrated from git.rz.ens.wtf)
Copy link

J'ai ajouté un support de webooks comme ça on peut faire des scripts de déploiement automatiques

J'ai ajouté un support de webooks comme ça on peut faire des scripts de déploiement automatiques
raito (Migrated from git.rz.ens.wtf) reviewed 2022-04-02 19:09:58 +02:00
raito (Migrated from git.rz.ens.wtf) left a comment
Copy link
  • faut utiliser acmeStaging de my à l'extérieur des modules que tu définis avec la technique lib.mkDefault, ce qui permet qu'on puisse overrider dans d'autres fichiers, s'il le faut ;
  • tu pourrais écrire qq lignes pour tes intentions avec le module webhook (que je renommerais bien webhooks en vrai), tu veux fournir de l'auto-déploiement à travers Git des sites statiques?

Sinon, c'est génial merci !

- faut utiliser `acmeStaging` de `my` à l'extérieur des modules que tu définis avec la technique `lib.mkDefault`, ce qui permet qu'on puisse overrider dans d'autres fichiers, s'il le faut ; - tu pourrais écrire qq lignes pour tes intentions avec le module `webhook` (que je renommerais bien `webhooks` en vrai), tu veux fournir de l'auto-déploiement à travers Git des sites statiques? Sinon, c'est génial merci !
hosts/hackens-org/modules/staticWebsite.nix
raito (Migrated from git.rz.ens.wtf) commented 2022-04-02 19:13:20 +02:00
Copy link

même commentaire que sur webhook

même commentaire que sur `webhook`
hosts/hackens-org/modules/staticWebsite.nix
raito (Migrated from git.rz.ens.wtf) commented 2022-04-02 19:14:17 +02:00
Copy link

je préfère staticWebsites (le pluriel)

je préfère `staticWebsites` (le pluriel)
hosts/hackens-org/modules/staticWebsite.nix
raito (Migrated from git.rz.ens.wtf) commented 2022-04-02 19:14:43 +02:00
Copy link

ça ressemble à un invariant sur la longueur qui me paraît plus propre

ça ressemble à un invariant sur la longueur qui me paraît plus propre
raito (Migrated from git.rz.ens.wtf) commented 2022-04-02 23:12:53 +02:00
Copy link

J'ai pas compris (j'avoue j'ai copié le module dokuwiki pour la condition)

J'ai pas compris (j'avoue j'ai copié le module `dokuwiki` pour la condition)
hosts/hackens-org/modules/webhook.nix
raito (Migrated from git.rz.ens.wtf) commented 2022-04-02 19:09:58 +02:00
Copy link

tu fais dépendre ce module d'un module de type my, c'est pas une bonne idée, utilise plutôt cfg.debug et crée un field debug, quitte à ce qu'à l'extérieur du module tu fasses:

services.webhook.debug = lib.mkDefault config.my.debug
tu fais dépendre ce module d'un module de type `my`, c'est pas une bonne idée, utilise plutôt `cfg.debug` et crée un field `debug`, quitte à ce qu'à l'extérieur du module tu fasses: ```nix services.webhook.debug = lib.mkDefault config.my.debug ```
hosts/hackens-org/modules/webhook.nix
raito (Migrated from git.rz.ens.wtf) commented 2022-04-02 19:10:11 +02:00
Copy link

package (c'est un semi standard)

`package` (c'est un semi standard)
hosts/hackens-org/modules/webhook.nix
raito (Migrated from git.rz.ens.wtf) commented 2022-04-02 19:10:41 +02:00
Copy link

endpoint (c'est un mot)

`endpoint` (c'est un mot)
sinavir commented 2022-04-02 23:08:10 +02:00 (Migrated from git.rz.ens.wtf)
Copy link

Tu as tout compris pour les webhooks. J'ai pas trouvé de soft qui fait ça tout seul donc je me suis dit qu'un script custom marcherait bien et c'est pas trop complexe normalement. J'ai pas écrit le script parce que j'ai des questions:

  • Est-ce que cd workingDir; git reset --hard HEAD; git checkout production; git pull est suffisant où il y a une subtilité ?
  • Est-ce que créer/gérer le dossier de sources avec systemd-tmpfiles est la façon canonique de faire ?
  • Est-ce que il faut créer un user ad-hoc qui fait git pull ?
Tu as tout compris pour les webhooks. J'ai pas trouvé de soft qui fait ça tout seul donc je me suis dit qu'un script custom marcherait bien et c'est pas trop complexe normalement. J'ai pas écrit le script parce que j'ai des questions: * Est-ce que `cd workingDir; git reset --hard HEAD; git checkout production; git pull` est suffisant où il y a une subtilité ? * Est-ce que créer/gérer le dossier de sources avec systemd-tmpfiles est la façon canonique de faire ? * Est-ce que il faut créer un user ad-hoc qui fait `git pull` ?
sinavir commented 2022-04-02 23:48:21 +02:00 (Migrated from git.rz.ens.wtf)
Copy link

Sinon, si j'ai bien compris, c'est pas à mes modules de gérer ACME, mais plutôt dans les fichiers de "configuration" (les modules dans lesquels je set les options)

Sinon, si j'ai bien compris, c'est pas à mes modules de gérer ACME, mais plutôt dans les fichiers de "configuration" (les modules dans lesquels je set les options)
tomate (Migrated from git.rz.ens.wtf) reviewed 2022-05-17 20:50:31 +02:00
hosts/hackens-org/modules/staticWebsites.nix
@ -0,0 +41,4 @@
value = (mkMerge [
{
serverName = conf.hostname;
forceSSL = if config.services.staticWebsites.debug then false else true;
tomate (Migrated from git.rz.ens.wtf) commented 2022-05-17 20:50:31 +02:00
Copy link

forceSSL = !config.services.staticWebsites.debug;

`forceSSL = !config.services.staticWebsites.debug;`
tomate (Migrated from git.rz.ens.wtf) reviewed 2022-05-17 21:05:37 +02:00
hosts/hackens-org/modules/webhooks.nix
@ -0,0 +49,4 @@
Description = "Small server for creating HTTP hooks";
Documentation = "https://github.com/adnanh/webhook/";
};
script = "${cfg.package}/bin/webhook -nopanic -ip \"127.0.0.1\" -port \"${toString cfg.internalPort}\" -verbose -hooks ${json.generate "conf.json" cfg.hooks}";
tomate (Migrated from git.rz.ens.wtf) commented 2022-05-17 21:05:37 +02:00
Copy link

Tu peux utiliser des simple quotes pour pas avoir à escape tes double quotes

Tu peux utiliser des simple quotes pour pas avoir à escape tes double quotes
This pull request has changes conflicting with the target branch.
  • hosts/hackens-org/configuration.nix
  • hosts/hackens-org/wiki.nix
View command line instructions

Checkout

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

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout master
git merge --no-ff static_website
git checkout static_website
git rebase master
git checkout master
git merge --ff-only static_website
git checkout static_website
git rebase master
git checkout master
git merge --no-ff static_website
git checkout master
git merge --squash static_website
git checkout master
git merge --ff-only static_website
git checkout master
git merge static_website
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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: HackENS/hackens-org-configurations#5
No description provided.
Delete branch "static_website"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?