*** armax has quit IRC | 00:10 | |
*** hoangcx has joined #openstack-requirements | 00:39 | |
*** ihrachys has joined #openstack-requirements | 00:48 | |
*** ihrachys_ has joined #openstack-requirements | 00:50 | |
*** armax has joined #openstack-requirements | 00:51 | |
*** ihrachys has quit IRC | 00:54 | |
tonyb | lbragstad: It's because https://review.openstack.org/#/c/439973/ also needs to be backported and I haven't done it yet :( | 01:31 |
---|---|---|
tonyb | dhellmann, lbragstad: I think we need the cap in stable/* and once we work though the "bad old days" of bumps and re-releases I think we'll be okay | 01:37 |
* tonyb is readign scrollback after public-holiday and virus :( | 01:39 | |
prometheanfire | tonyb: woo virus | 01:44 |
tonyb | prometheanfire: nothign terrible just a head full'o'cotton wool | 01:45 |
prometheanfire | ah | 01:46 |
*** ihrachys_ has quit IRC | 02:38 | |
prometheanfire | meetings are 5 central now, nice :D | 03:32 |
*** ihrachys has joined #openstack-requirements | 03:36 | |
*** ihrachys has quit IRC | 03:46 | |
*** karthikpr has joined #openstack-requirements | 04:15 | |
openstackgerrit | Merged openstack/requirements master: Add betamax-matchers to global-requirements https://review.openstack.org/443231 | 04:16 |
*** udesale has joined #openstack-requirements | 04:18 | |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.i18n to new release 3.14.0 https://review.openstack.org/445008 | 04:21 |
*** ihrachys has joined #openstack-requirements | 04:24 | |
*** karthikpr has quit IRC | 04:30 | |
*** karthikpr has joined #openstack-requirements | 04:30 | |
*** karthikpr has quit IRC | 04:37 | |
openstackgerrit | Merged openstack/requirements master: Bump oslo.messaging to 5.19 https://review.openstack.org/445302 | 04:45 |
*** ihrachys has quit IRC | 05:06 | |
*** karthikpr has joined #openstack-requirements | 05:10 | |
*** karthikpr has quit IRC | 05:16 | |
*** karthikpr has joined #openstack-requirements | 05:17 | |
*** udesale__ has joined #openstack-requirements | 05:17 | |
*** udesale has quit IRC | 05:21 | |
*** karthikpr has quit IRC | 05:21 | |
*** udesale has joined #openstack-requirements | 05:35 | |
*** udesale__ has quit IRC | 05:37 | |
*** udesale__ has joined #openstack-requirements | 05:44 | |
*** udesale has quit IRC | 05:46 | |
*** armax has quit IRC | 06:20 | |
tonyb | So I've been sick and I'm not feelign up to a late night tonight so I think I'm going to bail on the meeting. Sorry all. | 07:48 |
tonyb | Of course you're free to have one without me. | 07:48 |
*** udesale has joined #openstack-requirements | 08:13 | |
*** udesale__ has quit IRC | 08:14 | |
*** amoralej|off is now known as amoralej | 08:37 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.openstack.org/444773 | 08:38 |
*** jpich has joined #openstack-requirements | 08:39 | |
*** udesale__ has joined #openstack-requirements | 09:04 | |
*** udesale has quit IRC | 09:06 | |
amoralej | tonyb, question about https://review.openstack.org/#/c/442913/ | 09:09 |
*** amoralej has quit IRC | 09:54 | |
*** amoralej has joined #openstack-requirements | 10:00 | |
dirk | tonyb: ah, thanks for the update | 10:05 |
dirk | prometheanfire: do you want to do a meeting? and if so, in which timezone? :) | 10:05 |
prometheanfire | dirk: I want to go back to bed :P | 10:06 |
dirk | prometheanfire: do so :) | 10:06 |
* dirk doesn't mind | 10:06 | |
prometheanfire | k, back to bed :D | 10:08 |
*** udesale has joined #openstack-requirements | 10:14 | |
*** udesale has quit IRC | 10:16 | |
*** udesale has joined #openstack-requirements | 10:16 | |
*** udesale__ has quit IRC | 10:17 | |
*** openstackgerrit has quit IRC | 10:18 | |
*** sdague has joined #openstack-requirements | 10:27 | |
*** udesale has quit IRC | 10:41 | |
*** hoangcx has quit IRC | 10:46 | |
*** kornicameister has quit IRC | 12:09 | |
*** openstackgerrit has joined #openstack-requirements | 12:14 | |
openstackgerrit | Saad Zaher proposed openstack/requirements master: update global requirements constraint for python-freezerclient to new release 1.3.0 https://review.openstack.org/445905 | 12:14 |
openstackgerrit | Merged openstack/requirements master: Allow newer SQLAlchemy https://review.openstack.org/423192 | 12:26 |
*** jpich has quit IRC | 12:46 | |
*** jpich_ has joined #openstack-requirements | 12:46 | |
*** jpich_ is now known as jpich | 12:51 | |
*** amoralej is now known as amoralej|lunch | 13:34 | |
*** udesale has joined #openstack-requirements | 13:59 | |
dhellmann | tonyb, lbragstad : I'm happy to keep the cap for now, if it's working out ok. | 14:00 |
prometheanfire | upgrades seem to be doing alright | 14:01 |
lbragstad | dhellmann is this change alright then? https://review.openstack.org/#/c/439355/6/requirements.txt,unified | 14:02 |
dhellmann | lbragstad : it's what we've done elsewhere, although clarkb pointed out that not having overlap between ocata and pike for pbr may cause issues | 14:03 |
dhellmann | tonyb : I'd like your input on whether we should change pike's pbr settings back to >=1.8 because of clarkb's comments | 14:03 |
dhellmann | tonyb : see http://eavesdrop.openstack.org/irclogs/%23openstack-shade/%23openstack-shade.2017-03-14.log.html for the logs | 14:04 |
dhellmann | prometheanfire : ^^ | 14:04 |
lbragstad | dhellmann ok - i'm just double checking before i start approving those changes for identity related projects | 14:05 |
prometheanfire | dhellmann: ya, I remember him (or someone) pointing that out, but it hasn't caused problems for the upgrade gating that I know of | 14:07 |
*** ihrachys has joined #openstack-requirements | 14:08 | |
dhellmann | lbragstad : yeah, I can't give you a clear yes or no. clarkb raised some concerns, but not in this channel or with the requirements team, so we need to get them to think about it. | 14:08 |
prometheanfire | though if we can do it it'd just be preventitive care | 14:08 |
prometheanfire | which is good | 14:08 |
dhellmann | prometheanfire : apparently it caused some issues for shade somewhere, but I don't have the full details | 14:08 |
prometheanfire | ah | 14:08 |
prometheanfire | having overlap would make packaging a little easier | 14:08 |
dhellmann | if we do it, I think the flag for sphinx to treat warnings as errors will always be false, but that's what it was before | 14:09 |
dhellmann | so I think it's probably safe to lower the min version | 14:10 |
dhellmann | I haven't given it a lot of thought though | 14:10 |
prometheanfire | where does sphinx enter into this? | 14:11 |
prometheanfire | lbragstad: as far as the diff goes, the only thing that may change is pbr | 14:13 |
lbragstad | prometheanfire yeah - that's what i was thinking | 14:14 |
lbragstad | prometheanfire everything looks ok in IMO | 14:14 |
*** amoralej|lunch is now known as amoralej | 15:02 | |
*** karthikpr has joined #openstack-requirements | 15:10 | |
*** karthikpr has quit IRC | 15:13 | |
*** armax has joined #openstack-requirements | 15:37 | |
*** karthikpr has joined #openstack-requirements | 16:07 | |
*** karthikpr has quit IRC | 16:08 | |
*** udesale has quit IRC | 16:12 | |
*** robputt_ has quit IRC | 16:28 | |
openstackgerrit | Saad Zaher proposed openstack/requirements master: update constraint for python-freezerclient to new release 1.3.0 https://review.openstack.org/445905 | 16:30 |
*** robputt has joined #openstack-requirements | 16:30 | |
*** karthikpr has joined #openstack-requirements | 16:34 | |
*** strigazi is now known as strigazi_AFK | 16:39 | |
openstackgerrit | Sean McGinnis proposed openstack/requirements master: Raise python-cinderclient to 2.0.1 https://review.openstack.org/446064 | 16:53 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-cinderclient to new release 2.0.1 https://review.openstack.org/446066 | 16:55 |
*** karthikpr has quit IRC | 17:12 | |
*** rtheis has joined #openstack-requirements | 17:13 | |
*** rtheis has quit IRC | 17:16 | |
amoralej | dhellmann, question about https://review.openstack.org/#/c/442913/ apparently is adding a pike release into stable/ocata u-c, is that correct? | 17:24 |
*** karthikpr has joined #openstack-requirements | 17:30 | |
*** jpich has quit IRC | 17:31 | |
*** karthikpr has quit IRC | 17:32 | |
dhellmann | amoralej : it looks like that is in fact a pike release. tonyb ^^ | 17:37 |
dhellmann | prometheanfire : ^^ | 17:37 |
*** karthikpr has joined #openstack-requirements | 17:37 | |
dhellmann | dirk : ^^ | 17:37 |
prometheanfire | looking | 17:37 |
prometheanfire | 1.0.1? | 17:38 |
prometheanfire | and commit message makes me think it's targeted to ocata | 17:38 |
prometheanfire | anyway, need to get food while I can | 17:38 |
dhellmann | it looks like I should have been more careful with that 1.0.1 release tag | 17:40 |
amoralej | dhellmann, tagging 1.0.1 in master is effectively blocking new releases in stable/ocata, or am i missing something? | 17:41 |
dhellmann | amoralej : yes, I messed up. we can do a 1.0.2 from stable/ocata after the fixes are backported | 17:42 |
amoralej | also, commit message in https://review.openstack.org/#/c/442913/ is confusing as it seems 1.0.1 is required for messaging in pike, not ocata but they are adding it in u-c in stable/ocata because it's not properly updated in grenade jobs | 17:43 |
dhellmann | that part i dont understand | 17:43 |
amoralej | what seems not to be the right fix | 17:43 |
amoralej | dhellman, iiuc package is not being properly upgraded in grenade jobs when upgrading from ocata to master so they are forcing to install the required one in master since the beginning | 17:45 |
*** jd_ has joined #openstack-requirements | 17:45 | |
jd_ | howdy | 17:45 |
dhellmann | amoralej : I have no idea about any of that, I wasn't involved. I've asked gordc and jd__ to join us here. I'm going to work on improving validation in the releases repo, but they're going to have to work on fixing the rest of it. | 17:46 |
dhellmann | hi, jd_ | 17:46 |
dhellmann | https://review.openstack.org/#/c/442913/ has introduced a pike series library into the ocata series tests | 17:46 |
dhellmann | we need to undo that | 17:46 |
amoralej | thanks for your help dhellmann | 17:46 |
dhellmann | and then get a release from the ocata branch instead | 17:46 |
dhellmann | I should have noticed that 1.0.1 was on master, not ocata, and asked for a different version number there, which would have made the rest of this more obviously wrong | 17:47 |
jd_ | oh ok so the release was made in some bizzare way? | 17:48 |
jd_ | gordc might know better than me the root of this patch anyway | 17:49 |
dhellmann | jd_ : the release was cut from master, yeah | 17:49 |
jd_ | dhellmann: what can I do to help? | 17:49 |
jd_ | push a button? | 17:49 |
dhellmann | we need you to backport whatever fix went into 1.0.1 onto stable/ocata, then we can release 1.0.2 from stable/ocata and we can update the constraints to use that | 17:49 |
dhellmann | that way packagers watching that branch will actually get the released version | 17:50 |
prometheanfire | yep | 17:50 |
* jd_ digs into that | 17:51 | |
amoralej | jd_ according to message in https://review.openstack.org/#/c/442913/ 1.0.1 is not really required for ocata but for pike, u-c bump in stable/ocata was only to workaround issue in grenade jobs not updating it properly | 17:53 |
amoralej | that's my understanding | 17:53 |
jd_ | so IIUC it's https://review.openstack.org/#/c/446090/ that is required for Ocata too because the oslo.messaging stuff is too recent otherwise | 17:54 |
jd_ | and that's why gordc released 1.0.1 | 17:54 |
jd_ | and made it happen on Ocata | 17:54 |
*** armax has quit IRC | 17:55 | |
jd_ | amoralej: though what's the problem with 1.0.1 in the end being pulled? any error? | 17:56 |
*** karthikpr has quit IRC | 17:56 | |
amoralej | jd_ using library from master branch in stable/ocata builds, that's the problem | 17:57 |
*** gordc has joined #openstack-requirements | 18:12 | |
gordc | dhellmann: sorry, just came back from lunch. did you need more details on my patch? | 18:13 |
dhellmann | gordc : check the scrollback for this channel. we need a new 1.0.2 on stable/ocata to avoid having packagers put 1.0.1 from pike into the ocata series | 18:13 |
gordc | basically relates to this: http://lists.openstack.org/pipermail/openstack-dev/2017-March/113506.html | 18:13 |
gordc | yeah, i just read it. | 18:14 |
gordc | or up to whatever is in eavesdrop | 18:14 |
dhellmann | there may be a better fix to the original problem, but from the release team's perspective we need to deal with the release, too | 18:14 |
gordc | i see. didn't realise libs couldn't cross branches | 18:14 |
dhellmann | yeah, it causes issues downstream so we try to avoid it | 18:15 |
dhellmann | I missed that when I was reviewing the release request, so I've added some extra validation to the releases repo | 18:15 |
gordc | well the release was suppose to be for master/pike... it just happens that it causes issue with our grenade because ocata pseudo-cap doesn't play well with pike | 18:16 |
dhellmann | ok, then I should have asked you to make the version 1.1.0 to save space for patch releases on the stable branch | 18:17 |
gordc | i see. i'll keep that in mind next time as well. wasn't' aware libs couldnt' exists across branches downstream | 18:18 |
dhellmann | it sounds like the proper fix will need to be a change somewhere in devstack, unless we can put the middleware in the constraints list | 18:18 |
dhellmann | yeah, it should work but it makes life a lot more complicated for packagers so we try to help them out by at least keeping our releases in the right series | 18:19 |
dhellmann | gordc : so you're saying that because nothing upgrades ceilometermiddleware as a side-effect of upgrading a service, it's not upgraded at all? | 18:20 |
gordc | dhellmann: i didn't check if it was possible with fix in devstack. what was happening was that when grenade updates services, it will update the requirements. but ceilometermiddleware isn't in anyone requirements. not sure how to hack that. | 18:20 |
dhellmann | iirc, grenade just does a pip -U on the service, and the requirements come in for free. so it sounds like it needs to be smart enough to look for ceilometermiddleware somewhere in ther | 18:21 |
dhellmann | there | 18:21 |
gordc | dhellmann: right. so in our grenade, pike would be testing against pike requirements, except ceiloemtermiddleware which remains stuck at ocata | 18:21 |
dhellmann | right | 18:21 |
dhellmann | ok, I think you had 90% of the right fix. | 18:21 |
gordc | ceilometermiddleware isn't actually in an service's requirement, that's the issue i couldn't solve | 18:21 |
dhellmann | if you backport whatever change was needed in ceilometermiddleware to your stable/ocata branch, we can tag 1.0.2 and we can make the constraints point to that | 18:22 |
dhellmann | ideally devstack or grenade would try to explicitly upgrade the middleware, since that's the tool responsible for installing it | 18:22 |
dhellmann | because you're going to have the same problem the next time something in the middleware isn't compatible with the new series | 18:22 |
dhellmann | but that's probably a longer-term fix than just getting the release straightened out | 18:23 |
dhellmann | gordc : make sense? | 18:23 |
gordc | agreed. was banking on never touching ceilometermiddleware again ;) | 18:23 |
dhellmann | heh | 18:23 |
dhellmann | let me know if I can help with stable reviews, and the new release | 18:23 |
gordc | i'll backport patch | 18:23 |
* gordc assuming jd_ didn't do it yet | 18:23 | |
gordc | nevermind he did. | 18:24 |
dhellmann | oh, I think he did, so look first | 18:24 |
dhellmann | gordc : I think https://review.openstack.org/#/c/446090/ | 18:24 |
gordc | dhellmann: i'll add a 1.0.2 release when it merges. | 18:25 |
dhellmann | gordc : ++ | 18:25 |
dhellmann | amoralej : ^^ | 18:25 |
amoralej | gordc, dhellmann, thanks for fixing this! | 18:30 |
*** armax has joined #openstack-requirements | 18:30 | |
jd_ | yeah I just did not press the button until you were back gordc to be sure, but cool thanks :) | 18:30 |
dirk | dhellmann: thanks for the ping, will double check what happened in packaging | 18:42 |
*** karthikpr has joined #openstack-requirements | 19:13 | |
*** karthikpr has quit IRC | 19:14 | |
*** amoralej is now known as amoralej|off | 19:20 | |
*** gordc has quit IRC | 19:48 | |
*** stream10 has joined #openstack-requirements | 20:40 | |
*** stream10 has quit IRC | 21:06 | |
*** sdague has quit IRC | 21:09 | |
*** ihrachys has quit IRC | 21:27 | |
dasm | o/ | 21:47 |
dasm | hey, i'm trying to force gate to install latest version of neutron-lib | 21:47 |
dasm | https://review.openstack.org/#/c/416302/16/requirements.txt | 21:47 |
dasm | but it gives me hard time: http://logs.openstack.org/02/416302/16/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/edc4279/logs/devstacklog.txt.gz#_2017-03-15_20_41_15_332 | 21:47 |
dasm | neutron_lib is not installed at all | 21:47 |
dasm | is there any check, or maybe -e is prohibited from requirements? | 21:48 |
dasm | maybe there is a better solution to install latest lib? I've read about LIB_FROM_GIT=neutron_lib but I'm not sure how to force dsvm-neutron-full job to get it | 21:49 |
dasm | (probably there is a better solution :) ) | 21:49 |
dasm | btw. purpose of testing latest neutron-lib is to ensure, that we won't break anything with new release of library | 21:51 |
*** armax has quit IRC | 22:19 | |
*** ihrachys has joined #openstack-requirements | 22:56 | |
*** ihrachys has quit IRC | 23:07 | |
*** ihrachys has joined #openstack-requirements | 23:08 | |
*** ihrachys has quit IRC | 23:10 | |
*** ihrachys has joined #openstack-requirements | 23:12 | |
*** IgorYozhikov has quit IRC | 23:17 | |
*** IgorYozhikov has joined #openstack-requirements | 23:17 | |
*** ihrachys has quit IRC | 23:17 | |
*** ihrachys has joined #openstack-requirements | 23:39 | |
*** ihrachys has joined #openstack-requirements | 23:39 | |
*** ihrachys has quit IRC | 23:44 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!