| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-01 | |||
| 22:08:09 | efried | mriedem: tell it to review 666666 | |
| 22:11:42 | openstackgerrit | Eric Fried proposed openstack/nova master: Get rid of args to RBDDriver.__init__() https://review.opendev.org/668564 | |
| 22:48:22 | sean-k-mooney | aspiers: have you seen https://review.opendev.org/#/c/666769/1 yet | |
| 22:48:28 | sean-k-mooney | efried: ^ | |
| 22:50:58 | sean-k-mooney | i need to rereview it after fully rereading the sev spec. while not contradictory i think it makes more change then required at least for a v1 | |
| 23:11:51 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Set COMPUTE_STATUS_DISABLED trait from update_provider_tree flow https://review.opendev.org/668565 | |
| #openstack-nova - 2019-07-02 | |||
| 00:25:50 | openstackgerrit | Merged openstack/nova stable/queens: Fix live-migration when glance image deleted https://review.opendev.org/662154 | |
| 01:20:12 | openstackgerrit | Merged openstack/nova master: Drop pre-cinder 3.44 version compatibility https://review.opendev.org/621061 | |
| 01:20:20 | openstackgerrit | Merged openstack/nova master: Remove mox in virt/test_block_device.py https://review.opendev.org/566153 | |
| 01:36:02 | openstackgerrit | pengyuesheng proposed openstack/python-novaclient master: Add Python 3 Train unit tests https://review.opendev.org/668395 | |
| 01:52:31 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 02:07:02 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Add a guide to add a new microversion support https://review.opendev.org/667002 | |
| 02:09:14 | openstackgerrit | ya.wang proposed openstack/nova-specs master: Re-propose the spec to allow specifying a list of CPU models https://review.opendev.org/642030 | |
| 03:00:22 | openstackgerrit | Merged openstack/nova stable/queens: Move get_pci_mapping_for_migration to MigrationContext https://review.opendev.org/661571 | |
| 03:00:31 | openstackgerrit | Merged openstack/nova stable/queens: Allow driver to properly unplug VIFs on destination on confirm resize https://review.opendev.org/661572 | |
| 05:16:15 | openstackgerrit | ya.wang proposed openstack/nova-specs master: Re-propose the spec to allow specifying a list of CPU models https://review.opendev.org/642030 | |
| 05:23:01 | openstackgerrit | Merged openstack/nova-specs master: fix the spelling mistakes https://review.opendev.org/666244 | |
| 05:26:44 | openstackgerrit | Merged openstack/python-novaclient master: Add Python 3 Train unit tests https://review.opendev.org/668395 | |
| 07:30:52 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476 | |
| 07:35:12 | openstackgerrit | Josephine Seifert proposed openstack/nova-specs master: Spec for the Nova part of Image Encryption https://review.opendev.org/608696 | |
| 08:14:30 | yasemin | hi | |
| 08:15:12 | yasemin | we cannot start instances after server reboot, and the error is http://paste.openstack.org/show/753740/ | |
| 08:15:18 | yasemin | do you help me ? | |
| 08:15:30 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963 | |
| 08:17:32 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/667177 | |
| 08:17:32 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add neutron-tempest-iptables_hybrid job to experimental queue https://review.opendev.org/667154 | |
| 08:17:33 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Remove finish_revert_migration() signature try/catch https://review.opendev.org/668631 | |
| 08:29:28 | yasemin | hi, i cannot start instances after server reboot, and the error is http://paste.openstack.org/show/753740/ do you help me ? | |
| 08:40:23 | lyarwood | yasemin: I've got no idea what `16.0.13` is but make sure you have https://review.opendev.org/#/q/Iae2962bb86100f03fd3ad9aac3767da876291e74 | |
| 08:43:04 | yasemin | <lyarwood> the bug is 2017. does it related to my problem ? | |
| 08:56:05 | lyarwood | yasemin: if you're using QEMU > 2.10 then maybe | |
| 08:56:51 | lyarwood | yasemin: `16.0.13` isn't a released version of openstack-nova upstream AFAICT, it should contain that patch as it was included in actual early releases but I obviously can't tell | |
| 08:57:23 | lyarwood | yasemin: feel free to open a bug https://launchpad.net/nova/ listing your versions of QEMU, Libvirt and where you got this version of openstack-nova. | |
| 09:36:12 | jrosser | lyarwood: i think that is the 16.0.13 release of openstack-ansible which pulls in this SHA of nova https://github.com/openstack/openstack-ansible/commit/f678b3d9a237a0cad229cad6cbc5fd9254b5a6f4#diff-78907cb9f0dc957a6642505f5b9c135cR150 | |
| 09:43:16 | lyarwood | jrosser: ack thanks | |
| 09:43:42 | lyarwood | yasemin: ^ so you should have that fix, 09593628fda7a6cb947fff34c725e3ef29889dc7 includes https://review.opendev.org/#/q/Iae2962bb86100f03fd3ad9aac3767da876291e74 | |
| 10:14:02 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 10:17:26 | yaawang | Hello, could anyone review these spec? It's about expose auto-converge/post-copy and vcpu model selection :) https://review.opendev.org/651681 https://review.opendev.org/642030 | |
| 10:18:33 | kashyap | stephenfin: You back? | |
| 10:18:52 | kashyap | stephenfin: What is the problem in this rST snippet on the said line: http://paste.openstack.org/show/753749/ | |
| 10:19:53 | stephenfin | and yes, back and not _too_ sunburnt | |
| 10:19:58 | kashyap | Hehe | |
| 10:20:06 | kashyap | (Hope you had a good time) | |
| 10:20:23 | kashyap | The literal block, where do I put it on line-306? Why rST, why you do this to me... | |
| 10:20:34 | stephenfin | 306 needs to be indented more | |
| 10:20:54 | stephenfin | and 303 needs to take the form '(a)', I think | |
| 10:21:09 | stephenfin | since it's not smart enough to realize that the '3.' refers to the above point | |
| 10:21:17 | stephenfin | though you can force the HTML to render that way, of course | |
| 10:21:28 | kashyap | It "worked" :-( | |
| 10:21:37 | kashyap | Let me fiddle | |
| 10:22:37 | stephenfin | In general, I'm learning that even if it looks better in raw form with 2 or 3 spaces, you're better off using four spaces for all indentation since it's more predictable | |
| 10:23:23 | kashyap | Yeah, I hear you | |
| 10:54:42 | kashyap | stephenfin: Another error I don't understand :-( -- http://paste.openstack.org/show/753750/ | |
| 10:55:09 | kashyap | I compared with another spec, and the indentation and bullets in the "History" section look exactly same | |
| 10:56:00 | kashyap | Oh, I think I'm missing the "Introduced" keyword | |
| 10:56:56 | cdent | efried: can you point me to a quick and dirty example of cleanly retrying a set_inventory_for_provider that's gets a generation conflict and that is being called independently . I'd like to cargo cult^w^wlearn by example if possible. | |
| 10:57:32 | stephenfin | kashyap: Yeah, you're missing the description bullet point for the 'Train' release | |
| 10:57:40 | stephenfin | * - Train | |
| 10:57:46 | stephenfin | - Introduce | |
| 10:57:47 | stephenfin | *d | |
| 10:57:49 | kashyap | stephenfin: Yep, figured as much. Done. | |
| 10:57:58 | kashyap | Thanks, and sorry for bothering with these trivialities. | |
| 11:22:03 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Add "Secure Boot support for KVM & QEMU guests" spec https://review.opendev.org/506720 | |
| 11:22:44 | openstackgerrit | Huachang Wang proposed openstack/nova-specs master: We'd like to propose a way to create instance to take using of PCPU and VCPU resource in one instance. https://review.opendev.org/668656 | |
| 11:26:37 | kashyap | johnthetubaguy: As promised yesterday, reworked: Add "Secure Boot support for KVM & QEMU guests" spec https://review.opendev.org/506720 | |
| 11:26:49 | kashyap | johnthetubaguy: I added a short changelog in the Gerrit with what I addressed. | |
| 11:26:56 | kashyap | See if that matches our conversation from our past review. | |
| 11:31:27 | openstackgerrit | Huachang Wang proposed openstack/nova-specs master: Use PCPU and VCPU in one instance https://review.opendev.org/668656 | |
| 11:39:11 | alex_xu | stephenfin: so we won't support to request PCPU for the instance without numa topo in standard cpu resource tracking spec, right? | |
| 11:39:40 | sean-k-mooney | alex_xu: that is the same as it has always been | |
| 11:39:42 | stephenfin | alex_xu: I wasn't planning to uncouple the two features, no | |
| 11:40:10 | alex_xu | thanks, that is what I'm expecting | |
| 12:09:06 | openstackgerrit | Martin Midolesov proposed openstack/nova master: Implementing graceful shutdown. https://review.opendev.org/666245 | |
| 12:10:04 | maxbahosh3 | hi there, i have problem with numa and my flavors. | |
| 12:10:06 | maxbahosh3 | i have 2 socket servers and 4 socket servers, and i want to set numa to user all socket for memory | |
| 12:10:12 | maxbahosh3 | i set hw:numa_nodes='2' for 2 socket server and hw:numa_nodes='4' for 4 socket | |
| 12:10:18 | maxbahosh3 | is it any way to configure one flavor to use for both type of servers? | |
| 12:16:30 | sean-k-mooney | maxbahosh3: no | |
| 12:16:52 | sean-k-mooney | you need two flavors or just use hw:numa_nodes=2 for all of them | |
| 12:17:15 | sean-k-mooney | maxbahosh3: well that or just dont set hw:numa_nodes at all | |
| 12:17:29 | sean-k-mooney | but that also means you cant use cpu pinning or hugepages | |
| 12:18:20 | maxbahosh3 | if i use numa nodes 2, just use 2 socket and and oom maybe occur | |
| 12:18:56 | maxbahosh3 | if not set, does it use all of socket? | |
| 12:19:22 | sean-k-mooney | if not set it uses all soceckt | |
| 12:19:42 | sean-k-mooney | what oom errors are you seeing | |
| 12:20:02 | sean-k-mooney | you shoudl not see the kernel killing instances | |
| 12:20:29 | sean-k-mooney | you may not be able to fit a vm but that would just be beacuse of the normal ttris problem | |
| 12:21:05 | maxbahosh3 | i see oom kiiler error cause of it use one instance, use one socket | |
| 12:21:25 | maxbahosh3 | so, i use numa node to use two socket | |
| 12:21:38 | sean-k-mooney | that means you have not correctly reserved memoyr per numa node | |
| 12:22:34 | maxbahosh3 | how can i reserved memory? | |
| 12:22:38 | maxbahosh3 | do you mean reserved_host_memory_mb ? | |
| 12:22:46 | sean-k-mooney | the oom killer runs per numa node and will be trigered whenever the kernel needs memroy on a specific numa node even if there is memory free on a different hode | |
| 12:22:50 | sean-k-mooney | *node | |
| 12:23:05 | sean-k-mooney | the best way to avoid this is to use hugepages | |
| 12:24:13 | sean-k-mooney | reserved_host_memory_mb is not enough. you also neeed to use cgroups to limit the memoy available to qemu/libvirt so that it cannot use all memory on each indivigual node | |
| 12:24:20 | maxbahosh3 | mmm, i didnt ahave any configuration for hugepages ... | |
| 12:24:40 | sean-k-mooney | if you use vm with hugepages it will both improve teh performace of the vms and prevent the OOM issues | |
| 12:25:41 | maxbahosh3 | do you have good tuning link for it? (i'm going to search on google) | |
| 12:31:49 | maxbahosh3 | sean-k-mooney | |