Wednesday, 2018-09-05

prometheanfiretonyb: divergant?00:26
prometheanfireor is it ent00:26
tonyb*ent*00:26
prometheanfiredivergant it is00:29
*** hongbin_ has joined #openstack-requirements01:26
*** lbragstad has quit IRC02:39
*** hongbin_ has quit IRC03:46
openstackgerritTony Breeds proposed openstack/requirements master: Remove networking-odl from global-requirements  https://review.openstack.org/59927703:51
*** udesale has joined #openstack-requirements03:54
openstackgerritTony Breeds proposed openstack/requirements master: Remove networking-odl from global-requirements  https://review.openstack.org/59927704:35
*** adriant has quit IRC04:40
*** tonyb has quit IRC04:40
*** tonyb has joined #openstack-requirements04:40
*** adriant has joined #openstack-requirements04:41
*** shachar is now known as snapiri05:16
tonybOh cool.  I've found another problem with our integration job :/  Clearly it isn't urgent05:37
prometheanfiretonyb: not really surprising at this point unfortunately05:40
prometheanfiremeeting in 1:20?05:40
tonybprometheanfire: perhaps I'll see if I can move things around05:43
tonybhttp://eavesdrop.openstack.org/#Requirements_Team_Meeting still says UTC 2030 which is 0630 Thursday mornign for me05:43
* tonyb suspects the topic got reset when the bot restarted / reconnected05:44
prometheanfireah, that may have been it05:56
* tonyb will try to be there06:02
tonybIt looks like my hack to unblock our gate is technically correct but it's terrible06:03
prometheanfireit really us06:13
prometheanfires/us/is06:13
prometheanfireI feel like we need to be able to remove projects that do stupid things that break gate06:14
tonybIt's a complex situation.  They certainly didn't do it deliberately and we kinda did it to ourselves with the move to tox-siblings and everything being on pypi06:16
tonybit took a team of peopel to break it so it'll take a team to put it back together :/06:16
openstackgerritTony Breeds proposed openstack/requirements master: Remove networking-odl from g-r in the requirements-integration job  https://review.openstack.org/59927706:19
tonybprometheanfire: ^^^ that one for real ;P06:19
prometheanfireyep06:29
prometheanfiretonyb: I'm still more in favor of removing networking-odl from reqs, you do a bad thing, you get the consequences06:30
prometheanfirenot that they weren't forced or something, rolling back their add to the gate seems better (with a review submitted to readd them)06:31
prometheanfireif we go with your way I'd like a revert review up so we don't forget as well06:31
prometheanfirereview/bug06:31
tonybprometheanfire: Sure as soon as it merges I'll click revert in gerrit06:32
tonybprometheanfire: I don't think we *can* remove them them global-requirements without causing a cascde of failures06:32
tonybprometheanfire: that fact is "they" didn't do a bad thing, "we" did and this is the fallout06:33
tonybprometheanfire: We just need to come to grips with the way the world works now and given them a tool to do what they need06:34
prometheanfirethat's half true I think06:37
prometheanfireI don't think pointing to a git revision in reqs.txt should be allowed06:37
prometheanfirethere are mechanisms to gate against another project's head06:38
prometheanfirethat's my main gripe, that shouldn't have happened06:38
prometheanfire| networking-bgpvpn | test-requirements.txt   |   27 | networking-odl>=12.0.0,<13 # Apache-2.0                         |06:38
prometheanfirebgpvpn is about to remove networking-odl from their test-reqs06:39
prometheanfireso we COULD remove them from global reqs with no issue to other projects (bgpvpn is the only project I found that uses odl)06:39
tonybprometheanfire: Well if networking-bgpvpn removes them then yes we can drop them from g-r06:43
tonybprometheanfire: they're only pointing at the git URL because ceilometer doesn't publich to pypi06:44
tonyb(and we took away the tox_install.sh)06:44
tonybwhich we could but back06:44
prometheanfireya06:45
tonybwe should also check that networking-odl has the check-requirements job enabled but that's later06:45
prometheanfirejust a shit situation06:45
prometheanfireya, that's a good point06:45
prometheanfireI see it in their templates06:46
tonybYeah and it ran on the change that removed tox_install.sh06:48
tonybso if we don't allow urls we should fix that tool ;P06:48
prometheanfireheh06:50
prometheanfireso many edge cases06:51
prometheanfirehuh, icinga2 doesn't dep on yajl06:51
dirkMeeting?07:02
tonybdirk: Not sure.07:03
tonybprometheanfire: ?07:03
prometheanfireI'm up if you are07:08
prometheanfirenothing to do but look at tony's review and ponder our place in the world07:09
* tonyb moved stuff so we could do this07:10
prometheanfirewell07:11
prometheanfire#startmeeting requirements07:11
openstackMeeting started Wed Sep  5 07:11:15 2018 UTC and is due to finish in 60 minutes.  The chair is prometheanfire. Information about MeetBot at http://wiki.debian.org/MeetBot.07:11
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.07:11
*** openstack changes topic to " (Meeting topic: requirements)"07:11
openstackThe meeting name has been set to 'requirements'07:11
prometheanfire#topic rollcall07:11
*** openstack changes topic to "rollcall (Meeting topic: requirements)"07:11
prometheanfiretonyb, prometheanfire, number80, dirk, coolsvap, toabctl, smcginnis, dhellmann07:11
prometheanfireing07:11
prometheanfireo/07:11
tonyb\o07:11
dirko/07:12
prometheanfiregood nuf07:13
prometheanfire#topic bad things07:13
*** openstack changes topic to "bad things (Meeting topic: requirements)"07:13
prometheanfiretonyb: want to lead? I think your commit message summed it up nicely07:14
tonybhttps://review.openstack.org/59927707:14
tonybIt does bad things and removes networking-odl from g-r for the integration job07:15
tonybwe'er still horribly broken (for example propose-updates) isn't working07:15
tonybfor the same reason07:15
dirkSigh07:15
prometheanfireyep07:15
prometheanfirethat07:15
tonybI don't like that work around but fixing it correctly is a cross-project thing07:16
dirkWho needs opendaylight (warning sarcasm included)07:16
prometheanfiredirk: lol07:16
prometheanfiredirk: I did suggest removing it til they clean their own house07:16
dirkCan we drop ceilometerr from idl?07:16
dirkOdl07:16
tonybprometheanfire: said that networking-bgp is the only consumer so if they dropped networking-odl we could also and get things back on track07:16
tonybdirk: I don't think so07:16
dirkWhy?07:17
tonybwe could publisha straw-person revert but I doubt it'll land07:17
tonybdirk: the code's been in there for ages07:17
tonybit isn't like its a recnet feature07:17
tonybI'm pretty sure $comsumers are relying on the agent existing07:17
prometheanfireI'm curious how they added it at all07:17
dirkSure, ceilometer itself isn't the new kid on the block07:18
prometheanfiredon't entries in reqs.txt need to be in global-reqs?07:18
tonybprometheanfire: they added it ages ago and used the well established tox_install from neutron07:18
dirktonyb: 'pretty sure' sounds worth investigating imho07:18
prometheanfireya, right07:18
tonybdirk: it seems not for URL based requirements07:19
tonybdirk: we may have a bug in our tools there but that doesn't help move things forward07:19
dirktonyb: sure, as a stop gap we can get things going for now07:20
prometheanfirewe should have a bug for the tool not working as expect at the very least07:20
dirkAny attempt at a more complete fix will eat more time07:20
tonybdirk: right that's why I've faffed with the hack above07:20
prometheanfirehow about this order of operations07:22
prometheanfire1. do the hack07:22
prometheanfire2. fix reqs to reject url based reqs07:22
prometheanfire3. make sure odl is fixed07:22
tonyb(if it's a bug)07:22
prometheanfire4. undo the hack07:22
prometheanfiretonyb: sure07:23
tonybprometheanfire: Yup that's pretty much what we have to do07:23
tonyball of which will be fun during the PTG07:23
*** olivierb has joined #openstack-requirements07:23
tonybprometheanfire: but you can try visiting the neutron room next week07:24
prometheanfireI get to play the pixie from zelda07:24
tonyb;p07:24
prometheanfirenot much diferent than I just got done doing with keystone and nova :P07:24
prometheanfiretonyb: mind putting this into a bug just so we don't forget? I'm bedding after this07:25
tonybprometheanfire: Sure but it'll be tomorrow07:25
prometheanfiretonyb: eh, ok07:26
prometheanfire#topic open floor07:27
*** openstack changes topic to "open floor (Meeting topic: requirements)"07:27
prometheanfireptg is next week, I'm not sure we'll be able to do anything more than inpromptu meetings07:27
tonybYeah it'll be hard to get the 3 TZ awake at the same time07:27
prometheanfire#link https://etherpad.openstack.org/p/stein-PTG-requirements07:29
prometheanfiredougs thing is the main complex one07:29
prometheanfirebeyond that, constraints publishing would be nice07:29
tonybYup I'll ping the infra team PTG is over07:30
prometheanfireanyone else have anything else before ending?07:31
tonybnot for me07:31
prometheanfirek07:32
prometheanfire#endmeeting07:33
*** 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"07:33
openstackMeeting ended Wed Sep  5 07:33:02 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)07:33
openstackMinutes:        http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-07.11.html07:33
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-07.11.txt07:33
openstackLog:            http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-07.11.log.html07:33
*** ccamacho has joined #openstack-requirements07:51
*** jpich has joined #openstack-requirements08:08
*** dtantsur|afk is now known as dtantsur08:35
*** zigo has joined #openstack-requirements10:03
*** udesale has quit IRC10:21
*** udesale has joined #openstack-requirements10:21
*** udesale has quit IRC10:21
*** udesale has joined #openstack-requirements10:22
*** dtantsur is now known as dtantsur|brb11:08
*** udesale has quit IRC11:10
*** zigo has quit IRC11:45
*** odyssey4me has quit IRC11:53
*** odyssey4me has joined #openstack-requirements11:55
*** lbragstad has joined #openstack-requirements11:57
*** udesale has joined #openstack-requirements12:57
openstackgerritEric Fried proposed openstack/requirements master: oslo.utils 3.37.0 upper constraint  https://review.openstack.org/60003413:04
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.utils to new release 3.37.0  https://review.openstack.org/60004113:16
*** dtantsur|brb is now known as dtantsur13:17
*** udesale has quit IRC13:43
*** dtantsur is now known as dtantsur|brb14:24
lbragstadprometheanfire around for a quick requirements question?14:26
lbragstadthis is on master https://review.openstack.org/#/c/599003/ which fails due to oslo.messaging requirements checking14:27
lbragstadPackage 'oslo.messaging' is constrained to 5.29.0 which does not match the minimum version specifier 8.1.0 in test-requirements.txt14:27
lbragstadi checked openstack/requirements and the upper constraint is 8.1.0 on master14:28
lbragstaddo you know how 5.29.0 is getting involved when we dealing with only master?14:28
lbragstadwe're*14:29
lbragstadoh - i bet https://review.openstack.org/#/c/599003/1/tox.ini@87 is causing issues14:34
prometheanfirelbragstad: hi14:43
lbragstado/14:52
lbragstadi might have answered my own question14:52
lbragstadvia rubber duck debugging14:52
prometheanfire:D14:57
prometheanfirelbragstad: I commented as well14:57
lbragstadupdated the patch again, not sure what i did with ps2 made sense15:03
prometheanfirelbragstad: do you know why you need the latest oslo.messaging?15:06
prometheanfireiirc there's not been much change15:07
lbragstadapparently we need oslo.messaging for our tests, but i doubt we have a requirement for the latest15:07
prometheanfireI'd just have 5.29.0 (or whatever it was) in test-requirements instead15:08
lbragstadah - good point15:12
lbragstadok - i think i'm done mucking with it15:13
* lbragstad convinces himself https://review.openstack.org/#/c/599003/4 looks right until Zuul says differently15:13
dhellmannprometheanfire , tonyb , dirk : one of the changes we're making with the python3 version of the publishing job is that all packages will be pushed to pypi. So that  may help with the networking-odl / ceilometer issue, although it's going to be a while before that goes into effect15:39
prometheanfiredhellmann: one step at a time15:41
dhellmannyeah, we probably want to take action now, I'm just pointing out that things may get easier later, too15:41
prometheanfiredhellmann: you may want to go read the backlog as well, had one meeting at 2AM15:42
prometheanfiredhellmann: will have another in a couple hours, but don't expect much to be said at it15:42
dhellmannyeah, I was responding to the meeting scrollback15:42
prometheanfire:D15:43
*** dtantsur|brb is now known as dtantsur16:01
*** jpich has quit IRC16:30
*** olivierb has quit IRC16:44
openstackgerritMerged openstack/requirements master: Remove networking-odl from g-r in the requirements-integration job  https://review.openstack.org/59927716:51
openstackgerritMatthew Thode proposed openstack/requirements master: Revert "Remove networking-odl from g-r in the requirements-integration job"  https://review.openstack.org/60012316:53
*** dtantsur is now known as dtantsur|afk17:26
*** olivierb has joined #openstack-requirements17:32
*** olivierb has quit IRC17:38
*** ccamacho has quit IRC17:44
*** openstackgerrit has quit IRC18:00
lbragstadprometheanfire following up on https://review.openstack.org/#/c/599003/4 again19:44
lbragstadwe define audit_notifications in setup.py as an extra package19:44
lbragstadsetup.cfg*19:44
*** openstackgerrit has joined #openstack-requirements19:48
openstackgerritOpenStack Proposal Bot proposed openstack/requirements stable/rocky: update constraint for python-openstackclient to new release 3.16.1  https://review.openstack.org/60019619:48
prometheanfirelbragstad: was wondering about that20:12
lbragstadi added it back in to see if it fixes the problem20:15
openstackgerritOpenStack Proposal Bot proposed openstack/requirements stable/rocky: update constraint for python-neutronclient to new release 6.9.1  https://review.openstack.org/60020020:15
*** efried has joined #openstack-requirements20:18
efriedGreetings y'all. There's some kind of tool that lets me figure out what my requirements.txt should actually contain based on stuff I import from my project, right?20:19
prometheanfireefried: which project?20:21
efriedprometheanfire: openstack/placement is currently using nova's *requirements.txt, which is obviously waaay overkill.20:21
efriedI want to trim it down to only what's needed20:21
efriedand I thought there was a tool that would figure that out for me20:21
efriedyou know, something better than20:22
efriedpip uninstall20:22
efriedtox || pip install20:22
efriedrepeat20:22
prometheanfirejust find out what you import20:24
prometheanfiregrep -R import20:25
prometheanfireefried: you coppied the nova git repo to be placement's?20:26
efriedyeah, sort of20:27
efriedI mean, *I* didn't20:27
efriedthere was some kind of script that seeded the repo with a subset of files while preserving their git history20:27
prometheanfireefried: https://paste.pound-python.org/show/eosJiPPXxjUNwPKI67om/20:28
efriedand then about 50 patches later to rename files and fix import paths and manually remove leftovers that the script didn't get, we have the thing gating.20:28
prometheanfirewell, that's my 'grep -Rh import | sort -u'20:28
efriedprometheanfire: Just to confirm: [test-]requirements.txt doesn't need transitive deps, but lower-constraints.txt does20:29
prometheanfireyes20:29
efriedcool20:29
efriedThanks prometheanfire20:29
prometheanfire#startmeeting requirements20:30
openstackMeeting started Wed Sep  5 20:30:18 2018 UTC and is due to finish in 60 minutes.  The chair is prometheanfire. Information about MeetBot at http://wiki.debian.org/MeetBot.20:30
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.20:30
*** openstack changes topic to " (Meeting topic: requirements)"20:30
openstackThe meeting name has been set to 'requirements'20:30
prometheanfire#topic rollcall20:30
*** openstack changes topic to "rollcall (Meeting topic: requirements)"20:30
prometheanfireping tonyb, prometheanfire, number80, dirk, coolsvap, toabctl, smcginnis, dhellmann20:30
prometheanfireo/20:30
prometheanfirewe had a meeting 13.5 hours ago, so this one probably isn't needed20:32
prometheanfiremore of a just in case thing20:32
*** efried has left #openstack-requirements20:34
smcginnisprometheanfire: Saw the ping earlier but didn't get a chance to read all the scrollback.20:38
smcginnisprometheanfire: Anything from earlier of concern?20:38
prometheanfiresmcginnis: nope20:39
smcginnisGreat20:39
prometheanfiresmcginnis: just https://review.openstack.org/#/c/599277/20:40
prometheanfire#endmeeting20:40
*** 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"20:40
openstackMeeting ended Wed Sep  5 20:40:16 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)20:40
openstackMinutes:        http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-20.30.html20:40
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-20.30.txt20:40
openstackLog:            http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-20.30.log.html20:40
smcginnisOh right20:40

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!