revert product-os/flowzone/pull/953

[ab77]

• installing Python dependencies on runner breaks other dependencies (e.g. AWS-CLI)

change-type: patch

[klutchell]

I prefer this approach rather than hardcoding runner tags.

[ab77]

I prefer this approach rather than hardcoding runner tags.

How is this approach fixing the underlying condition that the setup-python action will only currently succeed on X64/Linux? I must be missing something, perhaps you can add you commit directly to this branch?

We have seven environment repositories that are affected by this. We could technically change them all to only run on X64/Linux, but I think a one line change here is better than having to change x7, especially given that it will be temporary.

[klutchell]

I prefer this approach rather than hardcoding runner tags.

How is this approach fixing the underlying condition that the setup-python action will only currently succeed on X64/Linux? I must be missing something, perhaps you can add you commit directly to this branch?

We have seven environment repositories that are affected by this. We could technically change them all to only run on X64/Linux, but I think a one line change here is better than having to change x7, especially given that it will be temporary.

HOW temporary? Do we know if/when this will be fixed upstream?

I see the benefit of fixing it in Flowzone as it's less work, but then we have to make assumptions about what the calling repository has access to (self-hosted vs GitHub hosted) AND we need to remember and come back later to remove all the places in Flowzone where we are ignoring the runs_on input to provide a hardcoded one.

Would this be a suitable compromise?

[ab77]

Would this be a suitable compromise?

Ugh..

[ab77]

Would this be a suitable compromise?

Ugh..

We'll have to do it upstream and revert when they get their sh*t together.