Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-17
11:15:59 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330
11:18:43 openstackgerrit Rikimaru Honjo proposed openstack/nova: Consolidate unit tests for shelve API https://review.openstack.org/421220
11:19:01 openstackgerrit Matt Riedemann proposed openstack/nova: Add more details when test_create_delete_server_with_instance_update fails https://review.openstack.org/421221
11:22:28 sahid hello mriedem, anychance you look at this serie when you have a moment https://review.openstack.org/#/c/409260/
11:24:12 mriedem i see you and pkoniszewski are having a debate on the top patch in the series
11:25:24 sahid yes the first one has already +2 and the last one need a third person
11:29:23 openstackgerrit Matthew Booth proposed openstack/nova: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859
11:38:37 openstackgerrit Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494
11:50:30 openstackgerrit Matt Riedemann proposed openstack/nova: Fanout the db sync to all cells if --local_cell is False https://review.openstack.org/420973
11:50:31 openstackgerrit Matt Riedemann proposed openstack/nova: Add --verbose option to discover_hosts command https://review.openstack.org/421228
11:53:31 openstackgerrit Matt Riedemann proposed openstack/nova: Add --verbose option to discover_hosts command https://review.openstack.org/421228
11:53:31 openstackgerrit Matt Riedemann proposed openstack/nova: Fanout the db sync to all cells if --local_cell is False https://review.openstack.org/420973
12:07:54 openstackgerrit Pavel Gluschak proposed openstack/nova: live-migration test hook for serial console https://review.openstack.org/347471
12:10:03 openstackgerrit Pavel Gluschak proposed openstack/nova: Added live migration test hook for VNC console https://review.openstack.org/421207
12:10:55 openstackgerrit Pavel Gluschak proposed openstack/nova: don't store VNC/SPICE listen addrs in migration data if disabled https://review.openstack.org/396218
12:40:28 mriedem claudiub|2: need to add hw_cpu_features to ImageMetaProps in a change before this https://review.openstack.org/#/c/398509/
12:44:07 dims "Pass 1265 Failure 60 Skip 85" on the first run of gate-tempest-dsvm-nova-py35-ubuntu-xenial (DSVM/Tempest/py35), not too shabby
12:44:18 dims Anyone interested in helping with this? ^
12:44:42 mriedem Kevin_Zheng: feel free to get a start on the pike spec for the tags length microversion, it should be a simple spec
12:44:50 mriedem and the nova-specs repo is available for posting pike specs now
12:45:12 Kevin_Zheng mriedem: thanks
12:45:23 Kevin_Zheng will do it if got time
12:48:21 claudiub|2 mriedem: yep. that is why I've put that patch on top of this: https://review.openstack.org/#/c/420960/1
13:20:12 openstackgerrit Lee Yarwood proposed openstack/nova: Compute: Reset bdm connection_info during LM rollback https://review.openstack.org/338929
13:20:13 openstackgerrit Lee Yarwood proposed openstack/nova: Live migration of instances with encrypted volumes https://review.openstack.org/389608
13:24:51 jaypipes *yawn*
13:26:56 jaypipes cdent: thx :)
13:27:21 openstackgerrit Chris Dent proposed openstack/nova: A demo script for updating resource inventory https://review.openstack.org/382613
13:37:32 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
13:38:29 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955
13:40:12 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964
13:40:40 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
13:41:26 openstackgerrit Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926
13:47:52 openstackgerrit garyk proposed openstack/nova: VMware: retry network lookup if network not ready https://review.openstack.org/391778
13:48:47 mdbooth I think it's fallen into the 'nobody really understands this code' hole, though.
13:49:20 mdbooth All the CI we have thinks it's good.
13:52:45 kashyap mdbooth: In such cases, shouldn't one just merge it, and not leave it in limbo? As the author, you understand it the deepest, and you're more than willing to explain any question to anyone.
13:53:06 kashyap I think it should simply be merged if there are no regressions, or obvious repurcussions -- which there are none at the moment.
13:54:27 kashyap Let's see what MattR has to say when he's here. He's usually quite clear with his stance.
13:56:05 tdurakov PaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov andreas_s abhishekk mrhillsman raj_singh wznoinsk siva_krishnan hi folks, live-migration subteam meeting in 5 minutes on #openstack-meeting-3
13:57:01 huanxie2 Hi all, I made a very small patch to correct the comment message about input param's data type https://review.openstack.org/#/c/419308/could you help have a review, thanks a lot
14:02:24 jaypipes huanxie2: approved it. thx for the fix.
14:02:46 huanxie2 Thanks jaypipes :)
14:02:50 jaypipes np
14:04:49 huanxie2 I also made another patch to fix the waiting vif-plug-event timeout problem when rescuing a VM https://review.openstack.org/#/c/413469/, could you also have a review if possible?
14:06:02 jaypipes huanxie2: yes, will do shortlyu.
14:08:05 huanxie2 Many thanks jaypipes
14:10:38 mriedem jaypipes: i took a look at your aggregate map patch, comments inline
14:11:02 jaypipes mriedem: already responded...
14:11:06 mriedem ok
14:11:09 jaypipes mriedem: thx, great review comments.
14:11:17 mriedem school is closed up here today so i'm going to be on and off
14:11:38 mdbooth mriedem: Snow day?
14:11:44 jaypipes mriedem: thx for letting me know. trying to perform a bunch of my day-job duties today :( (management stuff)
14:11:45 mriedem ice dsay
14:11:47 mriedem *day
14:13:32 mdbooth mriedem: I'm still banging on https://review.openstack.org/#/c/383859/ btw
14:13:32 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571
14:14:41 mdbooth I just updated it. sfinucan had asked for some docs, and I noticed an additional potential race while I was writing them. I've fixed the race and added docstrings.
14:15:17 mdbooth Jenkins thinks it's good, and I've run the updated patch against devstack nfs, which still passes.
14:17:12 lyarwood mdbooth: quick nit on that review around your use of os.path.ismounted
14:17:39 mdbooth The race was: if we lose and regain our libvirt connection, when we re-initialise the mount state we couldn't be certain that there were no ongoing operations from before we lost the connection.
14:17:40 lyarwood mdbooth: still looking at the rest, started there given the other change I'm working on
14:18:06 mdbooth lyarwood: Thanks for looking!
14:19:46 tdurakov johnthetubaguy: hi, are you around?
14:20:19 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
14:23:06 mdbooth lyarwood: That raises a follow-on question about is_mounted, btw.
14:23:23 mdbooth I'm going to tread carefully around that rabbit hole :) I don't want to go down it.
14:24:49 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330
14:25:14 openstackgerrit Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494
14:25:38 mriedem mdbooth: ack. are you ok with the latest version of https://review.openstack.org/#/c/312488/ now?
14:26:00 lyarwood mdbooth: I assume your going to ask if it did that check before right?
14:26:06 lyarwood you're*
14:26:27 mdbooth mriedem: Just checking again. I was nearly ok with with it last time iirc.
14:27:15 mdbooth lyarwood: I responded to your comment.
14:28:23 lyarwood mdbooth: yup good point
14:30:28 diana_clarke mriedem: Let me know if there is anything you want help with, given ice day and all. In particular, is there anything I can help with wrt what tripleo needs from nova?
14:30:56 diana_clarke man pages etc
14:31:31 mriedem ok, hm
14:31:53 openstackgerrit Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791
14:31:54 mriedem it would be nice to have man page entries for the cells v2 commands
14:32:16 mriedem in here http://docs.openstack.org/developer/nova/man/nova-manage.html
14:32:45 diana_clarke mriedem: yeah, I saw the ask in the nova-cells-v2 wiki, which is why I asked. I no one is doing that, I can take that on.
14:33:03 diana_clarke typo: *if* no one is doing that...
14:33:17 mriedem i don't think anyone is working on that so it's all yours
14:33:35 mriedem you might want to do them one at a time in a series just to make reviews easier
14:34:02 diana_clarke mriedem: okie, dokie. and I'll update the wiki, claiming that task
14:34:42 mriedem we might want to get https://review.openstack.org/#/c/420442/ merged to avoid conflicts too
14:34:50 mriedem sdague: super simple change https://review.openstack.org/#/c/420442/
14:35:17 sdague mriedem: bam
14:37:36 mriedem thanks emeril
14:38:25 mriedem sean-k-mooney: do you know if we can re-use _get_vif_name in here https://review.openstack.org/#/c/410737/ ? i.e. is the device field set on that type of vif for vhostuser?
14:39:44 mriedem i guess the vif does have a devname, "devname":"tap96429488-f7"
14:39:50 mriedem but it's not what's needed here i guess
14:43:24 markus_z tdurakov: sorry, I missed the meeting
14:44:03 tdurakov markus_z: ping me once update the hook
14:45:06 markus_z tdurakov: I pushed the teardown pkoniszewski asked for in ps42. https://review.openstack.org/#/c/347471/
14:45:21 tdurakov acked, let me check it again
14:45:28 markus_z thanks!
14:49:23 sean-k-mooney mriedem just looking now

Earlier   Later