Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-12
04:05:20 mriedem edleafe: got it cleaned up with regex magic
04:12:59 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle special characters in database connection URL netloc https://review.openstack.org/464088
04:13:00 mriedem smcginnis: thanks for the regex suggestion ^
05:22:31 deepak hi i am facing error status of Newly VM created on my multi node setup
05:22:49 deepak error :"No valid host was found. There are not enough hosts available "
05:52:14 openstackgerrit Ritesh proposed openstack/nova master: rbd: flatten images when unshelving https://review.openstack.org/457886
06:33:53 openstackgerrit Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023
08:02:35 openstackgerrit Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi v2 in nova https://review.openstack.org/453493
08:04:56 openstackgerrit D G Lee proposed openstack/nova master: Force to evacuate ACTIVE status instances failed if a host is provided https://review.openstack.org/463952
08:12:00 openstackgerrit Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023
08:12:45 openstackgerrit D G Lee proposed openstack/nova master: Force to evacuate ACTIVE status instances failed if a host is provided https://review.openstack.org/463952
08:35:35 openstackgerrit chenaidong1 proposed openstack/nova master: Trivial fix https://review.openstack.org/464154
08:56:19 openstackgerrit D G Lee proposed openstack/nova master: Force to evacuate ACTIVE status instances failed if a host is provided https://review.openstack.org/463952
09:11:34 openstackgerrit D G Lee proposed openstack/nova master: Force to evacuate ACTIVE status instances failed if a host is provided https://review.openstack.org/463952
10:03:40 openstackgerrit falseuser proposed openstack/nova master: [BugFix] Include the video ram when comparing the difference of two flavors https://review.openstack.org/460388
10:45:30 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3) https://review.openstack.org/452820
10:45:30 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818
10:45:31 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819
12:15:10 sean-k-mooney sfinucan: hi are you wake?
12:18:43 sean-k-mooney sfinucan: ill take that as a no. do you know if https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4001-L4069 is the only place the vcpu pin elements are generated?
12:19:44 sean-k-mooney sfinucan: nova is currently generating them in the libvirt xml when useing any libvirt backend but they are not supported by qemu without kvm e.g. virt type qemu in the nova conf
12:25:27 kashyap sean-k-mooney: That sounds like a bug
12:25:49 sean-k-mooney yep i think it is and im trying to figure out how to fix it
12:25:53 kashyap If the guest XML is generated for backends that can't / don't use it
12:26:14 kashyap Probably a conditional that, generate it, only if it's 'virt_type = kvm'?
12:26:59 sean-k-mooney kashyap: yep that would fix it im just looking at the libvirt/driver.py and trying to figure out exactly where to add it
12:28:14 sean-k-mooney the issue is that we auto matically generage vcpu pinning when you request a numa topology and we automaticlaly generate a numa topology if we request huge pages hence none of these features work with qemu as a backend since cpu pinning does not work
14:04:46 openstackgerrit Kevin L. Mitchell proposed openstack/python-novaclient master: Mark cloudpipe deprecated in novaclient https://review.openstack.org/464068
14:30:56 mriedem sfinucan: replied in https://review.openstack.org/#/c/349014/ - hope that makes sense
14:47:39 sean-k-mooney sfinucan: i figured out how to modify nova to not do set the invalid config element so ignore my previous messages. ill add you to the review when i open the bug and submit the patch
15:31:21 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: remove glusterfs volume driver https://review.openstack.org/463992
15:31:23 mriedem lyarwood: I wanted to keep those test_config tests, just changed the source_protocol from gluster to netfs ^
15:42:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3) https://review.openstack.org/452820
15:42:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818
15:42:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819
16:28:53 openstackgerrit John Griffith proposed openstack/python-novaclient master: Fix help message and description for volume-update https://review.openstack.org/464232
16:39:05 openstackgerrit John Griffith proposed openstack/python-novaclient master: Fix help message and description for volume-update https://review.openstack.org/464232
16:56:59 abhishek can someone please review https://review.openstack.org/#/c/462561/
16:58:16 openstackgerrit John Griffith proposed openstack/python-novaclient master: Fix help message and description for volume-update https://review.openstack.org/464232
17:21:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3) https://review.openstack.org/452820
17:21:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818
17:21:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819
17:34:41 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle special characters in database connection URL netloc https://review.openstack.org/464088
17:34:43 mriedem melwitt: dansmith (superdan): here is the fix for that upgrade issue with simple_cell_setup and special characters in the db connection url ^; we'll want to backport that to newton
17:35:18 mriedem i should probably add a 'fixes' reno on that for the backports
17:45:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle special characters in database connection URL netloc https://review.openstack.org/464088
18:00:01 mriedem Vek: if you're around can you get this one again https://review.openstack.org/#/c/461448/ you were +2 before
18:05:21 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Fix help message and description for volume-update https://review.openstack.org/464232
18:17:49 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Fix warning for deprecated cert commands https://review.openstack.org/464249
18:29:28 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Fix warning for deprecated cert commands https://review.openstack.org/464249
20:30:21 mriedem if there are any other cores around, it'd be good to get this novaclient change in since it fixes the functional tests which are blocking all other novaclient changes right now https://review.openstack.org/#/c/464068/
20:36:05 mriedem mtreinish: https://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template
20:37:48 mtreinish mriedem: sure, but if I was reading the nova docs how would I know to look in devstack?
20:39:52 mriedem mtreinish: you wouldn't :)
20:39:57 mriedem mtreinish: i need to link that in somewhere,
20:40:07 mriedem either nova devref for placement, or install guide, or both
20:41:03 mriedem mtreinish: you should report a bug somewhere
20:45:08 mtreinish mriedem: I will, it's on my list of things to do post summit
20:46:49 mriedem mtreinish: btw Vek deprecated the cloudpipe stuff in novaclient
20:46:56 mriedem that's https://review.openstack.org/#/c/464068/
21:30:59 stewie925 hello all, would like to hear your input regarding this comment: https://review.openstack.org/#/c/385644/33/nova/tests/unit/compute/test_compute_mgr.py
21:31:35 stewie925 am not sure how to assert the mock_notify in that paragraph
21:33:14 dtp stewie925 the one at line 2596?
21:33:59 stewie925 dtp yeah
21:34:14 stewie925 he had another comment at line 2637
21:34:18 stewie925 has
21:35:17 stewie925 let me share what I did... will post the link here.... I get errors when I tried to do mock_notify in that paragraph - brb
21:35:30 dtp i think he's asking you to ...ok
21:37:03 dtp also brb
21:39:12 dtp back
21:40:28 stewie925 dtp: here is the code change and the error I got - http://paste.openstack.org/show/609476/
21:40:34 stewie925 thank you!
21:44:12 dtp don't thank me yet! :P
21:45:51 stewie925 no rush hehe
21:46:43 dtp stewie925 if you look at the very bottom, it prints out what you thought the calls should be and what they actually are. you need to diff them to find what's different, then either adjust you test or figure out and fix why it's different
21:47:02 dtp for instance, it looks like [rescue_image_ref=None] is in the actual, but not your expected
21:48:42 stewie925 dtp: yes, that rescue_image_ref - my head hurts lol... will look further into that
21:49:06 stewie925 like I should pass the rescue_image_ref or something
21:49:32 dtp yeah, if that's what the notification call should look like, then just add that param to your mock.call
21:57:16 stewie925 dtp: thanks again - will work on that now (too many distractions lol)
21:57:30 dtp you're welcome
21:57:49 stewie925 whoo hoo no failures now after I added the rescue_image_ref
21:58:10 dtp * high five *
22:03:09 stewie925 dtp-afk: high-fives back, even while afk :D
22:21:36 openstackgerrit STEW TY proposed openstack/nova master: Transform rescue/unrescue instance notifications https://review.openstack.org/385644
22:31:46 stewie925 have a great weekend all! good night!
22:31:56 dtp night!
22:46:31 openstackgerrit Monty Taylor proposed openstack/nova master: WIP Show results of nested parameters in api-ref https://review.openstack.org/464277
22:50:36 openstackgerrit Monty Taylor proposed openstack/nova master: DNM Show results of nested parameters in api-ref https://review.openstack.org/464277
23:57:52 openstackgerrit Dan Peschman proposed openstack/nova master: Use uuid for id in Services API https://review.openstack.org/464280
#openstack-nova - 2017-05-13
05:03:05 abhi89 dansmith: can you please review https://review.openstack.org/#/c/462561/
14:14:56 openstackgerrit Monty Taylor proposed openstack/nova master: Show results of nested parameters in api-ref https://review.openstack.org/464277
14:49:10 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3) https://review.openstack.org/452820
14:49:10 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818
14:49:11 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819
14:52:32 abhi89 melwitt: can you please review https://review.openstack.org/#/c/462561/
15:15:51 openstackgerrit falseuser proposed openstack/nova master: [BugFix] Change the parameter of the exception error message https://review.openstack.org/461985
17:14:03 abhi89 bauzas: can you please review https://review.openstack.org/#/c/462561/
18:25:24 openstackgerrit Dan Peschman proposed openstack/nova master: Use uuid for id in Services API https://review.openstack.org/464280
22:54:19 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Fix help message and description for volume-update https://review.openstack.org/464232

Earlier   Later