Page 2 sur 3

Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 27 août 2022, 21:17
par marsante

oui pour le bcmath c'est tout nouveau et lié à la nouvelle version de wixel/gump. J'avais testé sur la branch master non synchronisé à upstream :?


Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 27 août 2022, 21:31
par marsante

J'avais oublié aussi que je n'avais pas mis de variable pour la path de medshake sur ce dépot.

Maintenant c'est wixel qui bloque, le passer en 1.5.7 permet d'aller au bout


Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 27 août 2022, 21:32
par marsante

et pour le côté magique oui ça pousse à la surconsommation de vm ^^ avec la fibre et un nvme on ne voit rien passer


Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 27 août 2022, 21:37
par marsante

pardon depuis tout à l'heure je parle de 1.5.7 mais pour php8 c'est 1.5.11, après ils ont mis en place les alpha_numeric_dash validators et c'est ce qui entraine des bugs d'url / d'installation


Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 28 août 2022, 00:02
par Bertrand

Bon voilà, le vagrant up ne donne plus d'erreur.
Par contre http://192.168.56.4/install.php ne donne rien.
L'adresse est transformée en https://msehr.local/install.php et not found pour finir.

Sinon si je comprends bien, à chaque up, les modifs du dépôt cloné initialement en local sont injectés de nouveau de la vm ?

B.


Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 28 août 2022, 10:18
par marsante

A oui le virtualhost fait une redirection vers le nom de domaine.
Soit il faut le rajouter à la main dans /etc/hosts.
Soit installer

Code : Tout sélectionner

vagrant plugin install vagrant-hostsupdater 

le mot de passe est demandé sur chaque nouvel session de terminal pour écrire dans le fichier hosts. Ou on peut rajouter cette ligne dans le vagrantfile pour conserver l'entrée malgré les up et les halt

Code : Tout sélectionner

hostsupdater.remove_on_suspend = false

.
Il faudra que je modifie les consignes. Je ne le voyais plus avec le plugin d'installé.


Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 28 août 2022, 12:15
par marsante

Concernant la synchronisation des sources,

C'est ce bloc qui gère :

Code : Tout sélectionner

- name: sync sources with working directory
      copy: 
        src: /vagrant/
        dest: /opt/ehr 
        remote_src: yes 
        directory_mode: yes
      when: stage == "testing"
 

Tant que la variable testing est dans le secret.yml, les sources seront synchronisées à chaque

Code : Tout sélectionner

vagrant provision

. Sur une machine arrêtée déjà provisionnée une première fois, qu'on voudrait synchroniser,

Code : Tout sélectionner

vagrant up --provision

Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 28 août 2022, 20:28
par Bertrand

Ok merci pour les explications.
Coincé je suis à nouveau, cette fois dans le navigateur à la page https://msehr.local/install.php où le bouton Suivant ne fait rien, sauf recharger la page.


Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 28 août 2022, 21:32
par marsante

alors là je ne sais pas.
ça marche toujours sur mon dépôt https://github.com/marsante/MedShakeEHR-vagrant/ avec la 7.1.1. Mais plus sur le dépot master (l'avantage aussi de cette méthode c'est qu'on obtient les mêmes bugs).
Je vais vérifier que ça ne soit pas une erreur d'inattention de ma part sur le yaml. Un vagrant ssh permet de rentrer dans la machine, j'ai bien trouvé les fichiers et le config.yml à leurs places


Re: Technique pour tester rapidement et intégrer vos PR.

Publié : 29 août 2022, 10:56
par marsante

J'ai vérifié, je reproduis le bug si je synchronise les sources master sur mon projet Vagrant et sur mon projet docker (avec php 7.4 sinon j'ai les messages d'erreurs soulevés précédemment). Il semblerait que ça vienne du dépôt.
J'en ai profité pour synchroniser mon projet vagrant avec la base. Une nouvelle pull request mais qui n'arrangera pas le comportement de la page d'installation.