Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-05
17:40:34 openstackgerrit Merged openstack/nova master: Stop using PlacementDirect https://review.opendev.org/640887
17:44:28 luyao lyarwood: before rollback live migration at destination ，it also check the do_cleanup flag，in this case，i think we need destroy the domain at target host manually right?
17:45:48 lyarwood luyao: just looking at the review now, I didn't get that we were talking about rollback sorry
17:46:26 lyarwood luyao: it depends on the failure tbh, we ask Libvirt to persist the domain but only on a successful migration
17:47:02 lyarwood https://github.com/openstack/nova/blob/19cbbbebddadb125b8123ce0350b7146b29a59c6/nova/virt/libvirt/driver.py#L913-L915
17:47:05 lyarwood https://github.com/openstack/nova/blob/19cbbbebddadb125b8123ce0350b7146b29a59c6/nova/virt/libvirt/guest.py#L609-L611
17:50:51 luyao lyarwood: I understand it may not trigger error in real env, but do_cleanup flag setting and the code path it controls is a little confusing
17:52:18 luyao since do_cleanup is just tell if the instance path file is shared
17:52:31 lyarwood luyao: yeah very, I've added a comment, I think we could look into calling rollback_live_migration_at_destination regardless of that value in a seperate change
17:52:54 lyarwood luyao: I don't know if that's going to break assumptions in other drivers that support LM
17:58:21 luyao lyarwood: thanks for your comments I’ll go into it further tomorrow :)
18:22:53 openstackgerrit Jason Anderson proposed openstack/nova master: Use fair locks in resource tracker https://review.opendev.org/711528
19:58:13 openstackgerrit Victor Coutellier proposed openstack/nova master: Non-Admin user can filter their instances by more filters https://review.opendev.org/701609
20:22:21 mloza I'm trying to do a live migrate on a instance between computes and both have same model but different processor series. I'm getting this error
20:22:24 mloza Live Migration failure: operation failed: guest CPU doesn't match specification: missing features: virt-ssbd: libvirtError: operation failed: guest CPU doesn't match specification: missing features: virt-ssbd
20:35:44 mriedem mloza: https://docs.openstack.org/nova/latest/admin/configuration/hypervisor-kvm.html#specify-the-cpu-model-of-kvm-guests might be helpful
21:25:12 mloza mriedem: tahnsk
21:25:15 mloza thanks*
21:33:00 openstackgerrit Merged openstack/nova master: Fix os-console-output policy to be admin_or_owner https://review.opendev.org/706725
23:28:18 mloza I specified cpu_mode = custom and cpu_models = EPYC-IBRS but I can't turn on the instance
23:28:23 mloza I get this error ': Invalid: Config requested a custom CPU model, but no model name was provided'
23:29:24 mloza I have x86_EPYC-IBRS.xml in /usr/share/libvirt/cpu_map/
23:49:43 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add a tests to check when legacy access is removed https://review.opendev.org/710813
#openstack-nova - 2020-03-06
00:55:47 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-deferred_delete https://review.opendev.org/707476
00:59:54 openstackgerrit Brin Zhang proposed openstack/nova master: Correct the actual target in os-instance-actions policy https://review.opendev.org/710411
00:59:55 openstackgerrit Brin Zhang proposed openstack/nova master: Add test coverage of existing os-instance-actions policies https://review.opendev.org/707777
01:05:50 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-admin-password https://review.opendev.org/701630
01:06:04 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-admin-password policies https://review.opendev.org/701639
01:06:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add a tests to check when legacy access is removed https://review.opendev.org/710813
01:06:32 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-deferred_delete https://review.opendev.org/707476
01:06:44 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-deferred_delete policies https://review.opendev.org/707485
01:09:25 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-deferred_delete https://review.opendev.org/707476
01:09:37 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-deferred_delete policies https://review.opendev.org/707485
01:23:21 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-deferred_delete policies https://review.opendev.org/707485
01:26:45 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing console_output policies https://review.opendev.org/706724
01:27:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-console-output https://review.opendev.org/707040
01:28:11 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing console_output policies https://review.opendev.org/706724
01:28:34 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-console-output https://review.opendev.org/707040
01:29:37 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-console-output https://review.opendev.org/707040
01:29:51 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-console-output policies https://review.opendev.org/707041
01:36:44 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-console-output policies https://review.opendev.org/707041
01:38:13 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing create_backup policies https://review.opendev.org/706726
01:38:56 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix os-create-backup policy to be admin_or_owner https://review.opendev.org/706727
01:39:03 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix os-create-backup policy to be admin_or_owner https://review.opendev.org/706727
01:39:23 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing create_backup policies https://review.opendev.org/706726
01:39:35 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-create-backup https://review.opendev.org/707038
01:40:54 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-create-backup https://review.opendev.org/707038
01:41:09 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-create-backup policies https://review.opendev.org/707039
01:45:52 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-create-backup policies https://review.opendev.org/707039
02:48:34 brinzhang damsmith: as mriedem said in https://review.opendev.org/#/c/694428/4, I think this issue was triggered when I modified the os-instance-actions APIs default policy [1]. I am not particularly familiar with @serialize_args. Can you suggest or submit some fixes?
02:50:15 brinzhang dansmith: [1]https://review.opendev.org/#/c/706470/7, it's functional error report https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5a7/706179/6/check/openstack-tox-py36/5a7826b/testr_results.html
02:53:17 brinzhang damsmith: when microversion > 2.51, we can show the instance action events for a server, but now the non-admin can get the ['host', 'traceback'], that should belong to admin role, this validates mriedem's comment.
02:58:02 openstackgerrit norman shen proposed openstack/nova stable/rocky: Relex python3 version for pep8 https://review.opendev.org/711577
03:10:54 brinzhang dansmith: I was said maybe be not correct, I am not sure they are the same issue, please check.
03:18:37 openstackgerrit Eric Xie proposed openstack/nova master: Catch exception when use invalid architecture of image https://review.opendev.org/711363
03:22:47 brinzhang dansmith: bug 1866292 mainly trace for non-admin can get the 'traceback'
03:22:47 openstack bug 1866292 in OpenStack Compute (nova) "non-admin shouldn't get the ['host', 'traceback'] of os-instance-actions's events" [Undecided,New] https://launchpad.net/bugs/1866292
03:30:09 openstackgerrit Merged openstack/nova stable/train: libvirt: Ignore DiskNotFound during update_available_resource https://review.opendev.org/711276
03:49:45 openstackgerrit Brin Zhang proposed openstack/nova master: Add SYSTEM_READER role to servers actions API https://review.opendev.org/706179
08:37:00 openstackgerrit Balazs Gibizer proposed openstack/nova stable/train: Reproduce bug 1862633 https://review.opendev.org/709166
08:37:00 openstack bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,Fix released] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer)
08:37:15 openstackgerrit Balazs Gibizer proposed openstack/nova stable/train: Clean up allocation if unshelve fails due to neutron https://review.opendev.org/709167
08:38:01 gibi elod, lyarwood: ^^ updated the commit message on the bottom backport patch as requested
08:57:30 lyarwood gibi: ack done, thanks
08:57:48 lyarwood elod: https://review.opendev.org/#/c/709167/ LGTM to me also now.
09:01:46 elod lyarwood: thanks, +W'd :]
09:19:00 gibi lyarwood, elod: thanks a bunch
09:19:26 openstackgerrit Brin Zhang proposed openstack/nova master: Fix os-volumes-attachments policy to be admin_or_owner https://review.opendev.org/709955
09:19:26 openstackgerrit Brin Zhang proposed openstack/nova master: Add test coverage of existing os-volumes-attachments policies https://review.opendev.org/709929
09:19:27 openstackgerrit Brin Zhang proposed openstack/nova master: Introduce scope_types in os-volumes-attachments policy https://review.opendev.org/709388
09:19:27 openstackgerrit Brin Zhang proposed openstack/nova master: Add new default roles in os-volumes-attachments policies https://review.opendev.org/710190
09:19:28 openstackgerrit Brin Zhang proposed openstack/nova master: Add PATCH volume attachments api to os-volume_attachments https://review.opendev.org/693828
09:19:28 openstackgerrit Brin Zhang proposed openstack/nova master: Add new policy to PATCH update volume API https://review.opendev.org/711194
09:19:29 openstackgerrit Brin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API https://review.opendev.org/710965
09:33:02 openstackgerrit Merged openstack/nova stable/train: Reproduce bug 1862633 https://review.opendev.org/709166
09:33:02 openstack bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,Fix released] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer)
09:35:23 brinzhang_ gibi: Do you have time to check this functional failed? https://review.opendev.org/#/c/706470/
09:36:05 brinzhang_ gibi: I am not sure whether is it a bug, I create a bug in launchpad, for bug 1866292
09:36:05 openstack bug 1866292 in OpenStack Compute (nova) "non-admin shouldn't get the ['host', 'traceback'] of os-instance-actions's events" [Undecided,New] https://launchpad.net/bugs/1866292
09:41:11 gibi brinzhang_: I will check..
09:41:30 brinzhang_ gibi: thanks ^^
09:57:22 gibi brinzhang_: I cannot reproduce the issue in devstack with demo user
09:57:25 gibi brinzhang_: http://paste.openstack.org/show/790383/
09:57:46 gibi brinzhang_: so I think it could be a specific problem in the func test en
09:57:49 gibi env
09:58:58 brinzhang_ gibi: yes, I was changed it's default policy, I think caused by the new policy
09:59:26 gibi I can pull down your patch in my devstack to see if it changes the behavior
09:59:51 brinzhang_ gibi: I debuged in my devstack, and it has the same issue with that CI result
10:00:00 brinzhang_ gibi: cool, thanks
10:02:25 openstackgerrit John Garbutt proposed openstack/nova master: Introduce scope_types in os-aggregates policy https://review.opendev.org/701652
10:02:40 lyarwood does anyone recall where the legacy zuul migration guide lives now? I'm getting 404's for the links I'm finding.
10:03:16 lyarwood https://docs.openstack.org/devstack/latest/zuul_ci_jobs_migration.html ah ha!
10:03:43 brinzhang_ lyarwood: can you provide the old.. cool, let me try
10:04:22 brinzhang_ lyarwood: ignore above word^^
10:06:25 lyarwood brinzhang_: https://docs.openstack.org/infra/manual/zuulv3.html
10:06:30 lyarwood brinzhang_: returns a 404 now
10:06:40 lyarwood brinzhang_: still cached by Google apparently
10:07:31 brinzhang_ lyarwood: https://docs.openstack.org/devstack/latest/zuul_ci_jobs_migration.html this is not what would you want to get?
10:08:08 lyarwood brinzhang_: yes sorry that's what I was looking for
10:08:48 brinzhang_ lyarwood: ^^

Earlier   Later