Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-20
22:06:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
22:06:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID https://review.openstack.org/523606
22:06:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400
22:14:02 openstackgerrit Chris Dent proposed openstack/nova master: Do not set allocation.id in AllocationList.create_all() https://review.openstack.org/529397
22:40:01 oomichi alex_xu: yeah, I reviewed that. Thanks for your work. The other https://review.openstack.org/#/c/529207 needs to be updated in your patch sequence
23:07:29 openstackgerrit Ed Leafe proposed openstack/nova master: Don't try to delete build request during a reschedule https://review.openstack.org/528835
23:07:30 openstackgerrit Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436
23:07:30 openstackgerrit Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358
23:07:37 edleafe mriedem: ^^ addressed your comments
23:07:44 edleafe and rebased your bottom patch
23:07:52 edleafe sorry
23:09:11 mriedem as long as i don't lose the +2 it's fine :)
23:13:55 edleafe you're still good :)
23:32:05 mriedem edleafe: so can we split the unnecessary-for-this-patch scheduler thing out of this change and out of the series and just have it on it's own?
23:41:50 edleafe mriedem: if I did that, I'd have to also undo the test cleanups that assumed that host_list[0] had an empty allocation_request
23:42:08 mriedem in test_filter_scheduler?
23:42:15 mriedem or test_conductor?
23:43:02 edleafe both
23:44:01 edleafe the test_filter_scheduler stuff could be broken out easily enough, but the conductor tests are more tangled
23:44:26 mriedem https://www.amazon.com/Orangutangled-Sudipta-Bardhan-Quallen/dp/1477847170
23:44:29 mriedem you said the word of the day
23:44:30 mriedem ahhhhhhhhhhhhhh
23:44:44 mriedem sorry, it's a book i've had to read several times
23:45:30 mriedem alright nevermind then
23:45:42 mriedem just makes it hard to review big changes with unrelated stuff mixed into them
23:45:52 mriedem especially if it's not pointed out in the commit message
23:46:03 edleafe I understand, but it is kinda related :)
23:49:31 mriedem it's certainly orangutangled
#openstack-nova - 2017-12-21
01:12:18 openstackgerrit Mr Rambo proposed openstack/nova master: Fix the problems that volume-backed server rebuild https://review.openstack.org/528740
02:02:27 openstackgerrit Merged openstack/nova stable/newton: Patch mkisofs calls https://review.openstack.org/529284
02:17:15 Sandy619 Hello Guys, Need some information.. How do nova collect console logs of instances? Some of my instances logs are not coming, i need to troubleshoot this.
02:19:43 Sandy619 ping #openstack-nova , can you please help me with above query :) if anybody have an idea.
02:33:51 openstackgerrit Chen Hanxiao proposed openstack/nova master: InstanceInfo: store more runtime information of instance https://review.openstack.org/526348
02:34:18 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Fix functional tests for USE_NEUTRON https://review.openstack.org/529456
02:35:59 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Fix functional tests for USE_NEUTRON https://review.openstack.org/529456
02:54:45 openstackgerrit Jianle He proposed openstack/nova master: Fix nova backup for volume-backed instance https://review.openstack.org/529458
03:00:45 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: doc: update supported drivers for cpu topology https://review.openstack.org/529294
04:51:13 openstackgerrit Merged openstack/nova master: Delete the TypeAffinityFilter https://review.openstack.org/528099
06:03:31 openstackgerrit Merged openstack/nova master: Use instance.project_id when creating request specs for old instances https://review.openstack.org/529184
06:03:37 openstackgerrit Merged openstack/nova master: Workaround missing RequestSpec.project_id when moving an instance https://review.openstack.org/529185
06:03:44 openstackgerrit Merged openstack/nova master: Implement set_and_clear_allocations in report client https://review.openstack.org/528089
06:04:09 Rambo Fix the problems that volume-backed server rebuild
06:04:10 Rambo .Could you please review it? https://review.openstack.org/#/c/528740/
06:07:24 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: implement NotImplementedError for _set_quiesced https://review.openstack.org/529278
06:14:49 openstackgerrit Mr Rambo proposed openstack/nova master: Fix the problems that volume-backed server rebuild https://review.openstack.org/528740
06:34:34 openstackgerrit Merged openstack/nova master: trivial: conf: libvirt: fix a typo https://review.openstack.org/529229
06:41:14 openstackgerrit Merged openstack/nova master: [placement] Separate API schemas (allocation) https://review.openstack.org/528630
07:09:24 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
07:21:55 ameeda Hello, Can you please review my code here https://review.openstack.org/#/c/528385/ ?
07:29:03 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: validate SCSI controller model https://review.openstack.org/525055
08:13:02 ameeda Can you please review my code here https://review.openstack.org/#/c/526900/ ?
08:23:29 openstackgerrit xhzhf proposed openstack/nova master: [cellv2] performance issue when querying compute nodes in a cell https://review.openstack.org/529504
08:47:16 tetsuro sahid: I have updated this document revision patch about cpu topology on qemu TCG https://review.openstack.org/#/c/529294/
08:59:15 sahid tetsuro: oh, thanks a lot. I will see that :)
09:15:00 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Clean up foreign key before root resource provider deleting https://review.openstack.org/529519
09:16:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
09:32:06 bauzas good morning Nova
10:28:10 openstackgerrit Ilya Shakhat proposed openstack/nova master: Initialize osprofiler in WSGI application https://review.openstack.org/519664
10:31:49 stephenfin Not sure who's about between now and new year, but I'd appreciate some eyes on this series to get it over the line https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/websocket-proxy-to-host-security
10:32:02 stephenfin It's been ongoing since 2013. Would be great to get the damn thing finished, heh
10:44:49 openstackgerrit Chen Hanxiao proposed openstack/nova master: InstanceInfo: store more runtime information of instance https://review.openstack.org/526348
10:53:16 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: validate SCSI controller model https://review.openstack.org/525055
11:08:16 openstackgerrit Alex Xu proposed openstack/nova master: Merge ResourceV21 obj into Resource obj https://review.openstack.org/529217
11:08:16 openstackgerrit Alex Xu proposed openstack/nova master: Remove the LoadedExtensionInfo object https://review.openstack.org/529207
11:08:17 openstackgerrit Alex Xu proposed openstack/nova master: Remove the inherits parameter for the Resource object https://review.openstack.org/529218
11:09:01 alex_xu oomichi: ^ thanks for the review, just updated
11:35:03 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix error when deleting any root resource provider https://review.openstack.org/529519
11:41:10 openstackgerrit xhzhf proposed openstack/nova master: [cellv2] performance issue when querying compute nodes in a cell https://review.openstack.org/529504
11:47:03 openstackgerrit Matthew Booth proposed openstack/nova master: Removed unused argument from LibvirtDriver._connect_volume https://review.openstack.org/529556
11:47:44 mdbooth lyarwood: Cleanup related to our earlier discussion ^^^
11:49:32 lyarwood mdbooth: ack nice!
11:50:20 lyarwood mdbooth: I wonder if we can do the same with disk_dev on _disconnect_volume
11:50:49 mdbooth lyarwood: Don't. I've been actively trying to limit the quantity of ocean I'm boiling lately.
11:52:00 mdbooth Cleanup as you go is a wonderful concept which I fully support, but in practice it seems to be where my patch series end up getting buried.
11:52:33 lyarwood meh, I'll do this as a standalone, if it lands great :)
11:53:08 mdbooth I wanted to do ^^^ because disk_info is relevant to my other series.
11:53:25 mdbooth I may actually incorporate it in there, as it probably reduces some test noise.
11:53:47 lyarwood kk
11:56:42 mdbooth lyarwood: FWIW, looks like disk_dev can also be removed. It's only used for logging.
11:57:01 mdbooth And the log messages aren't really required given the context.
11:57:16 mdbooth Meh, I'll do it anyway as I have it open
11:57:40 lyarwood wonderful
11:59:11 mdbooth lyarwood: Remind me, can connection_info contain secrets?
12:00:29 lyarwood mdbooth: yeah I think so
12:00:47 mdbooth I'd have thought so. I'll just confirm.
12:11:17 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls https://review.openstack.org/460243
12:12:46 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008
12:13:56 lyarwood stephenfin: sorry, was about to do that, just respinning the refactor one final time
12:14:25 stephenfin lyarwood: No problem :) I didn't realize you'd pushed a new revision
12:14:32 stephenfin Comments on said revision incoming though
12:14:43 lyarwood stephenfin: thanks
12:16:26 stephenfin lyarwood: and done
12:17:11 lyarwood stephenfin: cheers, yeah that's why I was respinning
12:17:23 stephenfin :) Cool
12:19:45 openstackgerrit Matthew Booth proposed openstack/nova master: Fix unit test failures when direct IO not supported https://review.openstack.org/529568
12:20:20 mdbooth If we can get ^^^ in sharpish that would be awesome. It's really annoying.
12:21:13 stephenfin mdbooth: (y)
12:23:41 stephenfin mdbooth: Regarding the preceding patch, do you know if we support out-of-tree volume drivers? https://review.openstack.org/#/c/529556/
12:24:51 stephenfin Also, the one you linked could probably be rebased onto master to get it in sooner. No dependency that I can see
12:27:14 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls https://review.openstack.org/460243

Earlier   Later