Thursday, 2017-09-21

*** openstackgerrit has quit IRC00:41
*** openstackgerrit has joined #openstack-requirements00:48
openstackgerritMerged openstack/requirements master: update constraint for oslo.log to new release 3.31.0  https://review.openstack.org/50553700:48
openstackgerritIan Wienand proposed openstack/requirements master: Avoid networkx 2.0  https://review.openstack.org/50590501:39
openstackgerritzhangyangyang proposed openstack/requirements master:  Update sphinx version in requirements  https://review.openstack.org/50590601:52
*** amotoki has quit IRC01:55
*** amotoki has joined #openstack-requirements01:56
*** strigazi has quit IRC01:56
*** strigazi has joined #openstack-requirements01:58
openstackgerritMerged openstack/requirements master: update constraint for oslo.context to new release 2.18.1  https://review.openstack.org/50553902:01
openstackgerritMerged openstack/requirements master: update constraint for oslo.policy to new release 1.27.0  https://review.openstack.org/50555102:02
openstackgerritMerged openstack/requirements master: update constraint for neutron-lib to new release 1.10.0  https://review.openstack.org/50575502:05
openstackgerritMerged openstack/requirements master: [tools] Add a README.txt  https://review.openstack.org/50579402:36
*** racsox has joined #openstack-requirements03:08
*** coolsvap has joined #openstack-requirements03:37
*** udesale has joined #openstack-requirements03:41
prometheanfireianw: can we just mask that particular release for now instead of capping it?04:20
ianwprometheanfire: well 2.0 is effectively a new branch that has broken backwards compat04:21
prometheanfirewe could do so with a comment in file to know to remove it once projects are updated04:21
prometheanfireok, in order to cap something we will need a bug to track to cap.04:21
ianwtaskflow uses upper-constraints it seems ... but it does fail with 2.004:22
ianwas for the other projects, i dunno04:22
ianwok, i can file a bug against what i see in codesearch04:22
prometheanfirewe can use the bug to notify consuming projects to test with another review (that sets UC to 2.0 and req >=2.0)04:22
prometheanfiremake it a requirements bug04:22
ianwit's really quite annoying because there's no sane way to support both versions in your code04:23
prometheanfireopenstack-requirements is the project04:23
prometheanfireyour? :P04:23
ianwthey probably should have just called it networkx204:24
prometheanfireeh, it's what major versions are for04:24
prometheanfireI'd rather them keep the same name04:24
*** racsox has left #openstack-requirements04:25
*** jrist has joined #openstack-requirements04:30
openstackgerritIan Wienand proposed openstack/requirements master: Avoid networkx 2.0  https://review.openstack.org/50590504:31
ianwoh for heavens sake, did marking it as a dup remove all the other bugs04:33
prometheanfirelol04:34
prometheanfireremoved the dup flag04:34
prometheanfirewell, if launchpad stops spinning04:34
prometheanfirelooks like that worked04:35
ianwi did too, and dup'd the existing dib bug to that one04:35
prometheanfiredups removed04:35
openstackgerritMatthew Thode proposed openstack/requirements master: [DNM] test review for consuming projects to use  https://review.openstack.org/50597304:39
ianwyou know, how am i going to merge the requirements update anyway if the gate is broken04:39
prometheanfiremake the requirements update depend on the gate fix04:40
ianwbut it won't merge until the gate fix is in, which won't merge till the requirements fix is in ... circle04:40
prometheanfiremake it all one review04:41
ianwi guess i can either mash them together, or turn it off the tests in project-config and then back on04:41
prometheanfireI'd just mash them together, but you know how I work...04:43
prometheanfireI tend to like more atomicy, but sometimes things gotta go fast04:43
prometheanfirehttp://i0.kym-cdn.com/photos/images/newsfeed/000/506/223/2ab.gif04:44
openstackgerritMatthew Thode proposed openstack/requirements master: [DNM] test review of networkx-2.0 for consuming projects to use  https://review.openstack.org/50597304:46
prometheanfiretonyb: dirk dims coolsvap, you around to look at https://review.openstack.org/505905 ?04:55
coolsvapprometheanfire: i will keep track04:56
prometheanfirethanks04:57
coolsvapnp!04:57
prometheanfireianw: once it merges you should be good, kinda wonder why you don't use upper-constraints though (if you are hitting this bug)04:57
ianwi don't think there's any particular reason, other than nobody's updated it04:58
ianwreviews welcome :)04:58
prometheanfireya, will be doing so04:58
dirkianw: can you add the bug reference to the commit message?05:19
dirkI wonder which of those projects are not using uc05:19
prometheanfireya, DIB isn't (made a review for that...)05:20
*** udesale__ has joined #openstack-requirements05:35
*** udesale has quit IRC05:38
*** udesale has joined #openstack-requirements05:50
*** udesale__ has quit IRC05:52
ianwdirk: which bug?05:56
openstackgerritIan Wienand proposed openstack/requirements master: Avoid networkx 2.0  https://review.openstack.org/50590505:58
ianwdirk: ^ i'm presuming that's what you mean05:58
*** florianf has joined #openstack-requirements06:19
*** udesale has quit IRC06:29
*** udesale has joined #openstack-requirements06:30
dirkianw: yes, thanks06:40
dirkianw: do you happen to know whether your networkx 2 porting patch is compatible with networkx<2 ?06:40
*** ralonsoh has joined #openstack-requirements06:45
ianwdirk: no, unfortunately it's not06:57
*** jpich has joined #openstack-requirements06:58
dirkianw: hmpf. Jumping from <2 to >=2 is not an easy switch06:58
dirkSo then I would prefer to not merge the cap06:59
ianwdirk: well it leaves dib in a not so nice situation, because the gate is broken for other reasons (the upstream kernel packaging changed)07:03
ianwso i can't merge a 2.0 fix without fixing that07:03
ianwbut i can't fix that without a 2.0 fix07:03
ianwand afaict, everyone else will be broken by it too, it's just not a backwards compat change.  so wouldn't it be better to undertake the upgrade in a more controlled fashion?07:04
*** ChanServ sets mode: +o openstack07:15
prometheanfiredirk: some of the breakage should be here https://review.openstack.org/50597307:19
prometheanfireianw: you really should make sure to merge my tox update into whatever final fix you have07:19
prometheanfirefor now though, I sleep07:19
openstackgerritMerged openstack/requirements master: update constraint for oslo.cache to new release 1.26.0  https://review.openstack.org/50554407:42
openstackgerritMerged openstack/requirements master: update constraint for oslo.rootwrap to new release 5.10.0  https://review.openstack.org/50553807:42
*** yolanda has joined #openstack-requirements07:48
yolandahi, morning, can we get https://review.openstack.org/#/c/505905/ approved? breaking dib...07:48
*** edmondsw has joined #openstack-requirements09:15
*** edmondsw has quit IRC09:20
*** udesale has quit IRC10:16
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints  https://review.openstack.org/50610910:47
openstackgerritMerged openstack/requirements master: Avoid networkx 2.0  https://review.openstack.org/50590510:49
*** sdague has joined #openstack-requirements11:38
dimsprometheanfire : looks like it got merged11:43
*** udesale has joined #openstack-requirements12:02
*** udesale has quit IRC12:04
*** udesale has joined #openstack-requirements12:04
*** edmondsw has joined #openstack-requirements12:13
openstackgerritDirk Mueller proposed openstack/requirements master: Updated from generate-constraints  https://review.openstack.org/50610912:32
*** udesale has quit IRC12:39
*** yolanda has left #openstack-requirements13:15
*** coolsvap has quit IRC13:39
*** openstackgerrit has quit IRC14:33
tonybprometheanfire: You'll want to +1 this: https://review.openstack.org/#/c/505817/14:39
tonybprometheanfire: Well review it, I'm not telling you how to vote P14:40
prometheanfirethat was a simple one :P14:45
*** openstackgerrit has joined #openstack-requirements15:05
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: update constraint for ovsdbapp to new release 0.6.0  https://review.openstack.org/50622915:05
prometheanfiretonyb: how often is the periodic queue run?15:11
tonybprometheanfire: daily IIRC it's at 0600 UTC15:12
tonybhttp://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n16715:14
tonybYup 0600 UTC15:14
prometheanfiretonyb: I do think we need to update reqs.txt faster than that15:14
prometheanfirethink of gate breakage as a scenario15:14
tonybprometheanfire: but the published things aren't used in the gate15:15
tonybprometheanfire: a commit to $branch will clear the gate15:15
tonybthe published URLs are used only by devs/consumers outside CI running tox15:16
tonybprometheanfire: *ideally* they'd also be used by installers but there aren't a lot of people installing OpenStack by hand from traballs15:16
prometheanfiretrue, forgot about the caching we do15:17
tonybprometheanfire: It isn't caching it's just job setup15:17
prometheanfireit's not updating the cache'd repo?15:17
tonybque?15:17
prometheanfireton of repos are cached on the test images15:18
prometheanfireincluding ours15:18
tonybOh sure that's releated but different15:18
prometheanfireok, what's the harm of publishing often?15:18
tonybeach job always gets the current branch from openstack/requiremenmts15:18
prometheanfireI think openstack-ansible uses the url only, will have to ask15:19
tonybharm? none I just think it's wasteful15:19
prometheanfireI think the low amount of waste does not offset the minor amount of goodness provided by having that file up to date15:21
prometheanfirewe would be making it harder for people to use upper-constraints externally for one thing (via longer turn around time)15:21
*** mtreinish has joined #openstack-requirements15:21
tonybprometheanfire: okay so say all that on the mailing list.  When I suggested mving to periodic I got *crikets*15:25
* tonyb waves at mtreinish15:25
prometheanfireI kinda want to isolate the periodic vs post, mind if I change the email subject in the reply?15:26
tonybprometheanfire: got nuts, just reply to the part wehere I suggested it15:27
tonybhuh, eaves drop doesn't update 'real-time' :(15:27
prometheanfireofcourse :P15:27
mtreinishtonyb: it's on puppet/cron iirc15:27
* mtreinish waves back15:27
mtreinishso every 15min or so15:28
tonybmtreinish: okay, you'll just have to wait to see the ridicule15:28
mtreinishheh, I've already got the log open and have been hitting refesh every couple of min :)15:28
tonybhehe15:30
mtreinishtonyb: hey, I install from tarballs!15:33
tonybmtreinish: \o/15:33
tonybmtreinish: good blog BTW15:33
mtreinishtonyb: thanks15:34
tonybprometheanfire: if OSA does use the URL they're broken and that needs to be fixed.  Howevere that's a tanget from the discussion15:36
prometheanfirethey don't15:36
prometheanfireI checked15:36
prometheanfireand ya, tangent15:36
odyssey4mewe use the URL for tox jobs, just like everyone else15:36
prometheanfireno one else uses the url :P15:37
prometheanfirethe UPPER_CONSTRAINTS_FILE var is set then tox is run15:37
prometheanfireso it's a locally updated file15:37
odyssey4mehttps://git.openstack.org/cgit/openstack/keystone/tree/tox.ini#n815:38
odyssey4meso yes, in openstack infra it doesn't use the url - but outside infra it does15:38
odyssey4mewe do the same thing15:38
prometheanfireright15:38
openstackgerritboden proposed openstack/requirements master: bump neutron-lib to 1.10.0  https://review.openstack.org/50626115:38
prometheanfireok, sent15:43
tonybodyssey4me: cool. I'm more worried about how repo_build.  tox is easy15:45
odyssey4meoh repo_build does it's own git clone so no worries there15:46
tonybodyssey4me: becuase I'm dense I can't see where it gets URL from15:46
odyssey4mehmm, but all our role tests use u-c via the URL though I think15:46
odyssey4mehttps://github.com/openstack/openstack-ansible-tests/blob/master/test-vars.yml#L485 <--- used for role tests15:47
odyssey4merepo_build is perfectly fine - it uses a git clone, checks out the sha... the new mechanism of publishing the head of the branch to a web server won't work for the way we sha pin right now... but we clone the repo and own that tech debt15:48
odyssey4mewe plan, in queens, to try to remodel the whole way the wheels are built anyway15:49
odyssey4mewhat's the worry though - with the git url no longer work for some reason?15:49
odyssey4me*will15:49
tonybodyssey4me: okay.  So you don't work with Depends-On now and wont after this change15:49
odyssey4meno, we only ever use the *committed* version of the file15:50
tonybodyssey4me: the URL will work as it does today15:50
tonybodyssey4me: cool.15:51
odyssey4mewell, if everything goes on fire we'll just blame you :p15:53
odyssey4mebut of course we appreciate the consideration!15:53
tonybodyssey4me: The bottom line is nothing will chnage for you.  You just don't get the goodness of being able to install after requirements goes EOL.15:54
tonybodyssey4me: but you can always factor that into your repo re-work15:54
odyssey4merequirements as a repo is going EOL?15:55
tonybodyssey4me: No the branches with in the repo (eg stable/liberty) will EOL along with the other projects15:56
odyssey4meoh yeah, that's fine - we expect that anyway15:56
tonybodyssey4me: requiremenst are FOR EVER!!!!!15:56
*** ralonsoh has quit IRC16:02
*** ralonsoh has joined #openstack-requirements16:02
*** ralonsoh has quit IRC16:02
*** florianf has quit IRC16:13
openstackgerritMerged openstack/requirements master: Updated from generate-constraints  https://review.openstack.org/50610916:29
*** jpich has quit IRC17:01
-openstackstatus- NOTICE: Gerrit OpenIDs have been accidentally overwritten and are in the process of being restored18:18
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly18:23
*** ChanServ changes topic to "The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly"18:23
*** ChanServ changes topic to "OpenStack Requirements - IRC meetngs on Wednesdays @ 12:00 UTC in #openstack-meeting-3 - See agenda @ http://tinyurl.com/h44ryuw - IRC channel is *LOGGED* @ http://tinyurl.com/j38rk24"19:21
-openstackstatus- NOTICE: OpenIDs for the Gerrit service have been restored from a recent backup and the service is running again; before/after table states are being analyzed now to identify any remaining cleanup needed for changes made to accounts today19:21
openstackgerritMerged openstack/requirements master: update constraint for oslo.versionedobjects to new release 1.27.0  https://review.openstack.org/50554719:43
openstackgerritMerged openstack/requirements master: update constraint for ovsdbapp to new release 0.6.0  https://review.openstack.org/50622919:45
*** edmondsw has quit IRC20:51
*** edmondsw has joined #openstack-requirements20:57
*** edmondsw has quit IRC21:01
openstackgerritMerged openstack/requirements master: update constraint for oslo.db to new release 4.27.0  https://review.openstack.org/50554521:30
*** coreycb has quit IRC21:46
*** mrhillsman has quit IRC21:47
*** rm_work has quit IRC21:47
*** coreycb has joined #openstack-requirements21:48
*** amotoki has quit IRC21:48
*** mrhillsman has joined #openstack-requirements21:50
*** amotoki has joined #openstack-requirements21:51
*** rm_work has joined #openstack-requirements22:02
*** rm_work has quit IRC22:02
*** rm_work has joined #openstack-requirements22:02
*** sdague has quit IRC22:34
*** edmondsw has joined #openstack-requirements22:58
*** edmondsw has quit IRC23:02

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