Posted	Nick	Remark
#openstack-nova - 2019-12-12
21:38:32	efried	It may be possible. But I'd leave it up to gibi
21:39:34	sean-k-mooney	well with your follow up patch every time we add a filed to the normal nova object the notificaiton object fileds and schema will auto update chaning the hash
21:40:14	sean-k-mooney	so if we apply that patch we need to rememebr to bump the notificion object version and document the new filed even though we are not modifying the class
21:40:33	efried	ohhh, good point, the version. That needs a -1.
21:40:57	sean-k-mooney	that is still less work then today however. ya i can put that in a review comment
21:41:28	efried	Yeah, we at least need to make a note in the class with that instruction.
21:41:41	efried	the test will fail, but you could get around it by just updating the hash without bumping the version.
21:41:52	sean-k-mooney	yep
21:41:54	efried	an astute reviewer ought to notice, but...
21:42:11	jroll	melwitt: it feels like it should be a required field for an ironic node
21:42:20	jroll	there's no way to guess at it
21:56:24	sean-k-mooney	jroll: the cpu architecture
21:57:07	sean-k-mooney	ya i mean some hyperisors can emulate other archs but that is kind of true for all virt drivers
22:00:32	mriedem	ironic != hypervisor
22:00:54	sean-k-mooney	also true
22:01:55	sean-k-mooney	i was makeing the point that most of the virt driver only support one archtecture at a time so you amost always want to know the cpu architecture if you care about check the image for compatablity
22:03:30	sean-k-mooney	there was that project denvor chip that nviad wanted to make but then never closed the license agreement with intel for x86
22:04:41	sean-k-mooney	iwas going to be an arm architecture chip with native traslation of x86 to arm opcode in hardware allowing them to run code comiled for either achtecture
22:05:21	sean-k-mooney	im sure that would have been a pain to debug but it would have been cool if it ever worked
22:11:28	openstackgerrit	sean mooney proposed openstack/nova master: Sync ImageMetaPropsPayload fields https://review.opendev.org/698613
22:15:36	openstackgerrit	Matt Riedemann proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate https://review.opendev.org/695220
22:23:39	sean-k-mooney	efried: actully my brain has already switched off today so ill do the FUP tomorrow the other FUP i promised is https://review.opendev.org/#/c/698260/ but there is no really rush with reviewing that
22:24:01	efried	ack
22:24:42	openstackgerrit	Merged openstack/nova master: trivial: Resolve (most) flake8 3.x issues https://review.opendev.org/695732
22:41:53	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add NodeOwnerFilter https://review.opendev.org/697331
22:41:53	openstackgerrit	Matt Riedemann proposed openstack/nova master: Allow overriding self.api in _build_minimal_create_server_request https://review.opendev.org/698813
22:51:58	openstackgerrit	Merged openstack/nova master: Don't hardcode Python versions in test https://review.opendev.org/698732
23:39:21	openstackgerrit	Merged openstack/nova master: Keep pre-commit inline with hacking and fix whitespace https://review.opendev.org/674057
23:42:35	openstackgerrit	Merged openstack/nova stable/queens: Add functional regression test for bug 1849409 https://review.opendev.org/690724
23:42:35	openstack	bug 1849409 in OpenStack Compute (nova) queens "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem)
#openstack-nova - 2019-12-13
00:25:22	openstackgerrit	Merged openstack/nova stable/queens: Fix listing deleted servers with a marker https://review.opendev.org/690725
00:25:28	openstackgerrit	Merged openstack/nova master: trivial: Remove dead code https://review.opendev.org/698703
00:25:35	openstackgerrit	Merged openstack/nova master: support pci numa affinity policies in flavor and image https://review.opendev.org/674072
00:26:44	openstackgerrit	jichenjc proposed openstack/nova master: libvirt: avoid cpu check at s390x arch https://review.opendev.org/696228
00:26:56	brinzhang	efried: Thanks, I send this to the ML :)
01:56:34	melwitt	jroll: thanks, I've started a ML thread to better organize my questions and get people's thoughts in one place http://lists.openstack.org/pipermail/openstack-discuss/2019-December/011558.html cc TheJulia
02:16:46	luyao	mriedem, efried : we don't patch anything on Intel OpenStack CI, it should be zuul problem
03:53:04	openstackgerrit	Mykola Yakovliev proposed openstack/nova master: Validate aggregate IDs before querying database https://review.opendev.org/698094
04:13:45	openstackgerrit	Merged openstack/nova stable/rocky: Add functional recreate test for bug 1829479 and bug 1817833 https://review.opendev.org/698106
04:13:45	openstack	bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Medium,In progress] https://launchpad.net/bugs/1829479 - Assigned to Matt Riedemann (mriedem)
04:13:46	openstack	bug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Medium,In progress] https://launchpad.net/bugs/1817833 - Assigned to Matt Riedemann (mriedem)
08:30:22	openstackgerrit	jichenjc proposed openstack/nova master: libvirt: avoid cpu check at s390x arch https://review.opendev.org/696228
08:32:08	openstackgerrit	Merged openstack/nova master: Follow up to I5b9d41ef34385689d8da9b3962a1eac759eddf6a https://review.opendev.org/698028
08:56:25	openstackgerrit	Guo Jingyu proposed openstack/nova master: Define and catch InstanceAgentNotRunning for set-password failure https://review.opendev.org/661466
09:42:11	yoctozepto	morning
09:42:53	yoctozepto	I assigned https://bugs.launchpad.net/nova/+bug/1856296 to nova, mostly to let you know, not sure if you can do too much about it, other than letting users know they might be impacted on upgrades
09:42:53	openstack	Launchpad bug 1856296 in kolla-ansible "upgrade to Train might fail due to mariadb row format" [Critical,Triaged]
09:54:11	openstackgerrit	Guo Jingyu proposed openstack/nova master: Make scheduling more debuggable https://review.opendev.org/698421
11:25:30	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Drop tables for removed services, features https://review.opendev.org/698706
11:25:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP: db: Remove placement tables https://review.opendev.org/698707
11:25:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Remove unused ec2 DB APIs https://review.opendev.org/698880
11:25:32	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Remove unused models https://review.opendev.org/698881
11:58:01	gibi	stephenfin: hi! what do you think about https://review.opendev.org/#/c/696574/10/nova/objects/instance.py@1239 ? shall I move this whole thing back to utils?
11:59:16	stephenfin	gibi: I guess if there's nowhere better, you've no choice :)
11:59:27	stephenfin	nova/compute/utils.py though, rather than nova/utils.py
12:07:27	gibi	stephenfin: ack. moving
12:42:26	openstackgerrit	Stephen Finucane proposed openstack/nova master: Revert "(Temporarily) readd bare support for py27" https://review.opendev.org/698890
12:42:45	stephenfin	tosky: we're good to merge that now, right? ^
12:43:18	sean-k-mooney	that drops support again since m1 was yesterday?
12:43:36	stephenfin	officially, yeah
12:43:45	stephenfin	it prevents you installing nova under python 2.7
12:43:51	stephenfin	pip will spit the dummy out
12:43:53	sean-k-mooney	oh i know what the patch does
12:44:07	sean-k-mooney	i was just checking m1 was yesterday
12:44:13	sean-k-mooney	i was 99% sure it was
12:44:47	sean-k-mooney	also that is such an irish saying
12:49:00	openstackgerrit	Stephen Finucane proposed openstack/nova master: Revert "(Temporarily) readd bare support for py27" https://review.opendev.org/698890
12:49:00	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove remaining Python 2.7-only dependencies https://review.opendev.org/698895
12:49:31	sean-k-mooney	stephenfin: abandoned by typeing module fixes you flake 8 changes already cover that right
12:49:51	stephenfin	yup
12:50:12	sean-k-mooney	actully i really dont like the if False
12:50:44	sean-k-mooney	can you just convert the comments in the native form and import them normally
12:50:57	sean-k-mooney	or import them normally and add the #noqa
12:51:06	sean-k-mooney	to silence the unused import thing
12:51:14	stephenfin	Once we merge https://review.opendev.org/698890. yup
12:51:21	stephenfin	Feel free to -1 to remind me :)
12:51:38	sean-k-mooney	oh your right py27
12:51:47	sean-k-mooney	ya we should merge that :)
12:52:03	stephenfin	and now, I'm nearly an hour late for lunch, but at least X-cell resize is reviewed finally. bbiab
12:54:51	openstackgerrit	jichenjc proposed openstack/nova master: libvirt: avoid cpu check at s390x arch https://review.opendev.org/696228
13:06:06	tosky	stephenfin: uhm, maybe
13:21:23	tobiash	hi, I'd like to revive discussion on this nova-spec which was proposed for rocky and abandoned: https://review.opendev.org/572805. What is the preferred way of doing this? Shall I create a new change or ask a nova-core to hit the restore button?
13:23:02	gibi	tobiash: restored the spec. Please update it (e.g. move it to the ussuri dir at least)
13:23:19	tobiash	gibi: thank you very much!
13:24:09	sean-k-mooney	tobiash: you will need to add the new feature liason field at a minium to get it to pass the gate checks
13:24:21	sean-k-mooney	there is a description of that in the readme
13:24:43	tobiash	sean-k-mooney: thanks for the hint
13:24:45	sean-k-mooney	there may be some other template changes too but i think that is the main one since rocky
13:26:11	openstackgerrit	Eric Xie proposed openstack/nova master: Fix error when live-migrating instance cause of gic version https://review.opendev.org/698905
13:38:40	openstackgerrit	Tobias Henkel proposed openstack/nova-specs master: Add spec for downloading images via RBD https://review.opendev.org/572805
13:54:58	openstackgerrit	Matthew Booth proposed openstack/nova master: Don't retry database connections from command line applications https://review.opendev.org/666892
14:01:46	mriedem	efried: have you seen this? https://bugs.launchpad.net/nova/+bug/1856311
14:01:46	openstack	Launchpad bug 1856311 in OpenStack Compute (nova) "server_external_events response status is always 'completed' instead of event status which requested as failed" [Undecided,New]
14:03:49	efried	mriedem: nope, /me clicks
14:04:01	efried	what, now that I'm a server-external-events expert?
14:04:18	mriedem	yes you own it now
14:04:26	mriedem	but it looks like it's not what i thought they were complaining about
14:04:27	efried	I haven't looked at the novaclient shim at all yet
14:05:00	sean-k-mooney	is that valid? i would have assuemd that status faild for network-vif-plugged ment the pluggin failed
14:05:46	sean-k-mooney	and code 200 ment the event was sucessfully recieved?

Earlier   Later