Code review comment for lp:~santhoshkumar/network-service/quantum_testing_framework

Revision history for this message
Santhosh Kumar Muniraj (santhoshkumar) wrote :

Hi Salvatore,

Thanks for the feedback. We fixed the ImportError and the usage of load_paste_app from config.py.

We had modified the load_paste_app as it has an advantage of loading the .conf file from a default set of locations similar to other Openstack projects. In addition to that, load_paste_app will also return the configuration variables defined in the .conf file which can be further used for other configurations.

In wsgi.py, the 'arg_dict' is already using 'request' key to pass the request variable to controller actions. After all these change we verified that app can be started and a simple request to get networks works fine.

We couldn't get the smoke tests passing both on the trunk and our branch. May be we are missing some setup on our side. All the functional tests were failing with the following error:

<code> TypeError: [NetworkInUse() is not JSON serializable]. </code>

We are not sure if these errors are due to our changes.

review: Needs Resubmitting

« Back to merge proposal