Re: [openstack-dev] [neutron] Runtime checks vs Sanity checks

2014-08-25 Thread Miguel Angel Ajo Pelayo

In spite of my +1 I actually agree. I had forgotten about the sanity
check framework. We put it in place to avoid an excessive (and
growing) amount of checks to be done in runtime.

In this case several agents need would be doing the same check.

We should do things either one way or another, but not mixed.

In this case, the check would require a privilege drop to be done:
(probably could be done by using a root_helper= 
"su - %(neutron_user)s %(root_helper)s")



Best regards,
Miguel Ángel.

- Original Message -
> Kevin Benton has proposed adding a runtime check for netns permission
> problems:
> 
> https://review.openstack.org/#/c/109736/
> 
> There seems to be consensus on the patch that this is something that we want
> to do at runtime, but that would seem to run counter to the precedent that
> host-specific issues such as this one be considered a deployment-time
> responsibility.  The addition of the sanity check  framework would seem to
> support the latter case:
> 
> https://github.com/openstack/neutron/blob/master/neutron/cmd/sanity_check.py
> 
> Thoughts?
> 
> 
> Maru
> 
> 
> ___
> OpenStack-dev mailing list
> OpenStack-dev@lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> 

___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [neutron] Runtime checks vs Sanity checks

2014-08-23 Thread Maru Newby
Kevin Benton has proposed adding a runtime check for netns permission problems:

https://review.openstack.org/#/c/109736/

There seems to be consensus on the patch that this is something that we want to 
do at runtime, but that would seem to run counter to the precedent that 
host-specific issues such as this one be considered a deployment-time 
responsibility.  The addition of the sanity check  framework would seem to 
support the latter case:

https://github.com/openstack/neutron/blob/master/neutron/cmd/sanity_check.py

Thoughts?


Maru


___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev