Friday, 2018-02-02

*** masayukig has quit IRC00:40
*** masayukig has joined #openstack-requirements00:41
prometheanfiredirk: tonyb: plskthnx https://review.openstack.org/54010501:47
tonybprometheanfire: done01:48
prometheanfirety01:48
*** derekjhyang has joined #openstack-requirements02:50
openstackgerritMerged openstack/requirements master: Update constraints for delayed Queens libs  https://review.openstack.org/54010504:28
prometheanfirewoot04:29
prometheanfirewe are now freezable04:29
*** derekjhyang has quit IRC05:23
*** derekjhyang has joined #openstack-requirements05:23
dirkNice05:51
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints  https://review.openstack.org/53855606:34
prometheanfiresmcginnis: python-blazarclient and python-kingbirdclient are being proposed by the bot, do I care? https://review.openstack.org/53855606:39
*** jpich has joined #openstack-requirements07:51
*** florianf has joined #openstack-requirements07:55
*** ralonsoh has joined #openstack-requirements08:15
openstackgerritMonty Taylor proposed openstack/requirements master: Update lower bound for openstacksdk to 0.11.2  https://review.openstack.org/54034311:16
*** jpich has quit IRC12:01
*** edmondsw has joined #openstack-requirements12:17
smcginnisprometheanfire: Hmm, not sure where the blazarclient one came from. I thought we had done that one several days ago.13:40
smcginnisprometheanfire: python-kingbirdclient must have been done by that team.13:40
smcginnisprometheanfire: I don't see the blazarclient proposal in the open reviews.13:42
openstackgerritQiming Teng proposed openstack/requirements master: Recap openstacksdk to 0.9.19  https://review.openstack.org/54038814:03
prometheanfiresmcginnis: ya, confused me too14:41
smcginnis¯\(°_o)/¯14:43
mordredprometheanfire: was just chatting with the release team about those two patches above ^^ - my preference would be to raise the lower bound since the constraints cap is already in place (so that non-gate and gate match each other) - what do you think?15:29
*** florianf has quit IRC15:37
prometheanfiremordred: what do you mean specifically?16:06
prometheanfiremordred: making the minimum change to what the current UC is?16:06
mordredprometheanfire: yah16:25
prometheanfiremordred: I don't think we'd like that :P16:26
prometheanfiremin bumps require specific reasoning for them16:27
mordredprometheanfire: oh - there is a specific reason - octavia-dashboard needs 0.11 - it contains features they need for the queens release16:27
johnsomIt's a very visible bug fix for health monitors to be exact.16:28
mordredprometheanfire: I'm guessing what I should have done is suggested we put a min-bump in with the constraints patch yesterday, yeah/16:29
prometheanfiremordred: probably16:30
prometheanfiremordred: I think it's too late at this point, o-sdk is used too many places (that'd cause re-releases for them too)16:31
mordredprometheanfire: kk. we can just keep it as a lesson learned for next time16:32
prometheanfiremordred: thanks :D16:32
johnsomprometheanfire So how do we handle it on the stable branch after release?16:32
prometheanfiremordred: we can keep the patch for master for after the freeze though16:32
johnsomThis fix: https://review.openstack.org/#/c/522962/ is needed or the dashboard will give users errors when they do anything with health monitors.16:33
prometheanfirejohnsom: These should be few and far between on stable branches, mainly masking known bad versions or in extreme adding a maximum version allowable for a package. We work to remove these caps as well. Raising effective minimums is only acceptable during Phase I, and only due to security issues.16:34
prometheanfirehttps://github.com/openstack/requirements/#global-requirements16:34
mordrednod16:35
johnsomYeah, that is my worry. We are effectively going out with a broken release because we didn't get an SDK patch out16:35
prometheanfirejohnsom: it's not a broken release, UC is what's tested against and what openstack states is supported16:35
mordredjohnsom: well - it's metadata is broken - most users when they install the release will get 0.11.2 of sdk so will get the error message upgrades16:35
johnsomprometheanfire Right now with the 0.9.x UC it's broken16:36
mordredthe thing that's incorrect is hat twe're telling people they *could* use an older version - and if they chose to do that it would be broken for them16:36
mordredjohnsom: yup. but UC bump landed16:36
mordredjohnsom: so you should be good on that front16:36
prometheanfireso 0.11.2 is what's broken?16:36
prometheanfireI'm confused16:37
prometheanfireI thought 0.11.2 is the UC16:37
mordredno - 0.9 is what's broken - if the UC was still pinned at 0.9 octavia-dashboard would be sad16:37
johnsomOh! Last time I looked UC didn't land. Ok, nevermind, I am fine!16:37
mordredbut the UC bumped, so it *should* be all good now16:37
prometheanfireright, but it's not :P16:37
prometheanfirejohnsom: lol, k16:37
mordred\o/16:37
*** ralonsoh has quit IRC18:30
*** edmondsw has quit IRC23:09

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