Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-24
22:47:01 mriedem dansmith: ^ addressed the comments in the quotas changetest_migrate_quota_limits
22:47:03 mriedem oops
22:47:07 mriedem https://review.openstack.org/410946
22:55:59 openstackgerrit Gage Hugo proposed openstack/nova master: Admin API Policy contingent on is_admin_project https://review.openstack.org/384148
23:16:59 melwitt mriedem, dansmith: thanks yall
23:17:23 melwitt I was at the dentist, being angry
23:28:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
23:29:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
23:30:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
23:38:14 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova master: Remove the useless fake ExtensionManager from API unittests https://review.openstack.org/486416
#openstack-nova - 2017-07-25
00:40:42 openstackgerrit Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730
00:55:45 openstackgerrit Merged openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
01:02:29 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
01:04:37 openstackgerrit Merged openstack/nova master: Allow wrapping of closures https://review.openstack.org/479801
01:05:57 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Flush RC_CACHE after each gabbit sequence https://review.openstack.org/486805
01:08:21 cdent dansmith: if you’re still around, this is probably a better fix for that 201 204 problem: https://review.openstack.org/#/c/486805/
01:08:30 cdent (I couldn’t sleep, got bored, read mail…)
01:08:31 alex_xu oomichi: appreciate if you can take a look at https://review.openstack.org/#/c/486414/, that patch is the goal of pike
01:15:19 mriedem cdent: i'll drop mine once yours passes functional tests
01:15:30 cdent mriedem: roger that
01:32:22 openstackgerrit Merged openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949
02:07:58 openstackgerrit Merged openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
02:28:54 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Using wsgi-intercept in OSAPIFixture https://review.openstack.org/486825
02:29:48 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Use wsgi_intercept in PlacementFixture https://review.openstack.org/486237
02:30:26 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Using wsgi-intercept in OSAPIFixture https://review.openstack.org/486825
02:40:57 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829
02:41:44 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Update the placement deployment instructions https://review.openstack.org/469048
02:44:43 openstackgerrit Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
02:44:44 openstackgerrit Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
02:44:44 openstackgerrit Tony Breeds proposed openstack/nova master: Move the last_bytes util method to libvirt https://review.openstack.org/472228
02:44:45 openstackgerrit Tony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py https://review.openstack.org/485907
02:44:45 openstackgerrit Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
02:44:46 openstackgerrit Tony Breeds proposed openstack/nova master: rootwrap.d cleanup mislabeled files https://review.openstack.org/486831
02:56:15 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Using wsgi-intercept in OSAPIFixture https://review.openstack.org/486825
03:01:20 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
03:01:47 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829
03:30:24 TheJulia hshiina: looks like https://review.openstack.org/215385 is already in merge conflict again :(
03:58:13 openstackgerrit Julia Kreger proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
03:58:23 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850
04:17:24 openstackgerrit Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995
04:47:19 openstackgerrit Merged openstack/nova master: request_log addition for running under uwsgi https://review.openstack.org/485602
05:17:22 openstackgerrit sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
06:17:28 openstackgerrit Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
06:17:28 openstackgerrit Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
06:17:29 openstackgerrit Tony Breeds proposed openstack/nova master: Move the last_bytes util method to libvirt https://review.openstack.org/472228
06:17:29 openstackgerrit Tony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py https://review.openstack.org/485907
06:17:30 openstackgerrit Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
06:17:30 openstackgerrit Tony Breeds proposed openstack/nova master: rootwrap.d cleanup mislabeled files https://review.openstack.org/486831
06:20:56 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850
06:30:38 trinaths hi, I'm using nova (mitaka) compute. when I spawn a VM, it will auto shutoff after 10 minutes. and at qemu logs, it was the error: 2017-07-25T00:52:49.334691Z qemu-system-aarch64: Failed initializing vhost-user memory map, consider using -object memory-backend-file share=on
06:30:38 trinaths qemu-system-aarch64: /root/qemu/hw/virtio/vhost.c:1186: vhost_virtqueue_mask: Assertion `vdev->use_guest_notifier_mask' failed. Please help me in resolving this issue.
06:33:45 openstackgerrit Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
06:37:06 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954
06:54:24 openstackgerrit Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
06:54:25 openstackgerrit Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
06:54:25 openstackgerrit Tony Breeds proposed openstack/nova master: Move the last_bytes util method to libvirt https://review.openstack.org/472228
06:54:26 openstackgerrit Tony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py https://review.openstack.org/485907
06:54:26 openstackgerrit Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
06:54:27 openstackgerrit Tony Breeds proposed openstack/nova master: rootwrap.d cleanup mislabeled files https://review.openstack.org/486831
07:47:18 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
08:03:22 openstackgerrit Rawan Herzallah proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640
08:07:42 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850
08:15:30 jangutter Hi, there are two patches left for netronome-smartnic-enablement. Would anyone have cycles to leave comments? 50 lines of code, 300 lines of tests, many litres of blood. https://review.openstack.org/#/c/486426/ and https://review.openstack.org/#/c/483459/
08:16:43 jangutter For that matter, Mellanox's got a similar patch: https://review.openstack.org/#/c/398265/
08:42:46 openstackgerrit Artem Tiumentcev proposed openstack/nova-specs master: SPEC: nova disk quota tracking https://review.openstack.org/486947
08:49:08 openstackgerrit Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
09:01:21 kashyap mdbooth: Thanks for the review here: https://review.openstack.org/#/c/485752/
09:01:32 kashyap mdbooth: Test - I will fix, that's also why I -1ed it myself.
09:01:55 kashyap mdbooth: But the core change itself is working; the downstream bug reporter has tested the fix and confirmed that they don't see the perf regression anymore with this fix
09:18:24 mdbooth kashyap: Yeah, I said it looked good :)
09:18:43 mdbooth Except for the bits which didn't, but you can sort those easily enough.
09:18:57 kashyap mdbooth: Sure, did notice your remark. :-)
09:19:17 kashyap mdbooth: The bits which didn't are - (a) missing tests; (b) adding in the config line that accidentally removed
09:19:21 kashyap Anything else you noticed?
09:19:42 mdbooth Nope.
09:20:05 mdbooth The code structure is kinda ugly, but sahid already noted that
09:20:22 openstackgerrit Alexandru Muresan proposed openstack/nova master: Ensure that vif ports are unplugged during cold migration https://review.openstack.org/486955
09:22:38 kashyap mdbooth: There's a comment from Sahid? I don't even see it, /me refreshes the URL
09:22:49 mdbooth Not a review comment
09:22:49 kashyap Maybe you mean in the code
09:22:53 kashyap Yeah, noted.
09:22:57 mdbooth A code comment from the distant past
09:23:56 kashyap Yep
09:25:33 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842
09:31:47 openstackgerrit Rikimaru Honjo proposed openstack/nova master: compute: Sweep duplicated bdm records https://review.openstack.org/290793
09:32:31 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/482557
09:36:02 bauzas gabor_antal: gibi: about https://review.openstack.org/#/c/467514/7/nova/compute/api.py@4977 honestly my only concern was that I think "import" is probably not a great RESTful action
09:37:10 bauzas gabor_antal: gibi: cdent reminded me previously that a PUT can create or update with a body so in theory there should only be one notification of creation
09:37:39 bauzas but anyway, seems I'm nitpicking
09:45:44 bauzas gibi: btw. any notifications patches I could help to land before FF ?
10:18:30 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from ``ip`` commands to ``pyroute2`` https://review.openstack.org/484386
10:29:33 gibi bauzas: hi! thanks for the feedback. I don't have direct user feedback so I cannot say for sure that we need import in the notification but it feels useful for me
10:29:43 bauzas cools
10:31:14 gibi bauzas: for patches to help in, I see on patch that needs a second +2 https://review.openstack.org/#/c/480984/
10:31:51 gibi bauzas: besides that we have a list of patches that looks good from subteam perspective (and has +1) here https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/versioned-notification-transformation-pike+label:Code-Review%253E%253D%252B1+label:Verified%253E%253D1+AND+NOT+label:Verified%253C0+AND+NOT+label:Code-Review%253C0
10:34:12 openstackgerrit Gábor Antal proposed openstack/nova master: Transform HostAPI-set_enabled notifications https://review.openstack.org/482622
10:34:15 gibi bauzas: any help is appreciated
10:38:28 bauzas gibi: going out for lunch but will review those afterwards
10:40:50 openstackgerrit Merged openstack/nova master: Remove check_detach https://review.openstack.org/446671
10:43:18 gibi bauzas: enjoy for lunch, I just brought mine in the meantime

Earlier   Later