| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-31 | |||
| 16:13:58 | mriedem | maybe, whatever it is will go into the SchedulerLimits object https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L774 | |
| 16:14:20 | tssurya | mriedem: looking | |
| 16:14:35 | mriedem | tssurya: just the same stuff i said in irc, but with detailed links | |
| 16:15:47 | oomichi | alex_xu: are you still online now? happy if you take a look at https://review.openstack.org/#/c/571335 | |
| 16:19:19 | melwitt | mriedem: I was thinking queens release will be a minor version bump because of this, do you agree? https://git.openstack.org/cgit/openstack/nova/commit/?h=stable/queens&id=45331d3ea185d08687e9b24e716f3d5411210874 | |
| 16:21:07 | mriedem | melwitt: umm, | |
| 16:21:29 | mriedem | i personally don't really see the need for a minor version bump, i don't know exactly what that will signal since we already have a release note | |
| 16:21:37 | melwitt | api change? I wasn't sure | |
| 16:21:51 | mriedem | about the only time i've seen stable branch minor version bumps is when we have to adjust dependencies, | |
| 16:21:52 | melwitt | okay | |
| 16:21:56 | mriedem | or maybe with a new db migration | |
| 16:22:04 | mriedem | the api change is to fix broken behavior | |
| 16:22:13 | mriedem | so 409 rather than screw up your data | |
| 16:23:00 | melwitt | okay, I've got the bugfix version bump proposed at https://review.openstack.org/571494 | |
| 16:24:25 | mriedem | i'd like to get https://review.openstack.org/#/c/571425/ in if we can | |
| 16:24:28 | mriedem | the patch below it needs a +W | |
| 16:24:37 | mriedem | that was related to the nova-lvm regression we backported | |
| 16:24:47 | mriedem | bauzas: can you hit this? https://review.openstack.org/#/c/571424/1 | |
| 16:24:56 | melwitt | k | |
| 16:30:50 | melwitt | mriedem: will do similar for pike and ocata. we also have this for pike and ocata that just need +W https://review.openstack.org/561850 | |
| 16:39:38 | melwitt | release proposed for pike https://review.openstack.org/571521 and ocata https://review.openstack.org/571522 | |
| 16:40:49 | mriedem | jaypipes: bauzas: thanks for the comments on the heal_allocations change, i've replied https://review.openstack.org/#/c/565886/ | |
| 16:41:33 | mriedem | melwitt: ok i'll look at those backpots | |
| 16:41:34 | mriedem | *ports | |
| 16:42:20 | melwitt | thanks | |
| 16:42:25 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'numa-aware-vswitches' spec https://review.openstack.org/541290 | |
| 16:51:10 | mriedem | melwitt: +W on both of those backports | |
| 16:51:29 | melwitt | sweet thanks | |
| 16:52:23 | melwitt | need to round up some reviews for the skip fetching virtual size patches | |
| 16:52:55 | melwitt | something in this one isn't passing unit test jobs tho https://review.openstack.org/571432 | |
| 16:55:14 | mriedem | yeah i already talked with lee about it | |
| 16:55:35 | mriedem | i can call in stable maint core ringers if needed on the rest | |
| 16:55:46 | mriedem | i.e. mtreinish | |
| 17:02:40 | jaypipes | mriedem: "The missing consumer records" <-- worst Nancy Drew title ever. | |
| 17:02:40 | mriedem | easy libvirt driver change for the trusted vf series https://review.openstack.org/#/c/458514/ | |
| 17:02:58 | mriedem | jaypipes: it was a dark and stormy night and the consumer records were....MISSING | |
| 17:03:01 | mriedem | duh duh duh | |
| 17:03:17 | jaypipes | :) | |
| 17:35:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Match ComputeNode.uuid to ironic node uuid in RT https://review.openstack.org/571535 | |
| 17:41:21 | mriedem | is artom around today? | |
| 17:46:10 | melwitt | he was around earlier this morning but logged off about an hour ago | |
| 17:46:10 | openstackgerrit | karim proposed openstack/nova master: Handle rebuild of instances with image traits https://review.openstack.org/571539 | |
| 17:55:45 | openstackgerrit | Matt Riedemann proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344 | |
| 17:55:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | |
| 17:57:11 | mriedem | melwitt: ok i saw you +2ed that trusted vf patch, the rest of the series has my +2, and the next 2 are pretty easy as well | |
| 17:57:24 | mriedem | the rest of the series is all pretty easy actually | |
| 17:57:40 | mriedem | it's all just plumbing | |
| 17:58:03 | melwitt | coolness, I'll go through the rest of it later | |
| 18:24:14 | openstackgerrit | Merged openstack/nova stable/pike: Avoid showing password in log https://review.openstack.org/561850 | |
| 18:24:54 | openstackgerrit | karim proposed openstack/nova master: Handle rebuild of instances with image traits https://review.openstack.org/571539 | |
| 18:48:26 | openstackgerrit | Merged openstack/nova stable/queens: libvirt: handle DiskNotFound during update_available_resource https://review.openstack.org/571424 | |
| 18:57:33 | mriedem | esberglu: you should add the on-demand volume check job to https://wiki.openstack.org/wiki/ThirdPartySystems/IBM_PowerVM_CI | |
| 18:57:40 | mriedem | but +W on https://review.openstack.org/#/c/526094/ | |
| 18:58:26 | esberglu | mriedem: ack. Thanks for the review! | |
| 19:01:00 | mriedem | melwitt: with ^ the powervm series can be taken out of the runway | |
| 19:01:32 | melwitt | awesome | |
| 19:02:24 | melwitt | so next is going to be cert validation or cpu features as traits ... cert validation recently landed the spec amendment so should be good to go. thoughts? | |
| 19:06:22 | cfriesen | eandersson: you asked earlier about CPU pinning. In upstream the only way to see it is in the nova-compute logs or by looking at the DB. | |
| 19:07:19 | cfriesen | eandersson: internally we found it useful information and added a way to get it via the API, but never created a spec/blueprint for it. | |
| 19:20:36 | mriedem | melwitt: yes to cert validation | |
| 19:20:55 | melwitt | thanks | |
| 19:21:23 | mriedem | cfriesen: to which API? | |
| 19:24:59 | dansmith | I would like to see cert validation get another go and hopefully get finished | |
| 19:26:19 | melwitt | me too. thanks for the input, will move that into the runway next | |
| 19:29:21 | melwitt | mriedem: I'm working on getting the osc queens + novaclient queens stuff working with these patches https://review.openstack.org/#/q/topic:openstackclient-novaclient+(status:open+OR+status:merged) but from what I understand, nothing in the gate will verify those go together | |
| 19:30:29 | melwitt | I was thinking I might need to do some DNM patches to devstack to pull osc from git stable/queens and then a DNM revert of this https://review.openstack.org/538541 in grenade with depends-on to do it. would that work? is there a better way? | |
| 19:34:13 | openstackgerrit | Dan Smith proposed openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696 | |
| 19:38:35 | mriedem | melwitt: your question is breaking my brain | |
| 19:39:08 | mriedem | you want to test osc 3.14.2 with novaclient 10.1.0? | |
| 19:39:29 | mriedem | in queens, novaclient 10.x broke grenade and horizon, which got fixed in queens, | |
| 19:39:34 | melwitt | sorry ... that was too low-level a way to ask. yes, but osc 3.14.2 is not released yet, I was hoping to be able to verify it solves the problem before landing it if possible | |
| 19:39:40 | mriedem | the only remaining things that i knew of from the 10.x fallout was osc | |
| 19:40:22 | melwitt | yeah, that's my understanding too. I think it will work but realized I don't think anything in the gate run on the constraint update patch will actually run anything with those two together to make super sure it works | |
| 19:42:12 | mriedem | so you're thinking revert https://review.openstack.org/#/c/538541/ on stable/queens and depend on https://review.openstack.org/#/c/571540/ to get novaclient 10.1.0 so osc in grenade will use the osc proxy code that used to rely on novaclient, | |
| 19:42:34 | mriedem | i think the problem with that is the grenade revert won't run with the osc fix until that's released in https://review.openstack.org/#/c/571537/ | |
| 19:42:59 | mriedem | i think you can hack around that, | |
| 19:43:11 | mriedem | by depending on a stable/queens devstack patch that installs osc from stable/queens source | |
| 19:43:19 | melwitt | yeah. I was thinking I could make that happen tho with a devstack DNM patch that does LIBS_FROM_GIT=python-openstackclient on branch stable/queens and then depends-on that | |
| 19:43:50 | mriedem | so you'd have something like, queens grenade revert -> depends-on -> devstack queens osc from source -> depends-on -> novaclient constraints update https://review.openstack.org/#/c/571540/ | |
| 19:43:51 | melwitt | yeah exactly. just wanted to run it by you since your depends-on patch-verification-fu is strong | |
| 19:44:12 | mriedem | that's what i'd try i guess | |
| 19:44:36 | mriedem | OR | |
| 19:44:45 | mriedem | just get a queens devstack local and kick the tires | |
| 19:45:57 | melwitt | seeing something pass in the gate is more reassuring since I might screw up something in devstack somehow. but point taken | |
| 19:48:21 | melwitt | and also other people reviewing the changes can see it work, otherwise it's just "I tested it in devstack, you can trust me *wink*" | |
| 19:48:42 | mriedem | trust by verify? | |
| 19:48:47 | mriedem | i didn't know you were so into reagan | |
| 19:55:32 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923 | |
| 20:00:04 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923 | |
| 20:02:01 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-virtual-interfaces REST API https://review.openstack.org/569923 | |
| 20:02:06 | eandersson | cfriesen, I see - yea was thinking of doing the same internally | |
| 20:02:33 | mriedem | eandersson: or, OR, you could work with the upstream community on digging ourself out of your hole | |
| 20:03:06 | mriedem | *yourself | |
| 20:04:35 | eandersson | haha yea that would probably be the more sensable thing to do =] | |
| 20:12:24 | mriedem | i'm trying hard to not make a starlingx comment here | |
| 20:13:26 | mriedem | if you called it rock-n-roll-racing-x then i could maybe buy into it | |
| 20:19:08 | cfriesen | mriedem: i'm probably corporately obligated to keep my mouth shut. :) | |
| 20:21:41 | mriedem | heh | |
| 20:21:50 | mriedem | it's not like huawei doesn't make changes | |
| 20:21:53 | cfriesen | mriedem: sorry, missed your earlier question. I was referring to providing CPU pinning (and hugepages) consumption data to an admin user via the API. | |