Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-24
22:34:34 dansmith mriedem: I mean your resource class gabbit thing
22:34:38 mriedem oh
22:34:42 mriedem yes it's worth trying
22:34:43 mriedem won't hurt
22:34:49 mriedem slamaro
22:34:52 mriedem *slamaroo
22:35:18 dansmith slamaroo'd
22:36:58 mriedem rebuild reno for the quotas change and then will push that up
22:37:00 mriedem *rebuilding
22:37:24 cfriesen what will be the ordering of steps when adding a new cellsv2 cell? create the new cell in the central DB and then start up the services or vice versa?
22:38:53 mriedem create the cell first
22:39:00 mriedem and then start services within it
22:39:04 mriedem just like with the single cell case
22:40:18 cfriesen okay, that makes sense
22:43:01 cfriesen mriedem: I guess we need the DB server for the new cell up and running before creating the new cell in nova
22:43:30 cfriesen or at least we need to know what it will be
22:46:07 mriedem yes
22:46:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
22:46:29 mriedem because when you run nova-manage cell_v2 create_cell you need to specify the MQ and DB URLs for the new cell
22:46:40 mriedem which get inserted into the cell_mappings table in the nova_api db
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

Earlier   Later