| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-21 | |||
| 15:52:34 | mriedem | http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/57/490057/25/check/dsvm-tempest-neutron-network/c802d48/logs/devstacklog.txt.gz | |
| 15:52:40 | mriedem | TypeError: __init__() got an unexpected keyword argument 'retry_on_request' | |
| 15:53:05 | efried | mordred Can you point me to a doc somewhere that says discovery URIs should be auth-less? | |
| 15:55:16 | mriedem | weird that TypeError was fixed a few months ago Ic98f8cfeb6f78a7e066613787e7cbcea02fcec5c | |
| 15:55:25 | mriedem | so why is the xenserver ci failing on that now? | |
| 15:56:52 | mriedem | Skip the change: openstack/nova:master:refs/changes/57/490057/25 | |
| 15:56:53 | mriedem | wtf | |
| 15:59:08 | mriedem | https://review.openstack.org/#/c/490057/ probably just hasn't been rebased in forever | |
| 16:00:41 | mordred | efried: https://review.openstack.org/#/c/459710/17/guidelines/discoverability.rst - line 90 | |
| 16:01:06 | mordred | efried: it hasn't landed yet because I still need to do another pass | |
| 16:01:57 | mriedem | efried: there https://review.openstack.org/#/c/490057/ | |
| 16:02:03 | jianghuaw | mriedem, yeah. That patch may need a rebased. I met one similar failure some days ago. | |
| 16:02:54 | mordred | efried: but, tl;dr - it's fundamental metadata about a service and there is nothing about it that is different per user or secret in any way, so protecting it behind auth makes things harder for no benefit | |
| 16:03:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 16:03:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | |
| 16:03:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | |
| 16:03:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 16:05:06 | jianghuaw | jaypipes, bauzas: may you help to check this patch? https://review.openstack.org/#/c/518294/ I think we got some discussion on it before. But do need your review. Eric has +1 on it. | |
| 16:06:08 | jaypipes | jianghuaw: yes, will get to that this afternoon. | |
| 16:06:20 | jianghuaw | jaypipes, thanks:-) | |
| 16:06:58 | efried | mordred Thanks | |
| 16:07:26 | jaypipes | efried: do you have any local changes on that refactor series? cuz I want to fix up the one thing alex_xu wants to see w.r.t. trait validation. | |
| 16:07:36 | efried | jaypipes Nope, it's all you. | |
| 16:07:43 | jaypipes | roger | |
| 16:07:51 | efried | jaypipes You saw my comment there too? | |
| 16:07:55 | jaypipes | yup | |
| 16:08:08 | jaypipes | efried: will try to accommodate both of you | |
| 16:08:12 | efried | :) | |
| 16:09:12 | jaypipes | efried: shall I rebase all the way to https://review.openstack.org/#/c/521189/ ? | |
| 16:09:29 | efried | jaypipes Yes please. | |
| 16:09:32 | jaypipes | you got it. | |
| 16:14:09 | jianghuaw | efried, jaypipes: After switching to update_provider_tree(), the implementation in driver should cover the things done in _normalize_inventory_from_cn_obj(). right? | |
| 16:14:34 | efried | jianghuaw That's my position, yes. Others have yet to confirm. | |
| 16:14:48 | jaypipes | jianghuaw: yep, exactly correct. | |
| 16:15:15 | jianghuaw | efried, jaypipes thanks. XenAPI will go with that way. Cheers:-) | |
| 16:15:52 | efried | jianghuaw Note that it's gonna be a while before update_provider_tree is ready... | |
| 16:16:14 | efried | jianghuaw Lots of code underneath it that jaypipes and I are working on. | |
| 16:17:28 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: GET /RPs with aggs, inventories, traits https://review.openstack.org/521875 | |
| 16:18:05 | jianghuaw | efried, I see. the change will depends your patches and Jay's patches. Just also try to prepare thing as much as possible for XenAPI. | |
| 16:18:47 | efried | jianghuaw Yup, that's a good idea. Just be prepared to be patient (and flexible - the API very well might change). | |
| 16:19:13 | openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | |
| 16:19:42 | jianghuaw | efried, sure. Thanks for the heads-up. | |
| 16:21:05 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386 | |
| 16:24:03 | moshele | stephenfin: respond to you comment in https://review.openstack.org/#/c/519066/ | |
| 16:25:19 | stephenfin | moshele: replied | |
| 16:26:02 | stephenfin | ralonsoh: Rebased your 'pyroute2' patch for os-vif. If you can move that to a lib dir, I think that should keep both sean-k-mooney and sahid happy | |
| 16:27:43 | ralonsoh | stephenfin: I'll do this, thanks | |
| 16:28:51 | moshele | stephenfin: so you want message like https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L517 ? | |
| 16:29:34 | stephenfin | moshele: Exactly. Something to give a little more context (and make me more comfortable with ignoring the exception) | |
| 16:29:53 | moshele | stephenfin: undestood | |
| 16:30:11 | openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | |
| 16:30:16 | efried | mriedem ^ | |
| 16:31:07 | openstackgerrit | Merged openstack/nova master: Deduplicate keypair notification samples https://review.openstack.org/521562 | |
| 16:31:39 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for neutron client https://review.openstack.org/509892 | |
| 16:31:55 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for keystone conf & requests https://review.openstack.org/507693 | |
| 16:32:22 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345 | |
| 16:32:38 | niraj_singh | i am working on service token for cinder nova interaction. | |
| 16:32:39 | niraj_singh | Please find the log :http://paste.openstack.org/show/626966/ | |
| 16:32:39 | niraj_singh | while creating snapshot i am getting error at compute side. | |
| 16:32:39 | niraj_singh | i am creating volume snapshot that is attached to an instance. | |
| 16:32:39 | niraj_singh | Conf option for nova and cinder: http://paste.openstack.org/show/626972/ | |
| 16:32:39 | niraj_singh | Cinder backend is NFS and Nova backend is LVM. | |
| 16:32:41 | niraj_singh | Any help? | |
| 16:33:18 | bauzas | ralonsoh: https://pypi.python.org/pypi/os-traits/0.4.0 is there | |
| 16:34:07 | bauzas | ralonsoh: could you please provide a new patchset that would use it and just depend on https://review.openstack.org/#/c/521883/ ? | |
| 16:34:23 | ralonsoh | bauzas: i'll submit another patch for https://review.openstack.org/#/c/466051/ | |
| 16:34:29 | bauzas | ralonsoh: cool thanks | |
| 16:37:56 | efried | sdague I think this was one you and I talked about: https://review.openstack.org/#/c/512329/ | |
| 16:43:32 | gibi | jianghuaw: I see I missed the party in https://review.openstack.org/#/c/512580 and now it is on the gate. Sorry for not having the bandwidth to check it earlier | |
| 16:48:52 | openstackgerrit | Moshe Levi proposed openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/519066 | |
| 16:49:45 | moshele | stephenfin: done ^ | |
| 16:52:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027 | |
| 16:52:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646 | |
| 16:52:14 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617 | |
| 16:52:14 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | |
| 16:52:14 | openstackgerrit | Jay Pipes proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119 | |
| 16:52:15 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | |
| 16:52:15 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | |
| 16:52:16 | openstackgerrit | Jay Pipes proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | |
| 16:52:18 | jaypipes | efried: et voila. ^ | |
| 16:52:25 | efried | jaypipes ack | |
| 16:54:11 | sean-k-mooney | jaypipes: did you just push shared RP support or just traits to model shared and non shared RPs? | |
| 16:57:32 | sean-k-mooney | stephenfin: regarding embedded niantics i believe Intel sells embedded mezzanine form factor cards to dell/hp and there is also a version that is integrated into the motherboard too. | |
| 16:57:55 | efried | sean-k-mooney We're kinda working shared in as we go - at least to the point of maintaining existing functionality. Though we won't officially "support" it in Q, I don't imagine. | |
| 16:58:32 | stephenfin | dansmith: Could you take a look at this? https://review.openstack.org/#/c/396184/ Think it's something I should backport after | |
| 16:59:14 | dansmith | stephenfin: oh gawd | |
| 16:59:21 | dansmith | stephenfin: yeah, will look when I get off this call | |
| 16:59:26 | stephenfin | (y) | |
| 16:59:28 | sean-k-mooney | efried: ah cool. | |
| 17:03:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix NoneType error when [service_user] is misconfigured https://review.openstack.org/521947 | |
| 17:03:47 | mriedem | efried: you'll like this ^ let's not fail with NoneType errors during re-auth | |
| 17:04:01 | mriedem | niraj_singh had a real issue and then left | |
| 17:04:43 | efried | mriedem https://review.openstack.org/#/c/512329/ | |
| 17:05:05 | sean-k-mooney | stephenfin: so looking at that you infer the request for pinning by checking "self.cpu_pinning is not None" what dose self.cpu_pinning contain? will that check work if i explcitly set the policy to shared in the flavour | |
| 17:05:36 | stephenfin | sean-k-mooney: We _used_ to do that, then we added a field to actually store the policy | |
| 17:05:47 | efried | mriedem And yeah, I just sent him an email. | |
| 17:06:19 | stephenfin | With recent object versions, we check the policy field. However, that patch ensure the older object versions continue to work | |
| 17:06:22 | efried | mriedem I bet he didn't realize the conf split that happened a couple months ago, and has his stuff in the wrong conf file. | |
| 17:06:50 | mriedem | efried: replied on yours | |
| 17:06:54 | mriedem | i don't like raising a random exception here | |