Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-04-14
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
14:21:22 sean-k-mooney unless your useing templated transport urls ya yuou likely need to update them
14:21:22 sean-k-mooney unless your useing templated transport urls ya yuou likely need to update them
14:23:52 noonedeadpunk ok, thanks, that worked! Now I need to review wtf we do in https://opendev.org/openstack/openstack-ansible-os_nova/src/branch/master/tasks/nova_db_setup.yml and https://opendev.org/openstack/openstack-ansible-os_nova/src/branch/master/tasks/nova_db_post_setup.yml
14:23:52 noonedeadpunk ok, thanks, that worked! Now I need to review wtf we do in https://opendev.org/openstack/openstack-ansible-os_nova/src/branch/master/tasks/nova_db_setup.yml and https://opendev.org/openstack/openstack-ansible-os_nova/src/branch/master/tasks/nova_db_post_setup.yml
14:24:33 sean-k-mooney noonedeadpunk: i can never recall if templated urls is a nova thing or ooo. i think its a nova feature
14:24:33 sean-k-mooney noonedeadpunk: i can never recall if templated urls is a nova thing or ooo. i think its a nova feature
14:26:02 noonedeadpunk * noonedeadpunk no idea what templated urls are
14:26:49 noonedeadpunk https://docs.openstack.org/nova/latest/user/cells.html#template-urls-in-cell-mappings ?
14:26:49 noonedeadpunk https://docs.openstack.org/nova/latest/user/cells.html#template-urls-in-cell-mappings ?

Earlier   Later