Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-02
06:33:19 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
06:53:29 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use plain routes list for os-consoles, os-console-auth-tokens endpoint instead of stevedore https://review.openstack.org/462175
07:26:41 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use plain routes list for block_device_mapping instead of stevedore https://review.openstack.org/462180
07:40:23 openstackgerrit Felix Ma proposed openstack/nova master: Handle bdm save failure in attach https://review.openstack.org/470181
07:50:19 openstackgerrit Michael Henkel proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183
07:51:36 openstackgerrit Michael Henkel proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183
08:27:20 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use plain routes list for user_data instead of stevedore https://review.openstack.org/462183
08:47:56 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
09:16:41 cdent alex_xu: if you're still around, can you add a bit more detail on the scenario that would cause the duplicate problems in https://review.openstack.org/#/c/469578/ ? I had though that the combination of the local process lock, and the writer transaction would handle it? Are you concerned about the situation where there are different versions of the os-traits library on different machines?
09:38:05 openstackgerrit sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951
09:39:08 alex_xu cdent: I mean when multiple placement-api deployed on multiple nodes. When two users request to list traits, both of them will trigger the sync
09:39:36 alex_xu only one of them can sync successfully
09:39:51 alex_xu the transaction doesn't help that case I think
09:40:03 cdent that's the part i don't understand, why not?
09:40:14 cdent I had thought the transaction would?
09:40:57 cdent but then I'm often confused by what actually happens with transactions
09:42:01 alex_xu the transcation only ensure the consistent view of the data, except you set the transaction isolation level to serializable
09:46:39 cdent alex_xu: so, is the simplest solution to catch the duplicate error, log it, and move on?
09:47:29 alex_xu cdent: yea, I think so
10:04:50 openstackgerrit Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524
10:04:51 openstackgerrit Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625
10:12:33 openstackgerrit Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/447505
10:27:44 openstackgerrit NidhiMittalHada proposed openstack/python-novaclient master: Reset-state's default parameter needs correction https://review.openstack.org/463477
10:41:31 kashyap mriedem: When you're about, as someone who dabbles with libvirt / block layers sometimes, might want to look at this - https://review.openstack.org/#/c/465205/
10:41:35 kashyap It is ready to merge
10:41:42 kashyap And, fairly straight-forward
10:49:10 openstackgerrit NidhiMittalHada proposed openstack/python-novaclient master: client.logger.warning wrongly used in migrations https://review.openstack.org/463508
11:01:28 openstackgerrit NidhiMittalHada proposed openstack/python-novaclient master: Help message for aggregate-update is ambiguous https://review.openstack.org/469753
11:03:25 openstackgerrit Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova. https://review.openstack.org/459166
11:15:58 openstackgerrit Dirk Mueller proposed openstack/nova-specs master: Fix citation references (Sphinx 1.6.x compatibility) https://review.openstack.org/470252
11:27:35 openstackgerrit Chris Dent proposed openstack/nova master: Fix ChanceScheduler returning a wrong node https://review.openstack.org/466725
11:31:06 openstackgerrit Dirk Mueller proposed openstack/nova-specs master: Fix citation references (Sphinx 1.6.x compatibility) https://review.openstack.org/470252
11:53:28 bauzas cdent: thanks for rebasing https://review.openstack.org/#/c/466725/
11:56:14 openstackgerrit Dirk Mueller proposed openstack/nova-specs master: Fix citation references (Sphinx 1.6.x compatibility) https://review.openstack.org/470252
12:09:17 openstackgerrit Dirk Mueller proposed openstack/nova-specs master: Fix citation references (Sphinx 1.6.x compatibility) https://review.openstack.org/470252
12:28:08 openstackgerrit Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/447505
12:47:26 mriedem sfinucan: kashyap: someone from your team (sahid?) should probably take a look at this libvirt realtime feature bug v
12:47:29 mriedem https://bugs.launchpad.net/nova/+bug/1695056
12:47:29 openstack Launchpad bug 1695056 in OpenStack Compute (nova) "libvirt realtime feature mlockall: Cannot allocate memory" [Undecided,New]
12:50:57 openstackgerrit Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526
12:51:20 openstackgerrit Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526
12:58:56 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660
13:07:29 openstackgerrit Alex Szarka proposed openstack/nova master: Refactor instance-exists docstring and unit test https://review.openstack.org/462738
13:13:00 openstackgerrit Eric Fried proposed openstack/nova master: Bump python-glanceclient minimum to 2.7.0 https://review.openstack.org/470281
13:21:19 openstack bug 1646896 in OpenStack Compute (nova) "System hangs when using NFS storage backend with loopback mounts" [Low,Confirmed] https://launchpad.net/bugs/1646896
13:21:19 openstackgerrit Mihail proposed openstack/nova master: proposed fix for bug 1646896 - System hangs when using NFS storage backend with loopback mounts, Closes-Bug: #1646896 https://review.openstack.org/422041
13:23:56 openstackgerrit Chris Friesen proposed openstack/nova master: show flavor info in server details https://review.openstack.org/434514
13:28:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Replace messaging.get_transport with get_rpc_transport https://review.openstack.org/469659
13:38:03 mriedem sfinucan: bauzas: one of you want to pull the +W on this libvirt volume driver change? https://review.openstack.org/#/c/256292/ it's pretty trivial
13:40:55 bauzas mriedem: ack, looking
13:41:49 rfolco mriedem, would you please look at https://review.openstack.org/#/c/460231/ and https://review.openstack.org/#/c/460147/, both with one +2.
13:42:44 mriedem rfolco: yeah i can in a bit
13:42:55 rfolco mriedem, thx++
13:43:51 openstack Launchpad bug 1439855 in OpenStack Compute (nova) "encrypted iSCSI volume fails to attach, name too long" [Medium,Confirmed]
13:43:51 mriedem lyarwood: for when you're off sabbatical, this is right up your encrypted volume alley https://bugs.launchpad.net/nova/+bug/1439855
13:44:19 openstackgerrit Mihail proposed openstack/nova master: proposed fix for bug 1646896 - System hangs when using NFS storage backend with loopback mounts, Closes-Bug: #1646896 https://review.openstack.org/422041
13:44:19 openstack bug 1646896 in OpenStack Compute (nova) "System hangs when using NFS storage backend with loopback mounts" [Low,Confirmed] https://launchpad.net/bugs/1646896
13:58:12 openstackgerrit Matt Riedemann proposed openstack/nova master: placement: Specific error for inventory in use https://review.openstack.org/460231
13:58:24 openstack bug 1646896 in OpenStack Compute (nova) "System hangs when using NFS storage backend with loopback mounts" [Low,Confirmed] https://launchpad.net/bugs/1646896
13:58:24 openstackgerrit Mihail proposed openstack/nova master: proposed fix for bug 1646896 - System hangs when using NFS storage backend with loopback mounts, Closes-Bug: #1646896 https://review.openstack.org/422041
14:00:54 bauzas mriedem: had a comment on https://review.openstack.org/#/c/256292/6
14:02:49 bauzas mriedem: out of curiosity, do you know if Tooz driver is still a thing for our SG API or whether we should consider etcd ?
14:04:22 openstackgerrit Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/447505
14:07:29 mlavalle mriedem: hey. I am looking at https://bugs.launchpad.net/nova/+bug/1694417
14:07:30 openstack Launchpad bug 1694417 in OpenStack Compute (nova) "dhcp_domain used when use_neutron is not set" [Undecided,Incomplete]
14:07:34 claudiub mriedem: hi. i've found a nice pretty little bug. :) https://review.openstack.org/#/c/447505/13/nova/virt/libvirt/driver.py
14:08:17 mlavalle mriedem: seems that for the time being, we should wait for reply from the submitter
14:09:02 sfinucan mriedem, melwitt: All ze comments in https://review.openstack.org/#/c/379524/ _should_ be addressed now, if you fancy taking a gander today
14:09:40 sfinucan bauzas: RE: comments in https://review.openstack.org/#/c/256292/6/nova/virt/libvirt/volume/drbd.py
14:09:50 sfinucan instance is automatically exposed by oslo.log
14:10:04 sfinucan by way of the default formatter
14:10:10 bauzas mmmm okay
14:10:29 melwitt yeah, you can do that on any log message
14:10:31 sfinucan at least, iirc. mriedem has suggested I do something similar a few times previously
14:10:37 sfinucan *prior?
14:13:10 claudiub mriedem: also, one question: the cinder 3rd party ci requirements state that "CI's do not need to kick off tests until Jenkin's has given a +1", is there a similar policy for nova as well? ( https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#Third_Party_CI_Requirements )
14:20:50 efried claudiub I have that same thought every couple months ;-) Problem is that it would make overall completion take longer. Which is the lesser evil?
14:23:27 claudiub efried: i guess it depends on the size of the ci. at some point, all the ci resources could be allocated / in use. anyways, even though a ci can be configured to wait for jenkins to vote +1, it can still be triggered manually via "check some-ci" if needed.
14:23:37 mriedem bauzas: we should probably deprecate/remove tooz service group api driver
14:23:40 mriedem bauzas: it's not tested,
14:23:45 mriedem and i don't know if anyone outside of yahoo was using it
14:23:51 mriedem might be worth asking harlowja
14:24:07 mriedem bauzas: etcd is probably a better idea, and is something we can test in the gate
14:24:19 mriedem like i the multinode job or nova-next job
14:24:21 mriedem *in
14:24:30 efried claudiub Looks like mriedem missed that one in https://review.openstack.org/#/c/452958/ -- you going to propose the fix?
14:25:10 mriedem bauzas: instance=instance is common in all of our logging :)
14:25:15 claudiub ah, sure. will do.
14:25:37 mriedem efried: claudiub: missed what?
14:25:56 mriedem claudiub: i already have a patch for that
14:26:05 claudiub ah, ok.
14:26:15 mriedem claudiub: efried: https://review.openstack.org/#/c/469958/
14:26:21 mriedem bauzas: sfinucan: melwitt: dansmith: jaypipes: ^
14:26:31 mriedem and https://review.openstack.org/#/c/469740/
14:26:36 mriedem live migration rollback is borked w/o those two
14:26:52 bauzas mriedem: about instance=instance, sorry for my ignorance
14:27:16 sfinucan mriedem: In a meeting, but I'll look at them soon as I'm done
14:27:39 efried claudiub Perhaps you can suggest how to use autospec for https://review.openstack.org/#/c/469958/1/nova/tests/unit/virt/libvirt/test_driver.py ?
14:30:33 efried claudiub And https://review.openstack.org/#/c/469740/2/nova/tests/unit/compute/test_compute_mgr.py
14:32:19 claudiub efried: well, mock.patch does accept an "autospec" argument, but it doesn't work properly under the current release of mock. I have sent a bug report and bug fix for it: https://github.com/testing-cabal/mock/issues/396

Earlier   Later