Earlier  
Posted Nick Remark
#openstack-nova - 2017-06-27
03:00:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730
03:10:16 openstackgerrit Spencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value https://review.openstack.org/447766
03:14:07 openstackgerrit Chason Chan proposed openstack/nova master: import the admin guide content from openstack-manuals https://review.openstack.org/477497
03:17:18 openstackgerrit Chason Chan proposed openstack/nova master: import installation guide pages from openstack-manuals https://review.openstack.org/477488
03:32:59 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vmops https://review.openstack.org/451614
03:39:46 openstackgerrit suzhengwei proposed openstack/nova-specs master: availability zone return all services in api https://review.openstack.org/468261
04:16:26 krypto hi all i am facing strange issue with hugepages enabled flavor and resize.When i create instances with hugepages enabled,its getting created with 2M hugepages but when i resize ,scheduler complains about "Page size 2048 is not supported by the host" i have "allow_resize_to_same_host" flag set.Any idea why scheduler is showing this error during resize but not during instance creation?
04:22:20 krypto i got a clue that nova hypervisor-show doesnt show pse flag but in /proc/cpuinfo i can see that... any idea why some flags are ignored by nova?
04:24:06 krypto ahh sorry thats wrong information
04:28:05 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/467173
04:30:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730
04:46:00 openstackgerrit Spencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value https://review.openstack.org/447766
05:13:35 openstackgerrit Chason Chan proposed openstack/nova master: switch to openstackdocstheme https://review.openstack.org/477751
05:33:19 openstackgerrit Chason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme https://review.openstack.org/477757
05:42:43 openstackgerrit Chason Chan proposed openstack/nova master: switch to openstackdocstheme https://review.openstack.org/477751
05:57:56 openstackgerrit Chason Chan proposed openstack/nova master: import the admin guide content from openstack-manuals https://review.openstack.org/477497
06:32:18 openstackgerrit Chason Chan proposed openstack/nova master: import the admin guide content from openstack-manuals https://review.openstack.org/477497
06:50:43 openstackgerrit Jeffrey Zhang proposed openstack/nova master: Cleanup local files in libvirt when using shared block storage https://review.openstack.org/477780
06:51:46 openstackgerrit Gao Yuan proposed openstack/nova master: emove mox from nova.tests.unit.api.openstack.compute.test_limits https://review.openstack.org/477781
06:54:47 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.api.openstack.compute.test_limits https://review.openstack.org/477781
06:58:53 gcb mriedem: Could you please revisit https://review.openstack.org/#/c/466787 ?
07:10:44 openstackgerrit Chason Chan proposed openstack/nova master: DOC: Import glossary from openstack-manuals https://review.openstack.org/477788
07:30:55 openstackgerrit Chason Chan proposed openstack/nova master: DOC: Import glossary from openstack-manuals https://review.openstack.org/477788
07:38:22 openstackgerrit Chason Chan proposed openstack/nova master: import the admin guide content from openstack-manuals https://review.openstack.org/477497
07:46:48 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
07:46:56 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
07:46:57 openstackgerrit Chason Chan proposed openstack/nova master: switch to openstackdocstheme https://review.openstack.org/477751
07:46:57 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
08:05:13 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Read datapath_type from VIF object https://review.openstack.org/474914
08:05:37 openstackgerrit Rikimaru Honjo proposed openstack/nova master: compute: Do not create bdms before successfully reserving a volume https://review.openstack.org/290793
08:07:54 openstackgerrit Merged openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622
08:09:36 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Rehome unit tests to ``tests\unit`` folder https://review.openstack.org/476623
08:13:17 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.api.openstack.compute.test_limits https://review.openstack.org/477812
08:16:36 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.compute.test_keypairs.py https://review.openstack.org/477812
08:29:14 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Document the scheduler workflow https://review.openstack.org/475810
08:29:14 openstackgerrit Stephen Finucane proposed openstack/nova master: Monkey patch the blockdiag extension https://review.openstack.org/476159
08:49:57 chandra_sekhar I have multiple compute nodes with LVM storage
08:50:08 chandra_sekhar will live migration work with this kind of setup ?
09:23:40 openstackgerrit Chason Chan proposed openstack/nova master: switch to openstackdocstheme https://review.openstack.org/477751
09:25:54 openstackgerrit Chason Chan proposed openstack/nova master: switch to openstackdocstheme https://review.openstack.org/477751
09:47:33 sfinucan artom: All of that series reviewed with just a few comments left
09:50:13 artom sfinucan, muchas gracias senor
09:51:12 artom sfinucan, I re-ordered the patches to put the delete nic one at the bottom so that it doesn't hold up the delete bdm one
09:51:30 artom If Matt's around and you both feel like re-+2ing it we can get it off our plate
09:52:02 artom I had to adjust the unit test since MagicMock didn't like the new 'tag' in vif construct
09:52:11 sfinucan artom: Yup, that's what I've done
09:53:57 sfinucan By "that", I mean +W'd. I don't think the unit test change is too controversial
09:59:18 artom sfinucan, in 10 years we'll find an obscure bug that would have been noticed right away had Matt reviewed the change and pointed out a tricky way in which the new unit test doesn't cover every case
10:00:03 sfinucan artom: Aye, but by then I'll have retired to southern Italy and you won't be able to find me
10:00:12 ralonsoh jaypipes: hi
10:03:23 openstackgerrit Merged openstack/nova master: Remove mox in test_availability_zone.py https://review.openstack.org/477439
10:20:04 openstackgerrit Rikimaru Honjo proposed openstack/nova master: compute: Do not create bdms before successfully reserving a volume https://review.openstack.org/290793
11:05:07 openstackgerrit iswarya vakati proposed openstack/nova master: Add python 3.5 in classifier https://review.openstack.org/477887
11:20:23 openstackgerrit Pavel Gluschak (scsnow) proposed openstack/nova master: VStorage: changed default log path https://review.openstack.org/458557
11:25:32 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
11:26:43 artom The hyperv tests are... weird
11:36:23 artom As far as I can tell everything, including parameters passed to methods, and said methods return values, are mocked out
11:36:38 artom I'm not sure it's testing anything substantial
11:41:33 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Use ``assert_has_calls`` to check function calls https://review.openstack.org/477469
11:49:21 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091
11:56:03 openstackgerrit Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.test_block_device https://review.openstack.org/477904
12:05:37 bhagyashri_s johnthetubaguy: Hi, Waiting for long time, Could you please review patch https://review.openstack.org/#/c/409644 ? address all review comment. Thank you.
12:05:59 artom alex_xu, hi, questions about your feedback in https://review.openstack.org/#/c/376021/71/nova/tests/unit/api/openstack/compute/test_attach_interfaces.py@251
12:07:06 artom Having those tests in InterfaceAttachTestsV21 allowed me to use all the scaffolding that's already there. Should the new class inherit from InterfaceAttachTestsV21? That would replicate a lot of tests needlessly
12:08:21 artom If it doesn't inherit, we have to copy-paste said scaffolding
12:08:33 artom Well, maybe not the *entire* scaffolding
12:08:40 artom Let me see what I can get away with
12:08:48 artom Err, thanks for being my rubber duck :/
12:09:18 artom And if I was talking to myself the whole time and no one's here, even better, it means I haven't wasted anyone's time :)
12:15:17 alex_xu artom: we can create a new class, and you needn't copy too much code, since your test is only about schema validation, so the api method will fail at very early stage, so you needn't mock or stub_out anything
12:16:33 alex_xu artom: the new class just inherit from test.NoDBTestCase
12:16:51 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/477982
12:17:26 artom alex_xu, yep, that totally works, thanks :)
12:17:36 alex_xu artom: np :)
12:19:45 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/467173
12:21:51 jaypipes ralonsoh: heya
12:22:10 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/478083
12:24:39 jaypipes ralonsoh: did you need me for something?
12:27:09 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
12:27:10 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
12:34:42 ralonsoh jaypipes: hi, I left a comment in https://review.openstack.org/#/c/476612/
12:34:54 ralonsoh jaypipes: about the abstraction of the ovsdb access
12:38:44 jaypipes ralonsoh: ah, gotcha
12:47:44 sfinucan sdague, alex_xu
12:47:49 sfinucan Can I kill this docs? https://docs.openstack.org/developer/nova/api_plugins.html
12:47:51 sfinucan *doc
12:48:45 alex_xu sfinucan: yea, but probably need new doc to teach people how to add new API to the nova
12:49:00 sfinucan Well, we don't do it as plugins anymore, right?
12:49:28 sfinucan alex_xu: Is this what you're looking for? https://docs.openstack.org/developer/nova/api_microversion_dev.html
12:55:19 alex_xu sfinucan: yea, although it is specific to the microversion
12:56:31 sfinucan alex_xu: Is there any other way to extend the API than with a microversion now?
12:56:41 sfinucan I thought we didn't let people extend the API anymore?
12:57:11 sdague sfinucan: we don't
12:57:21 alex_xu sfinucan: yes, that doc is for the nova developer add API to the upstream
12:57:32 sdague however the api_plugins page is the only one that talks about the how you add new bits of api code
12:57:38 sfinucan Ohhh, is it now
12:57:50 sdague the microversion page is just about changing responses in existing ones
12:57:56 sfinucan OK, I won't delete it then
12:58:05 sdague it definitely needs a rewrite
12:58:07 sfinucan But I think it will need to be reworked

Earlier   Later