Earlier  
Posted Nick Remark
#openstack-nova - 2018-02-03
00:39:14 Spazmotic Would love to get some eyes on it before I go to Korea and cannot do anything with it for 2 weeks
03:39:11 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161
05:41:13 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Enable NUMA Features for Libvirt/QEMU Driver https://review.openstack.org/533077
07:04:46 openstackgerrit Andreas Jaeger proposed openstack/nova master: Zuul: Remove project name https://review.openstack.org/540326
08:46:33 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: doc: merge numa.rst to cpu-topologies.rst https://review.openstack.org/540585
11:44:43 openstackgerrit int32bit proposed openstack/nova master: Refix disk size during live migration with disk over-commit https://review.openstack.org/536351
19:00:38 openstackgerrit Eric Fried proposed openstack/nova master: Ensure resource classes correctly https://review.openstack.org/539738
#openstack-nova - 2018-02-04
01:16:51 Spazmotic Morning weekend Novaers
01:16:53 Spazmotic Should you exist
06:33:47 openstackgerrit Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
06:34:30 Spazmotic I don't understand how this test is successfully completing
07:02:26 Spazmotic O.. i see.. interesting
07:40:44 openstackgerrit Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385
09:43:42 Roamer` (on some days it's not enough to convince even me)
09:49:56 Spazmotic I'd need to see the test_roamer_exists method to be sure..
10:00:04 Spazmotic The trouble with tests is the ability to Mock your way to victory.. so really it's unreliable.. i'd need someone else to write that test. When AIs right their own test it's a bad day
10:09:19 Roamer` that's why in some projects at work we don't rely on regression tests only, but we have a test framework that ties into the actual application code, then issues random requests (and records the random seed every now and then) and checks various aspects of the internal state
10:10:07 Roamer` and it's still not enough, because it only checks what it's been written to check - hence one of our lead developers saying "if <the random test framework> does not check it, it has bugs"
10:11:29 ameeda Hello, how can I delete network from horizon, when I try to delete it, it gives my error "can't delete network..."
10:29:42 Spazmotic Roamer` Unit tests always kind of bother me.. but it's also because i'm so new at them they're still half mysterious
17:04:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Update cells v2 layout doc caveats for Queens https://review.openstack.org/540667
18:23:50 openstackgerrit Merged openstack/nova stable/pike: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540555
22:02:46 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests https://review.openstack.org/540679
22:04:02 lyarwood mriedem: ^ was the missing test that I had in mind for https://review.openstack.org/#/c/539739/ , feel free to merge it into your change if you agree.
22:15:08 openstackgerrit Lee Yarwood proposed openstack/nova stable/ocata: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540680
23:36:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
23:36:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
23:37:21 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types https://review.openstack.org/430839
23:38:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unnecessary arguments in notification methods https://review.openstack.org/535599
23:39:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
23:39:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
23:39:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
23:40:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
23:40:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
23:41:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
23:41:29 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
23:41:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
23:48:53 Spazmotic Ugh pre flight sleeplessness
23:49:01 Spazmotic Morning I guess
23:52:28 Spazmotic Wake me up KPOP.. You sweet necter. I guess I should comment my commits that I will be gone for 2 weeks
#openstack-nova - 2018-02-05
00:11:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299
01:17:36 openstackgerrit Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
01:24:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625
02:05:17 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
02:20:41 openstackgerrit Spencer Yu proposed openstack/nova master: project name should be removed from zuul configuration https://review.openstack.org/540695
04:16:12 openstackgerrit Chiawei Xie proposed openstack/nova stable/ocata: Modify nova-manage db migrations base message https://review.openstack.org/540709
04:32:48 Spaz-Korea Flight in 5 hours. almost time :)
04:32:57 Spaz-Korea See you all in about 2 weeks
04:36:18 Spaz-Korea If you need me for anything drop me a mail at bhkaminski (at) gmail.com
05:33:22 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
05:33:23 openstackgerrit jichenjc proposed openstack/nova master: z/VM driver: add snapshot function https://review.openstack.org/534240
05:33:23 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
05:41:25 openstackgerrit jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458
05:45:45 openstackgerrit jichenjc proposed openstack/nova master: mv generate_glance_url to get_image_endpoint_url https://review.openstack.org/511400
06:27:50 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833
07:37:23 openstackgerrit jichenjc proposed openstack/nova-specs master: add z/VM Driver in Rocky https://review.openstack.org/536186
08:14:12 gibi edleafe: both zuul's and your environment is missing a bugfix from ovo that my patch is depends on (Depends-On: https://review.openstack.org/540344). If you pull down that bugfix in your env then the test will pass
08:14:40 gibi edleafe: this of course means that we can only merge the patch after the ovo fix is merged and the ovo version is bumped in nova
08:19:53 ameeda gibi: good morning :)
08:19:59 ameeda can you please review this for me ? https://review.openstack.org/#/c/528385/
08:35:22 openstackgerrit jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458
08:38:35 AJaeger nova cores, could you merge a small docs update, please? https://review.openstack.org/#/c/538163/ I need another +2..
08:39:36 AJaeger stephenfin, gibi, can you help, please? ^
08:41:17 openstackgerrit jichenjc proposed openstack/nova master: Remove quota reserve/commit/rollback https://review.openstack.org/521470
08:43:27 amorin hello all
08:43:38 gibi ameeda: will try to look at it today
08:43:56 gibi AJaeger: looking
08:44:36 ameeda gibi: thank you very much, please don't forget
08:46:05 gibi ameeda: I tend not to forget these. It is more like I run out of time
08:47:42 ameeda gibi: Thank you !!
08:49:50 AJaeger thanks, gibi !
08:52:18 amorin I would like to know if there is any way to create a neutron client from nova, which could use the retries parameter of neutron
08:52:23 amorin on nova side: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L137
08:52:33 amorin neutron side: https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L220
08:52:52 amorin seems that nova is not using retries at all
09:03:26 bauzas good morning folksq
09:05:01 gibi bauzas: good morning
09:05:10 amorin morning
09:17:41 openstackgerrit jichenjc proposed openstack/nova master: deprecate fping_path config option https://review.openstack.org/526602
09:43:14 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
09:48:07 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Trivial fix a missleading comment https://review.openstack.org/540796
10:03:37 openstackgerrit Merged openstack/nova master: Zuul: Remove project name https://review.openstack.org/540326
10:03:47 openstackgerrit Merged openstack/nova master: Fix SUSE Install Guide: Placement port https://review.openstack.org/538163
10:08:39 gibi the gate is super fast now :)
10:11:32 openstackgerrit Jianghua Wang proposed openstack/nova master: doc: update the GPU passthrough HPC feature entry https://review.openstack.org/539638
10:11:33 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: Provide support matrix and doc for VGPU https://review.openstack.org/540808
10:11:33 openstackgerrit Jianghua Wang proposed openstack/nova master: Provide support matrix and doc for VGPU https://review.openstack.org/539266
10:50:18 ameeda gibi: replied
11:11:48 gibi ameeda: OK, I will double check with mriedem about the double detach
11:12:27 ameeda gibi: thank you !!
11:12:55 gibi ameeda: until then, I will try to make the functional test work without the need for mocking the detach call
11:13:53 ameeda gibi: yes please , I try a lot with it with no lucks
11:16:06 ameeda gibi: I have 2 nova reviews needs for 2 +2 here https://review.openstack.org/#/c/539880/ and https://review.openstack.org/#/c/540274/
11:16:17 ameeda gibi: also I need help here https://review.openstack.org/#/c/526900/
11:16:40 ameeda sorry for inconvenience, thanks for you time !!
11:25:27 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have usable input for graphical console https://review.openstack.org/538003
11:26:09 hrw morning
11:33:35 rabel hi. can we merge https://review.openstack.org/#/c/535871/ ?
11:38:58 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
12:03:04 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests https://review.openstack.org/540679

Earlier   Later