Kubernetes playbooks for mediawiki #17
Conversation
There was a problem hiding this comment.
Commentary in ansibleplaybookbundle/postgresql-apb#23 (review) that impacts this change. Better to have the conversation in one place than 2.
rthallisey
force-pushed
the
k8s-playbook
branch
from
aeabfd0
to
139628e
Compare
rthallisey
force-pushed
the
k8s-playbook
branch
from
139628e
to
fc9dd8b
Compare
playbooks/provision.yml
Outdated
| - mediawiki | ||
| vars: | ||
| action: provision | ||
| cluster: openshift |
There was a problem hiding this comment.
In the other PR you used kubernetes as the default, why the difference?
There was a problem hiding this comment.
it should be kubernetes
roles/mediawiki/tasks/update.yml
Outdated
| value: '{{ route.route.spec.host }}' | ||
| image: docker.io/ansibleplaybookbundle/mediawiki123:latest | ||
| value: '{{ endpoint.endpoints.subsets[0].addresses[0].ip }}' | ||
| image: docker.io/jmontleon/mediawiki123:update |
There was a problem hiding this comment.
This doesn't look like a change you want to make.
There was a problem hiding this comment.
Shouldn't we be using the mediawiki image now instead of mediawiki123?
There was a problem hiding this comment.
My rebase scooped this up. It should be docker.io/ansibleplaybookbundle/mediawiki123:latest
There was a problem hiding this comment.
What I meant was the mediawiki123 image isn't being updated anymore, you should probably use the mediawiki image
|
Going to merge this. I need it for the k8s and we can always fix it later if there are issues. |
No description provided.