Posted	Nick	Remark
#openstack-nova - 2019-01-07
14:45:55	yonglihe	Jay, Matt please take a look this spec, thanks. https://review.openstack.org/#/c/612256/18
14:51:09	yonglihe	call for review this simple "server-group" spec: https://review.openstack.org/#/c/612255/
14:57:35	gibi	ShilpaSD: https://review.openstack.org/#/c/470847/8 this patch fails the test_versions locally
14:57:46	gibi	ShilpaSD: but it passes on the gate
14:58:08	gibi	ShilpaSD: so someting is wrong with the version test even without the notification patch
15:03:05	openstackgerrit	Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256
15:09:18	ShilpaSD	gibi: ya, just checked, its failing locally, how to resolve this, any pointers?
15:09:46	gibi	I guess you see some kind of race condition or test case interference
15:10:20	gibi	ShilpaSD: try to figure out why MyObj is sometimes in the object registry but other times not in the object registry
15:11:14	ShilpaSD	gibi: observed that sometimes notification classes are in the object registry but other times not in the object registry, reason will figure out
15:11:36	ShilpaSD	gibi: will look into this tomorrow, and get back to you, leaving for now
15:11:46	ShilpaSD	gibi: thanks for discussion
15:12:02	gibi	ShilpaSD: @register decorator puts the class to the registry so the registry's content depends on which files are imported
15:13:02	gibi	ShilpaSD: good luck
15:13:02	gibi	:)
15:13:44	ShilpaSD	gibi: thats there, except for MyObj and MyOwnedObject, and for notification we have @, yes will check tomorrow, thanks for wishesregister_notification
15:14:14	ShilpaSD	gibi: thats there, except for MyObj and MyOwnedObject, and for notification we have @register_notification, yes will check tomorrow, thanks for wishes
15:17:51	dansmith	bauzas: replied to your naming comment on the cross-cell-resize spec, if you could take a look
15:18:04	bauzas	dansmith: sure ok
15:18:30	bauzas	oh, mriedem is s/French/on PTO/ this week, right
15:19:27	dansmith	bauzas: yes, and the deadline is next week
15:19:34	dansmith	so we basically need to get this merged before he gets back
15:19:41	bauzas	yup
15:19:57	dansmith	he's had it up for a long time
15:20:06	bauzas	that's even this Thursday, right?
15:20:36	dansmith	so given that (a) it's complex and we'll have to figure out some stuff during implementation, (b) those of you that haven't reviewed until now will have to be a little less picky :)
15:20:47	bauzas	I agree, hence my comment
15:20:53	dansmith	I don't remember, I thought it was end of week, but.. it's soon
15:21:02	bauzas	that's not a blocking comment, more a discussion to have
15:21:10	bauzas	but I see your comment you said
15:21:34	bauzas	anyway, I'll reply to you and +W it with comments
15:21:54	dansmith	bauzas: I'll start a revision on top where we can collect things like the naming
15:22:23	bauzas	that's not really a revision I want, I just want to make sure to understand why we need to add a new field
15:22:24	bauzas	:)
15:22:34	bauzas	but your comment about performance is understood
15:27:28	dansmith	bauzas: just replied to your other comments
15:32:04	openstackgerrit	Ifat Afek proposed openstack/nova master: The field instance_name was added to InstanceCreatePayload https://review.openstack.org/627967
15:34:53	openstackgerrit	Merged openstack/nova-specs master: Spec for cross-cell resize https://review.openstack.org/616037
15:36:53	openstackgerrit	Corey Bryant proposed openstack/nova stable/ocata: Ensure rbd auth fallback uses matching credentials https://review.openstack.org/627012
15:41:37	openstackgerrit	Jan Gutter proposed openstack/nova-specs master: Spec to implement vRouter HW offloads https://review.openstack.org/567148
16:38:16	spotz	Hey all, we're trying to run a Liberty devstack within AWS. Everything works fine except the console. Is there anyway to change the access_url to the external IP vs the private IP of the host VM?
16:51:31	bauzas	dansmith: ack, will look
17:00:31	openstackgerrit	Maria Luisa Arches proposed openstack/nova master: Skip checking of target_dev for vhostuser https://review.openstack.org/627540
17:55:39	openstackgerrit	Merged openstack/nova master: Fix circular import in nova.privsep.utils https://review.openstack.org/628302
18:24:24	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025
18:36:09	redkrieg	Does anyone have experience using nova-lxd and getting a NoValidHost error? The host was working before I got glance working on a swift backend but since that change I'm getting NoValidHost with reason set to "" and nothing else related in nova-conductor.log even with debug turned on. can someone suggest where I might look next?
18:54:40	openstackgerrit	Ken'ichi Ohmichi proposed openstack/nova master: Update nova docs front page for placement removal https://review.openstack.org/628183
18:55:05	openstackgerrit	Merged openstack/os-resource-classes master: Don't use upper-constraints when installing package https://review.openstack.org/628442
18:59:48	openstackgerrit	Chris Dent proposed openstack/os-resource-classes master: Update hacking version to latest https://review.openstack.org/628599
19:07:09	openstackgerrit	Merged openstack/os-resource-classes master: Update author-email in setup.cfg https://review.openstack.org/624885
21:15:57	openstackgerrit	Merged openstack/nova master: Run nova-lvm job on nova/privsep/* changes https://review.openstack.org/628306
23:08:08	openstackgerrit	Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256
23:10:04	openstackgerrit	Yongli He proposed openstack/nova-specs master: add 'show-server-group' spec https://review.openstack.org/612255
#openstack-nova - 2019-01-08
00:39:51	Swami	quick question on placement-api with stein. Enabling placement-api on compute node fails while trying to recreate_database. Is there any other option in devstack to prevent the database from recreating while running in the compute node. The placement-api is working fine on all in one node.
01:37:40	openstackgerrit	Corey Bryant proposed openstack/nova stable/ocata: Ensure rbd auth fallback uses matching credentials https://review.openstack.org/627012
01:58:57	openstackgerrit	Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163
02:28:27	openstackgerrit	Hesam Chobanlou proposed openstack/nova master: unused images are always deleted (add to in-tree hper-v code) https://review.openstack.org/628793
02:37:18	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add descriptions of numbered resource classes and traits https://review.openstack.org/621494
04:44:39	openstackgerrit	Merged openstack/nova master: Change to debug repetitive info messages https://review.openstack.org/628058
04:56:05	openstackgerrit	sean mooney proposed openstack/nova master: Skip checking of target_dev for vhostuser https://review.openstack.org/627540
09:04:08	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025
09:06:10	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Cleanup device at the end of 'test_iproute_object_closes_correctly' test https://review.openstack.org/629112
11:39:41	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Support native TLS for migration and disks over NBD https://review.openstack.org/625216
11:52:15	openstackgerrit	Chris Dent proposed openstack/nova master: Remove placement from nova install docs https://review.openstack.org/628184
11:52:24	openstackgerrit	Chris Dent proposed openstack/nova master: Remove link to placement configuration from nova config docs https://review.openstack.org/628185
11:52:35	openstackgerrit	Chris Dent proposed openstack/nova master: Remove placement from contributor doc https://review.openstack.org/628186
12:01:16	kashyap	sean-k-mooney1: gibi: When any of you get a moment, wonder if I can trick you to look at this change — "libvirt: Support native TLS for migration and disks over NBD" — https://review.openstack.org/625216
12:03:52	kashyap	(Or anyone here who has time.)
12:09:15	kashyap	cdent: Hi, there! Hehe, don't underestimate yourself :D
12:40:06	sean-k-mooney1	kashyap sure ill take a look. i think i added this to my list yesterday but did not get a chance to review it
12:44:06	openstackgerrit	Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163
13:40:10	openstackgerrit	Chris Dent proposed openstack/nova master: Delete the placement code https://review.openstack.org/618215
13:40:11	openstackgerrit	Chris Dent proposed openstack/nova master: Update nova docs front page for placement removal https://review.openstack.org/628183
13:40:11	openstackgerrit	Chris Dent proposed openstack/nova master: Remove placement from nova install docs https://review.openstack.org/628184
13:40:12	openstackgerrit	Chris Dent proposed openstack/nova master: Remove link to placement configuration from nova config docs https://review.openstack.org/628185
13:40:12	openstackgerrit	Chris Dent proposed openstack/nova master: Remove placement from contributor doc https://review.openstack.org/628186
13:43:27	kashyap	sean-k-mooney1: Thanks!
13:46:45	gibi	kashyap: do I see correctly that the bluprint for https://review.openstack.org/#/c/625216 haven't been approved yet?
13:47:35	kashyap	gibi: Before I left for PTO, melwitt and co said they'd approve it. And I recall even seeing it is approved
13:47:59	gibi	kashyap: this is still in new phase https://blueprints.launchpad.net/nova/+spec/support-qemu-native-tls-for-live-migration
13:48:17	kashyap	gibi: Ah, the "Definition"
13:48:35	kashyap	Hmm
13:52:52	gibi	kashyap: no worries, just ping melwitt when she is up and get the administration done
13:53:52	gibi	kashyap: I've started reading the implementation patch...
13:53:57	kashyap	Yep, will do. Thanks for noticing. I saw the BP, but the "Accepted for Stein" decieved me.
13:54:27	kashyap	gibi: It is nothing complex; but the underlying setup is a buffalo
13:55:18	kashyap	gibi: I wrote a detailed end-to-end test here (some parts of it are already scripted by installer tools): https://kashyapc.fedorapeople.org/Native-TLS/Setup-for-NBD-and-migration-streams-over-TLS.rst.txt
13:56:47	gibi	kashyap: thanks for the pointers. I will dissapeare for a meeting but still have the intention to go through your patch
13:57:06	kashyap	Sure. I don't count on instant responses :-) Thanks for the time!
13:57:52	gibi	kashyap: thank you for your effort!
13:59:57	kashyap	melwitt: When you're about, following up from our pre-break chat here: Mind ACKing this one, assuming I answered all questions: https://blueprints.launchpad.net/nova/+spec/support-qemu-native-tls-for-live-migration
14:09:17	maciejjozefczyk	melwitt: jaypipes PTAL https://review.openstack.org/#/c/614167/ https://review.openstack.org/#/c/591607 :)
14:20:42	jaypipes	maciejjozefczyk: on it.
14:28:50	maciejjozefczyk	thanks Jay
14:36:26	yonglihe	Jay, thank you giving constructive advice for spec "show-server-numa-topology," , https://review.openstack.org/#/c/612256/20
14:36:41	jaypipes	yonglihe: no problem. I'll do a final review on that shortly.
14:36:50	jaypipes	yonglihe: just finishing up maciejjozefczyk's second patch review.
14:37:16	yonglihe	so efficiency -:)
14:37:32	jaypipes	:)
14:38:22	maciejjozefczyk	jaypipes: after coffee already?

Earlier   Later