| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-06 | |||
| 17:21:30 | jamesbenson | I've looked at the docs (kolla, ceph, everywhere) as well and tried a bunch. Having trouble and hoping someone who has it running might have some insight. Thanks :-) | |
| 17:25:53 | bauzas | #nova clean | |
| 17:26:05 | bauzas | shit, wrong chan | |
| 17:54:22 | jamesbenson | ugh, wrong channel too. :-( | |
| #openstack-nova - 2022-04-07 | |||
| 00:34:52 | brinzhang_ | bauzas:about tenant_id topic, imo, there is nothing need to discuss, I have been doing it since wallaby, it's clear what we should do | |
| 00:36:05 | brinzhang_ | and the time it's later for me, sorry I will absent. | |
| 07:56:14 | bauzas | brinzhang_: no worries, we already discussed about your series and I think it's prioritized | |
| 08:35:56 | brinzhang_ | bauzas:ack, thx | |
| 09:07:54 | opendevreview | Wenping Song proposed openstack/nova-specs master: qUsage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/836583 | |
| 10:19:06 | opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Fix eventlet.tpool import https://review.opendev.org/c/openstack/nova/+/836788 | |
| 11:34:01 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Reproduce bug 1944759 https://review.opendev.org/c/openstack/nova/+/836733 | |
| 11:53:41 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Store old_flavor already on source host during resize https://review.opendev.org/c/openstack/nova/+/836734 | |
| 12:34:20 | Uggla | hello | |
| 12:34:51 | sean-k-mooney | o/ | |
| 12:35:15 | Uggla | sean-k-mooney, I need help. | |
| 12:36:24 | Uggla | I try to test my code with devstack. I have an issue with the migration of DB. It seems that nove-manage db sync did the correctly on cell0 but not cell1. | |
| 12:36:44 | sean-k-mooney | you have to run it twice | |
| 12:36:56 | Uggla | hum no | |
| 12:37:03 | Uggla | I don't think so | |
| 12:37:15 | sean-k-mooney | you have to run the sycn for each db | |
| 12:37:23 | sean-k-mooney | cell0 is kind of special | |
| 12:37:33 | sean-k-mooney | but in general you run the sync per cell | |
| 12:37:50 | sean-k-mooney | devstack should already do this | |
| 12:37:58 | sean-k-mooney | how exactly did it fail | |
| 12:39:35 | Uggla | it seems it doesn't fail. But cell1 has not the share_mapping table and the version within alambic_version table has not evolved. | |
| 12:39:50 | Uggla | I may miss something but .... | |
| 12:40:01 | sean-k-mooney | where did you create it | |
| 12:40:12 | sean-k-mooney | do you have the link to your patch | |
| 12:40:24 | Uggla | sure | |
| 12:40:48 | Uggla | https://review.opendev.org/c/openstack/nova/+/831193 | |
| 12:41:13 | sean-k-mooney | ok you modified the main db model | |
| 12:41:19 | sean-k-mooney | that should be correct | |
| 12:41:32 | sean-k-mooney | i just wanted to make sur eyou did not put it in the api db by mistake | |
| 12:41:35 | Uggla | All tests unit + func looks fine with tox. | |
| 12:42:03 | Uggla | I mean a functional test using the api + db is working fine. | |
| 12:42:30 | Uggla | But on devstack the migration seemed to not fan out to all cell. | |
| 12:42:37 | Uggla | It happens only on cell0 | |
| 12:43:43 | Uggla | sean-k-mooney, we can share a gmeet session, if you would like to see it. | |
| 12:43:45 | sean-k-mooney | the dbs are set up here | |
| 12:43:46 | sean-k-mooney | https://github.com/openstack/devstack/blob/master/lib/nova#L818-L871= | |
| 12:44:56 | Uggla | oh ok I get it . | |
| 12:45:03 | Uggla | If necessary, the --config-file argument can be used to point to the correct nova.conf file for the given cell. | |
| 12:45:52 | sean-k-mooney | yes | |
| 12:46:01 | sean-k-mooney | https://github.com/openstack/devstack/blob/master/lib/nova#L843= | |
| 12:46:09 | sean-k-mooney | is where the db sync shoudl happen for cell 1 | |
| 12:46:23 | sean-k-mooney | using the cell1 conductor config | |
| 12:46:28 | Uggla | I did not understand the doc correctly | |
| 12:47:32 | sean-k-mooney | so we are doing $NOVA_BIN_DIR/nova-manage --config-file $conffile db sync --local_cell | |
| 12:47:55 | sean-k-mooney | --local_cell disables faning out ot other celsl as part of the sync | |
| 12:47:58 | chateaulav | bauzas: great PTG so far! I am otherwise held up today, but i will be there tomorrow just fyi | |
| 12:48:09 | sean-k-mooney | ack | |
| 12:48:34 | sean-k-mooney | Uggla: have you tried runnign that manually | |
| 12:48:39 | Uggla | yep it is a bit ambiguous IMHO. | |
| 12:48:47 | Uggla | I'm gonna try. | |
| 12:49:16 | sean-k-mooney | nova-manage --config-file /etc/nova/nova-conductor-1.conf db sync --local_cell | |
| 12:49:25 | sean-k-mooney | i think that woudl be the right path but its something like that | |
| 12:49:58 | sean-k-mooney | ah its /etc/nova/nova_cell1.conf | |
| 12:51:01 | Uggla | nova-manage --config-file /etc/nova/nova_cell1.conf db sync correct ? | |
| 12:51:06 | sean-k-mooney | yes | |
| 12:51:20 | sean-k-mooney | that is what devstack ran | |
| 12:51:24 | Uggla | ERROR: Could not access cell0. | |
| 12:51:52 | sean-k-mooney | odd... | |
| 12:51:56 | sean-k-mooney | let me try that myself | |
| 12:52:42 | bauzas | chateaulav: no worries | |
| 12:53:18 | Uggla | sean-k-mooney, I got this: https://paste.openstack.org/show/bw9C4K0JEaD2eH2kM3TF/ | |
| 12:53:19 | sean-k-mooney | Uggla: you are missing --local_cell | |
| 12:53:40 | sean-k-mooney | that disablise it trying to connect to cell0 | |
| 12:53:46 | sean-k-mooney | when syncign cell 1 | |
| 12:54:06 | sean-k-mooney | stack@controller:/home/vagrant$ nova-manage --config-file /etc/nova/nova_cell1.conf db sync --local_cell | |
| 12:54:10 | sean-k-mooney | that should work | |
| 12:54:11 | Uggla | sean-k-mooney, sounds good ! I'm gonna check | |
| 12:54:38 | bauzas | correct ^ | |
| 12:55:19 | Uggla | it is ok regarding the tables \o/ | |
| 12:56:06 | sean-k-mooney | so how did you configure devstack to use your patch | |
| 12:56:08 | Uggla | but I'm unclear if this is specific onlyfor devstack ? | |
| 12:56:22 | sean-k-mooney | no you have to do this for all installers | |
| 12:57:09 | sean-k-mooney | nova-manage can try and discover all the cells via the api db technially but that assuem either you store the password in the db which we discurage or you use the same password on all db instances | |
| 12:57:11 | bauzas | https://github.com/openstack/devstack/blob/master/lib/nova#L818 | |
| 12:57:46 | bauzas | and https://github.com/openstack/devstack/blob/master/lib/nova#L847 | |
| 12:58:00 | sean-k-mooney | in generall its simpler to have one config file per cell and run the comand once per cell | |
| 12:58:04 | Uggla | for devstack I use my patch by configuring a git remote on my repo and "pulling" it | |
| 12:58:10 | bauzas | context : https://docs.openstack.org/nova/latest/admin/cells.html#configuring-a-new-deployment | |
| 12:58:27 | sean-k-mooney | Uggla: ok so you set NOVA_REPO and NOVA_BRANCH before you stacked | |
| 12:58:48 | sean-k-mooney | and you did not already have /opt/stack/nova right | |
| 12:58:56 | sean-k-mooney | before you ran stack.sh | |
| 12:59:09 | Uggla | not sorry, I had a already installed devstack and replace the code with mine + restart | |
| 12:59:19 | sean-k-mooney | oh ok | |
| 12:59:41 | sean-k-mooney | so doing sudo systemctl restart devstack@n-* | |
| 12:59:47 | Uggla | yep | |
| 12:59:50 | sean-k-mooney | works only for python changes | |
| 12:59:59 | sean-k-mooney | if you chagne the db you have to manually sync again | |
| 13:00:15 | Uggla | ack | |
| 13:00:43 | sean-k-mooney | 99% of the time we are not touching the db so we dont usually call that out | |
| 13:00:52 | Uggla | do we agree that the doc for db migration is this one: https://docs.openstack.org/nova/latest/admin/upgrades.html | |
| 13:01:07 | sean-k-mooney | yes it is | |
| 13:01:34 | sean-k-mooney | """Using the newly installed nova code, run the DB sync. First run nova-manage api_db sync, then nova-manage db sync. nova-manage db sync should be run for all cell databases, including cell0. If necessary, the --config-file argument can be used to point to the correct nova.conf file for the given cell.""" | |
| 13:01:45 | Uggla | any objection if I propose a minor update to make it clearer ? | |
| 13:02:15 | sean-k-mooney | im not sure what is not clear but sure we can review it | |
| 13:02:28 | sean-k-mooney | its always ok to propose updates to docs ectra | |
| 13:02:34 | sean-k-mooney | we will just review it as normal | |