Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-16
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 virt.test_virt_drivers.AbstractDriverTestCase on Python 3 https://review.openstack.org/414419
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: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 lyarwood mdbooth: someone already had - https://bugzilla.redhat.com/show_bug.cgi?id=1405238
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: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
11:00:21 lyarwood so it doesn't really matter what we do in connect_volume as long as the volume is mounted in the end
11:00:26 lyarwood urgh
11:00:32 mdbooth It's a terrible test
11:00:58 mdbooth I wouldn't rathole on it. I'd change the is_mounted mock to return False, because that's the previous behaviour, and remove the os.stat mock.
11:01:10 mdbooth Fixing it isn't what you're doing here.
11:01:16 lyarwood yeah true
11:01:16 mdbooth Would you like me to make that change, too?
11:01:22 lyarwood if you have everything open already
11:01:26 mdbooth Yeah.
11:01:29 lyarwood cool thanks
11:06:34 openstackgerrit Takashi NATSUME proposed openstack/nova-specs: Fix formatting errors and warnings https://review.openstack.org/407364
11:18:01 mdbooth lyarwood: Trying trying to work out what that quobyte test did before in case we're missing something.
11:18:36 mdbooth I wonder if it wasn't doing what the author thought it was doing.
11:19:36 openstackgerrit Chris Dent proposed openstack/nova: [placement] add api-ref for GET /resource_providers https://review.openstack.org/418503
11:19:50 lyarwood mdbooth: meh, it might be an older test that tested something useful in the past that's just been left in tbh
11:20:09 lyarwood mdbooth: asserting that we raise the exception that's thrown by the final call in the method is just odd
11:20:19 lyarwood mdbooth: what else could we possibly do?
11:20:31 mdbooth Yeah, I wonder if the exception was raise by something else
11:20:38 mdbooth Checking now, shouldn't take too long
11:20:53 lyarwood kk
11:20:56 lyarwood thanks again
11:25:27 mdbooth lyarwood: The previous code returned True, and didn't handle the os.stat() error in the test
11:25:54 mdbooth It returned true because of fake_execute
11:26:01 lyarwood ahhhh
11:26:37 mdbooth The os.stat() error is handled by the surrounding code
11:27:29 mdbooth Yup, the os.stat() mock can just be removed
11:27:39 mdbooth It remains a terrible test
11:30:05 mdbooth lyarwood: You never did add anything to openstack-dev-hacks, btw
11:31:11 lyarwood mdbooth: yeah sorry keep meaning to
11:31:25 lyarwood mdbooth: I'll try to get around to it this morning
11:31:43 mdbooth Don't both spending time cleaning it up, btw, just chuck it in there
11:31:56 mdbooth We can cleanup later if required
11:32:10 lyarwood yeah I wanted to remove the grep foo | grep bar crap etc ;)
11:32:49 mdbooth Meh, just drop it in there ;)
11:32:59 openstackgerrit Matthew Booth proposed openstack/nova: libvirt: workaround findmnt behaviour change https://review.openstack.org/420146
11:46:23 lyarwood mdbooth: pushed, also dumped my current crop of git aliases in there, not sure it's that useful so feel free to rm
11:47:36 markus_z Would someone please review a tempest test for l-m: https://review.openstack.org/#/c/346815/22 ?
11:51:38 markus_z tonyb: Would you have a look at the newton backports for bug 1455252 please? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:%22bug+1455252%22
11:51:38 openstack bug 1455252 in OpenStack Compute (nova) "enabling serial console breaks live migration" [High,Fix released] https://launchpad.net/bugs/1455252 - Assigned to Markus Zoeller (markus_z) (mzoeller)
11:52:18 lyarwood markus_z: I can take a look this afternoon
11:52:42 markus_z lyarwood: Thanks, that would be awesome.
11:57:33 mdbooth lyarwood: Thanks!

Earlier   Later