Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-03
15:12:21 mriedem which points you here docs.ceph.com/docs/master/rbd/rbd-openstack/#configuring-cinder
15:13:06 satyar mriedme that means the cinder.conf will have the rbd_secret_uuid?
15:13:25 jaypipes satyar: yes, looks like it.
15:13:34 satyar hmmm
15:13:40 cdent rfolco: so a couple of issues: running one test with subunit won't work because the way test generation and discovery works, but you can get at just one from testr with: tox -efunctional test_placement_api.inventory_inventories_for_missing_provider
15:13:53 satyar now i have to change the things :)
15:14:08 satyar Thanks a lot jaypipes and mriedem
15:14:12 mriedem yw
15:14:17 satyar will update how it goes
15:14:24 cdent rfolco: however you can't really use the debugger in that setup because the stdin/out of the debugger is on the wrong side of the wsgi-intercept running the placement application
15:14:27 jaypipes satyar: not sure I did much, but you're welcome anyway :)
15:14:53 cdent rfolco: and because of the stdout capturing that is happening. if you can work around that, it might be doable.
15:14:59 rfolco cdent, that describes exactly what my problem is :)
15:15:01 cdent I tend to just sys.stderr.write all over the place :)
15:15:33 rfolco good advice cdent thx
15:15:48 cdent if you turn off the stdout/in capturing fixture you might be able to make some headway, but I'm not sure
15:16:34 cdent rfolco: i need to run off for a little while but will be back in about an hour, let me know how it goes and if I can be of more help
15:16:56 openstackgerrit Samantha Blanco proposed openstack/nova: Add support matrix for attach and detach interfaces https://review.openstack.org/409967
15:26:20 openstackgerrit Jay Pipes proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472
15:26:21 openstackgerrit Jay Pipes proposed openstack/nova: rt: explicitly pass compute node to _update() https://review.openstack.org/398472
15:26:23 openstackgerrit Jay Pipes proposed openstack/nova: rt: use a single ResourceTracker object instance https://review.openstack.org/398473
15:31:12 openstackgerrit Julian proposed openstack/nova: Create schema generation for NetworkModel https://review.openstack.org/408737
15:32:58 openstackgerrit Samantha Blanco proposed openstack/nova: Add support matrix for attach and detach interfaces https://review.openstack.org/409967
15:34:16 dansmith jaypipes: did you ever return to this to read my comment? https://review.openstack.org/#/c/407238/
15:34:41 jaypipes dansmith: hey Dan, yeah actually I just had that up...
15:35:00 openstackgerrit Rafael Folco proposed openstack/nova-specs: DELETE all inventories for a resource provider https://review.openstack.org/415885
15:35:01 jaypipes dansmith: I'm cool fixing just that area.
15:39:25 openstackgerrit Rafael Folco proposed openstack/nova-specs: DELETE all inventories for a resource provider https://review.openstack.org/415885
15:59:42 kashyap sahid (or anyone interested in this area): Hey there, when you have a moment, mind looking at this -- https://review.openstack.org/#/c/411936/ (libvirt: avoid generating script with empty path)
16:00:44 jaypipes dansmith: k, changed to +2 and recheck'd all the patches in that series up to the WIP ones.
16:01:10 dansmith jaypipes: thanks.. I figure they might need a rebase anyway, but wanted to make sure we resolved that -1 before I did
16:05:45 mriedem dansmith: is there a reason we can't hide that in the Instance object?
16:06:06 dansmith mriedem: hide what?
16:06:06 mriedem to generically use the cell mapping context when lazy-loading attrs?
16:06:32 dansmith mriedem: well, the lazy loading code can be run inside the cell too, which can't look up mappings
16:07:04 mriedem ok so it's done in the view because that's api which we know has an api db connection
16:07:24 dansmith right
16:13:14 mriedem dansmith: do you want to make a comment on https://review.openstack.org/#/c/405948/ about using the network_info for this instead of a new column?
16:13:35 dansmith sure
16:21:07 openstackgerrit Neil Jerram proposed openstack/nova: libvirt: avoid generating script with empty path https://review.openstack.org/411936
16:30:05 cdent rfolco: I'm back now, you get things working like you want?
16:38:32 rfolco cdent, not yet, was stuck on a call, then I had to restore my environment, so running tox -e for the first time takes forever
16:39:07 cdent rfolco: heh, yeah, that can be painful
16:39:19 rfolco cdent, I am adding sys.stderr.write as you suggested....
16:40:06 cdent Are you having issues with getting gabbi to behave like you want, or is that the handler code you're writing isn't doing what you expect?
16:41:31 rfolco cdent, both. I added trhe DELETE method to call set_inventories, just like PUT does. I created a delete_all in gabbits, but I am getting HTTP 500 response...
16:43:22 cdent when you get the 500 you'll get two different tracebacks: one from gabbi saying the test failed (this is the client side of the wsgi-intercept), and then a bit further up, the one from the wsgi app itself (the server side). Usually the latter is the one that will actually be useful and point out where things have actually gone wrong
16:45:28 rfolco cdent, I assume the gabbits should run in sequence so the delete works only if you create a resource and inventory first for example.
16:46:13 cdent rfolco: yes, that's right: each file is one sequence. even if you ask for just one test, the one's before it in the same file will be run, to make sure those prereqs are done
16:46:35 cdent If you want to push your code to gerrit as a wip I can look at it too?
16:47:18 rfolco cdent, let me do my homework, won't bother you much at this point. It's been a nice learning process.
16:47:54 cdent sure, but if you get to a point where it is no fun, feel free to ping me. Is no bother at all.
16:48:11 rfolco cdent, thanks for the support
16:57:12 openstackgerrit Stephen Finucane proposed openstack/nova: Stop handling differences in registerCloseCallback https://review.openstack.org/416297
17:05:08 mriedem sfinucan: see my reply in https://review.openstack.org/#/c/415707/ ?
17:06:08 sfinucan mriedem: I did not. Checking
17:06:56 mriedem sfinucan: basically i'd rather not subclass and duplicate the non-network-specific tests
17:07:15 sfinucan mriedem: gotcha
17:07:17 mriedem those quota tests are hittin the nova-net floating IP manager code,
17:07:32 sfinucan just those two, right?
17:07:37 mriedem yeah i think so,
17:08:06 mriedem so i think what you were suggesting was the base class handles the non-network quota stuff, and that runs with either nova-net or neutron config, and then sublcass that to just run the nova-net specific tests
17:08:15 mriedem but that also duplicates the network agnostic quota tests
17:08:38 mriedem it could be a separate test class altogether, idk, doesn't really seem worth the churn
17:08:46 sfinucan yeah, exactly
17:08:47 sfinucan but
17:08:57 sfinucan I didn't realise the other tests were actually network agnostic
17:09:39 sfinucan I thought they were executing different (but duplicated) code paths for some reason
17:10:29 sfinucan Lemme take a second look at it. Defo not worth the churn if what you say is correct
17:24:31 mriedem dansmith: while you're looking at newton-y things :) https://review.openstack.org/#/c/408727/
17:24:50 dansmith ack
17:24:55 openstackgerrit Chris Dent proposed openstack/nova: Add an "Adding a Handler" section to placement_dev https://review.openstack.org/416304
17:24:58 mriedem ^ is something i'd like to get in and backported to newton so we can release it as it's an upgrade issue
17:27:30 openstackgerrit Chris Dent proposed openstack/nova: [placement] Add an "Adding a Handler" section to placement_dev https://review.openstack.org/416304
17:38:02 sfinucan mriedem: Saw the error of my ways
17:48:19 openstackgerrit Claudiu Belu proposed openstack/nova: Add support for setting boot order in Hyper-V https://review.openstack.org/248029
18:00:30 openstackgerrit Diana Clarke proposed openstack/nova: api-ref: use the examples with paging links https://review.openstack.org/416312
18:17:01 stewie925 happy new year everyone
18:17:52 stewie925 hey I was trying to rebase a merge conflict - fixed the conflict but ran into a test issue - http://paste.openstack.org/show/593804/
18:21:27 jaypipes stewie925: source .tox/epy27/bin/activate && pip install -r test-requirements
18:21:54 stewie925 jaypipes: thank you so much
18:22:02 jaypipes stewie925: any time :)
18:22:40 jaypipes stewie925: sorry, that should be:
18:22:43 jaypipes stewie925: source .tox/py27/bin/activate && pip install -r test-requirements
18:22:53 jaypipes (no e in the tox env...)
18:23:05 stewie925 ahh hehe :)
18:26:56 stewie925 jaypipes: it erroed with "Could not open requirements file: [Errno 2] No such file or directory: 'test-requirements' "
18:27:29 jaypipes stewie925: you running this from the root of your nova development tree?
18:27:54 stewie925 jaypipes: yes I am ... its in the ~/nova
18:28:03 stewie925 am pointing to ~/nova I mean
18:28:08 frickler test-requirements.txt ?
18:28:12 jaypipes stewie925: lol, sorry... should be a ".txt" on the end of the test-requirements
18:28:17 stewie925 ahhh hehehe no worries
18:28:28 jaypipes yeah, sorry stewie925.. relying too much on my TAB memory :)
18:28:34 stewie925 yeah I was gonna mention the *.txt file too
18:28:48 stewie925 jaypipes: frickler: thank you :)
18:28:52 jaypipes np
18:30:26 stewie925 ahhh it now installed ddt :)
18:30:35 stewie925 ddt = pesticide? heh
18:35:07 jaypipes stewie925: data-driven testing -- it's testing scenarios library...

Earlier   Later