| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-14 | |||
| 12:02:51 | zigo | Thanks. | |
| 12:02:51 | zigo | Thanks. | |
| 12:18:13 | stephenfin | ooh, we have our 23.0.0 tag \o/ | |
| 12:18:13 | stephenfin | ooh, we have our 23.0.0 tag \o/ | |
| 12:18:24 | stephenfin | gibi: Does this mean we can start merging larger patches again now? | |
| 12:18:24 | stephenfin | gibi: Does this mean we can start merging larger patches again now? | |
| 12:18:41 | sean-k-mooney | stephenfin: the release was yesterday so yes | |
| 12:18:41 | gibi | stephenfin: yes it is | |
| 12:18:41 | sean-k-mooney | stephenfin: the release was yesterday so yes | |
| 12:18:41 | gibi | stephenfin: yes it is | |
| 12:18:44 | stephenfin | * stephenfin has an alembic migration series _almost_ completed locally | |
| 12:18:50 | gibi | sounds cool | |
| 12:18:50 | stephenfin | hurrah | |
| 12:18:50 | gibi | sounds cool | |
| 12:18:50 | stephenfin | hurrah | |
| 12:19:17 | stephenfin | * stephenfin will try wrap this up and spend tomorrow on reviews so | |
| 12:19:50 | stephenfin | sean-k-mooney: I fixed https://review.opendev.org/c/openstack/nova/+/729918/ and added a follow-up explaining where nova.conf comes from | |
| 12:19:50 | stephenfin | sean-k-mooney: I fixed https://review.opendev.org/c/openstack/nova/+/729918/ and added a follow-up explaining where nova.conf comes from | |
| 12:20:22 | stephenfin | just fyi | |
| 12:20:22 | stephenfin | just fyi | |
| 13:01:09 | gibi | core: there are two patches needed on master before we get deep into xena: | |
| 13:01:09 | gibi | core: there are two patches needed on master before we get deep into xena: | |
| 13:01:13 | gibi | https://review.opendev.org/c/openstack/nova/+/782171 | |
| 13:01:13 | gibi | https://review.opendev.org/c/openstack/nova/+/782171 | |
| 13:01:17 | gibi | https://review.opendev.org/c/openstack/nova/+/778923 | |
| 13:01:17 | gibi | https://review.opendev.org/c/openstack/nova/+/778923 | |
| 13:01:28 | gibi | both needs a second core | |
| 13:01:28 | gibi | both needs a second core | |
| 13:12:10 | openstackgerrit | Johannes Kulik proposed openstack/nova master: vmware: OptionValue.value cannot be Boolean https://review.opendev.org/c/openstack/nova/+/786246 | |
| 13:12:10 | openstackgerrit | Johannes Kulik proposed openstack/nova master: vmware: OptionValue.value cannot be Boolean https://review.opendev.org/c/openstack/nova/+/786246 | |
| 13:18:00 | bauzas | gibi: ack lookinf | |
| 13:18:00 | bauzas | gibi: ack lookinf | |
| 13:18:06 | bauzas | lol | |
| 13:18:27 | bauzas | * bauzas just clicked and saw the changes :p | |
| 13:19:03 | gibi | :D | |
| 13:19:38 | sean-k-mooney | we likely should look at doing the libvirt min microverion bumps early too | |
| 13:19:38 | sean-k-mooney | we likely should look at doing the libvirt min microverion bumps early too | |
| 13:19:45 | sean-k-mooney | before m1 in any case | |
| 13:19:45 | sean-k-mooney | before m1 in any case | |
| 13:20:06 | sean-k-mooney | not as urgent though | |
| 13:20:06 | sean-k-mooney | not as urgent though | |
| 13:20:43 | sean-k-mooney | for https://review.opendev.org/c/openstack/nova/+/782171/2/nova/objects/service.py#205 | |
| 13:20:43 | sean-k-mooney | for https://review.opendev.org/c/openstack/nova/+/782171/2/nova/objects/service.py#205 | |
| 13:20:54 | sean-k-mooney | should we really be removing the victoria alias | |
| 13:20:54 | sean-k-mooney | should we really be removing the victoria alias | |
| 13:21:06 | sean-k-mooney | i would have just appended wallaby | |
| 13:21:06 | sean-k-mooney | i would have just appended wallaby | |
| 13:21:22 | sean-k-mooney | otherwise what is the point of having a dict | |
| 13:21:22 | sean-k-mooney | otherwise what is the point of having a dict | |
| 13:29:47 | gibi | sean-k-mooney: I can remove the dict. I thought originally that we need a list but the code only uses that latest one so the dict is not needed | |
| 13:29:47 | gibi | sean-k-mooney: I can remove the dict. I thought originally that we need a list but the code only uses that latest one so the dict is not needed | |
| 13:30:39 | gibi | but we both need a service number and the name | |
| 13:30:40 | gibi | but we both need a service number and the name | |
| 13:31:29 | sean-k-mooney | yep well we could leave it as is too it just feels a little clunky currently | |
| 13:31:29 | sean-k-mooney | yep well we could leave it as is too it just feels a little clunky currently | |
| 13:32:52 | sean-k-mooney | not that i really want to open this can of worms but our pms keep brinign up the idea of having nova support more then n-1 computes | |
| 13:32:52 | sean-k-mooney | not that i really want to open this can of worms but our pms keep brinign up the idea of having nova support more then n-1 computes | |
| 13:33:15 | sean-k-mooney | im partly wondering if we want this to be confiugrable at some point | |
| 13:33:15 | sean-k-mooney | im partly wondering if we want this to be confiugrable at some point | |
| 13:33:29 | sean-k-mooney | not that i actully want to really support n-2 or n-3 | |
| 13:33:29 | sean-k-mooney | not that i actully want to really support n-2 or n-3 | |
| 13:33:56 | sean-k-mooney | im happy we now codify the n-1 support | |
| 13:33:56 | sean-k-mooney | im happy we now codify the n-1 support | |
| 13:34:53 | gibi | sean-k-mooney: we cannot simply make the n-2 support configurable, as n-2 support means keeping the compatibilty code and service version checks longer in the code, so it needs in change of the behavior of the developers | |
| 13:34:53 | gibi | sean-k-mooney: we cannot simply make the n-2 support configurable, as n-2 support means keeping the compatibilty code and service version checks longer in the code, so it needs in change of the behavior of the developers | |
| 13:36:20 | gibi | but anyhow if we want to support a bigger version gap later then it make sense to keep the Victoria version mapping in the dict | |
| 13:36:20 | gibi | but anyhow if we want to support a bigger version gap later then it make sense to keep the Victoria version mapping in the dict | |
| 13:36:49 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Update min supported service version for Xena. https://review.opendev.org/c/openstack/nova/+/782171 | |
| 13:36:49 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Update min supported service version for Xena. https://review.opendev.org/c/openstack/nova/+/782171 | |
| 13:37:01 | gibi | sean-k-mooney: ^^ | |
| 13:37:01 | gibi | sean-k-mooney: ^^ | |
| 13:38:36 | sean-k-mooney | ya it is more work then that | |
| 13:38:36 | sean-k-mooney | ya it is more work then that | |
| 13:38:48 | sean-k-mooney | gibi: we always have a tension downstreeam | |
| 13:38:48 | sean-k-mooney | gibi: we always have a tension downstreeam | |
| 13:39:05 | sean-k-mooney | from osp 16 on we move to a FFU only model | |
| 13:39:05 | sean-k-mooney | from osp 16 on we move to a FFU only model | |
| 13:39:27 | sean-k-mooney | so the only way to go from train is to FFU directly to wallaby when 17 comes out | |
| 13:39:27 | sean-k-mooney | so the only way to go from train is to FFU directly to wallaby when 17 comes out | |
| 13:39:56 | sean-k-mooney | that is an issue for a mulit sight multi cell deployment as all nodes need to be upgrade at once | |
| 13:39:56 | sean-k-mooney | that is an issue for a mulit sight multi cell deployment as all nodes need to be upgrade at once | |
| 13:40:47 | gibi | I see | |
| 13:40:51 | sean-k-mooney | so the tension is between custoemr saying we want to do upgrades on a slower cadance and the fact that we can only support n-1 upstream | |
| 13:40:51 | sean-k-mooney | so the tension is between custoemr saying we want to do upgrades on a slower cadance and the fact that we can only support n-1 upstream | |
| 13:41:15 | sean-k-mooney | we have other customer that want new feature faster too so not all are the same obviously | |
| 13:41:15 | sean-k-mooney | we have other customer that want new feature faster too so not all are the same obviously | |
| 14:15:48 | noonedeadpunk | Hi there! Have a question regarding nova-conductor and rabbit... We had a deployment on V with enabled SSL for rabbitmq (but without trusted cert or rootca). And while upgrade to W, we disabled ssl for rabbitmq. However, conductor still tries to reach rabbit via ssl and fails with CERTIFICATE_VERIFY_FAILED (whic was the reason why ssl was disabled at the first place) | |
| 14:15:48 | noonedeadpunk | Hi there! Have a question regarding nova-conductor and rabbit... We had a deployment on V with enabled SSL for rabbitmq (but without trusted cert or rootca). And while upgrade to W, we disabled ssl for rabbitmq. However, conductor still tries to reach rabbit via ssl and fails with CERTIFICATE_VERIFY_FAILED (whic was the reason why ssl was disabled at the first place) | |
| 14:16:28 | noonedeadpunk | Am I missing so nova-manage command to make conductor migrate and re-read config? | |
| 14:16:28 | noonedeadpunk | Am I missing so nova-manage command to make conductor migrate and re-read config? | |
| 14:16:35 | noonedeadpunk | *some | |
| 14:16:35 | noonedeadpunk | *some | |
| 14:17:19 | noonedeadpunk | as with same config code that's deployed just on W works nicely | |
| 14:17:19 | noonedeadpunk | as with same config code that's deployed just on W works nicely | |
| 14:17:59 | gibi | noonedeadpunk: look at the cell_mappings table in your nova api database. I guess it still contains rabbit urls with ssl | |
| 14:17:59 | gibi | noonedeadpunk: look at the cell_mappings table in your nova api database. I guess it still contains rabbit urls with ssl | |
| 14:18:48 | noonedeadpunk | gibi: yep, that's absolutely true | |
| 14:18:48 | noonedeadpunk | gibi: yep, that's absolutely true | |
| 14:19:55 | gibi | noonedeadpunk: you can use nova-manage cell_v2 update_cell to make the adjustments | |
| 14:19:55 | gibi | noonedeadpunk: you can use nova-manage cell_v2 update_cell to make the adjustments | |