Posted	Nick	Remark
#openstack-nova - 2020-08-13
17:57:29	sangeet	You are correct .. it is without ssl
17:57:45	sean-k-mooney	yep so you could just update it and then it would work
17:59:30	sangeet	going back to useing --local-cell, do you see any issue with it? Also when you say "you need to run nova-manage db sync on one contoler on each cell wehn you use --local-cell"
17:59:31	sean-k-mooney	by should work i mean nova manage shoudl work without --local-cell and as a result the automated update should work too
17:59:39	sangeet	does it mean I have to run on each node
17:59:59	sean-k-mooney	no so each cell has its own db right
18:00:07	sangeet	yes
18:00:16	sean-k-mooney	so each cell db needs to be updated
18:00:27	sean-k-mooney	wehn you use --local-cell it only update well the locall cell db
18:00:39	sean-k-mooney	not all deployment have db access between cells
18:00:41	sangeet	will "nova-manage db sync --local-cell" not update all cells?
18:01:02	sean-k-mooney	correct it only updates the cell pointed to in the database section
18:01:21	sangeet	but "nova-manage db sync " does all?
18:01:31	sean-k-mooney	actully no
18:01:38	sean-k-mooney	it does the local cell and cell0
18:02:04	sean-k-mooney	you need to run nova-manage in each cell anyway
18:02:14	sean-k-mooney	https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L185-L186
18:03:05	sangeet	So it mean I run the command with our without --local-cell, it will update same cells?
18:04:18	sean-k-mooney	no the default behavor is to update the cell in [database]/connection and cell0
18:04:50	sean-k-mooney	so if you run the command on the host with the conducto for cell1 it will update cell1 and cell0
18:05:12	sean-k-mooney	ran on the host with the conductor for cell2 it will update cell2 and cell0
18:05:29	sean-k-mooney	if you add --local-cell it will skip updating cell0
18:05:51	sean-k-mooney	unless [database]/connection point to cell0
18:06:53	sean-k-mooney	basically the nova.conf used by the nova api should have [database]/connection pointing at cell0, each conductors nova.conf should have [database]/connection pointing at the cell db
18:07:21	sean-k-mooney	and the compute services should not have the database details defiend at all
18:08:06	sean-k-mooney	openstack helm shoudl then be automatinn runing the db sync once for each cell db using the correct cofigs
18:08:55	sean-k-mooney	and the simplet way to do that is to delegate runing nova-magange db sync to the first conductor host in each cell
18:09:13	sean-k-mooney	you can safly run it on each conductor host
18:09:28	sean-k-mooney	as it is not destructive its just wasteful to do it
18:09:45	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/746210
18:12:43	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/746210
19:21:31	sangeet	sean-k-mooney .. thanks a lot for your help
19:39:24	sean-k-mooney	sangeet: no worries hopefully if admin0 comes back and sees this it might help them too
20:56:01	mnaser	does nova run ceph jobs on every commit?
21:03:08	dansmith	yes
21:29:50	eandersson	btw I have a PR open for cloud-init for the OpenStack Datasource to increase the retry limit when identifying the metadata service. The current defaults only allows a single failure before giving up. https://github.com/canonical/cloud-init/pull/501
21:30:12	eandersson	If anyone has any thoughts feel free to drop a comment.
21:45:41	openstackgerrit	Merged openstack/nova master: Fix FIP policy for admin_or_owner https://review.opendev.org/741811
21:47:59	prometheanfire	can we get more cores on https://review.opendev.org/744992 (oneliner, with comment)
21:58:58	melwitt	prometheanfire: done
22:01:13	prometheanfire	thanks
22:21:51	openstackgerrit	Merged openstack/nova master: Add new default roles in multinic policies https://review.opendev.org/742330
22:21:57	openstackgerrit	Merged openstack/nova master: Correct the check_str and pass actual target in FIP pools policy https://review.opendev.org/741797
22:22:05	openstackgerrit	Merged openstack/nova master: Handle Neutron errors in _post_live_migration() https://review.opendev.org/729763
#openstack-nova - 2020-08-14
02:21:53	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of FIP policies https://review.opendev.org/741812
02:22:09	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in FIP policy https://review.opendev.org/741813
03:47:24	openstackgerrit	Merged openstack/nova master: Add test coverage of FIP policies https://review.opendev.org/741812
03:48:12	openstackgerrit	Brin Zhang proposed openstack/nova stable/ussuri: [Trivial] Remove wrong format_message() conversion https://review.opendev.org/746264
05:32:32	openstackgerrit	Merged openstack/nova master: Introduce scope_types in FIP policy https://review.opendev.org/741813
06:55:11	aarents	stephenfin: Hello, if you can recheck this one, when you can, I fixed comments and done test refacto: https://review.opendev.org/#/c/696084/
08:23:56	openstackgerrit	Merged openstack/nova master: Add lsscsi to bindep https://review.opendev.org/744992
09:19:40	tony_su	stephenfin: the remaining 3 patches are ready for review. meanwhile, i am looking into how to upgrade nova-manage command. but this will not block other patches. thanks.
17:09:34	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: migrate testing to ubuntu focal https://review.opendev.org/738126
#openstack-nova - 2020-08-15
07:45:26	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.PY2 and six.PY3 https://review.opendev.org/726512
07:45:41	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.add_metaclass https://review.opendev.org/726565
07:45:54	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898
07:46:10	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224
07:46:23	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757
07:46:37	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777
07:46:54	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094
07:47:08	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109
07:47:45	openstackgerrit	Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117
#openstack-nova - 2020-08-16
15:41:36	openstackgerrit	Takashi Natsume proposed openstack/nova-specs master: Create specs directory for Wallaby https://review.opendev.org/746408
16:24:25	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: [goal] Migrate testing to ubuntu focal https://review.opendev.org/738126
#openstack-nova - 2020-08-17
00:55:29	brinzhang	gibi_pto_24th, gmann, stephenfin, alex_xu, bauzas: I was update the runway slot, because some patches have reached the dealine in the slot
07:50:16	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Raising a failure in case of 403 https://review.opendev.org/735068
09:12:35	lyarwood	stephenfin: https://review.opendev.org/#/c/746210/ - would you mind hitting this today? simple fup from https://review.opendev.org/#/q/I7eb86edc130d186a66c04b229d46347ec5c0b625 as we all missed we are using < v4.1.0 in CI at the moment.
09:12:48	openstackgerrit	Stephen Finucane proposed openstack/nova stable/ussuri: [Trivial] Remove wrong format_message() conversion https://review.opendev.org/746464
09:13:57	stephenfin	sure
09:25:13	lyarwood	stephenfin: are you rebasing the vtpm stuff today btw?
09:25:34	stephenfin	yup, in the middle of that atm
09:25:38	lyarwood	wonderful
09:30:58	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Add helpers for suspend, resume and reboot of server https://review.opendev.org/741285
09:30:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Pass context, instance to '_create_guest' https://review.opendev.org/741286
09:30:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Reject non-spawn operations for vTPM https://review.opendev.org/741500
09:31:03	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Add emulated TPM support to Nova https://review.opendev.org/631363
09:31:03	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Add docs for vTPM support https://review.opendev.org/739213
09:31:04	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.compute.manager' https://review.opendev.org/742863
09:31:05	openstackgerrit	Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/741995
09:31:05	openstackgerrit	Stephen Finucane proposed openstack/nova master: privsep: Add support for recursive chown, move_tree operations https://review.opendev.org/742864
09:31:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.virt.libvirt.utils' https://review.opendev.org/742865
09:31:09	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
11:32:01	openstackgerrit	Hao Sun proposed openstack/nova master: Add checks for volume status when rebuilding https://review.opendev.org/734734
11:41:55	lyarwood	stephenfin: do you plan on blocking non-owner ops with vTPM instances in the API?
12:07:08	openstackgerrit	Merged openstack/nova master: Remove dead volume driver code https://review.opendev.org/745393
13:33:08	lyarwood	melwitt: https://review.opendev.org/#/c/746210/ - Morning, when you're online would you mind taking a look at this please?
14:20:29	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Mock LoopingCallBase._sleep in rbd tests https://review.opendev.org/745512
14:20:45	lyarwood	stephenfin: ^ trivial one for you if you have bandwidth
14:22:28	stephenfin	lyarwood: done
14:23:17	stephenfin	as for your earlier question (sorry, missed it then), yeah, that probably makes sense. I assume all non-owner ops except delete, of course
14:29:45	lyarwood	stephenfin: well any move ops really
14:30:02	lyarwood	stephenfin: hard reboot as well thinking about it
14:32:30	gmann	lyarwood: stephenfin can you check this, Focal migration patch is ready- https://review.opendev.org/#/c/738126/
14:36:24	openstackgerrit	Merged openstack/nova stable/ussuri: [Trivial] Remove wrong format_message() conversion https://review.opendev.org/746464
14:36:39	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in baremetal nodes policies https://review.opendev.org/741716
14:36:52	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in baremetal nodes policy https://review.opendev.org/741717
14:40:32	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add scope and new default roles in extensions policies https://review.opendev.org/743046
14:42:47	lyarwood	gmann: so what about that change moves us to focal? Wouldn't it be better to say that this is just bumping our reqs so we *can* move to focal in the future?
14:44:06	gmann	lyarwood: with the depends-on in previous PS tested all devstack based jobs on focal to it also made sure everything worked fine here so whenever base jobs moves to Focal nova automatically start tested on Focal
14:47:36	lyarwood	gmann: shouldn't we have a .DNM change on top of this to ensure it's still working?

Earlier   Later