[ovs-dev] [PATCH] github: Fix handling of python packages.

[Ilya Maximets]

On 3/15/21 12:53 PM, Aaron Conole wrote:
> Ilya Maximets <i.maximets at ovn.org> writes:
> 
>> GitHub Actions doesn't have python locations in PATH and different
>> runners might have different configuration for default python
>> location and versions.  For example, on some runners python2 might
>> be installed or not.
>>
>> Missing PATH causes weird situations on older branches where during
>> one run our scripts can locate just installed flake8 and can't do
>> that on a different run.  But this might also create other
>> unpredictable issues on all branches.
> 
> This is strange.  We do an install via the package manager - is it not
> installing to the correct PATH location?
> 
>> It's required to use actions/setup-python at v2 in order to have
>> predictable version of python installed and paths correctly configured.
>> Due to some bugs in GHA itself it doesn't set $HOME/.local/bin into
>> PATH, so we have to do that manually for now in order to use '--user'.
>> This might be fixed later in actions/setup-python or in base runners.
>> We already setting it for DPDK 20.11 (I think the issue was spotted
>> but not fully investigated).  Moving PATH updates to a separate step
>> to make them more explicit and avaialble for all steps of the job.
> 
> *available
> 
> Do you know if GHA team has a plan to fix this?
> 
> It's a strange syntax, but it looks like the manual requires that we do
> PATH updates via 'echo "..." >> $GITHUB_PATH' so, LGTM.

Can I treat that as 'Acked-by'?

Bets regards, Ilya Maximets.