-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for de-novo creation of all the OpenShift / NFS fixtures (in namespace wwp-int) #107
base: master
Are you sure you want to change the base?
Conversation
J'ai voulu recheck la PR pour tester mais je n'ai trouvé aucune modification suite à mes commentaires ni aucune réponse à ceux-ci. La documentation n'était pas clair quand j'ai fait mon premier test et elle n'a pas changé depuis donc je ne peux toujours pas tester... |
Bonjour, les modifications demandées ont été apportées: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alors j'ai essayé d'installer sur MiniShift mais ça ne passe pas... J'y suis allé un peu à l'aveugle car il n'y a rien dans la doc pour faire ça... Il faudrait donc compléter la doc pour ajouter la partie minishift.
J'ai trouvé la doc faite par @zuzu59 est la plus parlante car détaillée.
Voir aussi les commentaires sur les fichiers.
wwp-int_zuzu_doc.md
Outdated
|
||
Il faut en premier se connecter sur OC: | ||
|
||
![Image](https://raw.githubusercontent.com/epfl-idevelop/wp-ops/feature/wp-int/img/oc_login.gif) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zuzu59 Faudrait voir pour virer les GIF car on peu bien y voir ton token de connexion et c'est l'équivalent d'avoir mis un username/password 😉
wwp-int_zuzu_doc.md
Outdated
Pour pouvoir récupérer les sauvegardes des sites Wordpress de la prod, il nous faut un *pod* de management qui aura une connexion *read only* sur le NAS (NFS) des sauvegarde de la prod. | ||
``` | ||
cd wp-ops/ansible | ||
./wpsible -l wp-blue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cette commande ne passe pas. ça passe que si je fais ./wpsible
wp_veritas) echo "-i inventory/wp-veritas" ;; | ||
esac | ||
CONFIG=$(oc config current-context) | ||
if [[ $CONFIG == "myproject/192-168-99-"*"developer" ]] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
La détection de MiniShift n'est pas assez dynamique car la plage d'IP n'est pas forcément celle définie dans le script.
Proposition de changement qui part du principe que minishift
se trouve dans /usr/sbin/
et qui récupère dynamiquement les infos dont on a besoin pour savoir si le projet correspond
if [[ $CONFIG == "myproject/192-168-99-"*"developer" ]] | |
minishift_project="" | |
# If minishift installed (we assume it is here) | |
if [[ -e /usr/sbin/minishift ]] | |
then | |
# looking for information to build project string | |
minishift_ip_port=`minishift console --url | awk -F\/ '{print $3}'` | |
minishift_ip=`echo $minishift_ip_port| awk -F: '{print $1}' | sed -e "s/\./-/g"` | |
minishift_port=`echo $minishift_ip_port| awk -F: '{print $2}'` | |
minishift_project="myproject/${minishift_ip}:${minishift_port}/developer" | |
fi | |
if [[ $CONFIG == $minishift_project ]] |
CONFIG=$(oc config current-context) | ||
if [[ $CONFIG == "myproject/192-168-99-"*"developer" ]] | ||
then | ||
echo "-i inventory/minishift -e " IPADRS="`curl https://api.ipify.org`" " " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L'utilisation de api.ipify.org ne fonctionne à priori pas quand on est dans une VM en NAT, correct? dans ce cas-là, il faudrait récupérer l'IP de la VM non ? ou celle de la VM de minishift ?
J'ai cherché des références/infos à IPADRS
dans le code et sur le net mais rien trouvé...
wwp-int_zuzu_doc.md
Outdated
``` | ||
./utils/install_oc.sh | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#### MiniShift (uniquement si on veut faire tourner en local) | |
La documentation d'installation se trouve ici : https://www.novatec-gmbh.de/en/blog/getting-started-minishift-openshift-origin-one-vm/ | |
A la fin de l'installation, copier le fichier `minishift` dans `/usr/sbin` ou créer un lien symbolique `/usr/sbin/minishift -> <pathToMiniShift>` |
wwp-int_zuzu_doc.md
Outdated
``` | ||
|
||
#### Ansible | ||
Ansible va nous permettre d'automatiser l'installation de l'infra de tests.<br> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai dû installer ceci avant Ansible car ça ne fonctionnait pas sans.
Ansible va nous permettre d'automatiser l'installation de l'infra de tests.<br> | |
Installer libffi | |
> sudo apt-get install -y libffi-dev | |
Ansible va nous permettre d'automatiser l'installation de l'infra de tests.<br> |
Quelques modifications sont proposées pour l'utilisation de Minishift. Si je les fais, ça me permet d'avancer dans la détection du fait que je suis bien sur Minishift mais je me heurte ensuite aux warning suivant et il ne se passe rien... La commande passée, comme dans la doc de @zuzu59 :
|
Ces 2 lignes sont à double dans
|
Hello Lucien,
Minishift n'a plus été utilisé depuis fin juillet, après nous avons
concentré tous nos efforts sur Openshift !
C'est pourquoi, la docu ainsi que certains scripts ne sont peut-être
plus compatibles avec Minishift, car nous ne les avons pas testés quand
on travaillait à donf sur Openshift.
Mohamed va regarder tout cela ce mercredi pour remettre en état la
partie Minishift.
Désolé pour ce contretemps.
@+
Christian
…_____
Christian Zufferey, Head of Students Development Factory, Vice
Présidence SI, iDEVELOP
Ecole Polytechnique Fédérale de Lausanne, EPFL, Station 14, CH-1015 Lausanne
Tel: +41 (0)21 693 4598, Fax: +41 (0)21 693 2220
mailto:[email protected]
http://people.epfl.ch/christian.zufferey
Le 25.11.19 à 13:22, LuluTchab a écrit :
Quelques modifications sont proposées pour l'utilisation de Minishift.
Si je les fais, ça me permet d'avancer dans la détection du fait que
je suis bien sur Minishift mais je me heurte ensuite aux warning
suivant et il ne se passe rien...
Il semble manquer quelque chose en rapport avec MiniShift dans |inventory|
|[WARNING]: Unable to parse
/home/lulutchab/git-repos/wp-ops/ansible/inventory/minishift as an
inventory source [WARNING]: No inventory was parsed, only implicit
localhost is available [WARNING]: provided hosts list is empty, only
localhost is available. Note that the implicit localhost does not
match 'all' [WARNING]: Could not match supplied host pattern,
ignoring: wp-blue PLAY [Initial checks]
******************************************************************************************************************************
skipping: no hosts matched [WARNING]: Could not match supplied host
pattern, ignoring: all-openshift-namespaces PLAY [OpenShift
namespaces]
************************************************************************************************************************
skipping: no hosts matched [WARNING]: Could not match supplied host
pattern, ignoring: all-wordpress-deployments PLAY [Wordpress
deployments]
***********************************************************************************************************************
skipping: no hosts matched [WARNING]: Could not match supplied host
pattern, ignoring: all-wordpresses PLAY [WordPress instances]
*************************************************************************************************************************
skipping: no hosts matched [WARNING]: Could not match supplied host
pattern, ignoring: awx-poc-vpsi PLAY [Ansible Tower (AWX)
configuration]
***********************************************************************************************************
skipping: no hosts matched PLAY RECAP
*****************************************************************************************************************************************
|
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#107>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADVCMUPMO2P53IZADDLSCG3QVO7P3ANCNFSM4IQGMSVQ>.
|
…path need to be fixed
- Pick up sshd keys out of today's running mgmt container - Set up authorized_keys exactly like in the test instance (wwp-dev) - Remove guard on mgmt.yml (thereby causing a conflict with wwp-int-mgmt.yml; still TODO: merge both)
- PVC is identical to production (even though we didn't previously handle this object with Ansible yet) - DeploymentConfig is currently smarter on the wwp-int-mgmt.yml side, and cannot be unforked just yet (until such time that we can set database secrets in production like we do in wwp-int)
…e for restore tasks (cf Restore-Connector-doc.md)
… and service accounts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bravo pour l'excellent travail ! Encore un peu de nettoyage de commentaires, et il faut que @LuluTchab revalide; ensuite ce sera bon.
@@ -16,6 +17,11 @@ | |||
# Can we use a handler instead? | |||
- config | |||
|
|||
# TODO: Tu peux shooter les tâches (et leurs dépendances) "Backup", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Selon ta décision, il faut virer le TODO.
|
||
- name: Serving infrastructure | ||
when: wp_can.configure | ||
tags: | ||
- config | ||
import_tasks: "serving-infrastructure.yml" | ||
|
||
# TODO: À partir d'ici, à garder; avant de livrer, ces tâches doivent |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Idem — Selon ta décision, il faut virer le TODO.
@LuluTchab les changements concernant MiniShift ont été abandonnés dans le cadre de cette PR; @batKem s'y atellera dans une autre. Merci d'avance pour ta code review, et bon week-end ! |
9b8fa96
to
09df19f
Compare
89e81dd
to
e8140d7
Compare
facec54
to
76d9133
Compare
Pull-request for wwp-int feature: