search

plone / ansible.plone_server

Ansible role to create a plone server
Other
9 stars 16 forks source link

Fix venv path for Plone 4 buildouts #106

Closed Gomez closed 5 years ago

Gomez commented 7 years ago

For multiple instances on one server i moved the venv in the instance dir. Everything related to the instance is now better isolated. AFAIK this is already done in the Plone5 part.

Gomez commented 7 years ago

@smcmahon @djowett I noticed with this change (don't know before) a rolling update is not possible as the python bin is in use ([Errno 26] Text file busy). Should i add a check if the bin is already there and skip the creation of the venv?

djowett commented 7 years ago

Hi @Gomez,

you don't have a creates: "{{ plone_instance_home }}/bin/python" on the first change, you should probably add that (or just merge the 2 plays). Otherwise seems a reasonable thing to fix, but I forked at 1.2.10, so I'm not up to date to test this. D

On 18 April 2017 at 18:25, Steffen Lindner notifications@github.com wrote:

@smcmahon https://github.com/smcmahon @djowett https://github.com/djowett I noticed with this change (don't know before) a rolling update is not possible as the python bin is in use ([Errno 26] Text file busy). Should i add a check if the bin is already there, skip the creation of the venv?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/plone/ansible.plone_server/pull/106#issuecomment-294918147, or mute the thread https://github.com/notifications/unsubscribe-auth/ABZuRameutqwkgS5fYaF3eNpX8Vw3k_iks5rxPIbgaJpZM4NAkjK .

-- Daniel Jowett

Jowett Enterprises Ltd Tel: 0333 3 553 773 daniel@jowettenterprises.com www.jowettenterprises.com

jensens commented 5 years ago

There was no activity on this for a while. I close this request for now. If you plan to work further on this please reopen and go on.