| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-09 | |||
| 16:00:48 | sfinucan | johnthetubaguy: I forget - are we killing the feature that uses these opts? https://review.openstack.org/#/c/323313/13/nova/conf/libvirt.py | |
| 16:01:11 | sfinucan | ...or was that something else | |
| 16:01:45 | pkoniszewski | avolkov: hey, are you around? | |
| 16:04:12 | mriedem | sfinucan: this? https://review.openstack.org/#/c/429798 | |
| 16:04:31 | sfinucan | mriedem: Aye, that's the one | |
| 16:04:59 | avolkov | pkoniszewski hi | |
| 16:05:35 | pkoniszewski | avolkov: could you briefly describe what you have changed in the lm claims patch? | |
| 16:06:19 | avolkov | I just rebased, main change is service version from 16 to 17, because of placement api | |
| 16:06:28 | pkoniszewski | ah, okay | |
| 16:06:49 | pkoniszewski | asking whether i need to change something in my local changes | |
| 16:06:54 | pkoniszewski | still working on rollback issue | |
| 16:11:46 | pkoniszewski | avolkov: thanks a lot of this! will save me some time :) | |
| 16:11:54 | pkoniszewski | for this | |
| 16:12:36 | mriedem | sdague: i've got all of those backports up https://review.openstack.org/#/q/Ice78a0982bcce491f0c9690903ed2c6b6aaab1be,n,z | |
| 16:15:46 | mriedem | ildikov: smcginnis: for the nova/cinder meeting today i mostly wanted to just talk about PTG planning | |
| 16:16:00 | smcginnis | mriedem: +1 | |
| 16:16:04 | mriedem | we know the local.conf passthrough in infra is already on the infra agenda | |
| 16:16:10 | mriedem | for testing with cinder v3 | |
| 16:16:26 | ildikov | mriedem: sounds good | |
| 16:16:47 | ildikov | mriedem: I talked to the infra guys briefly about that patch, they were very positive about the progress | |
| 16:18:01 | mriedem | johnthetubaguy: on https://review.openstack.org/#/c/429798/ is the live migration progress timeout mostly just an issue for live block migrations? | |
| 16:18:14 | pkoniszewski | mriedem: for all of them | |
| 16:19:02 | ildikov | mriedem: can we at some point look into the remove check_attach patch too? | |
| 16:19:21 | pkoniszewski | its just wrong, doesnt matter whether it is block live migration or live migration | |
| 16:19:25 | openstackgerrit | John Garbutt proposed openstack/nova master: Placement api: set custom json_error_formatter in allocations https://review.openstack.org/395194 | |
| 16:19:52 | ildikov | mriedem: I need to figure out one notification test failure, but it would be great to have the other parts figured out too so we can merge it and finally move on | |
| 16:20:02 | sdague | mriedem: bam! | |
| 16:20:13 | mriedem | ildikov: yeah i was just looking it up | |
| 16:20:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | |
| 16:20:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211 | |
| 16:20:33 | ildikov | mriedem: cool, tnx | |
| 16:20:59 | mriedem | sdague: thanks | |
| 16:25:25 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |
| 16:26:47 | johnthetubaguy | mriedem: I think its technically a problem for all live-migrations | |
| 16:26:55 | johnthetubaguy | mriedem: but the block migrations have it worse | |
| 16:27:10 | johnthetubaguy | mriedem: heh, what pkoniszewski said | |
| 16:29:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor InstancePayload creation https://review.openstack.org/418488 | |
| 16:29:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | |
| 16:29:16 | mriedem | johnthetubaguy: ok, +W | |
| 16:29:25 | mriedem | johnthetubaguy: i've also sent a heads up to the operators list | |
| 16:29:46 | johnthetubaguy | mriedem: awesome, thanks | |
| 16:29:58 | johnthetubaguy | mriedem: I can get the backport in the queue to speed that up | |
| 16:30:47 | mriedem | johnthetubaguy: already done | |
| 16:31:06 | johnthetubaguy | oh. cool, thanks | |
| 16:39:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Rework '_get_cpu_topology_constraints' https://review.openstack.org/407173 | |
| 16:39:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Rework get_number_of_serial_ports https://review.openstack.org/407174 | |
| 16:39:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Log when conflicting image/flavor metadata found https://review.openstack.org/407175 | |
| 16:48:12 | auggy | mriedem: if someone else wanted to do the i18n liaison they could. There really wasn't anything to do. | |
| 16:48:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add device_id when creating ports https://review.openstack.org/431545 | |
| 16:56:52 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |
| 17:01:04 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Reserve NUMA nodes with PCI devices attached https://review.openstack.org/364468 | |
| 17:03:35 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Fix additional mistakes in the NUMA spec document https://review.openstack.org/323252 | |
| 17:04:55 | BobBall | johnthetubaguy: sorry - not sure how I missed the comment. Investigating. | |
| 17:05:27 | BobBall | johnthetubaguy: Citrix CI looks health to me? http://ci-watch.tintri.com/project?project=nova | |
| 17:05:52 | johnthetubaguy | BobBall: OK, strange. | |
| 17:08:57 | openstackgerrit | John Garbutt proposed openstack/nova master: Use admin context to create neutron ports https://review.openstack.org/357540 | |
| 17:12:06 | openstackgerrit | John Garbutt proposed openstack/nova master: Use admin context to create neutron ports https://review.openstack.org/357540 | |
| 17:14:06 | openstackgerrit | John Garbutt proposed openstack/nova master: Use admin context to create neutron ports https://review.openstack.org/357540 | |
| 17:37:56 | oomichi | johnthetubaguy: hi, can you take a look at https://review.openstack.org/#/c/389338 again ? I replied your question | |
| 17:42:14 | openstackgerrit | Maxim Nestratov proposed openstack/nova-specs master: libvirt: reintroduce runtime image type detection https://review.openstack.org/363547 | |
| 17:43:22 | openstackgerrit | John Garbutt proposed openstack/nova master: Assume neutron port_binding extensions enabled https://review.openstack.org/357726 | |
| 17:44:57 | openstackgerrit | John Garbutt proposed openstack/nova master: Use admin context to create neutron ports https://review.openstack.org/357540 | |
| 17:45:59 | openstackgerrit | melanie witt proposed openstack/nova master: Request zero root disk for boot-from-volume instances https://review.openstack.org/428481 | |
| 17:46:00 | openstackgerrit | melanie witt proposed openstack/nova master: Claim and report zero root disk for boot-from-volume instances https://review.openstack.org/428505 | |
| 17:53:00 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: Prepare for using standard python tests https://review.openstack.org/429391 | |
| 17:54:11 | cdent | jaypipes, edleafe: I'm on PTO tomorrow. If one of you is feeling inspired on the resource providers update front, please do one. | |
| 17:54:49 | jaypipes | cdent: unfortunately, I won't have the time to do it :( | |
| 17:54:56 | jaypipes | edleafe: would you have the bandwidth? | |
| 17:55:45 | cdent | jaypipes: no worries. I was hoping to do it this evening, but getting https://review.openstack.org/#/c/421846/ ready used up all my capacity for processing | |
| 17:55:46 | dansmith | we should merge this now that pike is open, before we land any other migrations: https://review.openstack.org/#/c/427900/ | |
| 17:56:00 | edleafe | jaypipes: cdent: yes | |
| 17:56:06 | cdent | edleafe++ | |
| 17:56:20 | jaypipes | edleafe: thank you Ed, I appreciate that very much. | |
| 17:56:23 | cdent | thanks | |
| 17:57:16 | cdent | I'm offf to birmingham to see my son before he heads off to southeast asia for a few months. He'll probably end up deciding not to come back to the western world. (I wouldn't) | |
| 17:59:06 | edleafe | jaypipes: Anything in particular besides https://review.openstack.org/#/c/404472/ ? | |
| 18:00:35 | jaypipes | edleafe: not from me, no. I need to get that rebased, along with the entire nested resource providers series. | |
| 18:00:49 | jaypipes | edleafe: also, need to work on traits bp/spec and PoC reviews before PTG | |
| 18:01:08 | edleafe | jaypipes: roger that | |
| 18:01:24 | cdent | edleafe: my usual strategy (if you're talking about the update email) is look for patches from the usual suspects that appear to be related, and then look at any pending specs, and then see what bugs tagged placement or scheduler are being worked on | |
| 18:02:38 | mriedem | johnthetubaguy: if you go through the multiattach nova spec again before the ptg, we should maybe highlight some sticky points from that for discussion | |
| 18:02:45 | mriedem | like i think evacuate, upgrade, | |
| 18:02:58 | mriedem | something about changing status on the attachment from the nova side which i disagreed with | |
| 18:04:06 | johnthetubaguy | mriedem: thats a good idea, get a list of sticking point | |
| 18:05:56 | mriedem | sdague: i can use your local_conf builder in project-config now right? it just won't do anything until the d-g changes are in | |
| 18:09:42 | cfriesen | can someone explain why we need to check "if 'id' in self" in objects/flavor.py? Do we not populate the "id" field when retrieving from the api DB? | |
| 18:10:35 | mriedem | has to do with the api db lookup doesn't it? | |
| 18:12:43 | melwitt | id gets generated only after a create(). so if a flavor object wasn't looked up and hasn't yet been create()'d, it won't have 'id' | |
| 18:13:10 | dansmith | all of this needs to diaf now | |
| 18:13:24 | dansmith | it's long since unused | |
| 18:13:37 | cfriesen | dansmith: I'm in newton | |
| 18:13:47 | mriedem | but...when i upgrade directly from mitaka to pike...i might need this to work dansmith | |
| 18:14:02 | dansmith | mriedem: oh well in that case, I'll kill it right now | |
| 18:14:13 | mriedem | cfriesen: melwitt answered your question | |
| 18:14:21 | mriedem | cfriesen: and dansmith is talking about killing it in pike | |
| 18:14:41 | cfriesen | melwitt: thanks. yes, killing in pike is fine. | |
| 18:15:24 | mriedem | well i know what dan is going to be doing later today | |
| 18:16:30 | dansmith | s/later today/right fucking now/ | |
| 18:30:53 | openstackgerrit | Maciej Szankin proposed openstack/nova master: A demo script for updating resource inventory https://review.openstack.org/382613 | |
| 18:51:36 | mriedem | raj_singh: johnthetubaguy: fyi https://review.openstack.org/#/c/431713/ | |