Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[Corey Bryant]

On Thu, Feb 16, 2017 at 1:30 AM, Chen CH Ji  wrote:

> Should be this one https://bugs.launchpad.net/nova/+bug/1664913 and
> *https://review.openstack.org/434179*
>  ?
>
> Hey Kevin, sorry I didn't notice your patch/bug.  It looks like this is on
it's way to getting merged:  https://review.openstack.org/#/c/434533
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[Chen CH Ji]

Should be this one https://bugs.launchpad.net/nova/+bug/1664913 and
https://review.openstack.org/434179 ?

Best Regards!

Kevin (Chen) Ji 纪 晨

Engineer, zVM Development, CSTL
Notes: Chen CH Ji/China/IBM@IBMCN   Internet: jiche...@cn.ibm.com
Phone: +86-10-82451493
Address: 3/F Ring Building, ZhongGuanCun Software Park, Haidian District,
Beijing 100193, PRC



From:   Corey Bryant <corey.bry...@canonical.com>
To: "OpenStack Development Mailing List (not for usage questions)"
<openstack-dev@lists.openstack.org>
Date:   02/16/2017 05:37 AM
Subject:        Re: [openstack-dev] [nova] scaling rabbitmq with cells v2
    requires manual database update





On Wed, Feb 15, 2017 at 4:34 PM, Corey Bryant <corey.bry...@canonical.com>
wrote:

  On Wed, Feb 15, 2017 at 4:26 PM, melanie witt <melwi...@gmail.com> wrote:
   On Wed, 15 Feb 2017 16:12:14 -0500, Corey Bryant wrote:
 This works but you have to specify all of the args (--cell_uuid,
 --name,
 --transport-url and --database_connection).  Otherwise you'll hit
 this: http://paste.ubuntu.com/24003055/

   Corey,

   Thanks for pointing that out.

   It looks like the command intended to have --transport-url and
   --database_connection as optional but missed defaulting them to None in
   the function parameter list.

   -melanie

  I think I figured it out.  I'll submit a patch in a little bit.



You're correct btw :)


--
Regards,
Corey
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[melanie witt]

On Wed, 15 Feb 2017 16:34:57 -0500, Corey Bryant wrote:

I think I figured it out.  I'll submit a patch in a little bit.


Great, post a link to the patch after you upload it and we'll review.

Also note earlier I forgot to mention --name should also be optional for 
the command (everything except --cell_uuid). I think you could also get 
fancy and require at least one of those things, otherwise it's just a no-op.


-melanie

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[Corey Bryant]

On Wed, Feb 15, 2017 at 4:34 PM, Corey Bryant 
wrote:

>
> On Wed, Feb 15, 2017 at 4:26 PM, melanie witt  wrote:
>
>> On Wed, 15 Feb 2017 16:12:14 -0500, Corey Bryant wrote:
>>
>>> This works but you have to specify all of the args (--cell_uuid, --name,
>>> --transport-url and --database_connection).  Otherwise you'll hit
>>> this: http://paste.ubuntu.com/24003055/
>>>
>>
>> Corey,
>>
>> Thanks for pointing that out.
>>
>> It looks like the command intended to have --transport-url and
>> --database_connection as optional but missed defaulting them to None in the
>> function parameter list.
>>
>> -melanie
>
>
> I think I figured it out.  I'll submit a patch in a little bit.
>
>
>
You're correct btw :)


-- 
Regards,
Corey
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[Corey Bryant]

On Wed, Feb 15, 2017 at 4:26 PM, melanie witt  wrote:

> On Wed, 15 Feb 2017 16:12:14 -0500, Corey Bryant wrote:
>
>> This works but you have to specify all of the args (--cell_uuid, --name,
>> --transport-url and --database_connection).  Otherwise you'll hit
>> this: http://paste.ubuntu.com/24003055/
>>
>
> Corey,
>
> Thanks for pointing that out.
>
> It looks like the command intended to have --transport-url and
> --database_connection as optional but missed defaulting them to None in the
> function parameter list.
>
> -melanie


I think I figured it out.  I'll submit a patch in a little bit.

-- 
Regards,
Corey
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[melanie witt]

On Wed, 15 Feb 2017 16:12:14 -0500, Corey Bryant wrote:

This works but you have to specify all of the args (--cell_uuid, --name,
--transport-url and --database_connection).  Otherwise you'll hit
this: http://paste.ubuntu.com/24003055/


Corey,

Thanks for pointing that out.

It looks like the command intended to have --transport-url and 
--database_connection as optional but missed defaulting them to None in 
the function parameter list.


-melanie

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[Corey Bryant]

On Wed, Feb 15, 2017 at 10:20 AM, Dan Smith  wrote:

> > The problem is there's no way to update an existing cell's transport_url
> > via nova-manage.
>
> There is:
>
> https://review.openstack.org/#/c/431582/


Dan,

This works but you have to specify all of the args (--cell_uuid, --name,
--transport-url and --database_connection).  Otherwise you'll hit this:
http://paste.ubuntu.com/24003055/

-- 
Regards,
Corey


>
>
> > It appears the only way to get around this is manually deleting the old
> > cell1 record from the db.
>
> No, don't do that :)
>
> > I'd like to hear more opinions on this but it really seems like this
> > should be a priority to fix prior to the Ocata final release.
>
> Already done!
>
> --Dan
>
> __
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[Corey Bryant]

On Wed, Feb 15, 2017 at 10:20 AM, Dan Smith  wrote:

> > The problem is there's no way to update an existing cell's transport_url
> > via nova-manage.
>
> There is:
>
> https://review.openstack.org/#/c/431582/
>
> > It appears the only way to get around this is manually deleting the old
> > cell1 record from the db.
>
> No, don't do that :)
>
> > I'd like to hear more opinions on this but it really seems like this
> > should be a priority to fix prior to the Ocata final release.
>
> Already done!
>
>
Awesome thanks!

-- 
Regards,
Corey
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[Dan Smith]

> The problem is there's no way to update an existing cell's transport_url
> via nova-manage.

There is:

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

> It appears the only way to get around this is manually deleting the old
> cell1 record from the db.

No, don't do that :)

> I'd like to hear more opinions on this but it really seems like this
> should be a priority to fix prior to the Ocata final release.

Already done!

--Dan

__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

[openstack-dev] [nova] scaling rabbitmq with cells v2 requires manual database update

[Corey Bryant]

Hi all,

Cells v2 in Ocata requires manual database updates each time you scale
rabbitmq.  In my opinion this is a pretty significant regression.  Prior to
cells v2 (no cells) rabbitmq could be scaled without manual database
updates.

The problem is there's no way to update an existing cell's transport_url
via nova-manage.

If you run 'nova-manage cell_v2 create_cell --name cell1' a second time,
the database ends up with two cell1 db records.

It appears the only way to get around this is manually deleting the old
cell1 record from the db.

More details are in the bug at:  https://bugs.launchpad.net/bugs/1664759

I'd like to hear more opinions on this but it really seems like this should
be a priority to fix prior to the Ocata final release.

-- 
Regards,
Corey
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev