Friday, 2019-10-04

*** brinzhang has joined #openstack-requirements00:01
*** brinzhang_ has joined #openstack-requirements01:00
*** brinzhang has quit IRC01:04
*** CrayZee has joined #openstack-requirements01:10
*** shachar has quit IRC01:13
*** brinzhang has joined #openstack-requirements01:25
*** brinzhang_ has quit IRC01:29
*** brinzhang_ has joined #openstack-requirements01:54
*** brinzhang has quit IRC01:58
*** gouthamr has quit IRC04:15
*** gouthamr has joined #openstack-requirements04:16
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints  https://review.opendev.org/68138206:12
*** spsurya has joined #openstack-requirements06:33
*** udesale has joined #openstack-requirements06:46
*** brinzhang has joined #openstack-requirements06:50
*** brinzhang_ has quit IRC06:53
*** brinzhang_ has joined #openstack-requirements07:10
*** brinzhang has quit IRC07:13
*** tosky has joined #openstack-requirements07:18
*** e0ne has joined #openstack-requirements07:22
*** e0ne has quit IRC07:22
*** brinzhang has joined #openstack-requirements07:51
*** brinzhang_ has quit IRC07:55
*** brinzhang has quit IRC08:09
*** zbr is now known as zbr|ruck08:23
*** e0ne has joined #openstack-requirements08:59
*** ccamacho has joined #openstack-requirements09:19
*** mugsie has quit IRC10:03
*** mugsie has joined #openstack-requirements10:05
*** prometheanfire has quit IRC10:22
*** prometheanfire has joined #openstack-requirements10:23
*** zbr|ruck is now known as zbr|lunch11:15
*** ccamacho has quit IRC11:45
*** spsurya has quit IRC12:10
*** udesale has quit IRC12:16
*** udesale has joined #openstack-requirements12:16
*** camelCaser has quit IRC12:53
*** camelCaser has joined #openstack-requirements12:55
*** udesale has quit IRC12:56
*** camelCaser has quit IRC12:59
openstackgerritOpenStack Proposal Bot proposed openstack/requirements stable/stein: update constraint for python-troveclient to new release 2.17.1  https://review.opendev.org/68661113:10
openstackgerritOpenStack Proposal Bot proposed openstack/requirements stable/queens: update constraint for oslo.messaging to new release 5.35.6  https://review.opendev.org/68661513:13
openstackgerritOpenStack Proposal Bot proposed openstack/requirements stable/rocky: update constraint for oslo.messaging to new release 8.1.4  https://review.opendev.org/68661713:14
*** zbr|lunch is now known as zbr|ruck13:25
*** camelCaser has joined #openstack-requirements13:43
*** gmann_afk is now known as gmann13:58
*** bnemec is now known as beekneemech14:54
*** efried has joined #openstack-requirements15:15
efriedprometheanfire: mordred and I have an interesting requirements dilemma, got a sec?15:16
prometheanfireefried: sure15:17
efriedIt looks like this:15:17
efriedThere's a fix in keystoneauth1 that, when applied, exposes a latent bug in openstacksdk.15:17
efriedWe have a fix in sdk which, as long as it's applied before/with the ksa fix, makes everything fine.15:17
efriedBut there's no requirement from ksa to sdk (which makes sense -- it's the other way around).15:17
efriedSo the question is: would adding the new rev of sdk to the lower-constraints of ksa do the trick, despite sdk not being in requirements.txt of ksa?15:18
*** mordred has joined #openstack-requirements15:18
* mordred waves to efried15:18
efriedo/ I got started without you15:18
mordredones15:18
mordredwait15:19
mordredo noes15:19
efriedand it'll be another 10 minutes before eavesdrop catches up, so you'll just have to wonder.15:19
* mordred is on the edge of his seat15:19
efriedprometheanfire: In case it matters, the changes in question are:15:20
efriedksa: https://review.opendev.org/#/c/685042/15:20
efriedsdk: https://review.opendev.org/#/c/685999/15:20
prometheanfireok, lower-constraints are used to have something to test against for lower bounds testing and to actually set a requirement in the requirements.txt file15:21
prometheanfireso if it's not in their requirements.txt it won't help much15:22
mordredefried: based on your comment on the change ... the only reason I'm saying that the ksa change depends on the sdk change is that landing and releasing the ksa change before the sdk change will break sdk15:22
mordredbut - like - mostly just in the gate15:22
prometheanfirethe best we can do is just fix it separately and have a release note stating that x-interaction was fixed in release y15:22
efriedmordred: I thought if in the wild you applied the ksa fix without the sdk fix you would break your compute proxy15:22
mordredthe ksa change doesn't ACTUALLY depend on the sdk change from a real-world pov15:22
mordredno - in the wild if you did that it should be fine - except for creating server groups with policies15:23
efriedwell yeah :)15:23
mordredso - ok - it would break in the wild :)15:24
prometheanfirenot a good way to avoid that :(15:24
efriedIf that's the case, then probably the ksa reno should call that out and point to a bug15:24
mordredbut - I think it should be _fairly_ safe if we just land and release the sdk fix before we land and release he ksa one15:24
mordredso you'd have to do some explicit work to break yourself15:24
mordredefried: good point15:24
efriedyeah, for the most part it shouldn't happen, but it'd be nice to warn15:24
mordredI'll update the reno15:24
efriedcool, thanks mordred & prometheanfire, I feel better :)15:25
prometheanfire:D15:26
mordredefried: how does https://review.opendev.org/#/c/685042/ look?15:30
efriedmordred: It would be nice to link to *something* with more detail. Ideally a bug (story?) would be the thing, but if not, at least the change set(s).15:33
efriedbut otherwise, yeah, lgtm.15:33
mordredefried: you people with your actual bug/story tracking ability15:38
efriedI know, I'm sorry, I'm SORRY!15:38
mordredefried: do you think https://review.opendev.org/#/c/685999/ has enough info to be useful to link to?15:38
efriedmordred: yeah, I think that's good enough15:39
efriedI'm just thinking from an operator's perspective they need to be able to understand what they need to do.15:39
efriedand what will break if they don't15:39
mordred++15:39
efriedconversely, when something breaks, something they'll be able to recognize when they go back and read the release notes (for the first time)15:40
*** adriant has quit IRC15:59
*** tosky has quit IRC16:05
*** e0ne has quit IRC16:32
*** zbr|ruck has quit IRC16:39
*** zbr has joined #openstack-requirements16:51
*** efried has quit IRC17:56
*** zbr has quit IRC19:46
*** e0ne has joined #openstack-requirements19:50
openstackgerritMerged openstack/requirements stable/stein: update constraint for python-troveclient to new release 2.17.1  https://review.opendev.org/68661119:57
openstackgerritMerged openstack/requirements stable/queens: update constraint for oslo.messaging to new release 5.35.6  https://review.opendev.org/68661519:57
*** e0ne has quit IRC20:09
*** prometheanfire has quit IRC21:16
*** prometheanfire has joined #openstack-requirements21:17
*** tosky has joined #openstack-requirements21:31
*** tosky has quit IRC23:39

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