eWaterCycle / infra

Instructions for system administrators to deploy the eWaterCycle platform
Apache License 2.0
0 stars 0 forks source link

Install pyOpenSSL==22.1.0 #135

Closed sverhoeven closed 1 year ago

sverhoeven commented 1 year ago

Between boot and roles/terria/tasks/main.yml an incompatible version of pyopenssl is installed.

The launcher is the only place where pip is run so a compatible pyopenssl is installed there.

Fixes #133

RolfHut commented 1 year ago

should we add something in docs or workflow to check if next time we update, this has been resolved and we can continue to use the latest version of pyOpenSSL? I'm afraid that if security risks not yet identified get fixed in future releases of pyOpenSSL, we might miss out on those.

sverhoeven commented 1 year ago

Thanks for the suggestion, I changed == to >= so we get at least the fixed version.

sverhoeven commented 1 year ago

To test it in Research Cloud we have to merge it as the SRC is configure to use main branch. (or reconfigure SRC to use 133-pyopenssl branch)

Peter9192 commented 1 year ago

To test it in Research Cloud we have to merge it as the SRC is configure to use main branch. (or reconfigure SRC to use 133-pyopenssl branch)

I thought pointing to this branch from SRC was probably the easiest way to test if the original problem was solved, but if merging it first is helpful then that's ok with me as well.

RolfHut commented 1 year ago

I'll wait until you give me a sign to test, given timezones: most likely after the weekend. Have a good weekend!

Rolf -- PS my working hours may not be your working hours. Please only reply in your working hours

-- dr. ir. Rolf Hut MacGyver Associate Professor in Water Resources and Maker Education at Delft University of Technology http://www.tudelft.nl/ Public speaker on Technology and Innovation for Everyone Columnist at De Ingenieur https://www.deingenieur.nl/artikel/quarantandem Writer of Rolfs Maakbare Wereld http://rolfhut.nl/rolfsmaakbarewereldbolcom and Knotsgekke Surprises voor Iedereen http://rolfhut.nl/surpriseboek. www.rolfhut.nl/english @RolfHut @.***>

Recent papers Hut et al.: Easy to build low-power GPS drifters with local storage and a cellular modem made from off-the-shelf components https://doi.org/10.5194/gi-9-435-2020, Geoscientific Instrumentation, Methods and Data Systems 2020 Hut et al.: Teaching a hands-on course during corona lockdown: from problems to opportunities https://doi.org/10.1088/1361-6552/abb06a, Physics Education 2020 Hut et al.: Taking a Breath of the Wild: are geoscientists more effective than non-geoscientists in determining whether video game world landscapes are realistic? https://doi.org/10.5194/gc-2-117-2019, Geoscience Communication 2019

On Fri, Oct 7, 2022 at 4:48 AM Peter Kalverla @.***> wrote:

To test it in Research Cloud we have to merge it as the SRC is configure to use main branch. (or reconfigure SRC to use 133-pyopenssl branch)

I thought pointing to this branch from SRC was probably the easiest way to test if the original problem was solved, but if merging it first is helpful then that's ok with me as well.

— Reply to this email directly, view it on GitHub https://urldefense.com/v3/__https://github.com/eWaterCycle/infra/pull/135*issuecomment-1271306765__;Iw!!PAKc-5URQlI!5pS0EMZYrA9RDPtc4quvRc_r90ZkTdheOIrRQRtMDDW1Byy2vAh7wMvU6qfqKlv8M3j7GT1X2lckPf9qnOAAGvMAmsE$, or unsubscribe https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AA5BYSCG5T77XHK2OPWZI7TWB7PYJANCNFSM6AAAAAAQ6MFDEE__;!!PAKc-5URQlI!5pS0EMZYrA9RDPtc4quvRc_r90ZkTdheOIrRQRtMDDW1Byy2vAh7wMvU6qfqKlv8M3j7GT1X2lckPf9qnOAAk0TMgoU$ . You are receiving this because you were mentioned.Message ID: @.***>

Peter9192 commented 1 year ago

@RolfHut please go ahead with spinning up a machine

RolfHut commented 1 year ago

machine spun up: works. Thanks all!