prometheanfire | tonyb: divergant? | 00:26 |
---|---|---|
prometheanfire | or is it ent | 00:26 |
tonyb | *ent* | 00:26 |
prometheanfire | divergant it is | 00:29 |
*** hongbin_ has joined #openstack-requirements | 01:26 | |
*** lbragstad has quit IRC | 02:39 | |
*** hongbin_ has quit IRC | 03:46 | |
openstackgerrit | Tony Breeds proposed openstack/requirements master: Remove networking-odl from global-requirements https://review.openstack.org/599277 | 03:51 |
*** udesale has joined #openstack-requirements | 03:54 | |
openstackgerrit | Tony Breeds proposed openstack/requirements master: Remove networking-odl from global-requirements https://review.openstack.org/599277 | 04:35 |
*** adriant has quit IRC | 04:40 | |
*** tonyb has quit IRC | 04:40 | |
*** tonyb has joined #openstack-requirements | 04:40 | |
*** adriant has joined #openstack-requirements | 04:41 | |
*** shachar is now known as snapiri | 05:16 | |
tonyb | Oh cool. I've found another problem with our integration job :/ Clearly it isn't urgent | 05:37 |
prometheanfire | tonyb: not really surprising at this point unfortunately | 05:40 |
prometheanfire | meeting in 1:20? | 05:40 |
tonyb | prometheanfire: perhaps I'll see if I can move things around | 05:43 |
tonyb | http://eavesdrop.openstack.org/#Requirements_Team_Meeting still says UTC 2030 which is 0630 Thursday mornign for me | 05:43 |
* tonyb suspects the topic got reset when the bot restarted / reconnected | 05:44 | |
prometheanfire | ah, that may have been it | 05:56 |
* tonyb will try to be there | 06:02 | |
tonyb | It looks like my hack to unblock our gate is technically correct but it's terrible | 06:03 |
prometheanfire | it really us | 06:13 |
prometheanfire | s/us/is | 06:13 |
prometheanfire | I feel like we need to be able to remove projects that do stupid things that break gate | 06:14 |
tonyb | It'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 pypi | 06:16 |
tonyb | it took a team of peopel to break it so it'll take a team to put it back together :/ | 06:16 |
openstackgerrit | Tony Breeds proposed openstack/requirements master: Remove networking-odl from g-r in the requirements-integration job https://review.openstack.org/599277 | 06:19 |
tonyb | prometheanfire: ^^^ that one for real ;P | 06:19 |
prometheanfire | yep | 06:29 |
prometheanfire | tonyb: I'm still more in favor of removing networking-odl from reqs, you do a bad thing, you get the consequences | 06:30 |
prometheanfire | not 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 |
prometheanfire | if we go with your way I'd like a revert review up so we don't forget as well | 06:31 |
prometheanfire | review/bug | 06:31 |
tonyb | prometheanfire: Sure as soon as it merges I'll click revert in gerrit | 06:32 |
tonyb | prometheanfire: I don't think we *can* remove them them global-requirements without causing a cascde of failures | 06:32 |
tonyb | prometheanfire: that fact is "they" didn't do a bad thing, "we" did and this is the fallout | 06:33 |
tonyb | prometheanfire: We just need to come to grips with the way the world works now and given them a tool to do what they need | 06:34 |
prometheanfire | that's half true I think | 06:37 |
prometheanfire | I don't think pointing to a git revision in reqs.txt should be allowed | 06:37 |
prometheanfire | there are mechanisms to gate against another project's head | 06:38 |
prometheanfire | that's my main gripe, that shouldn't have happened | 06:38 |
prometheanfire | | networking-bgpvpn | test-requirements.txt | 27 | networking-odl>=12.0.0,<13 # Apache-2.0 | | 06:38 |
prometheanfire | bgpvpn is about to remove networking-odl from their test-reqs | 06:39 |
prometheanfire | so 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 |
tonyb | prometheanfire: Well if networking-bgpvpn removes them then yes we can drop them from g-r | 06:43 |
tonyb | prometheanfire: they're only pointing at the git URL because ceilometer doesn't publich to pypi | 06:44 |
tonyb | (and we took away the tox_install.sh) | 06:44 |
tonyb | which we could but back | 06:44 |
prometheanfire | ya | 06:45 |
tonyb | we should also check that networking-odl has the check-requirements job enabled but that's later | 06:45 |
prometheanfire | just a shit situation | 06:45 |
prometheanfire | ya, that's a good point | 06:45 |
prometheanfire | I see it in their templates | 06:46 |
tonyb | Yeah and it ran on the change that removed tox_install.sh | 06:48 |
tonyb | so if we don't allow urls we should fix that tool ;P | 06:48 |
prometheanfire | heh | 06:50 |
prometheanfire | so many edge cases | 06:51 |
prometheanfire | huh, icinga2 doesn't dep on yajl | 06:51 |
dirk | Meeting? | 07:02 |
tonyb | dirk: Not sure. | 07:03 |
tonyb | prometheanfire: ? | 07:03 |
prometheanfire | I'm up if you are | 07:08 |
prometheanfire | nothing to do but look at tony's review and ponder our place in the world | 07:09 |
* tonyb moved stuff so we could do this | 07:10 | |
prometheanfire | well | 07:11 |
prometheanfire | #startmeeting requirements | 07:11 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 07:11 |
*** openstack changes topic to " (Meeting topic: requirements)" | 07:11 | |
openstack | The meeting name has been set to 'requirements' | 07:11 |
prometheanfire | #topic rollcall | 07:11 |
*** openstack changes topic to "rollcall (Meeting topic: requirements)" | 07:11 | |
prometheanfire | tonyb, prometheanfire, number80, dirk, coolsvap, toabctl, smcginnis, dhellmann | 07:11 |
prometheanfire | ing | 07:11 |
prometheanfire | o/ | 07:11 |
tonyb | \o | 07:11 |
dirk | o/ | 07:12 |
prometheanfire | good nuf | 07:13 |
prometheanfire | #topic bad things | 07:13 |
*** openstack changes topic to "bad things (Meeting topic: requirements)" | 07:13 | |
prometheanfire | tonyb: want to lead? I think your commit message summed it up nicely | 07:14 |
tonyb | https://review.openstack.org/599277 | 07:14 |
tonyb | It does bad things and removes networking-odl from g-r for the integration job | 07:15 |
tonyb | we'er still horribly broken (for example propose-updates) isn't working | 07:15 |
tonyb | for the same reason | 07:15 |
dirk | Sigh | 07:15 |
prometheanfire | yep | 07:15 |
prometheanfire | that | 07:15 |
tonyb | I don't like that work around but fixing it correctly is a cross-project thing | 07:16 |
dirk | Who needs opendaylight (warning sarcasm included) | 07:16 |
prometheanfire | dirk: lol | 07:16 |
prometheanfire | dirk: I did suggest removing it til they clean their own house | 07:16 |
dirk | Can we drop ceilometerr from idl? | 07:16 |
dirk | Odl | 07:16 |
tonyb | prometheanfire: said that networking-bgp is the only consumer so if they dropped networking-odl we could also and get things back on track | 07:16 |
tonyb | dirk: I don't think so | 07:16 |
dirk | Why? | 07:17 |
tonyb | we could publisha straw-person revert but I doubt it'll land | 07:17 |
tonyb | dirk: the code's been in there for ages | 07:17 |
tonyb | it isn't like its a recnet feature | 07:17 |
tonyb | I'm pretty sure $comsumers are relying on the agent existing | 07:17 |
prometheanfire | I'm curious how they added it at all | 07:17 |
dirk | Sure, ceilometer itself isn't the new kid on the block | 07:18 |
prometheanfire | don't entries in reqs.txt need to be in global-reqs? | 07:18 |
tonyb | prometheanfire: they added it ages ago and used the well established tox_install from neutron | 07:18 |
dirk | tonyb: 'pretty sure' sounds worth investigating imho | 07:18 |
prometheanfire | ya, right | 07:18 |
tonyb | dirk: it seems not for URL based requirements | 07:19 |
tonyb | dirk: we may have a bug in our tools there but that doesn't help move things forward | 07:19 |
dirk | tonyb: sure, as a stop gap we can get things going for now | 07:20 |
prometheanfire | we should have a bug for the tool not working as expect at the very least | 07:20 |
dirk | Any attempt at a more complete fix will eat more time | 07:20 |
tonyb | dirk: right that's why I've faffed with the hack above | 07:20 |
prometheanfire | how about this order of operations | 07:22 |
prometheanfire | 1. do the hack | 07:22 |
prometheanfire | 2. fix reqs to reject url based reqs | 07:22 |
prometheanfire | 3. make sure odl is fixed | 07:22 |
tonyb | (if it's a bug) | 07:22 |
prometheanfire | 4. undo the hack | 07:22 |
prometheanfire | tonyb: sure | 07:23 |
tonyb | prometheanfire: Yup that's pretty much what we have to do | 07:23 |
tonyb | all of which will be fun during the PTG | 07:23 |
*** olivierb has joined #openstack-requirements | 07:23 | |
tonyb | prometheanfire: but you can try visiting the neutron room next week | 07:24 |
prometheanfire | I get to play the pixie from zelda | 07:24 |
tonyb | ;p | 07:24 |
prometheanfire | not much diferent than I just got done doing with keystone and nova :P | 07:24 |
prometheanfire | tonyb: mind putting this into a bug just so we don't forget? I'm bedding after this | 07:25 |
tonyb | prometheanfire: Sure but it'll be tomorrow | 07:25 |
prometheanfire | tonyb: eh, ok | 07:26 |
prometheanfire | #topic open floor | 07:27 |
*** openstack changes topic to "open floor (Meeting topic: requirements)" | 07:27 | |
prometheanfire | ptg is next week, I'm not sure we'll be able to do anything more than inpromptu meetings | 07:27 |
tonyb | Yeah it'll be hard to get the 3 TZ awake at the same time | 07:27 |
prometheanfire | #link https://etherpad.openstack.org/p/stein-PTG-requirements | 07:29 |
prometheanfire | dougs thing is the main complex one | 07:29 |
prometheanfire | beyond that, constraints publishing would be nice | 07:29 |
tonyb | Yup I'll ping the infra team PTG is over | 07:30 |
prometheanfire | anyone else have anything else before ending? | 07:31 |
tonyb | not for me | 07:31 |
prometheanfire | k | 07:32 |
prometheanfire | #endmeeting | 07: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 | |
openstack | Meeting ended Wed Sep 5 07:33:02 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 07:33 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-07.11.html | 07:33 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-07.11.txt | 07:33 |
openstack | Log: http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-07.11.log.html | 07:33 |
*** ccamacho has joined #openstack-requirements | 07:51 | |
*** jpich has joined #openstack-requirements | 08:08 | |
*** dtantsur|afk is now known as dtantsur | 08:35 | |
*** zigo has joined #openstack-requirements | 10:03 | |
*** udesale has quit IRC | 10:21 | |
*** udesale has joined #openstack-requirements | 10:21 | |
*** udesale has quit IRC | 10:21 | |
*** udesale has joined #openstack-requirements | 10:22 | |
*** dtantsur is now known as dtantsur|brb | 11:08 | |
*** udesale has quit IRC | 11:10 | |
*** zigo has quit IRC | 11:45 | |
*** odyssey4me has quit IRC | 11:53 | |
*** odyssey4me has joined #openstack-requirements | 11:55 | |
*** lbragstad has joined #openstack-requirements | 11:57 | |
*** udesale has joined #openstack-requirements | 12:57 | |
openstackgerrit | Eric Fried proposed openstack/requirements master: oslo.utils 3.37.0 upper constraint https://review.openstack.org/600034 | 13:04 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.utils to new release 3.37.0 https://review.openstack.org/600041 | 13:16 |
*** dtantsur|brb is now known as dtantsur | 13:17 | |
*** udesale has quit IRC | 13:43 | |
*** dtantsur is now known as dtantsur|brb | 14:24 | |
lbragstad | prometheanfire around for a quick requirements question? | 14:26 |
lbragstad | this is on master https://review.openstack.org/#/c/599003/ which fails due to oslo.messaging requirements checking | 14:27 |
lbragstad | Package 'oslo.messaging' is constrained to 5.29.0 which does not match the minimum version specifier 8.1.0 in test-requirements.txt | 14:27 |
lbragstad | i checked openstack/requirements and the upper constraint is 8.1.0 on master | 14:28 |
lbragstad | do you know how 5.29.0 is getting involved when we dealing with only master? | 14:28 |
lbragstad | we're* | 14:29 |
lbragstad | oh - i bet https://review.openstack.org/#/c/599003/1/tox.ini@87 is causing issues | 14:34 |
prometheanfire | lbragstad: hi | 14:43 |
lbragstad | o/ | 14:52 |
lbragstad | i might have answered my own question | 14:52 |
lbragstad | via rubber duck debugging | 14:52 |
prometheanfire | :D | 14:57 |
prometheanfire | lbragstad: I commented as well | 14:57 |
lbragstad | updated the patch again, not sure what i did with ps2 made sense | 15:03 |
prometheanfire | lbragstad: do you know why you need the latest oslo.messaging? | 15:06 |
prometheanfire | iirc there's not been much change | 15:07 |
lbragstad | apparently we need oslo.messaging for our tests, but i doubt we have a requirement for the latest | 15:07 |
prometheanfire | I'd just have 5.29.0 (or whatever it was) in test-requirements instead | 15:08 |
lbragstad | ah - good point | 15:12 |
lbragstad | ok - i think i'm done mucking with it | 15:13 |
* lbragstad convinces himself https://review.openstack.org/#/c/599003/4 looks right until Zuul says differently | 15:13 | |
dhellmann | prometheanfire , 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 effect | 15:39 |
prometheanfire | dhellmann: one step at a time | 15:41 |
dhellmann | yeah, we probably want to take action now, I'm just pointing out that things may get easier later, too | 15:41 |
prometheanfire | dhellmann: you may want to go read the backlog as well, had one meeting at 2AM | 15:42 |
prometheanfire | dhellmann: will have another in a couple hours, but don't expect much to be said at it | 15:42 |
dhellmann | yeah, I was responding to the meeting scrollback | 15:42 |
prometheanfire | :D | 15:43 |
*** dtantsur|brb is now known as dtantsur | 16:01 | |
*** jpich has quit IRC | 16:30 | |
*** olivierb has quit IRC | 16:44 | |
openstackgerrit | Merged openstack/requirements master: Remove networking-odl from g-r in the requirements-integration job https://review.openstack.org/599277 | 16:51 |
openstackgerrit | Matthew Thode proposed openstack/requirements master: Revert "Remove networking-odl from g-r in the requirements-integration job" https://review.openstack.org/600123 | 16:53 |
*** dtantsur is now known as dtantsur|afk | 17:26 | |
*** olivierb has joined #openstack-requirements | 17:32 | |
*** olivierb has quit IRC | 17:38 | |
*** ccamacho has quit IRC | 17:44 | |
*** openstackgerrit has quit IRC | 18:00 | |
lbragstad | prometheanfire following up on https://review.openstack.org/#/c/599003/4 again | 19:44 |
lbragstad | we define audit_notifications in setup.py as an extra package | 19:44 |
lbragstad | setup.cfg* | 19:44 |
*** openstackgerrit has joined #openstack-requirements | 19:48 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/rocky: update constraint for python-openstackclient to new release 3.16.1 https://review.openstack.org/600196 | 19:48 |
prometheanfire | lbragstad: was wondering about that | 20:12 |
lbragstad | i added it back in to see if it fixes the problem | 20:15 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/rocky: update constraint for python-neutronclient to new release 6.9.1 https://review.openstack.org/600200 | 20:15 |
*** efried has joined #openstack-requirements | 20:18 | |
efried | Greetings 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 |
prometheanfire | efried: which project? | 20:21 |
efried | prometheanfire: openstack/placement is currently using nova's *requirements.txt, which is obviously waaay overkill. | 20:21 |
efried | I want to trim it down to only what's needed | 20:21 |
efried | and I thought there was a tool that would figure that out for me | 20:21 |
efried | you know, something better than | 20:22 |
efried | pip uninstall | 20:22 |
efried | tox || pip install | 20:22 |
efried | repeat | 20:22 |
prometheanfire | just find out what you import | 20:24 |
prometheanfire | grep -R import | 20:25 |
prometheanfire | efried: you coppied the nova git repo to be placement's? | 20:26 |
efried | yeah, sort of | 20:27 |
efried | I mean, *I* didn't | 20:27 |
efried | there was some kind of script that seeded the repo with a subset of files while preserving their git history | 20:27 |
prometheanfire | efried: https://paste.pound-python.org/show/eosJiPPXxjUNwPKI67om/ | 20:28 |
efried | and 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 |
prometheanfire | well, that's my 'grep -Rh import | sort -u' | 20:28 |
efried | prometheanfire: Just to confirm: [test-]requirements.txt doesn't need transitive deps, but lower-constraints.txt does | 20:29 |
prometheanfire | yes | 20:29 |
efried | cool | 20:29 |
efried | Thanks prometheanfire | 20:29 |
prometheanfire | #startmeeting requirements | 20:30 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 20:30 |
*** openstack changes topic to " (Meeting topic: requirements)" | 20:30 | |
openstack | The meeting name has been set to 'requirements' | 20:30 |
prometheanfire | #topic rollcall | 20:30 |
*** openstack changes topic to "rollcall (Meeting topic: requirements)" | 20:30 | |
prometheanfire | ping tonyb, prometheanfire, number80, dirk, coolsvap, toabctl, smcginnis, dhellmann | 20:30 |
prometheanfire | o/ | 20:30 |
prometheanfire | we had a meeting 13.5 hours ago, so this one probably isn't needed | 20:32 |
prometheanfire | more of a just in case thing | 20:32 |
*** efried has left #openstack-requirements | 20:34 | |
smcginnis | prometheanfire: Saw the ping earlier but didn't get a chance to read all the scrollback. | 20:38 |
smcginnis | prometheanfire: Anything from earlier of concern? | 20:38 |
prometheanfire | smcginnis: nope | 20:39 |
smcginnis | Great | 20:39 |
prometheanfire | smcginnis: just https://review.openstack.org/#/c/599277/ | 20:40 |
prometheanfire | #endmeeting | 20: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 | |
openstack | Meeting ended Wed Sep 5 20:40:16 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 20:40 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-20.30.html | 20:40 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-20.30.txt | 20:40 |
openstack | Log: http://eavesdrop.openstack.org/meetings/requirements/2018/requirements.2018-09-05-20.30.log.html | 20:40 |
smcginnis | Oh right | 20:40 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!