Earlier  
Posted Nick Remark
#openstack-nova - 2018-06-11
19:21:43 mriedem besides just the attach_volume cleanup
19:22:25 artom I'll try to dig into it more
19:22:44 artom Not now though, don't want to context switch from NUMA LM
19:31:16 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_util https://review.openstack.org/574403
19:31:17 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.handlers.test_aggregate https://review.openstack.org/574404
19:31:18 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_requestlog https://review.openstack.org/574405
19:31:19 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_fault_wrap https://review.openstack.org/574406
19:31:20 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_handler https://review.openstack.org/574407
19:31:21 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_microversion https://review.openstack.org/574408
19:31:22 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_deploy https://review.openstack.org/574409
19:33:08 openstackgerrit Zack Cornelius proposed openstack/nova master: Refactor libvirt get_memory_used_mb() https://review.openstack.org/571030
19:33:09 openstackgerrit Zack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876
19:41:48 mriedem dansmith: is dash working for you?
19:42:01 dansmith mriedem: not at the moment but it was a few minutes ago
19:42:14 dansmith this happens when they have zuul down and it returns a {}
19:42:16 dansmith so I assume that's it
19:42:32 mriedem yeah looks like from infra chatter zuul is down
19:43:10 dansmith "sweet"
19:45:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Keep attach_mode as top-level field in _translate_attachment_ref https://review.openstack.org/574413
20:17:33 openstackgerrit Dan Smith proposed openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696
21:09:10 mriedem dansmith: did you forget to git add nova/conf/rpc.py?
21:09:16 dansmith eff
21:09:30 mriedem heh
21:09:40 openstackgerrit Dan Smith proposed openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696
21:13:05 mriedem did you think about creating a new [rpc] group?
21:16:50 openstackgerrit Julia Kreger proposed openstack/nova master: ironic: bugfix: ensure a host is set for volume connectors https://review.openstack.org/571982
21:27:06 dansmith mriedem: I can't move rpc_response_timeout, so it didn't seem worth the trouble
21:27:09 dansmith and potential confusion
21:27:50 openstackgerrit Eric Fried proposed openstack/nova master: Direct Placement API: ksa Adapter https://review.openstack.org/574470
21:31:05 openstackgerrit Eric Fried proposed openstack/nova master: Direct Placement API: ksa Adapter https://review.openstack.org/574470
21:35:26 openstackgerrit Merged openstack/nova master: Document the internal online_migrations function behaviors https://review.openstack.org/574268
21:44:16 openstackgerrit Dan Smith proposed openstack/nova master: Use oslo.messaging per-call monitoring https://review.openstack.org/566696
21:44:34 openstackgerrit Eric Fried proposed openstack/nova master: Direct Placement API: ksa Adapter https://review.openstack.org/574470
22:01:21 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_util https://review.openstack.org/574403
22:01:22 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.handlers.test_aggregate https://review.openstack.org/574404
22:01:23 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_requestlog https://review.openstack.org/574405
22:01:24 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_fault_wrap https://review.openstack.org/574406
22:01:25 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_handler https://review.openstack.org/574407
22:01:26 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_microversion https://review.openstack.org/574408
22:01:27 openstackgerrit Chris Dent proposed openstack/nova master: Do not use nova.test in placement.test_deploy https://review.openstack.org/574409
23:15:00 openstackgerrit Matt Riedemann proposed openstack/nova master: Time how long pre_live_migration() takes https://review.openstack.org/574481
23:15:01 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: test heartbeat code in pre_live_migration https://review.openstack.org/574482
23:19:36 openstackgerrit Eric Fried proposed openstack/nova-specs master: Spec: Handling Reshaped Provider Trees https://review.openstack.org/572583
23:48:38 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Heal allocations with missing/incomplete consumer information https://review.openstack.org/574488
23:49:37 openstackgerrit Merged openstack/nova master: placement: always create consumer records https://review.openstack.org/567678
#openstack-nova - 2018-06-12
01:40:06 naichuans mriedem: Hi, Matt. We can set one or more hosts' SR to LVM to check it. But look it can't refect to the test result. Currently, we checked it on an independent host in our company, all the tests are passed.
01:55:02 openstackgerrit Merged openstack/nova master: Fix typo in enable_certificate_validation config option help https://review.openstack.org/572185
02:04:31 openstackgerrit Merged openstack/nova master: Mention running rootwrap in daemon mode if hitting vif plug timeouts https://review.openstack.org/571577
02:29:48 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case https://review.openstack.org/574240
03:09:29 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398
03:09:30 openstackgerrit Artom Lifshitz proposed openstack/nova master: libvirt live migration: fit NUMA topology on dest https://review.openstack.org/567242
03:09:31 openstackgerrit Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723
03:34:12 openstackgerrit chenxiangui proposed openstack/nova master: Replace a extra_specs with an extra_specs https://review.openstack.org/574535
03:42:50 openstackgerrit qingszhao proposed openstack/os-traits master: change default python 3 env in tox to 3.5 https://review.openstack.org/574539
04:59:18 openstackgerrit Naichuan Sun proposed openstack/nova master: DNM: default [xenserver]/image_handler to vdi_remote_stream https://review.openstack.org/574318
05:34:27 openstackgerrit garyk proposed openstack/nova master: VMware: save VC reads for information that is static https://review.openstack.org/574552
05:37:38 openstackgerrit garyk proposed openstack/nova master: VMware: remove reading resourcePool data https://review.openstack.org/574553
06:33:16 openstackgerrit jiang wei proposed openstack/nova master: Add action initiator attribute to the payload https://review.openstack.org/536243
07:14:19 openstackgerrit Pooja Jadhav proposed openstack/nova master: Remove unnecessary parameters from create volume API https://review.openstack.org/574598
07:26:04 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
08:02:23 openstackgerrit jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_util https://review.openstack.org/574403
08:02:24 openstackgerrit jichenjc proposed openstack/nova master: Do not use nova.test in placement.handlers.test_aggregate https://review.openstack.org/574404
08:02:25 openstackgerrit jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_requestlog https://review.openstack.org/574405
08:02:26 openstackgerrit jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_fault_wrap https://review.openstack.org/574406
08:02:27 openstackgerrit jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_handler https://review.openstack.org/574407
08:02:28 openstackgerrit jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_microversion https://review.openstack.org/574408
08:02:29 openstackgerrit jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_deploy https://review.openstack.org/574409
08:35:37 openstackgerrit XiaojueGuan proposed openstack/nova master: Update links in README https://review.openstack.org/574644
08:52:11 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092
09:14:58 Kevin_Zheng ping gibi
09:21:27 gibi Kevin_Zheng: pong
09:21:49 Kevin_Zheng Hi, thanks for the advise on exception payload patch
09:22:12 Kevin_Zheng the notification_sample_base.NotificationSampleTestBase.ANY is good
09:22:54 Kevin_Zheng but the sample file still seems hard to implement
09:23:24 Kevin_Zheng a typical traceback is at at least 10 lines
09:23:51 gibi Kevin_Zheng: do you mean it will be ugly in the sample file?
09:23:54 Kevin_Zheng it will be review-able to put it in the json file
09:23:57 Kevin_Zheng yeah
09:24:10 Kevin_Zheng I didn't update the sample yet
09:24:12 Kevin_Zheng just local
09:24:26 Kevin_Zheng I'm going to fix all the samples in next patch
09:24:30 Kevin_Zheng patchset
09:24:58 gibi Kevin_Zheng: does the uglyness came from the fact that a traceback is naturally a multiline string and it is now force into a single line string in json
09:25:01 gibi ?
09:25:33 Kevin_Zheng yeah, and I cannot find a good way to make it multiline
09:25:35 Kevin_Zheng in json
09:42:44 gibi Kevin_Zheng: sorry got distracted locally
09:43:38 gibi Kevin_Zheng: could you just add the beginning of a traceback maybe the first line including the \n at the end then '...' to signal to the reader the there is more
09:43:53 Kevin_Zheng yeah, that might work
09:44:07 Kevin_Zheng I will add them tonight
09:44:30 gibi Kevin_Zheng: thanks a lot!
09:46:04 ratailor bauzas, Do you think, this comment is valid on https://review.openstack.org/#/c/401009/11/nova/compute/api.py
09:56:35 lyarwood lyarwood
09:56:37 lyarwood argh
10:24:31 openstackgerrit garyk proposed openstack/nova master: VMware: populate datastore refs at init https://review.openstack.org/574688
10:57:27 openstackgerrit Chris Dent proposed openstack/nova master: Move db using provider unit tests to functional https://review.openstack.org/572280
10:57:28 openstackgerrit Chris Dent proposed openstack/nova master: Adjust db using allocation unit tests https://review.openstack.org/572281
10:57:29 openstackgerrit Chris Dent proposed openstack/nova master: Remove nova dependencies from test_resource_provider https://review.openstack.org/572282
11:03:41 openstackgerrit jiang wei proposed openstack/nova master: Add action initiator attribute to the payload https://review.openstack.org/536243
11:21:55 openstackgerrit Merged openstack/nova stable/queens: Change consecutive build failure limit to a weigher https://review.openstack.org/573239

Earlier   Later