[OpenStack-Infra] Checking release approvals automatically

James E. Blair corvus at inaugust.com
Mon Dec 16 18:28:05 UTC 2019


Thierry Carrez <thierry at openstack.org> writes:

> James E. Blair wrote:
>> [...]
>> But back on the first hand, I think that installing python packages in a
>> virtualenv is too heavyweight for a job to run on the executor.  The
>> candidates we usually look for are things that can run with what's
>> already installed.  Happily, yaml is already installed, because it's
>> kind of a big deal on the executor.  Unhappily, openstack-governance is
>> not merely a repo you need to have on-disk, but is actually a python
>> package you need installed (wow, when did that happen?).
>>
>> We were so close.  If you just needed to run a python script that
>> imported yaml and read a file out of governance, I'd say it would be a
>> great candidate for running on the executor.  But I think the
>> installation of openstack-governance (which has its own requirements
>> that are not installed on the executor) pushes this over the line, and
>> we should run it on a full node.
>
> Actually the script only uses openstack-governance to parse YAML files
> that are in the governance repository... So if YAML is available and
> the contents of the governance repo are accessible, that can easily
> work.
>
> The only drawback compared to using the governance lib is that it will
> not survive a change in the YAML format of governance files... but
> then it's not the only thing that would break if we did that.
>
> So it looks like a simple Python script that only imports yaml would
> work on the executor. The script uses requests as well, but I can make
> it use urllib instead (unless requests is pre-installed on the
> executor too ?)

Yes, requests is installed too.

> Thanks for the full analysis, I learned a couple of things :)

You're welcome!

-Jim



More information about the OpenStack-Infra mailing list