Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-11
21:38:47 openstackgerrit Ed Leafe proposed openstack/nova master: Make the method to put allocations public https://review.openstack.org/464086
21:38:57 edleafe ^^ Hey placement folks: here is the stuff to make the scheduler do claims
21:39:37 edleafe ^^ cdent, jaypipes, bauzas, dansmith
21:39:58 edleafe I'll be on PTO Friday
21:50:18 mriedem edleafe: know of a fancy way to encode a url that has ? in the creds in the netloc?
21:50:29 mriedem so that when you split it, it doesn't think the ? is part of the path
21:50:47 mriedem e.g. mysql+pymysql://nova:abcd0123?AB@controller/nova
21:58:30 edleafe mriedem: can't say I do, but let me play with it. I assume you've tried urlparse?
21:59:17 mriedem yup
21:59:20 mriedem and urlencode
21:59:24 mriedem well, urllib things
21:59:37 mriedem urlparse is kind of the problem, it splits on the ?
21:59:50 mriedem so you get
21:59:51 mriedem SplitResult(scheme='mysql+pymysql', netloc='nova:abcd0123', path='', query='AB@controller/nova', fragment='')
22:02:22 edleafe yeah, well, that's how it's supposed to work :)
22:04:35 mriedem right. urllib.quote doesn't seem to be very helpful though
22:06:39 mriedem SplitResult(scheme='mysql+pymysql', netloc='nova:abcd0123%3FAB@controller', path='/nova', query='', fragment='')
22:06:39 mriedem >>> urlparse.urlsplit(urllib.quote_plus(connection, safe="%/:=&~#+!$,;'@()*[]"))
22:07:06 mriedem that pretty much sucks...but it fixes this particular issue
22:07:30 mriedem if your password has @ or : or / in it, still a problem
22:07:42 edleafe it isn't a hack if that's the only way to make it work :)
22:20:21 lyarwood mriedem: looks like you missed a few references to gluster in the support-matrix and nova/tests/unit/virt/libvirt/test_config.py , would you mind if I respin?
22:20:49 pooja Hi, Can someone comment on impact of enabling neutron metadata agent on nova guest ssh-key injection? We had ssh keys working fine for root user earlier but since we enabled metadata service, the vm's root user authorized_keys file contains this restriction:
22:20:59 pooja # The following ssh key was injected by Nova no-port-forwarding,no-agent-forwarding,no-X11-forwarding,command="echo 'Please login as the user \"cloud-user\" rather than the user \"root\".';echo;sleep 10" ssh-rsa xxx
22:21:07 mriedem lyarwood: sure
22:21:36 pooja In nova-compute, I don't see these restrictions being added while debugging, so trying to understand what prepends that to key_data
22:21:46 mriedem edleafe: i think i've got something
22:21:53 mriedem which handles all of those thigns
22:26:04 edleafe mriedem: do tell
22:27:51 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: remove glusterfs volume driver https://review.openstack.org/463992
22:28:19 mriedem wip :)
22:28:31 mriedem i'll post quick and then have to run
22:30:17 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Handle special characters in database connection URL netloc https://review.openstack.org/464088
22:30:19 mriedem ^ need to tease it out a bit but that's the general idea
22:32:52 edleafe mriedem: doesn't look as hairy as I had imagined
22:53:36 openstackgerrit Ed Leafe proposed openstack/nova master: Pass all instance IDs to scheduler https://review.openstack.org/463623
23:32:47 openstackgerrit Ed Leafe proposed openstack/nova master: Pass all instance UUIDs to scheduler https://review.openstack.org/463623
#openstack-nova - 2017-05-12
01:07:29 openstackgerrit Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023
03:24:31 openstackgerrit Ritesh proposed openstack/nova master: rbd: flatten images when unshelving https://review.openstack.org/457886
03:47:34 openstackgerrit Ed Leafe proposed openstack/nova master: Have scheduler claim resources https://review.openstack.org/463993
03:47:35 openstackgerrit Ed Leafe proposed openstack/nova master: Make the method to put allocations public https://review.openstack.org/464086
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 https://review.openstack.org/452818
10:45:30 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3) https://review.openstack.org/452820
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 https://review.openstack.org/452818
15:42:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3) https://review.openstack.org/452820
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 (2) https://review.openstack.org/452819
17:21:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818
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

Earlier   Later