| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-06 | |||
| 12:06:57 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Avoid deleting allocations for instances being built https://review.openstack.org/517879 | |
| 12:09:16 | mriedem | johnthetubaguy: want to take a look at these backports? https://review.openstack.org/#/c/517876/ https://review.openstack.org/#/c/517879/ | |
| 12:09:19 | mriedem | actually pretty nasty issue | |
| 12:11:36 | johnthetubaguy | I was actually worried about the code around https://review.openstack.org/#/c/517879/, we had a DB outage that seemed to trigger a loss of all allocations, but I wasn't 100% who did it | |
| 12:16:32 | mriedem | your old friend the RT | |
| 12:16:33 | mriedem | that's who | |
| 12:39:11 | efried | niraj_singh That stuff looks okay - but if it's truly in your cinder.conf, it won't be doing you any good. It needs to be in your compute conf. | |
| 12:41:49 | niraj_singh | efried:i am working on service token for cinder to nova interaction so is it necessary to put these conf options in compute conf? | |
| 12:42:29 | efried | niraj_singh I don't know what's needed from the cinder side tbh, but I know for sure that you need this stuff on the nova side. | |
| 12:43:00 | niraj_singh | efried: thanks. i gonna try this. :) | |
| 12:43:06 | efried | niraj_singh Your issue was that you were doing some kind of long-running operation that was talking *from* nova *to* cinder, right? | |
| 12:43:39 | efried | niraj_singh That's what these service tokens were designed to handle from the nova side. | |
| 12:44:23 | niraj_singh | efried: NO thats what i telling you, i am doing operation that is talking *from* cinder to nova. :) | |
| 12:45:09 | efried | niraj_singh Oh, well I have no idea if cinder implemented service tokens the same way (or at all). | |
| 12:45:33 | niraj_singh | efrid: i am implementing service token for cinder. As it is already implemented in nova thats why i asked you that am i missing something or any suggession from your side. | |
| 12:46:10 | niraj_singh | efried: nop. | |
| 12:46:38 | openstackgerrit | edan david proposed openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/505613 | |
| 12:49:18 | efried | niraj_singh Have you looked at the old specs? | |
| 12:51:33 | efried | niraj_singh ocata: https://review.openstack.org/387711 pike: https://review.openstack.org/426933 | |
| 12:53:26 | efried | niraj_singh And the keystone side: https://specs.openstack.org/openstack/keystone-specs/specs/keystone/ocata/allow-expired.html | |
| 12:54:06 | efried | So perhaps you could get better help in #openstack-keystone (if they're not all in Sydney, sleeping) | |
| 12:55:41 | niraj_singh | efried: ok i am looking into the links you sent. And i will also try to connect #openstack-keyston. thanks for help :) | |
| 12:55:51 | efried | niraj_singh Good luck! | |
| 13:03:55 | efried | Morning jaypipes! | |
| 13:04:17 | jaypipes | efried: mornin. I am yet to caffeinate. bear with me please. | |
| 13:04:37 | efried | jaypipes FYI, taking my youngest to get fillings this a.m. (cause my wife can't handle watching him get nitrous) so will be out for a couple hours. | |
| 13:04:45 | efried | jaypipes ^ will give you plenty of time to caffeinate. | |
| 13:05:03 | jaypipes | efried: heh, sounds good. | |
| 13:57:34 | openstackgerrit | Elod Illes proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 14:31:26 | moshele | stephenfin, jaypipes: hi can you review https://review.openstack.org/#/c/505613/ ? | |
| 14:32:21 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif https://review.openstack.org/422512 | |
| 14:37:11 | jaypipes | moshele: done | |
| 14:37:22 | jaypipes | moshele: fix that up and I'm +2 | |
| 14:38:47 | openstackgerrit | Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | |
| 14:41:39 | moshele | jaypipes: thanks :) will fix | |
| 14:58:37 | Lirt | Hello guys, can I have one small question about default_schedule_zone an AZs? | |
| 15:01:17 | openstackgerrit | edan david proposed openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/505613 | |
| 15:01:28 | jaypipes | Lirt: FYI, it's OpenStack summit week, so this channel is particularly vacant this week... also, for usage questions, it's best to ask questions on the openstack@ mailing list. | |
| 15:02:37 | jaypipes | moshele: done | |
| 15:02:59 | moshele | jaypipes: thanks :) | |
| 15:03:03 | jaypipes | np :) | |
| 15:03:32 | Lirt | Ah OK, I won't interrupt and use mailing list. Yeah I saw new uploads on Openstack Youtube channel so I will watch it later :-) | |
| 15:04:31 | efried | Ye gods, that was awful. | |
| 15:05:37 | openstackgerrit | Elod Illes proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 15:18:51 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819 | |
| 15:18:51 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicated instance.(un)pause notification samples https://review.openstack.org/452820 | |
| 15:18:52 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate instance.power-off notification samples https://review.openstack.org/475860 | |
| 15:18:53 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate instance.interface_attach samples https://review.openstack.org/515740 | |
| 15:18:53 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate instance.live_migration notification samples https://review.openstack.org/515745 | |
| 15:18:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate instance.reboot notification samples https://review.openstack.org/515751 | |
| 15:20:32 | gibi | gmann: hi! I renamed the payloads dir to common_payloads as you suggested in https://review.openstack.org/#/q/topic:refactor-notification-samples | |
| 15:42:16 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: reno for notification-transformation-pike https://review.openstack.org/518018 | |
| 15:56:55 | efried | Hey jaypipes, would you be willing to tackle a PowerVM review this week? https://review.openstack.org/409404 | |
| 16:14:24 | openstackgerrit | Mike Lowe proposed openstack/nova master: Take no chance an assigned disk device address will be overwritten during migration https://review.openstack.org/518022 | |
| 17:06:30 | efried | Thanks jaypipes! | |
| 17:09:16 | efried | gibi If you're still good with https://review.openstack.org/#/c/515151/ could I get a +Wutwut? | |
| 17:16:38 | gibi | efried: yepp, still looks good +wd | |
| 17:41:51 | efried | gibi Thanks! | |
| 19:06:15 | openstackgerrit | Merged openstack/nova master: Parse granular resources/traits from extra_specs https://review.openstack.org/515151 | |
| 19:06:22 | openstackgerrit | Merged openstack/nova master: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223 | |
| 19:18:34 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404 | |
| 19:46:31 | efried | jaypipes How can I best be of service this afternoon? If nothing else calls, I'll start writing some functional tests for GET /allocation_candidates - the remainder of the granular work is going to be queued behind the refactor series anyway. | |
| 19:49:16 | openstackgerrit | Eric Fried proposed openstack/nova master: ResourceRequest.to_querystring() https://review.openstack.org/515811 | |
| 19:51:05 | jaypipes | efried: func tests for the allocation candidates would be great, thanks. FYI, I'm kinda in and out doing chores and helping my wife with her rental property this afternoon. | |
| 19:52:39 | efried | ack | |
| 19:56:53 | crlb | Hello melwitt, are you there? | |
| 20:01:26 | jaypipes | crlb: melwitt is in Sydney | |
| 20:04:31 | crlb | Hi jaypipes, Thank you. I was having trouble with git review after amending a previous review. I'm doing some reading now. May I trouble you if I still have questions? | |
| 20:04:44 | jaypipes | crlb: sure, np | |
| 20:05:27 | crlb | I'm being called away, so maybe 1 to 2 hours from now. | |
| 20:10:19 | openstackgerrit | Merged openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 20:23:34 | openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Granular requests to get_allocation_candidates https://review.openstack.org/515811 | |
| 20:24:00 | efried | jaypipes Since the dependency was going to be in place anyway, it was trivial to change this ^ to just do the call with the new syntax as well. | |
| 21:46:57 | mriedem | o/ | |
| 21:48:06 | mriedem | sdague: can you hit https://review.openstack.org/#/c/517879/ and https://review.openstack.org/#/c/517876/ on stable/pike please? pretty nasty bugs. | |
| 21:51:15 | sdague | mriedem done | |
| 21:51:33 | sdague | hope sydney is treating you well, and you've managed a kangaroo steak | |
| 21:53:07 | mriedem | sdague: thanks. no kangaroo steak yet - actually only interested in sausage form. | |
| 21:53:20 | mriedem | i've been warned that the steak is super easy to overcook | |
| 21:54:49 | sdague | yeh, you have to eat it rare, medium rare at most, otherwise it hardens up | |
| 21:58:14 | mriedem | i did have some tim tams | |
| 21:58:25 | mriedem | and have experienced their oversized coinage | |
| 22:12:30 | efried | sdague You're not in Sydney? | |
| 22:15:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle InstanceNotFound when setting password via metadata https://review.openstack.org/472933 | |
| 22:18:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add error message on metadata API https://review.openstack.org/517152 | |
| 22:35:54 | mriedem | lyarwood: want to take a look at these backports when you get a chance? https://review.openstack.org/#/q/I29fac80d08baf64bf69e54cf673e55123174de2a | |
| 22:36:01 | mriedem | libvirt resize booboo with shared storage | |
| 22:36:08 | mriedem | red hat might care about that for ceph, maybe a little | |
| 22:45:46 | openstackgerrit | C Leavett-Brown proposed openstack/nova master: Modify nova routing table to process null requests https://review.openstack.org/518119 | |
| 23:26:18 | openstackgerrit | Eric Fried proposed openstack/nova master: Extract allocation candidates functional tests https://review.openstack.org/518123 | |
| 23:26:19 | openstackgerrit | Eric Fried proposed openstack/nova master: Refactor test_allocation_candidates https://review.openstack.org/518124 | |
| 23:26:41 | efried | jaypipes ^ No new tests yet; just a 33% reduction in LOC (so I can see an entire test on one screen (sometimes)) | |
| 23:27:20 | jaypipes | efried: reviewing your get_by_requests() patch now.. | |
| 23:27:29 | efried | jaypipes Cool. | |
| 23:27:41 | jaypipes | efried: couple little niggles, but overall excellent work. | |
| 23:27:47 | efried | jaypipes Thanks :) | |
| 23:28:45 | jaypipes | efried: k, done with review. | |
| 23:29:44 | efried | jaypipes I'd be happy to do that rebase; I'm now used to refactoring these tests. | |
| 23:30:05 | jaypipes | efried: no worries, I can do the rebasing. I'm also quite used to it by now ;) | |
| 23:41:30 | openstackgerrit | Eric Fried proposed openstack/nova master: placement: AllocCands.get_by_{filters => requests} https://review.openstack.org/514092 | |