| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-02 | |||
| 11:22:48 | alex_xu | efried: I just reverted the patch, still want to base on that version sql to implement all your cases, but now, I'm waiting for jay, see what is his plan | |
| 11:44:15 | efried | alex_xu Does our database backend not support the ALL function? | |
| 11:46:30 | sileht | sahid, hi, I see your name around the code that introduces this issue: https://bugs.launchpad.net/nova/+bug/1729584 | |
| 11:46:31 | openstack | Launchpad bug 1729584 in OpenStack Compute (nova) "boot from volume + configdrive broken" [Undecided,New] | |
| 11:46:49 | sileht | sahid, it's just to keep you informed | |
| 11:50:50 | sahid | sileht: ok thanks | |
| 11:52:54 | sileht | sahid, I guess you fixed this bug: https://bugs.launchpad.net/nova/+bug/1524301 | |
| 11:52:55 | openstack | Launchpad bug 1524301 in OpenStack Compute (nova) "image_meta scsi model ignored " [Medium,Triaged] | |
| 11:53:46 | sileht | sahid, did you change the number of virtio-scsi disks from 6 to 256 ? | |
| 11:54:00 | sahid | sileht: yes exaclty, normally it should be marked as closed | |
| 11:54:15 | sileht | sahid, thx for that ! | |
| 11:54:23 | sileht | a two years old bug ! | |
| 11:54:37 | sahid | :) | |
| 11:54:40 | openstackgerrit | Chen proposed openstack/nova master: Trivial grammar fix https://review.openstack.org/517283 | |
| 11:54:56 | sahid | sileht: on the bug you reported you also proposed a fix which seems to be reasonable | |
| 11:55:02 | sahid | don't you want to send a patch? | |
| 11:56:46 | sileht | sahid, not really, I'm on holiday currently :) | |
| 11:58:21 | sahid | sileht: noted, i will do it for you so :) | |
| 11:58:44 | sahid | thanks for the report and investigation | |
| 12:28:21 | efried | jaypipes Good morning! | |
| 12:28:34 | efried | I think alex_xu had questions for ya. | |
| 12:28:54 | jaypipes | efried: morning. I need to caffeinate and breakfast. please leave q's and I'll get to them in about 30 mins. | |
| 12:33:05 | efried | dansmith If you have a little breathing room today, I would like to get your opinion on http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-11-01.log.html#t2017-11-01T14:25:13 | |
| 12:38:07 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Create specs directory for Rocky https://review.openstack.org/514101 | |
| 12:54:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Create specs directory for Rocky https://review.openstack.org/514101 | |
| 12:56:01 | openstackgerrit | jichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | |
| 13:22:29 | niraj_singh | efried:Is there any difference in request headers when we use service token? | |
| 13:22:50 | efried | niraj_singh I did actually see your question earlier, but didn't answer it because... I don't know :) | |
| 13:23:11 | niraj_singh | efried: ok nop. | |
| 13:23:47 | efried | niraj_singh I suppose you could try it out and see. We log the headers in the API log, I believe. | |
| 13:35:49 | jaypipes | efried: dansmith is in the air. :) | |
| 13:36:12 | efried | jaypipes Excuses. | |
| 13:41:21 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: instance mapping records exist even after the instances are archived https://review.openstack.org/515034 | |
| 13:49:11 | mriedem | stephenfin: bauzas: need a final +2 on this so we can start backports https://review.openstack.org/#/c/516395/ | |
| 13:50:13 | mriedem | lyarwood: can you go through these backports? https://review.openstack.org/#/q/topic:bug/1702454+status:open | |
| 13:51:48 | openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | |
| 13:52:26 | lyarwood | mriedem: Yeah sure, sorry for not keeping up with stable reviews recently btw, a few more weeks of tripleo and I'll be back full-time on nova, hopefully.. | |
| 13:52:50 | mriedem | np | |
| 13:54:41 | mriedem | nova team meeting in ~5 minutes | |
| 13:55:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove 'vendordata_driver' opt https://review.openstack.org/397835 | |
| 13:55:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Validate '[api] vendordata_providers' options https://review.openstack.org/513318 | |
| 13:56:00 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | |
| 13:59:38 | jaypipes | efried: you didn't push any changes to my refactor series yesterday or this morning did you? | |
| 13:59:46 | efried | jaypipes Nope. | |
| 14:00:11 | jaypipes | k, just checking... finally identified what the damn problem was with the py35 func tests... | |
| 14:01:01 | efried | Dict ordering? | |
| 14:01:18 | jaypipes | nope. | |
| 14:01:32 | efried | jaypipes nova meeting | |
| 14:09:00 | openstackgerrit | Hironori Shiina proposed openstack/nova master: [WIP] Ironic: Get IP address for volume connector from ports/portgroups https://review.openstack.org/517322 | |
| 14:09:54 | mriedem | bauzas: you can drop the -2 on this https://review.openstack.org/#/c/491000/ | |
| 14:21:30 | mriedem | johnthetubaguy: you'll be in sydney right? | |
| 14:36:56 | mriedem | johnthetubaguy: added the issue to L37 here: https://etherpad.openstack.org/p/sydney-cinder-nova-cp-multi-attach | |
| 14:36:58 | mriedem | ildikov: ^ | |
| 14:37:47 | johnthetubaguy | mriedem: oh, good call, I missed that when I reviewed it | |
| 14:38:08 | ildikov | mriedem: thanks | |
| 14:38:09 | johnthetubaguy | mriedem: given the timezone upside down ness, I wasn't going to try remote join | |
| 14:38:16 | mriedem | johnthetubaguy: stvnoyes was the one that actually brought it up, i just wrote the tempest test that verified the issue | |
| 14:38:30 | mriedem | johnthetubaguy: oh so you won't be there, i thought you were for some reason | |
| 14:38:31 | mriedem | alright | |
| 14:38:45 | johnthetubaguy | mriedem: sorry, watching sally's bump | |
| 14:38:50 | mriedem | ah right | |
| 14:39:11 | mriedem | i don't blame you | |
| 14:40:08 | ildikov | mriedem: I will try to sync up with jgriffith before the session | |
| 14:40:14 | johnthetubaguy | could be "marriage limiting" to do something different | |
| 14:40:31 | ildikov | mriedem: as we have options on how to handle this | |
| 14:46:36 | jaypipes | johnthetubaguy: oddly, "watching sally's bump" is a great Hipster band name. | |
| 14:46:48 | johnthetubaguy | jaypipes: heh | |
| 14:47:02 | jaypipes | johnthetubaguy: a band which could have a tremendous Tuba section. | |
| 15:01:29 | tssurya | mriedem : regarding the destroying of request specs records for archived instances ; I remember you telling me to do it similar to how instance_mappings are removed (https://review.openstack.org/#/c/515034/ is the review for removing instance_mappings), | |
| 15:02:16 | mriedem | tssurya: ok | |
| 15:02:22 | tssurya | however, regarding the case of request spec records, as we know there is an open review which destroys the records differently - | |
| 15:02:24 | tssurya | https://review.openstack.org/#/c/391060/ | |
| 15:02:41 | mriedem | yeah i don't really agree with sylvain's patch | |
| 15:02:44 | tssurya | I don't know if I should open a new review | |
| 15:02:48 | tssurya | for the same bug | |
| 15:03:17 | tssurya | so then there would be two reviews closing the same bug | |
| 15:03:25 | mriedem | because while he asserts that we recreate reqspecs later in the code if we don't find one, like during evacuate/live migrate/resize, the reqspec we create on the fly is often missing important details, and honestly that was all compat code from newton that is going to be removed anyway | |
| 15:03:32 | mriedem | i'm going to -1 his patch, i'll comment in there | |
| 15:03:59 | tssurya | mriedem : okay thanks! :) | |
| 15:04:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818 | |
| 15:04:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819 | |
| 15:04:37 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicated instance.(un)pause notification samples https://review.openstack.org/452820 | |
| 15:04:37 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate instance.power-off notification samples https://review.openstack.org/475860 | |
| 15:04:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate instance.interface_attach samples https://review.openstack.org/515740 | |
| 15:04:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate instance.live_migration notification samples https://review.openstack.org/515745 | |
| 15:04:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate instance.reboot notification samples https://review.openstack.org/515751 | |
| 15:07:30 | mriedem | done | |
| 15:08:00 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | |
| 15:08:01 | openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | |
| 15:08:01 | openstackgerrit | Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159 | |
| 15:08:02 | openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707 | |
| 15:08:02 | openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | |
| 15:08:03 | openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | |
| 15:08:13 | tssurya | mriedem : ack | |
| 15:10:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove instance.keypairs migration code https://review.openstack.org/517158 | |
| 15:13:16 | edleafe | mriedem: The legacy_find_hosts fix you have is something I fixed in a later patch in my series. https://review.openstack.org/#/c/510159/22/nova/scheduler/filter_scheduler.py@365 | |
| 15:13:37 | edleafe | mriedem: how about I move that fix earlier in the series, so that your fix isn't needed? | |
| 15:15:56 | edleafe | the problem is that the drivers shouldn't be messing with the return type. That should be done by the manager at the RPC boundary | |
| 15:24:24 | mriedem | edleafe: i think it should be fixed outside of your series | |
| 15:24:36 | mriedem | because it's busted for anyone picking this up today from master | |