Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-15
21:57:21 mriedem ok, so grenade patch is updated, nova change depends on it with renos
21:57:29 mriedem and my list and delete cells patches are all up
21:57:34 dansmith cool, thanks
22:00:03 mriedem and https://wiki.openstack.org/wiki/Nova-Cells-v2 should be all updated
22:00:12 mriedem ok, f'ing football time
22:04:39 mriedem btw, ftr, i hate aaron rodgers' stupid hair marying face
22:04:47 mriedem *hail
#openstack-nova - 2017-01-16
00:12:21 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters and response in os-quota-sets.inc https://review.openstack.org/416461
00:12:39 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911
00:13:18 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882
00:14:53 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix a parameter in os-availability-zone.inc https://review.openstack.org/417233
00:15:20 openstackgerrit Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738
00:15:31 openstackgerrit Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436
00:15:56 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
00:15:59 openstackgerrit Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876
00:17:03 openstackgerrit Takashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec https://review.openstack.org/397193
01:10:38 openstackgerrit Matt Riedemann proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198
01:55:16 openstackgerrit Alex Szarka proposed openstack/nova: Transform instance-live_migration_abort notification https://review.openstack.org/420453
02:14:38 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955
02:15:29 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964
02:17:42 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
02:17:52 openstackgerrit Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926
02:18:01 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host https://review.openstack.org/406707
02:53:07 openstackgerrit Jay Pipes proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215
02:53:07 openstackgerrit Jay Pipes proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807
02:53:08 openstackgerrit Jay Pipes proposed openstack/nova: placement: integrate ProviderTree to report client https://review.openstack.org/415921
02:53:08 openstackgerrit Jay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768
02:53:09 openstackgerrit Jay Pipes proposed openstack/nova: placement: adds ProviderTree for nested resources https://review.openstack.org/415920
02:53:09 openstackgerrit Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769
02:53:10 openstackgerrit Jay Pipes proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138
02:53:10 openstackgerrit Jay Pipes proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693
03:47:31 openstackgerrit Tristan Cacqueray proposed openstack/nova: Mask the token used to allow access to consoles https://review.openstack.org/220622
03:56:54 openstackgerrit Tuan Luong-Anh proposed openstack/os-vif: H803 hacking have been deprecated https://review.openstack.org/420489
04:10:04 openstackgerrit Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494
04:17:18 openstackgerrit Tuan Luong-Anh proposed openstack/os-vif: H803 hacking have been deprecated https://review.openstack.org/420489
05:45:45 openstackgerrit Tristan Cacqueray proposed openstack/nova: Mask the token used to allow access to consoles https://review.openstack.org/220622
05:51:52 openstackgerrit jichenjc proposed openstack/nova: Fix broken link of Doc https://review.openstack.org/420504
06:18:58 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330
06:25:35 satyar Hi jeffrey4l
06:26:01 Jeffrey4l hi satyar. ( nova channel ;) )
06:26:34 satyar opps lets go to our challel :P
06:26:40 satyar channel
06:28:06 openstackgerrit Yoshihiko Atsumi proposed openstack/nova: Add resize server functional negative tests https://review.openstack.org/420514
06:32:02 openstackgerrit jichenjc proposed openstack/nova: Fix broken link of Doc https://review.openstack.org/420504
07:12:02 openstackgerrit ChangBo Guo(gcb) proposed openstack/nova: Enable more virt.xenapi tests on Python 3 https://review.openstack.org/414421
07:12:02 openstackgerrit ChangBo Guo(gcb) proposed openstack/nova: Enable virt.test_virt_drivers.AbstractDriverTestCase on Python 3 https://review.openstack.org/414419
07:12:03 openstackgerrit ChangBo Guo(gcb) proposed openstack/nova: Enable virt.vmwareapi test cases on Python https://review.openstack.org/414420
07:17:24 openstackgerrit huangtianhua proposed openstack/python-novaclient: Make _console() public https://review.openstack.org/414364
07:31:52 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330
08:10:45 openstackgerrit Roman Dobosz proposed openstack/nova-specs: Provide detailed error information for placement API https://review.openstack.org/418393
08:16:23 snikitin alex_xu: Hi, Alex! I updated diagnostics patch to fix problem with upgrade. https://review.openstack.org/#/c/355540/ Can you look, please?
08:31:34 trananhkma hi nova core reviewers
08:32:10 trananhkma I have 2 patches which have +2 from Stephen Finucane in last week
08:32:59 trananhkma would you mind helping me review these patches: https://review.openstack.org/#/c/322003/ https://review.openstack.org/#/c/321326/
08:43:25 openstackgerrit Andrey Volkov proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713
08:47:44 openstackgerrit Balazs Gibizer proposed openstack/nova: handle uninited fields in notification payload https://review.openstack.org/415857
08:52:33 openstackgerrit wanghao proposed openstack/nova: Set min_disk in the image meta based on the root volume's size https://review.openstack.org/407397
08:55:00 openstackgerrit Andrey Volkov proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713
08:57:58 openstackgerrit Dao Cong Tien proposed openstack/nova: Stop using mox in virt/xenapi/image/test_glance.py https://review.openstack.org/327457
09:11:57 openstackgerrit Yoshihiko Atsumi proposed openstack/nova: Add resize server functional negative tests https://review.openstack.org/420514
09:12:12 bauzas p!spy
09:13:31 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330
09:43:15 openstackgerrit Dao Cong Tien proposed openstack/nova: Stop using mox in compute/test_hypervisors.py https://review.openstack.org/326814
09:54:20 openstackgerrit Claudiu Belu proposed openstack/nova: WIP: Hyper-V PCI Passthrough https://review.openstack.org/420614
10:06:47 openstackgerrit Béla Vancsics proposed openstack/nova: Reduced the complexity of the _init_instance method (3) https://review.openstack.org/420619
10:18:33 openstackgerrit Lee Yarwood proposed openstack/nova: libvirt: workaround findmnt behaviour change https://review.openstack.org/420146
10:22:15 openstackgerrit Lee Yarwood proposed openstack/nova: libvirt: workaround findmnt behaviour change https://review.openstack.org/420146
10:23:09 lyarwood mdbooth: ^ that's the hackaround for the findmnt change we spoke about on Friday btw
10:23:46 mdbooth lyarwood: Did you report a bug against util-linux, btw?
10:24:36 openstack bugzilla.redhat.com bug 1405238 in util-linux "findmnt --target behaviour changed in 7.3, shows all mount-points in chroot" [Unspecified,Post] - Assigned to kzak
10:24:36 lyarwood mdbooth: someone already had - https://bugzilla.redhat.com/show_bug.cgi?id=1405238
10:25:23 mdbooth lyarwood: Reading. Might add REGRESSION to the bz title if it's not getting attention.
10:25:50 lyarwood mdbooth: seems to be POST'd for 7.4, I'll ask for 7.3.z now
10:28:19 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters https://review.openstack.org/420626
10:31:49 mdbooth lyarwood: Brilliant, thanks.
10:34:01 openstackgerrit Spencer Yu proposed openstack/nova: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/420629
10:39:32 takashin bauzas: alex_xu: Are you around? Could you review https://review.openstack.org/#/c/408955/ again and check my comments?
10:39:34 openstackgerrit Béla Vancsics proposed openstack/nova: Reduced the complexity of the _init_instance method (4) https://review.openstack.org/420632
10:44:55 mdbooth lyarwood: Reviewed. Would you like me to add the requested test case and resubmit? It'll only take a couple of minutes.
10:45:09 mdbooth Also the os.stat() question.
10:45:28 lyarwood mdbooth: yup sure
10:45:35 mdbooth Ok, will do.
10:48:42 openstackgerrit zhangyujun proposed openstack/nova: Query parameter validation for hosts API https://review.openstack.org/418880
10:50:12 lyarwood mdbooth: thanks btw, I've answered the os.stat question in the review, https://github.com/openstack/nova/blob/ad1c7ac2b102112280a25927d731edb168f80998/nova/virt/libvirt/volume/quobyte.py#L120 is the reason.
10:51:02 openstackgerrit ChangBo Guo(gcb) proposed openstack/nova: Set access_policy for messaging's dispatcher https://review.openstack.org/408494
10:51:23 lyarwood hmmmmm this might not work actually
10:53:11 lyarwood mdbooth: what do you make of L115 from the link above?
10:53:24 lyarwood mdbooth: I don't think that was even valid with findmnt
10:53:43 mdbooth lyarwood: I was just reading it more carefully myself.
10:53:50 mdbooth Surely it previously returned false?
10:53:57 mdbooth I don't think it's mocked in the test.
10:56:02 mdbooth lyarwood: That depends how quobyte volumes are known to mount
10:56:07 lyarwood mdbooth: I think mounted was False previously in this test so os.stat was nnenver called
10:56:39 mdbooth lyarwood: Right. I suspect it was false because it wasn't mocked, and therefore was never going to work in a test environment
10:57:50 mdbooth I think this test needs it to be false, though.
10:57:58 mdbooth That test is bizarre
10:58:06 mdbooth Still trying to wrap my head round what it's doing
10:59:05 mdbooth It's raising an exception if validate_volume() is called with any arguments?
10:59:20 mdbooth Why?
11:00:04 lyarwood validate_volume is calling command = ['getfattr', "-n", "quobyte.info", mnt_base] so I guess it's just mocking a failure of that

Earlier   Later