Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-02
09:46:09 openstackgerrit guanzuoyu proposed openstack/nova master: Modify incorrect debug meaasge in _inject_data https://review.openstack.org/517227
10:27:19 niraj_singh is there any difference between request headers when we use service token??
10:53:59 efried alex_xu Greetings.
10:54:18 efried I haven't made it all the way through my gerrit emails yet; but is there anything we need to discuss?
10:55:54 efried alex_xu Now I've made it through the emails. Can we talk about https://review.openstack.org/#/c/514197/ ?
11:07:14 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273
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

Earlier   Later