search

ansible / ansible-runner-service

Other
200 stars 66 forks source link

Release 1.0.3 #72

Closed mnecas closed 4 years ago

mnecas commented 4 years ago

@mwperina @jmolmo @pcuzner

jmolmo commented 4 years ago

It seems ok for me. Just one question: Have you tried to build the nginx container with the new "psutil" dependency and check that it starts?

mwperina commented 4 years ago

AFAIK the psutil dependency raised up only when tried to use ARS RPM under Apache on CentOS Stream, but we didn't saw the issue on standard CentOS 8.1. That's why I we haven't touched spec file for nginx container, am I right Martine?

Anyway let's check also nginx container build

@mnecas

mnecas commented 4 years ago

Yes, that is correct. @jmolmo if you want I can add it to nginx just to be safe.

jmolmo commented 4 years ago

Yes, that is correct. @jmolmo if you want I can add it to nginx just to be safe.

@mnecas : Yes, it will be nice. Probably is safer. Once "psutil" dependency will be included i think that it is ok to create the new release. Thank you