| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-23 | |||
| 21:30:24 | openstackgerrit | Michael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module. https://review.openstack.org/557552 | |
| 21:30:25 | openstackgerrit | Michael Still proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | |
| 21:30:26 | openstackgerrit | Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | |
| 21:32:34 | arvindn0_ | efried: btw, GET /allocations/{instance_uuid} does not provide traits? | |
| 21:32:46 | arvindn0_ | that would have made it much easier...any plans to add this? | |
| 21:36:14 | efried | arvindn0_: I've proposed a similar "improvement" in the past (https://review.openstack.org/#/c/521875/), but the PTB deemed this unnecessary, expressing a preference for lots of smaller calls and reduced API breadth/complexity. | |
| 21:36:31 | efried | I should abandon that... | |
| 21:37:40 | arvindn0_ | cool...thanks the history helps so i dont spend time exploring that possibility | |
| 21:38:49 | efried | nod | |
| 21:40:22 | efried | arvindn0_: I suspect your biggest overhead is going to be building up the client. Once you have the comm channel to placement open, the individual calls should be very fast. | |
| 21:41:07 | efried | arvindn0_: ...which is why I asked if you had a resource tracker or scheduler client or any other way to get at an already-built SchedulerReportClient. mriedem, any way to get at those things from a filter without having to build afresh? | |
| 21:42:23 | arvindn0_ | i think the implementation hasnt been figured out...we could implement in the scheduler, so we dont need to recreate some clients...if we do it in the filter, then we have the overhead yes | |
| 21:43:22 | efried | gotcha | |
| 21:43:39 | efried | Well, I leave it in your capable hands. Let me know if further questions. Good luck! | |
| 21:46:27 | arvindn0_ | thanks! | |
| 23:19:17 | openstackgerrit | Merged openstack/nova stable/pike: libvirt: disconnect volume from host during detach https://review.openstack.org/560690 | |
| 23:46:47 | openstackgerrit | Zack Cornelius proposed openstack/nova-specs master: Libvirt file backed memory https://review.openstack.org/563704 | |
| #openstack-nova - 2018-04-24 | |||
| 00:02:11 | alex_xu | mriedem: yes, it is change the API behavior, do you know what happened now if I didn't specific boot_index? | |
| 00:02:23 | alex_xu | for now | |
| 00:04:18 | alex_xu | or I will check then when I reach to the office | |
| 00:36:58 | mriedem | alex_xu: yeah https://review.openstack.org/#/c/563732/ i have to update that, tomorrow | |
| 00:54:35 | openstackgerrit | Adam Harwell proposed openstack/nova stable/pike: Add support for listing hosts in cellv2 https://review.openstack.org/563797 | |
| 01:09:32 | openstackgerrit | Merged openstack/nova master: Add CPUWeigher https://review.openstack.org/379525 | |
| 01:09:41 | openstackgerrit | Merged openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | |
| 01:10:55 | openstackgerrit | Merged openstack/nova master: Move some tests into nova.tests.unit.notifications.objects.test_instance https://review.openstack.org/562793 | |
| 01:11:09 | melwitt | Kevin_Zheng: your blueprint was removed from the review runway today because all of the related code merged. please feel free to add comments to the log area at L144 if you have any comments or feedback about your experience with runways https://etherpad.openstack.org/p/nova-runways-rocky | |
| 01:11:53 | Kevin_Zheng | melwitt cool, I will | |
| 01:44:16 | gmann | alex_xu: testing in tempest too - https://review.openstack.org/#/c/563801/ | |
| 02:22:23 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Address nits in Iac23a574a05e62a99aab20ffce4265f4fb5a44d9 https://review.openstack.org/563802 | |
| 02:46:25 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | |
| 02:46:26 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | |
| 02:46:27 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | |
| 02:47:43 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Address nits in Iac23a574a05e62a99aab20ffce4265f4fb5a44d9 https://review.openstack.org/563802 | |
| 03:36:15 | openstackgerrit | Michael Still proposed openstack/nova master: Use os.rename, not mv. https://review.openstack.org/562463 | |
| 04:45:50 | openstackgerrit | Julia Kreger proposed openstack/nova master: Ironic: Lock the instance if early vif plug https://review.openstack.org/563714 | |
| 06:16:11 | pratik | deletion of snapshot created from volume attached to an instance is getting stuck in 'deleting' state with NFS backend. can anyone help? https://developer.openstack.org/api-ref/compute/#delete-assisted-volume-snapshot this API returns 204 response but not sure why it gets stuck | |
| 06:26:37 | openstackgerrit | XiaojueGuan proposed openstack/nova master: reduce complex if clause to nice or clause https://review.openstack.org/563834 | |
| 06:29:06 | kashyap | stephenfin: Morning | |
| 06:29:31 | kashyap | stephenfin: It's in the commit message :-) | |
| 06:30:51 | openstackgerrit | Merged openstack/nova master: Record the host info in EventReporter https://review.openstack.org/556746 | |
| 06:31:33 | kashyap | stephenfin: IOW: | |
| 06:31:36 | kashyap | The advertized NEXT_MIN_{LIBVIRT,QEMU} versions during | |
| 06:31:36 | kashyap | [quote] | |
| 06:31:37 | kashyap | weren't actually bumped for "Queens". So let's actually bump them for | |
| 06:31:37 | kashyap | "Pike" release were set to: libvirt 1.3.1 and QEMU 2.5.0 -- but they | |
| 06:31:40 | kashyap | the "Rocky" release. | |
| 06:31:42 | kashyap | [/quote] | |
| 06:31:45 | kashyap | (From the commit message) | |
| 07:10:58 | alex_xu | jaypipes, ericyoung, we miss this case in this thread http://lists.openstack.org/pipermail/openstack-dev/2018-April/129763.html | |
| 07:15:10 | kashyap | stephenfin: And for "Stein", we've picked the NEXT_MIN versions here (this is merged): https://review.openstack.org/#/c/558171/ | |
| 07:53:27 | ygl | anyone here can help me with my issue please | |
| 07:54:01 | ygl | i have created a vm using a heat stack. when I delete that stack, I want a user script to be trigerred automatically in the VM before its deltion | |
| 07:54:16 | ygl | is that possible in openstack as of now ? | |
| 08:06:54 | bauzas | ygl: looks like it's not a Nova feature | |
| 08:07:36 | bauzas | ygl: AFAIK | |
| 08:16:02 | stephenfin | kashyap: Ah, so it was a different commit. That's what caught me out :) | |
| 08:16:18 | kashyap | stephenfin: Maybe I should've mentioned it | |
| 08:16:26 | kashyap | stephenfin: Because it requires discussion on the Operators / Dev list | |
| 08:17:39 | stephenfin | kashyap: Ah, it's fine. +Wd now | |
| 08:18:01 | kashyap | stephenfin: Most excellent. I can build other clean-up patches on top of it | |
| 08:18:04 | kashyap | Thanks | |
| 08:18:21 | stephenfin | kashyap: Cool. Feel free to add me as a reviewer to those when you do | |
| 08:18:38 | openstackgerrit | sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168 | |
| 08:18:45 | openstack | Launchpad bug 1766208 in OpenStack Compute (nova) "Lift the restriction on choices for `cpu_model_extra_flags` config attribute" [Undecided,New] - Assigned to Kashyap Chamarthy (kashyapc) | |
| 08:18:45 | kashyap | Definitely. But before that I'm trying to get this out of the way: https://bugs.launchpad.net/nova/+bug/1766208 | |
| 08:25:01 | openstackgerrit | sahid proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | |
| 09:19:59 | Kevin_Zheng | alex_xu Thanks for the review, but Error instances could also have Ip, so I guess we should not skip cell0? | |
| 09:28:54 | alex_xu | Kevin_Zheng: cell0 means the instance didn't get right scheduling, so they shouldn't have IP I think | |
| 09:29:25 | Kevin_Zheng | let me think | |
| 09:29:32 | alex_xu | even if they go to a cell and fail, the IP should be released? | |
| 09:29:34 | Kevin_Zheng | error instance will still be cell1? | |
| 09:29:46 | alex_xu | Kevin_Zheng: yes | |
| 09:29:55 | Kevin_Zheng | No, Ip will still be on instances records | |
| 09:30:04 | alex_xu | Kevin_Zheng: one of case I'm sure is the pre-created ported | |
| 09:30:55 | Kevin_Zheng | then you should comment erlier :) | |
| 09:30:58 | alex_xu | Kevin_Zheng: if the instance turn to error after booting, they shouldn't be in cell0, and yes they has IP | |
| 09:31:10 | Kevin_Zheng | yeah, I got that | |
| 09:31:14 | alex_xu | Kevin_Zheng: yea, but i'm not -1 :) | |
| 09:32:00 | alex_xu | Kevin_Zheng: if that makes sense, I still think they should be separated patch | |
| 09:40:41 | Kevin_Zheng | alex_xu I will consider it :) | |
| 09:50:59 | openstackgerrit | XiaojueGuan proposed openstack/nova master: change complex if clause to nice or clause https://review.openstack.org/563872 | |
| 09:53:43 | openstackgerrit | Merged openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Rocky" https://review.openstack.org/558783 | |
| 10:17:21 | openstackgerrit | Merged openstack/nova master: Drop MIN_LIBVIRT_SET_ADMIN_PASSWD https://review.openstack.org/562312 | |
| 10:17:28 | openstackgerrit | Merged openstack/nova master: Remove vestigial system_metadata param from info_from_instance() https://review.openstack.org/561724 | |
| 10:56:45 | openstackgerrit | Merged openstack/nova master: Improve performance when list instances with IP filter https://review.openstack.org/539469 | |
| 11:23:33 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | |
| 12:08:40 | jaypipes | alex_xu: will respond on the ML thread, ok? | |
| 12:12:05 | kashyap | Goddamnit | |
| 12:12:09 | kashyap | The Gerrit UI | |
| 12:12:33 | kashyap | It _keeps_ jumping around. | |
| 12:13:37 | kashyap | No wonder I still like the blazing fast e-mail workflow. /me whips up the 'ncurses'-based 'Gertty' tool to see if it has gotten a bit less crashy | |
| 12:13:41 | sean-k-mooney | kashyap: matt found a workaround for that. in your preferences set render mode to slow | |
| 12:13:57 | kashyap | sean-k-mooney: Oh really? /me goes to look | |
| 12:13:59 | jaypipes | kashyap: I know. It's infuriating. | |
| 12:14:11 | kashyap | This just makes the throw the laptop out the fucking window | |
| 12:14:19 | jaypipes | sean-k-mooney: I have that set in my prefs and still happens occasionally. | |
| 12:14:28 | cdent | sean-k-mooney, kashyap the slow render mode helps some, but not all of it | |
| 12:14:32 | cdent | jinx | |
| 12:14:47 | sean-k-mooney | jaypipes: ya it seams to depend on gerrit version | |
| 12:15:19 | sean-k-mooney | there is a new gerrit ui based on javascitp on the latest realease called polygerrit | |