cell_v2 update_cell cell0 get transport_url from config file

[hanchl]

There is an error record in my nova_api database so I use nova-manage cell_v2 update_cell --name cell0 --database_connection XXX ,then the I found it tired to modify the value of transport-url according to the config file.However, the value should be "none:///",right?
I checked the source code ,it is found that when the modification of 'enable' or 'disable' operation is performed, it will be verified whether it is operation to cell0, but updating of database_connection will not check.
code:
 if disable and enable:
            print(_('Cell cannot be disabled and enabled at the same time.'))
            return 4
        if disable or enable:
            if cell_mapping.is_cell0():

[Balazs Gibizer (balazs-gibizer)]

Yeah, we could add a similar check for the transport_url update too. https://github.com/openstack/nova/blob/9a95288ef1dd693c152f2a0eea362bcdc2b30067/nova/cmd/manage.py#L1374

[Daniel de Oliveira Pereira (danielpereira01)]

Hi, I'm a new contributor and would like to take this one.

[Daniel de Oliveira Pereira (danielpereira01)]

Hi,

I have submitted a patch for this bug: https://review.opendev.org/#/c/762433/5

Could you please help review it?

Thanks!