*** altlogbot_1 has quit IRC | 00:10 | |
*** altlogbot_1 has joined #openstack-requirements | 00:12 | |
openstackgerrit | Tony Breeds proposed openstack/requirements master: update warlock due to bug on 1.3.1 https://review.opendev.org/660529 | 00:28 |
---|---|---|
*** dangtrinhnt has quit IRC | 02:21 | |
*** dangtrinhnt has joined #openstack-requirements | 02:22 | |
*** dangtrinhnt has quit IRC | 02:23 | |
*** udesale has joined #openstack-requirements | 04:11 | |
prometheanfire | tonyb: so, one of the items on my todo is to write out what happened, are we talking about having to place caps on py2 stuff or other things? | 04:39 |
tonyb | prometheanfire: Why we need it, why there was a sudden rash of them rather than at release time how the timing of the bandit change didn't help | 04:40 |
tonyb | to restore faith in teh team and identify things we can chnage to avoid it happenign again | 04:41 |
tonyb | prometheanfire: I wrote a thing: http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006478.html | 04:42 |
* tonyb really hopes it makes sense | 04:42 | |
prometheanfire | oh, nice | 04:44 |
* tonyb has done abot 20% of the "fix lc env" work but once they're submitted we'll see that nearly all projects have broken lower-constraints | 04:50 | |
tonyb | We should document the best way to setup tox.ini | 04:51 |
tonyb | and the right way to deal with service dependancies | 04:51 |
tonyb | there is some real cruft out there | 04:52 |
prometheanfire | tonyb: heh, pgp could not be verified, I thought we cross signed at some point | 04:54 |
tonyb | prometheanfire: I don't think so | 04:55 |
tonyb | prometheanfire: I know we talked about doing after the last Vancouver summit but I don't think we ever did | 04:55 |
tonyb | prometheanfire: We should do it at the next North American summit | 04:56 |
prometheanfire | ah | 04:56 |
prometheanfire | ya | 04:56 |
prometheanfire | I always have my pgp slips for keysigning in my bag | 04:56 |
tonyb | hehe | 04:59 |
* tonyb is not that organised | 04:59 | |
* tonyb also pretty much never has a pssport on him | 05:00 | |
prometheanfire | well, I know I should check but a regular ID would work well enough, feel like I know you somehow | 05:02 |
prometheanfire | cross glance failed on the uc update :| | 05:19 |
*** mwhahaha has quit IRC | 05:30 | |
*** rm_work has quit IRC | 05:31 | |
*** dustinc has quit IRC | 05:31 | |
*** mwhahaha has joined #openstack-requirements | 05:33 | |
dirk | So wer don't run a python3 devstack on our uc changes? | 05:34 |
*** dustinc has joined #openstack-requirements | 05:34 | |
*** rm_work has joined #openstack-requirements | 05:34 | |
dirk | tonyb: that email is missing the UPPER/TOX constraints file change that I suggested to do at the same time to not have two sets of 600 patches to propose | 05:39 |
dirk | tonyb: should I be replying? | 05:40 |
tonyb | dirk: I thought that got pushback from infra so I figured it was shelved | 05:40 |
prometheanfire | USE_PYTHON3 is false | 05:40 |
prometheanfire | dirk: looks like our tempest-full is py2 | 05:41 |
dirk | tonyb: did it? There were some comments from AJ which I addressed | 05:41 |
tonyb | Yeah we probably do more with py2 than we should but that's a cycle goal for train to switch tp py3 so ... we can do that :) | 05:41 |
dirk | prometheanfire: we should add a py3 job | 05:42 |
prometheanfire | dirk: indeed | 05:42 |
prometheanfire | I still wonder why pip was pulling in py3 packages for some things | 05:43 |
tonyb | prometheanfire: why wouldn't it? | 05:43 |
tonyb | dirk: I don't see a +2 there | 05:46 |
tonyb | dirk: I'd rather not hold up the constraints stuff as we're getting more and more "bogus" changes | 05:47 |
tonyb | dirk: https://review.opendev.org/#/q/owner:zhangyuan1%2540chinaunicom.cn | 05:47 |
prometheanfire | tonyb: I just expect it to be smarter, if it's metadata says Requires: Python >=3.5, that info should be used, I guess | 05:47 |
prometheanfire | though I guess it's not used | 05:48 |
tonyb | Yeah clearly not used and pip can't use it ... until it gets a depsolver | 05:49 |
prometheanfire | yep | 05:49 |
* prometheanfire is used to gentoo too much | 05:51 | |
prometheanfire | seems like the only fix would be to fix constraints usage (or non-usage) | 05:52 |
* prometheanfire shouldn't have taken that nap, will be up all night | 05:54 | |
dirk | tonyb: well, a -1 can be put on stuff now that the email is out | 05:55 |
dirk | Let me see if I can lobby for reviews | 05:56 |
tonyb | prometheanfire: constraints get us a long way but we still need the py2 caps | 05:56 |
prometheanfire | ya, it's a two pronged thing | 05:57 |
prometheanfire | https://etherpad.openstack.org/p/reqs-py2-caps | 05:58 |
prometheanfire | that's my email (more or less) | 05:58 |
tonyb | prometheanfire: I'm not sure that adds any value :/ | 05:59 |
tonyb | prometheanfire: it doesn't discuss or mention that we weer using older wheels unil May 10 and that's why we didn't hit the sphinx thing until then | 06:00 |
prometheanfire | I didn't really think it does, it's more a small tag on to your email | 06:00 |
prometheanfire | I didn't know that | 06:00 |
prometheanfire | I don't have all the context as to what was fixed | 06:00 |
tonyb | prometheanfire: okay don't worry then | 06:01 |
prometheanfire | sorry, it's a starting point at least | 06:01 |
tonyb | I think there is still value but it's nearly 2 weeks since things went bang | 06:01 |
prometheanfire | so it was an issue with mirroring? | 06:01 |
prometheanfire | as well | 06:01 |
tonyb | AFS wouldn't release and it wasn't noticed | 06:01 |
tonyb | once that was fixed we suddenly tarted building/using new wheels | 06:02 |
tonyb | there *may* be some work to do around NOT building py2.py3 wheels for things that are py3 only but I don't understand most of that | 06:02 |
tonyb | I think we really need py2 and py3 integration job so we have a chance of detecting this next time | 06:03 |
prometheanfire | added * AFS was locked (and not noticed), causing wheel builds / mirroring issues infra side | 06:03 |
tonyb | cool | 06:03 |
tonyb | I s'psoe we can flesh it out tomorrow | 06:03 |
prometheanfire | ya, I agree (about py3 job) | 06:03 |
prometheanfire | for some reason I thought dirk already proposed the py3 tempest change | 06:04 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/660566 | 06:13 |
*** dtantsur|afk is now known as dtantsur | 06:28 | |
*** e0ne has joined #openstack-requirements | 06:29 | |
dirk | prometheanfire: we used to have a tempest-full-py3 on opensuse job but it seems it got removed from the checks | 06:48 |
dirk | no idea why | 06:48 |
openstackgerrit | Dirk Mueller proposed openstack/requirements master: Add tempest-python3 job https://review.opendev.org/660580 | 06:54 |
openstackgerrit | Dirk Mueller proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/660566 | 06:54 |
*** e0ne has quit IRC | 06:57 | |
*** tosky has joined #openstack-requirements | 07:21 | |
*** hberaud|gone is now known as hberaud | 07:44 | |
*** e0ne has joined #openstack-requirements | 08:12 | |
evrardjp | hey | 08:20 |
evrardjp | consider me john snow here ... I have a problem with latest bump of M2Crypto, for which the pip wheel doesnt' seem to build | 08:21 |
evrardjp | https://github.com/openstack/requirements/commit/d2699d3a8de1a24a2358d1dc360de2ae15d056c0 | 08:22 |
evrardjp | There are two things I want to discuss: 1) Can we make sure, before we merge things, to have a job to build requirements with pip wheel? | 08:22 |
evrardjp | 2) Where does this M2Crypto bump come from? | 08:23 |
*** ccamacho has joined #openstack-requirements | 08:23 | |
evrardjp | I can understand that 3 days ago docker bump was triggered, and from that, a few things were bumped, but none of those have M2Crypto in their dependency chain. | 08:24 |
evrardjp | Oh I guess it comes from pywbem on the same day | 08:26 |
evrardjp | let me dig deeper... (question 1 still stands) | 08:26 |
evrardjp | LOCI has a build wheel job if you like | 08:33 |
evrardjp | but this can also be a one liner thing in some new job | 08:33 |
tonyb[m] | evrardjp: I'll look tomorrow at the wheel thing but we shouldn't be able to land anything that doesn't fundamentally exist and install | 09:28 |
tonyb[m] | evrardjp: in terms of where bit came from it may be as simple at it was published to pypi so we added it | 09:29 |
tonyb[m] | evrardjp: can you explain the error you're seeing in a little more detail? | 09:29 |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.log to new release 3.44.0 https://review.opendev.org/660469 | 09:31 |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.rootwrap to new release 5.16.0 https://review.opendev.org/660468 | 09:31 |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.cache to new release 1.35.0 https://review.opendev.org/660467 | 09:31 |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.service to new release 1.39.0 https://review.opendev.org/660466 | 09:37 |
openstackgerrit | Merged openstack/requirements master: update constraint for taskflow to new release 3.6.0 https://review.opendev.org/660465 | 09:37 |
evrardjp | evrardjp: sure thing, we have two issues in loci right now. | 09:37 |
evrardjp | The first one will be solved here: https://review.opendev.org/#/c/660254 -- basically warlock fails to `pip wheel` in 1.3.1. That code was just a broken thing, which was fixed in 1.3.3, so let's skip that for now. (However if we had a test to ensure wheels can be built, then that would have been fine, as 1.3.1 would not have been bumped, and issue would have been fixed earlier upstream) | 09:38 |
evrardjp | the second is more complex. By adding pywbem https://github.com/openstack/requirements/commit/1970ac7a9234fa095d82d432fda4857f565de545 we basically pulled its requirements https://github.com/openstack/requirements/commit/d2699d3a8de1a24a2358d1dc360de2ae15d056c0 | 09:40 |
evrardjp | that's fine, except that M2Crypto fails to build its wheels under centos/ubuntu in LOCI, because we are using the distro provided openssl-devel libraries | 09:40 |
evrardjp | (first log here, let me create a cleaner log for you: http://logs.openstack.org/93/660493/1/check/loci-requirements/e217427/job-output.txt.gz#_2019-05-21_19_08_34_839694 ) | 09:42 |
openstackgerrit | Merged openstack/requirements master: update constraint for metalsmith to new release 0.12.0 https://review.opendev.org/660470 | 09:52 |
openstackgerrit | Merged openstack/requirements master: update constraint for ironic-lib to new release 2.17.0 https://review.opendev.org/660476 | 09:52 |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.messaging to new release 9.7.0 https://review.opendev.org/660475 | 09:52 |
evrardjp | https://gist.github.com/evrardjp/1b6b52aef5b68b04abe09ce080a31d48 | 09:59 |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.db to new release 5.0.0 https://review.opendev.org/660480 | 10:05 |
openstackgerrit | Merged openstack/requirements master: Updated from generate-constraints https://review.opendev.org/660254 | 10:05 |
*** hberaud is now known as hberaud|lunch | 10:13 | |
openstackgerrit | Matthew Thode proposed openstack/requirements master: Add tempest-python3 job https://review.opendev.org/660580 | 10:24 |
openstackgerrit | Matthew Thode proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/660566 | 10:24 |
*** toabctl has joined #openstack-requirements | 10:58 | |
*** hberaud|lunch is now known as hberaud | 11:10 | |
*** udesale has quit IRC | 11:30 | |
*** udesale has joined #openstack-requirements | 11:31 | |
openstackgerrit | Dirk Mueller proposed openstack/requirements master: Add tempest-python3 job https://review.opendev.org/660580 | 12:10 |
*** zul has joined #openstack-requirements | 13:01 | |
*** dtantsur is now known as dtantsur|brb | 13:29 | |
*** dtantsur|brb is now known as dtantsur | 14:29 | |
*** udesale has quit IRC | 14:44 | |
*** udesale has joined #openstack-requirements | 14:44 | |
*** tosky has quit IRC | 14:45 | |
*** tosky has joined #openstack-requirements | 14:46 | |
*** udesale has quit IRC | 16:38 | |
*** udesale has joined #openstack-requirements | 16:43 | |
*** e0ne has quit IRC | 16:51 | |
*** udesale has quit IRC | 17:12 | |
openstackgerrit | Dirk Mueller proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/660566 | 17:32 |
*** samueldmq has joined #openstack-requirements | 17:51 | |
*** hberaud is now known as hberaud|gone | 17:58 | |
*** tosky has quit IRC | 18:30 | |
*** tosky has joined #openstack-requirements | 18:30 | |
openstackgerrit | Douglas Mendizábal proposed openstack/requirements master: Stop Requirements Sync for castellan-ui https://review.opendev.org/660807 | 19:13 |
*** e0ne has joined #openstack-requirements | 19:15 | |
tonyb[m] | evrardjp: thanks I understand now. | 20:30 |
tonyb[m] | prometheanfire: I'll start the meeting from my phone until I can make coffee | 20:30 |
prometheanfire | tonyb[m]: kk | 20:31 |
prometheanfire | #startmeeting requirements | 20:31 |
openstack | Meeting started Wed May 22 20:31:35 2019 UTC and is due to finish in 60 minutes. The chair is prometheanfire. Information about MeetBot at http://wiki.debian.org/MeetBot. | 20:31 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 20:31 |
*** openstack changes topic to " (Meeting topic: requirements)" | 20:31 | |
openstack | The meeting name has been set to 'requirements' | 20:31 |
tonyb[m] | O/ | 20:31 |
prometheanfire | #topic rollcall | 20:31 |
*** openstack changes topic to "rollcall (Meeting topic: requirements)" | 20:31 | |
prometheanfire | tonyb, prometheanfire, number80, dirk, coolsvap, toabctl, smcginnis, dhellmann ping | 20:32 |
prometheanfire | o/ | 20:32 |
tonyb[m] | smcginnis: is in Barcelona ATM so probably won't be here | 20:32 |
tonyb[m] | ... as is dhellmann | 20:32 |
prometheanfire | yep | 20:32 |
prometheanfire | #topic controversies in the queue | 20:34 |
*** openstack changes topic to "controversies in the queue (Meeting topic: requirements)" | 20:34 | |
*** e0ne has quit IRC | 20:34 | |
prometheanfire | I think we are generally fine here | 20:35 |
tonyb | Yeah | 20:36 |
prometheanfire | #topic email the list for issues we had a couple of weeks ago | 20:36 |
*** openstack changes topic to "email the list for issues we had a couple of weeks ago (Meeting topic: requirements)" | 20:36 | |
prometheanfire | #link https://etherpad.openstack.org/p/reqs-py2-caps | 20:36 |
tonyb | I'll make time to add my thoughtts today. | 20:38 |
prometheanfire | ok, thanks | 20:38 |
prometheanfire | #topic open floor | 20:38 |
*** openstack changes topic to "open floor (Meeting topic: requirements)" | 20:38 | |
* tonyb has 2 things | 20:39 | |
tonyb | 1) ... following on from the courtesy pings discussion on the m/l | 20:39 |
tonyb | I think we should clean out our list | 20:39 |
*** e0ne has joined #openstack-requirements | 20:40 | |
prometheanfire | our list? | 20:40 |
tonyb | I suggest we update the wiki with "ping list started Jun15" and opt in again | 20:40 |
tonyb | then from Jun 15 we stop pinging the existing list | 20:40 |
prometheanfire | oh, for roll call, ya | 20:40 |
tonyb | as I don't see number80, tobctl or coolsvap at all | 20:40 |
prometheanfire | :D | 20:41 |
tonyb | Shall I do a picard ... and make it so? | 20:41 |
*** e0ne has quit IRC | 20:41 | |
prometheanfire | please do | 20:42 |
prometheanfire | the other item? | 20:42 |
tonyb | sorry multitasking | 20:43 |
prometheanfire | np | 20:43 |
tonyb | evrardjp: brought up that our 'check-uc' jobs install things but don't build wheels | 20:43 |
smcginnis | tonyb[m]: Actually not in Barcelona. Plans were changed for me. | 20:43 |
tonyb | (which I need to investigate but assuming that's right) | 20:43 |
tonyb | smcginnis: Oh dear! | 20:43 |
smcginnis | Yeah, it's been an interesting few days. | 20:44 |
tonyb | so do we feel about adding a LOCI job or similar to build the wheels in our check and gate pipelines? | 20:44 |
dirk | o/ | 20:44 |
prometheanfire | tonyb: is it our job to build/ship wheels? | 20:45 |
prometheanfire | I understand we may want to check if they are available | 20:45 |
tonyb | prometheanfire: no but it is our job to ensure things work and are installable | 20:45 |
prometheanfire | right, adding a job for checking that makes sense | 20:45 |
tonyb | IMO adding a job like this will have our consumers pain so it's something we shoudl consider | 20:46 |
dirk | so the issue was that osh got a constraints update where the wheels was not buildable? | 20:46 |
dirk | on some distribution? | 20:46 |
dirk | or on any distribution? I guess I'm missing some detail | 20:46 |
tonyb | dirk: Yeah, it looks like perhaps only centos was/is broken | 20:46 |
tonyb | ... evrardjp says: M2Crypto fails to build its wheels under centos/ubuntu in LOCI | 20:47 |
prometheanfire | tonyb: so would we be building wheels (I assume so) and if so would we be shipping them (that's releases / infra's job right?) | 20:48 |
tonyb | so it's worth looking at why (and he provides a log) and if it's LOCI specific | 20:48 |
prometheanfire | if we ship them how do we not become the mirror everyone uses | 20:48 |
tonyb | prometheanfire: we'd just build them and discard them | 20:48 |
tonyb | prometheanfire: infra will do the building for our internal mirrors | 20:48 |
smcginnis | Seems a little outside our scope. | 20:48 |
dirk | I guess I'm not following why we'd care about that | 20:49 |
dirk | do we want to avoid that the infra build-wheels job gets broken? | 20:49 |
prometheanfire | tonyb: ok, build only makes more sense | 20:49 |
tonyb | smcginnis: I see it as a completion of what check-uc* does | 20:49 |
prometheanfire | ya, co-installability extension | 20:50 |
tonyb | smcginnis: to be completely open I *thought* that check-uc did build wheels as part of the install process | 20:50 |
prometheanfire | I think it's worth doing, possibly in the same job as check-uc (since it has time and it's an extension) | 20:52 |
dirk | hmm, I thought I've seen bindep related issues there as well in the past | 20:52 |
tonyb | So it sounds like if I do some research and then write up the plan we can discuss it again noone is *hard* against it | 20:53 |
prometheanfire | sgtm | 20:53 |
prometheanfire | tonyb: so, three action items then | 20:53 |
prometheanfire | etherpad, pings and wheels | 20:54 |
tonyb | it could be a corner case where what we do in check-uc is fine but slightly diffeent to what LOCI needs/does in which case it might just be we add them as a cross job like nova or keystone | 20:54 |
prometheanfire | yep | 20:55 |
tonyb | huzzah | 20:55 |
prometheanfire | anyone have anything else? | 20:55 |
dirk | yeah | 20:55 |
dirk | I've seen the requests CVE bump abandoned, what did I miss again? | 20:55 |
prometheanfire | I didn't abandon it iirc | 20:56 |
prometheanfire | you have links to the reviews? abandon reason should be there | 20:57 |
dirk | https://review.opendev.org/#/c/640727/ | 20:57 |
prometheanfire | ugh | 20:57 |
prometheanfire | gertty was being stupid | 20:57 |
dirk | I am not sure I understand what "python is dumb" means | 20:57 |
prometheanfire | I was trying to abandon my 'test python intel' patch | 20:58 |
prometheanfire | no clue why it selected those | 20:58 |
dirk | I guess mistakes happen | 20:58 |
prometheanfire | ya | 20:58 |
dirk | can you unabandon it? I forgot to follow the mailthread, I'll do so | 20:58 |
prometheanfire | ya | 20:59 |
prometheanfire | just did | 20:59 |
dirk | there was some other ping that you sent me a week ago or so, but I wasn't able to reread it as I lost the irc history (was migrating bouncer implementation) | 20:59 |
dirk | what was the action item for me? | 20:59 |
tonyb | https://review.opendev.org/#/c/640710/ ? | 20:59 |
prometheanfire | so, I'm not sure we are any further for being able to do security patches | 20:59 |
prometheanfire | that one too, thanks | 21:00 |
prometheanfire | ok, all restored now (there were a couple of others) | 21:01 |
tonyb | okay | 21:01 |
tonyb | looks like https://review.opendev.org/#/q/topic:tox_constraints+is:open is moving along | 21:02 |
prometheanfire | anything else? | 21:02 |
tonyb | so I'll include that in the updates I do | 21:02 |
prometheanfire | thanks | 21:03 |
tonyb | dirk: do you mind if I add some Depends-On to you changes so we can things up safely? | 21:03 |
dirk | tonyb: the comment about django upper-constraints.. I think horizon installs unconstraint, so I guess changing upper-constraints is mostly a documentation exercise | 21:04 |
dirk | I'll add a link to the review, I wanted to double check things and maybe see if a fix can be done first | 21:05 |
tonyb | dirk: okay | 21:05 |
dirk | tonyb: you mean the tox_constraings part? yes, sure, go ahead | 21:05 |
dirk | tonyb: I wanted to lobby on -infra for a 2nd core review, but it seems people are distracted with some infra issue | 21:05 |
dirk | or rather, I didn't want to distract them from fixing the outage | 21:06 |
tonyb | dirk: yeah basially make sure we can't land things in a breaking order | 21:06 |
tonyb | dirk: cool beans | 21:06 |
tonyb | dirk: I can +1 the release update | 21:06 |
prometheanfire | ok, gonna end it here | 21:08 |
tonyb | +1 | 21:08 |
prometheanfire | #endmeeting | 21:09 |
*** openstack changes topic to "OpenStack Requirements - IRC meetngs on Wednesdays @ 07:00 UTC in here in #openstack-requirements - See agenda @ http://tinyurl.com/h44ryuw - IRC channel is *LOGGED* @ http://tinyurl.com/j38rk24" | 21:09 | |
openstack | Meeting ended Wed May 22 21:09:25 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:09 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/requirements/2019/requirements.2019-05-22-20.31.html | 21:09 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/requirements/2019/requirements.2019-05-22-20.31.txt | 21:09 |
openstack | Log: http://eavesdrop.openstack.org/meetings/requirements/2019/requirements.2019-05-22-20.31.log.html | 21:09 |
dirk | tonyb: I would hope the TOX_CONSTRAINTS rename is finished relatively quickly, and then we can include that cleanup in one go, that was my main concern | 21:17 |
tonyb[m] | Yeah it they merge I can do it in a single change if not we can do it at the same time | 21:37 |
tonyb[m] | We need to look at branches and zuul jobs | 21:37 |
prometheanfire | dirk: nice replies :D | 22:16 |
*** hberaud|gone has quit IRC | 22:36 | |
*** tosky has quit IRC | 23:11 | |
*** tosky has joined #openstack-requirements | 23:11 | |
prometheanfire | dirk: I'll review them tonight hopefully (D&D may interfere) | 23:29 |
*** tosky has quit IRC | 23:34 | |
openstackgerrit | Dirk Mueller proposed openstack/requirements stable/ocata: Cap keyring and scikit-learn on python2 https://review.opendev.org/660860 | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!