Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-01
20:34:10 openstackgerrit Merged openstack/nova stable/queens: Fix retry of instance_update_and_get_original https://review.opendev.org/659320
20:34:16 openstackgerrit Merged openstack/nova stable/queens: DRY up test_rollback_live_migration_set_migration_status https://review.opendev.org/662470
20:34:23 openstackgerrit Merged openstack/nova stable/queens: Include all network devices in nova diagnostics https://review.opendev.org/666152
20:34:32 openstackgerrit Merged openstack/nova stable/queens: Add regression recreate test for bug 1830747 https://review.opendev.org/662774
20:34:32 openstack bug 1830747 in OpenStack Compute (nova) queens "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem)
20:35:00 efried aspiers: SEV series in merge conflict after the first patch :(
20:49:16 spatel sean-k-mooney: yt
20:51:24 spatel getting this issue - https://bugzilla.redhat.com/show_bug.cgi?id=1176020
20:51:25 openstack bugzilla.redhat.com bug 1176020 in libvirt "libvirt should do a right check for numa cpus set" [Medium,Closed: errata] - Assigned to mprivozn
21:03:14 openstackgerrit Eric Fried proposed openstack/nova master: Rename CinderFixtureNewAttachFlow to CinderFixture https://review.opendev.org/668561
21:07:11 mriedem i see what you're doing
21:10:59 mriedem bbiab
21:19:47 openstackgerrit Merged openstack/nova stable/queens: Workaround missing RequestSpec.instance_group.uuid https://review.opendev.org/663110
21:22:20 efried o/ jaypipes, how goes?
21:30:25 sean-k-mooney spatel: you are getting that because you are livemigrating instance with a numa toplogy
21:30:45 sean-k-mooney spatel: when you do that we do not recalulate the guest xml currently
21:31:13 sean-k-mooney as a result if you live migrate form a host with more cpus to one with less you can have an xml on the dest that reference cpus that are not present
21:31:35 spatel sean-k-mooney: now i am stuck and not sure how to get out
21:31:55 sean-k-mooney spatel: cold migration should work
21:32:08 spatel i tried all option but getting same error
21:32:27 spatel now thinking to re-build it...
21:32:57 spatel when you said - you are getting that because you are livemigrating instance with a numa toplogy
21:33:11 spatel what does that means?
21:33:28 sean-k-mooney i assume you are getting that error becasue you tried to do a live migration
21:33:37 spatel if i have machine and i said live migrate, where numa comes in picture
21:33:39 sean-k-mooney if that is not the case how did you hit this issue
21:34:04 sean-k-mooney if your instance use hugepage or cpu pinning it will have a numa toplogy
21:34:19 spatel someone from my team trying to do and it endup in this situation, i have to talk to him to get full picture
21:35:29 spatel oh.. yes that instance had hugepage/cpu pinning
21:35:57 spatel you are saying we can't do live migration if instance has hugepage/cpu pinning?
21:36:01 spatel sean-k-mooney: ^
21:36:18 sean-k-mooney technically it can work but its not properly supported by nova
21:36:31 sean-k-mooney we dont currently update teh xml when migrating
21:36:37 sean-k-mooney for live migrations
21:37:12 sean-k-mooney spatel: its a knwon issue that is being adressed by https://review.opendev.org/#/q/topic:bp/numa-aware-live-migration+(status:open+OR+status:merged)
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

Earlier   Later