Posted	Nick	Remark
#openstack-nova - 2017-08-13
00:32:57	openstackgerrit	Jianghua Wang proposed openstack/nova master: WIP: support os_xenapi's VDI streaming https://review.openstack.org/486475
18:40:24	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
18:40:25	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285
18:40:25	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
#openstack-nova - 2017-08-14
00:05:51	openstackgerrit	Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
00:05:51	openstackgerrit	Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
00:05:52	openstackgerrit	Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
00:05:52	openstackgerrit	Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190
00:05:53	openstackgerrit	Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438
00:05:55	openstackgerrit	Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
00:05:56	openstackgerrit	Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
00:05:57	openstackgerrit	Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
00:05:57	openstackgerrit	Michael Still proposed openstack/nova master: WIP: Read from console ptys using privsep. https://review.openstack.org/489486
00:05:58	openstackgerrit	Michael Still proposed openstack/nova master: Refactor libvirt.utils.execute() away. https://review.openstack.org/489816
01:41:11	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
05:00:45	openstackgerrit	Thomas Bechtold proposed openstack/nova master: Handle deleted instances when refreshing the info_cache https://review.openstack.org/398308
05:00:53	openstackgerrit	Thomas Bechtold proposed openstack/nova master: Handle deleted instances when refreshing the info_cache https://review.openstack.org/398308
05:01:56	toabctl	oomichi, updated https://review.openstack.org/#/c/398308
05:25:35	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
05:30:44	openstackgerrit	jichenjc proposed openstack/nova master: Raise MigrationPreCheckError if instance location is unavailable https://review.openstack.org/469779
05:41:51	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
05:59:28	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
05:59:41	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
06:01:11	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - List/Show all server migration types https://review.openstack.org/430839
06:11:53	openstackgerrit	Hesam Chobanlou proposed openstack/nova master: adding cli documentation for online_data_migrations to clarify when the command is complete. https://review.openstack.org/493442
06:36:06	openstackgerrit	jichenjc proposed openstack/nova master: Avoid race for status update and field value https://review.openstack.org/493448
06:42:11	openstackgerrit	jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458
07:09:11	openstackgerrit	jichenjc proposed openstack/nova master: WIP:check query param for service's index function https://review.openstack.org/489492
07:09:11	openstackgerrit	jichenjc proposed openstack/nova master: Move common definition into common layer https://review.openstack.org/489491
07:14:34	openstackgerrit	jichenjc proposed openstack/nova master: Add device tag support info in support matrix https://review.openstack.org/481478
07:14:34	openstackgerrit	jichenjc proposed openstack/nova master: Add description for tag support in attach https://review.openstack.org/481479
07:20:33	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix messages in functional tests https://review.openstack.org/493460
07:31:28	openstackgerrit	Balazs Gibizer proposed openstack/nova master: test shelve and shelve offload with placement https://review.openstack.org/493062
08:02:10	openstackgerrit	Merged openstack/nova master: doc: address review comments in stable-api guide updates https://review.openstack.org/492690
09:48:17	openstackgerrit	Kevin Zhao proposed openstack/nova master: Make uefi as the default boot perproties for AArch64 https://review.openstack.org/489951
10:53:17	openstackgerrit	Dmitry Tantsur proposed openstack/nova master: Fix reporting inventory for provisioned nodes in the Ironic driver https://review.openstack.org/492964
10:53:18	dtantsur	vdrok: mind re-testing ^^^ ?
10:53:34	dtantsur	essentially, this is commit message and debug logging only change, I've figured out other concerns
11:05:47	vdrok	dtantsur: hey, I think I can, but if it's only debug logging addition, is there a reason? :)
11:11:36	dtantsur	dunno, I'm paranoid :)
11:11:40	dtantsur	yeah, maybe it's fine
12:48:23	openstackgerrit	Balazs Gibizer proposed openstack/nova master: delete allocation of evacuated instance https://review.openstack.org/493037
12:58:54	cdent	gibi: you seen this https://review.openstack.org/#/c/493448/
12:59:56	openstackgerrit	Chris Dent proposed openstack/nova master: Reset client session when placement endpoint not found https://review.openstack.org/493536
13:00:49	cdent	gibi: also your ideas on that ^ would be helpful
13:03:34	cdent	efried: you too on that ^^
13:04:00	efried	cdent Which?
13:04:10	cdent	placement endpoint not found
13:04:21	efried	cdent rgr, looking.
13:04:59	cdent	efried: also, did you see my response to you on the -dev list ? make sense or did I miss the boat?
13:05:11	efried	cdent I did, and it does... somewhat.
13:05:49	cdent	only somewhat?
13:05:56	efried	cdent I gather a lot of folks set up their endpoints at http://example.com:1234/ instead of http://example.com/service
13:06:12	efried	Though I understand the former is now discouraged.
13:06:14	cdent	yes
13:06:27	efried	So in the former case, the `self` links will come back *without* the prefix?
13:06:52	cdent	yes, the presence of the prefix is unrelated to the service catalog entry, it is based entirely on the web server config
13:07:26	efried	cdent Well, I hadn't assumed it had to do with the service catalog; but was hoping it wasn't hardcoded somewhere :)
13:07:31	gibi	cdent: for the first, I noticed that this morning
13:07:39	gibi	cdent: test_evacuate is unstable
13:08:04	cdent	efried: it’s not hardcoded in placement itself. The prefix comes from the environ[‘SCRIPT_NAME’]
13:08:15	gibi	cdent: I will pushed a fix for that in the evacuate bugfix but then I will remove that and leave comment on https://review.openstack.org/#/c/493448/ instead
13:08:26	cdent	gibi: I tested it out myself quite a bit and from what I could tell it was a matter of the loop timing out too soon
13:08:49	cdent	with a longer loop it worked
13:10:01	gibi	cdent: it happens because instance already in ACTIVE state when the the loop starts
13:10:10	gibi	cdent: at least in my trial
13:10:13	efried	cdent Okay, so (and perhaps this is in the devref (is that up yet?), but) how is a consumer supposed to use the links? Seems like nontrivial url dicing would be required.
13:10:15	gibi	cdent: but anyhow
13:10:26	gibi	cdent: my suggestion is to wait for both the status and the host
13:10:35	gibi	cdent: and we already have a function for that
13:11:11	cdent	huh, I’ll take your word for it gibi, I could consistently get it to fail with short timeout, and not with long, but maybe that was just (bad-) luck
13:11:18	gibi	cdent: https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L221
13:11:32	gibi	cdent: I suggest to use this ^^
13:11:47	gibi	cdent: this way we wait for both the ACTIVE status and the new host name to appeare on th REST API
13:13:38	cdent	efried: a) if the consumer is using ksa then it is ksa which is making the decision to not treat an absolute link as an absolute link, b) I don’t reckon anybody uses the links anyway, certainly not in any code that I’ve seen. The code I’ve seen behaves as if it is all “well known urls”, which is probably best practice when using a url-manipulating client like ksa
13:14:11	efried	cdent Hmph. Then what's the point in having 'em in there in the first place?
13:14:12	efried	But okay.
13:14:23	gibi	cdent: I left a comment in https://review.openstack.org/#/c/493448
13:15:23	cdent	efried: it (things like ksa) does bugger the concept of HATEOAS. The reason for having the links is if you are using a client that doesn’t do a requests style mount
13:15:41	cdent	of which there could easily be
13:15:50	cdent	the server needs to be client agnostic
13:16:08	efried	Okay, fair enough. Thanks for the explanation.
13:16:43	cdent	efried: I’m totally with you that it is weird
13:17:34	cdent	gibi, makes sense
13:18:04	cdent	these non-atomic updates are bewildering
13:18:13	cdent	but not surprising, just hard to track
13:20:10	gibi	cdent: an extra complication that the propose change in the _wait_for_state_change function is no the one that is called by the actual failing test
13:20:31	cdent	oops
13:20:34	cdent	:)
13:20:59	gibi	now I start reading the your report client patch :)
13:21:14	cdent	it should be a little more straightforward...maybe
13:39:12	gibi	cdent: your report client patch looks good to me but I have limited knowledge about ksa
13:39:54	jangutter	Hi, we're testing a third-party CI to test OpenStack on Netronome hardware (specifically with regard to https://review.openstack.org/#/c/491502/ ). Would anyone be willing to give feedback?
13:40:44	cdent	jangutter: just a heads up this week a significan number of nova cores are away on holiday, so the amount of feedback may be limited
13:41:24	jangutter	cdent: I saw, this isn't on our critical path, I'll re-post next week or so.
13:42:23	jangutter	cdent: more or less looking for "run, you fools!" kind of feedback now.
13:42:36	cdent	that’s my default state
13:42:40	cdent	so I’ll look!\
13:50:24	edleafe-	Scheduler subteam meeting in 10 minutes in #openstack-meeting-alt
13:51:07	openstackgerrit	Merged openstack/nova master: Handle addition of new nodes/instances in ironic flavor migration https://review.openstack.org/487954

Earlier   Later