Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-13 Thread Russell Bryant
On 01/12/2014 05:30 PM, Dugger, Donald D wrote:
 So I have 25 patches that I need to push to backport changes that have
 been made to the nova tree that apply to the gantt tree.  The problem is
 how do we want to approve these patches?  Given that they have already
 been reviewed and approved in the nova tree do we have to go through the
 overhead of doing new reviews in the gantt tree and, if not, how do we
 bypass that mechanism?

For sync commits, how about we just allow +A with a single +2.

-- 
Russell Bryant

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


[openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-12 Thread Dugger, Donald D
So I have 25 patches that I need to push to backport changes that have been 
made to the nova tree that apply to the gantt tree.  The problem is how do we 
want to approve these patches?  Given that they have already been reviewed and 
approved in the nova tree do we have to go through the overhead of doing new 
reviews in the gantt tree and, if not, how do we bypass that mechanism?

--
Don Dugger
Censeo Toto nos in Kansa esse decisse. - D. Gale
Ph: 303/443-3786

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


Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-12 Thread Doug Hellmann
On Sun, Jan 12, 2014 at 5:30 PM, Dugger, Donald D donald.d.dug...@intel.com
 wrote:

  So I have 25 patches that I need to push to backport changes that have
 been made to the nova tree that apply to the gantt tree.  The problem is
 how do we want to approve these patches?  Given that they have already been
 reviewed and approved in the nova tree do we have to go through the
 overhead of doing new reviews in the gantt tree and, if not, how do we
 bypass that mechanism?


Why is code being copied from nova directly?

Doug





 --

 Don Dugger

 Censeo Toto nos in Kansa esse decisse. - D. Gale

 Ph: 303/443-3786



 ___
 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


Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-12 Thread Clint Byrum
Excerpts from Doug Hellmann's message of 2014-01-12 14:45:11 -0800:
 On Sun, Jan 12, 2014 at 5:30 PM, Dugger, Donald D donald.d.dug...@intel.com
  wrote:
 
   So I have 25 patches that I need to push to backport changes that have
  been made to the nova tree that apply to the gantt tree.  The problem is
  how do we want to approve these patches?  Given that they have already been
  reviewed and approved in the nova tree do we have to go through the
  overhead of doing new reviews in the gantt tree and, if not, how do we
  bypass that mechanism?
 
 
 Why is code being copied from nova directly?
 

I suspect because gantt forked a while ago, but development has been
allowed to continue in nova's scheduler code. Seems like that should be
stopped at some point soon to reduce the extra sync effort.

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


Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-12 Thread Jay Lau
According to
https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg12973.html,
it is said that we should not stop nova scheduler working for now.

BTW: I think that we may need to consider the following patches to enable
Nova use scheduler from Oslo first to reduce future work before sync up to
Gantt.

https://review.openstack.org/#/c/66105/
https://review.openstack.org/#/c/65418/
https://review.openstack.org/#/c/65424/

Thanks,

Jay


2014/1/13 Clint Byrum cl...@fewbar.com

 Excerpts from Doug Hellmann's message of 2014-01-12 14:45:11 -0800:
  On Sun, Jan 12, 2014 at 5:30 PM, Dugger, Donald D 
 donald.d.dug...@intel.com
   wrote:
 
So I have 25 patches that I need to push to backport changes that have
   been made to the nova tree that apply to the gantt tree.  The problem
 is
   how do we want to approve these patches?  Given that they have already
 been
   reviewed and approved in the nova tree do we have to go through the
   overhead of doing new reviews in the gantt tree and, if not, how do we
   bypass that mechanism?
  
 
  Why is code being copied from nova directly?
 

 I suspect because gantt forked a while ago, but development has been
 allowed to continue in nova's scheduler code. Seems like that should be
 stopped at some point soon to reduce the extra sync effort.

 ___
 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


Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-12 Thread Dugger, Donald D
Jay-

Those patches are not a problem.  Once they are approved and pushed into the 
nova tree I will backport them over to gantt.  To re-iterate, the idea is to 
keep the current development going in nova and only when the gantt tree is 
functional and provides the same functionality as the scheduler inside the nova 
will we consider moving all development to gantt.

--
Don Dugger
Censeo Toto nos in Kansa esse decisse. - D. Gale
Ph: 303/443-3786

From: Jay Lau [mailto:jay.lau@gmail.com]
Sent: Sunday, January 12, 2014 5:39 PM
To: OpenStack Development Mailing List (not for usage questions)
Subject: Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current 
nova tree

According to 
https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg12973.html, 
it is said that we should not stop nova scheduler working for now.
BTW: I think that we may need to consider the following patches to enable Nova 
use scheduler from Oslo first to reduce future work before sync up to Gantt.

https://review.openstack.org/#/c/66105/
https://review.openstack.org/#/c/65418/
https://review.openstack.org/#/c/65424/

Thanks,
Jay

2014/1/13 Clint Byrum cl...@fewbar.commailto:cl...@fewbar.com
Excerpts from Doug Hellmann's message of 2014-01-12 14:45:11 -0800:
 On Sun, Jan 12, 2014 at 5:30 PM, Dugger, Donald D 
 donald.d.dug...@intel.commailto:donald.d.dug...@intel.com
  wrote:

   So I have 25 patches that I need to push to backport changes that have
  been made to the nova tree that apply to the gantt tree.  The problem is
  how do we want to approve these patches?  Given that they have already been
  reviewed and approved in the nova tree do we have to go through the
  overhead of doing new reviews in the gantt tree and, if not, how do we
  bypass that mechanism?
 

 Why is code being copied from nova directly?

I suspect because gantt forked a while ago, but development has been
allowed to continue in nova's scheduler code. Seems like that should be
stopped at some point soon to reduce the extra sync effort.

___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.orgmailto: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


Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-12 Thread Jay Lau
Cool! Thanks Don.

Regards,

Jay


2014/1/13 Dugger, Donald D donald.d.dug...@intel.com

  Jay-



 Those patches are not a problem.  Once they are approved and pushed into
 the nova tree I will backport them over to gantt.  To re-iterate, the idea
 is to keep the current development going in nova and only when the gantt
 tree is functional and provides the same functionality as the scheduler
 inside the nova will we consider moving all development to gantt.



 --

 Don Dugger

 Censeo Toto nos in Kansa esse decisse. - D. Gale

 Ph: 303/443-3786



 *From:* Jay Lau [mailto:jay.lau@gmail.com]
 *Sent:* Sunday, January 12, 2014 5:39 PM
 *To:* OpenStack Development Mailing List (not for usage questions)
 *Subject:* Re: [openstack-dev] [gantt[ Patches to sync gantt up to the
 current nova tree



 According to
 https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg12973.html,
 it is said that we should not stop nova scheduler working for now.

 BTW: I think that we may need to consider the following patches to enable
 Nova use scheduler from Oslo first to reduce future work before sync up to
 Gantt.

 https://review.openstack.org/#/c/66105/
 https://review.openstack.org/#/c/65418/
 https://review.openstack.org/#/c/65424/


 Thanks,

 Jay



 2014/1/13 Clint Byrum cl...@fewbar.com

 Excerpts from Doug Hellmann's message of 2014-01-12 14:45:11 -0800:

  On Sun, Jan 12, 2014 at 5:30 PM, Dugger, Donald D 
 donald.d.dug...@intel.com
   wrote:
 
So I have 25 patches that I need to push to backport changes that have
   been made to the nova tree that apply to the gantt tree.  The problem
 is
   how do we want to approve these patches?  Given that they have already
 been
   reviewed and approved in the nova tree do we have to go through the
   overhead of doing new reviews in the gantt tree and, if not, how do we
   bypass that mechanism?
  
 
  Why is code being copied from nova directly?
 

 I suspect because gantt forked a while ago, but development has been
 allowed to continue in nova's scheduler code. Seems like that should be
 stopped at some point soon to reduce the extra sync effort.


 ___
 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 mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-12 Thread Sylvain Bauza
I think we should still review the patches in case of any merging issues
with Gantt code. If the volume of reviews is high, then we have to increase
the number of reviewers.

I was part of the discussions for an external scheduler, I will dedicate
some time for +1...

Afaik, the current weekly meeting for Gantt is during sched subteam
meeting, correct ?
Le 12 janv. 2014 23:33, Dugger, Donald D donald.d.dug...@intel.com a
écrit :

  So I have 25 patches that I need to push to backport changes that have
 been made to the nova tree that apply to the gantt tree.  The problem is
 how do we want to approve these patches?  Given that they have already been
 reviewed and approved in the nova tree do we have to go through the
 overhead of doing new reviews in the gantt tree and, if not, how do we
 bypass that mechanism?



 --

 Don Dugger

 Censeo Toto nos in Kansa esse decisse. - D. Gale

 Ph: 303/443-3786



 ___
 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


Re: [openstack-dev] [gantt[ Patches to sync gantt up to the current nova tree

2014-01-12 Thread Robert Collins
On 13 January 2014 11:45, Doug Hellmann doug.hellm...@dreamhost.com wrote:



 On Sun, Jan 12, 2014 at 5:30 PM, Dugger, Donald D
 donald.d.dug...@intel.com wrote:

 So I have 25 patches that I need to push to backport changes that have
 been made to the nova tree that apply to the gantt tree.  The problem is how
 do we want to approve these patches?  Given that they have already been
 reviewed and approved in the nova tree do we have to go through the overhead
 of doing new reviews in the gantt tree and, if not, how do we bypass that
 mechanism?


 Why is code being copied from nova directly?

Because (like for cinder) during the migration period we're
maintaining two copies one in Nova and one in Gantt. If we're
successful then once  I is out we can delete the Nova copy.

-Rob

-- 
Robert Collins rbtcoll...@hp.com
Distinguished Technologist
HP Converged Cloud

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