Open Stack

On Fri, Apr 5, 2019 at 2:59 AM Sean Mooney <smooney at redhat.com> wrote:
>
> On Thu, 2019-04-04 at 15:20 -0800, Eric K wrote:
> > On 4/4/19, 1:51 PM, "Matthew Thode" <mthode at mthode.org> wrote:
> >
> > > On 19-04-04 14:30:51, Eric K wrote:
> > > > Requesting exception to lower a requirement to resolve a packaging
> > > > issue.
> > > >
> > > > https://review.openstack.org/#/c/650100/
> > > >
> > >
> > > This is more of a stable question, it's late in the cycle but it sounds
> > > possibly needed.  I think this is fine technically, but not sure about
> > > it policy wise.  To me it sounds more of CentOS's problem than
> > > Openstack's (to be a bit brash).  Added the stable tag to the thread.
> > >
> > > --
> > > Matthew Thode
> >
> > Thanks Matthew. I'm not so worried about stability because even if it
> > breaks
> > Something it affects only a new (Stein) experimental feature no one is
> > relying on.
> congresses lower constraints job
> https://github.com/openstack/congress/blob/master/tox.ini#L103-L108
>
> is broken the same way as novas was because of the install command that is used
> https://github.com/openstack/congress/blob/master/tox.ini#L9
>
> so https://review.openstack.org/#/c/650100/
> sets the lower constatit to something that is invalid for the gate job
> which runs on ubuntu. if you look at the output of the job
> http://logs.openstack.org/00/650100/1/check/openstack-tox-lower-constraints/13666ba/tox/lower-constraints-1.log
>
> we can see its using psycopg2
> Collecting psycopg2===2.7.7 (from -c /home/zuul/src/git.openstack.org/openstack/requirements/upper-constraints.txt (line
> 81))
>   Downloading
> http://mirror.ord.rax.openstack.org/pypifiles/packages/37/25/53e8398975aa3323de46a5cc2745aeb4c9db11352ca905d3a15c53b6a816/psycopg2-2.7.7-cp36-cp36m-manylinux1_x86_64.whl
>  (2.7MB)
>
> it is technically true that psycopg2==2.5.1 will work on centos but until this is restored to 2.7 the gate job cannot be
> fixed.
>
> pep-0508 defines a set of markers we can use for packageing
> https://www.python.org/dev/peps/pep-0508/
>
> the only marker that i know of that contains the disto name is "platform_version"
>
> we should be able to do something like
>
> psycopg2==2.5.1; platform_version~=CENTOS
> psycopg2==2.7;
>
> but i dont think we should be breaking one distor to fix another when the lower constatint.txt wont be used to build the
> centos packages anyway lower-constiatns will only be used when installing from pip in which case the distro versions is
> not relevent unless i have missed something.
> >
> >
> >
Thank you, Sean. Sorry I'm ignorant re: the lower constraints job.
Where can we look-up what the valid lower-constraints are?

Specifically, I looked at another earlier patch without the change to
psycopg2 version (lower constraint 2.7) but there also it installed
2.7.7.
http://logs.openstack.org/63/649763/4/check/openstack-tox-lower-constraints/8eb07bd/tox/lower-constraints-1.log
https://review.openstack.org/#/c/649763/

Does that mean psycopg2==2.7 is also invalid?