[Starlingx-discuss] Analysis of patch 039705e for StarlingX upstreaming

Qin, Kailun kailun.qin at intel.com
Tue Nov 6 14:38:25 UTC 2018


Allain,

Thanks a lot for the confirmation and clarification. Yes, the host driver is being redesigned. I'll drop the patch accordingly.

BR,
Kailun

From: Legacy, Allain [mailto:Allain.Legacy at windriver.com]
Sent: Tuesday, November 6, 2018 8:16 PM
To: Qin, Kailun <kailun.qin at intel.com>; Peters, Matt <Matt.Peters at windriver.com>; starlingx-discuss at lists.starlingx.io
Subject: RE: [Starlingx-discuss] Analysis of patch 039705e for StarlingX upstreaming

I agree.   These changes can all be dropped.

The keystone config opts change was needed for the host driver to communicate back to our sysinv subsystem, but, as far as I know, that is being redesigned to avoid this requirement.

Regards,
Allain


Allain Legacy, Software Developer, Wind River
direct 613.270.2279  fax 613.492.7870 skype allain.legacy
350 Terry Fox Drive, Suite 200, Ottawa, Ontario, K2K 2W5


[WIND]<http://www.windriver.com/>

From: Qin, Kailun [mailto:kailun.qin at intel.com]
Sent: Tuesday, November 06, 2018 2:10 AM
To: Peters, Matt; starlingx-discuss at lists.starlingx.io<mailto:starlingx-discuss at lists.starlingx.io>
Subject: [Starlingx-discuss] Analysis of patch 039705e for StarlingX upstreaming

Hi Matt,

We're looking into the patch 039705e for upstreaming, which contains several base changes.


1.      For the oslo.db related change, upstream has already dropped this override in neutron-server to make it use default values from oslo.db (see https://review.openstack.org/456419/ for detailed information).

2.      For the keystone opts registration, I guess it is used for connecting to external services. From upstream's perspective, this should be maintained along with the specific service to extend rather than make it globally.

3.      For the others, I think they're StarlingX specific.
Thus, I suggest that we should not upstream these changes.

What do you think? Let me know if any question, thanks!

BR,
Kailun
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.starlingx.io/pipermail/starlingx-discuss/attachments/20181106/3aeefeb9/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image001.png
Type: image/png
Size: 1807 bytes
Desc: image001.png
URL: <http://lists.starlingx.io/pipermail/starlingx-discuss/attachments/20181106/3aeefeb9/attachment-0001.png>


More information about the Starlingx-discuss mailing list