Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-12
05:30:14 openstackgerrit Merged openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105
06:03:12 openstackgerrit Merged openstack/nova master: test server evacuation with placement https://review.openstack.org/492548
06:16:46 openstackgerrit Merged openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112
18:29:08 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285
18:29:08 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
18:29:08 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
18:32:37 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Add attachment_complete call to volume/cinder.py https://review.openstack.org/493323
18:32:38 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
18:32:38 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285
#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: Tweak connection_info translation for the new Cinder attach/detach API https://review.openstack.org/493324
18:40:25 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP - Implement new attach Cinder flow https://review.openstack.org/330285
#openstack-nova - 2017-08-14
00:05:51 openstackgerrit Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
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:52 openstackgerrit Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190
00:05:52 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
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: WIP: Read from console ptys using privsep. https://review.openstack.org/489486
00:05:57 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
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: Move common definition into common layer https://review.openstack.org/489491
07:09:11 openstackgerrit jichenjc proposed openstack/nova master: WIP:check query param for service's index function https://review.openstack.org/489492
07:14:34 openstackgerrit jichenjc proposed openstack/nova master: Add description for tag support in attach https://review.openstack.org/481479
07:14:34 openstackgerrit jichenjc proposed openstack/nova master: Add device tag support info in support matrix https://review.openstack.org/481478
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

Earlier   Later