| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-01 | |||
| 21:38:36 | spatel | so it not merge yet? | |
| 21:38:41 | sean-k-mooney | cold migrations are supported but until ^ is implemented nova will not update teh libvirt xml and you can get that error amoung others | |
| 21:38:45 | sean-k-mooney | no its not | |
| 21:39:07 | sean-k-mooney | for stien we specifcialy disable live migration for instnace with numa toplogies | |
| 21:39:17 | sean-k-mooney | i think we started backporting that too | |
| 21:39:57 | spatel | oh! | |
| 21:40:04 | spatel | this is a big issue then.. | |
| 21:40:38 | spatel | how to migrate instance if my compute node is dead and all vm running on shared storage? | |
| 21:41:05 | sean-k-mooney | if the compute node is dead and the vms are using shared storage you should use evacuate | |
| 21:41:28 | spatel | hmm.. | |
| 21:41:30 | sean-k-mooney | it will rebuild the instance on another host | |
| 21:41:42 | sean-k-mooney | but you should only use evacuate if the instance is stopped | |
| 21:41:56 | spatel | what if i want to migrate specific vm ? any heck there? | |
| 21:42:28 | spatel | sean-k-mooney: so you are saying i can shutdown instance and do evaluate ? | |
| 21:42:30 | sean-k-mooney | evacuate works for a specifc vm | |
| 21:43:03 | spatel | This is very important information, i was totally not aware of it.. | |
| 21:43:55 | spatel | I need to make big banner now to tell everyone don't do live migration period.. | |
| 21:44:13 | sean-k-mooney | if you have not already read it the go to text on evacuate is http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/ | |
| 21:44:23 | sean-k-mooney | what release of openstack are you running | |
| 21:44:46 | spatel | my old cloud running on queens and new cloud i am build on stein | |
| 21:45:46 | sean-k-mooney | looks we have the config option on rocky https://github.com/openstack/nova/blob/stable/rocky/nova/conf/workarounds.py#L223 | |
| 21:46:58 | sean-k-mooney | ah we havnt merged https://review.opendev.org/#/c/629597/ | |
| 21:48:31 | spatel | hmm is it config option which i can turn on to protect from accident ? | |
| 21:48:47 | sean-k-mooney | spatel: live migration with hugepage works most of the time. cpu pinning is much less likely to work correctly | |
| 21:49:17 | sean-k-mooney | with that patch by default we will not allow instace with a numa toplogy to be live migrated | |
| 21:49:24 | sean-k-mooney | so its safe by default | |
| 21:49:26 | jaypipes | efried: ahoy! :) going well, ty! | |
| 21:49:39 | efried | good to hear | |
| 21:50:15 | spatel | sean-k-mooney: hmm we used both pinning/hugepage :) | |
| 21:51:10 | sean-k-mooney | :) yes. this si something we have wanted to fix for several relese. it will get done this cycle one way or another. | |
| 21:51:19 | sean-k-mooney | unfortuanetly its not backportable | |
| 21:52:19 | spatel | damn.. | |
| 21:52:36 | sean-k-mooney | jaypipes: life treating you well in aws land | |
| 22:07:00 | mriedem | efried: i did your deed | |
| 22:07:05 | mriedem | with lots of comments about why i didn't want to | |
| 22:07:27 | mriedem | also, i'll have to redo https://review.opendev.org/#/c/541420/ which is 3 months older than takashin's de-mox-ification | |
| 22:07:34 | efried | Thanks mriedem. You will be rewarded in the hereafter. | |
| 22:07:42 | mriedem | satan won't care about this | |
| 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: Add neutron-tempest-iptables_hybrid job to experimental queue https://review.opendev.org/667154 | |
| 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: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. | |