| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-20 | |||
| 18:09:21 | edleafe | dansmith: you could inspect the returned a-c, but that's always too complicated to write in gabbi | |
| 18:09:46 | dansmith | edleafe: yeah, I'm doing that in my real test and getting back [], but I dunno how to do much with the gabbit | |
| 18:09:47 | cdent | edleafe: untrue, either make the test verbose: True or make it fail intentionally in the response_json_paths | |
| 18:09:54 | openstackgerrit | Merged openstack/nova master: Remove old flavor_destroy db api method https://review.openstack.org/539729 | |
| 18:10:00 | dansmith | but it's checking the length of allocation_requests | |
| 18:10:04 | cdent | and the error message will show the full response | |
| 18:10:23 | openstackgerrit | Merged openstack/nova master: Remove old flavor_get_by_flavor_id db api method https://review.openstack.org/539733 | |
| 18:10:35 | openstackgerrit | Merged openstack/nova master: Remove old flavor_get_by_name db api method https://review.openstack.org/544379 | |
| 18:10:36 | dansmith | cdent: ah, duh, trying | |
| 18:10:47 | openstackgerrit | Merged openstack/nova master: Remove old flavor_get db api method https://review.openstack.org/544621 | |
| 18:10:58 | openstackgerrit | Merged openstack/nova master: Remove old flavor_get_all db api method https://review.openstack.org/544688 | |
| 18:11:10 | openstackgerrit | Merged openstack/nova master: Remove old flavor_create db api method https://review.openstack.org/544703 | |
| 18:11:35 | openstackgerrit | Merged openstack/python-novaclient master: Fix local test fails with pypy https://review.openstack.org/553426 | |
| 18:11:45 | dansmith | cdent: I tried just setting the asserted length to something wrong, but it doesn't show me anything else other than 17 != 1 | |
| 18:12:03 | cdent | dansmith: yeah, I meant something more breaking that that: | |
| 18:12:19 | cdent | response_json_paths:\n $: foo | |
| 18:12:39 | cdent | will try to compre the whole json object to foo | |
| 18:14:11 | vivsoni__ | In case of 'nova create' - NO cinder API is called | |
| 18:14:18 | vivsoni__ | In case of 'nova delete' - NO cinder API is called | |
| 18:15:10 | vivsoni__ | Hi Team, please correct if my understanding is wrong | |
| 18:15:25 | mriedem | if you're booting from volume then of course cinder is called | |
| 18:15:44 | mriedem | if you've attached volumes before you delete the instance, of course cinder is called | |
| 18:15:46 | vivsoni__ | mriedem: ok | |
| 18:16:36 | edleafe | cdent: /me learns new gabbi trick | |
| 18:16:51 | dansmith | cdent: edleafe: yeah, well, that clearly shows it's working as expected | |
| 18:16:56 | dansmith | so I be stumped | |
| 18:17:04 | dansmith | if I don't pass member_of, I get back the things I expect | |
| 18:17:42 | vivsoni__ | mriedem: so if my instance is attached to volume, then if i shutoff the nova instance and restart the instance... then cinder api of detach and attach is called is it ? | |
| 18:17:53 | dansmith | interestingly, the request logging from placement is urlencoded from inside my functional test, but not when the gabbit runs | |
| 18:18:10 | dansmith | so I wonder if it's getting more than the uuid | |
| 18:18:12 | mriedem | vivsoni__: no, if you're just stopping and starting the instance, nova doesn't detach the volume | |
| 18:19:00 | edleafe | dansmith: are you sending a single uuid string, or a 1-element list? | |
| 18:19:30 | dansmith | edleafe: I've tried both, initially just one, then tried in:$uuid | |
| 18:20:11 | vivsoni__ | mriedem: ok... i wanted to understand more on 'nova create/delete/live-migration' w.r.t cinder api call.. do you have some reference link, if yes, please share | |
| 18:21:27 | mriedem | vivsoni__: not really | |
| 18:21:44 | mriedem | vivsoni__: hemna has some diagrams of those flows though i think | |
| 18:21:52 | mriedem | not sure if they are published | |
| 18:22:26 | vivsoni__ | ok | |
| 18:22:34 | dansmith | edleafe: yeah I know | |
| 18:22:54 | dansmith | edleafe: I printed member_of from inside the normalize function and it is properly [$uuid] | |
| 18:23:19 | cdent | dansmith, edleafe: any chance that this is the result of some base database condition that is different between the dan tests and the ed tests? | |
| 18:23:31 | dansmith | cdent: such as what? | |
| 18:24:09 | dansmith | cdent: I can get /resource_providers/$rp/aggregates and I get back $uuid | |
| 18:24:23 | dansmith | so I feel like placement is working and storing the association I made | |
| 18:25:07 | dansmith | ohhhhhhh | |
| 18:25:22 | dansmith | I may be completely stupid | |
| 18:25:23 | dansmith | maybe | |
| 18:27:34 | dansmith | edleafe: I deserve whatever was coming before you bit your tongue for this one | |
| 18:29:14 | dansmith | good news, member_of seems to work.. EOM | |
| 18:29:24 | efried | Well now you gotta tell us what you were doing wrong. | |
| 18:29:31 | efried | Not just so we can make fun of you - although that too. | |
| 18:29:40 | efried | But mainly so we can avoid similar pratfalls ourselves. | |
| 18:29:53 | efried | And therefore avoid being made fun of in turn. | |
| 18:30:18 | edleafe | uh, yeah - for *education* | |
| 18:31:00 | dansmith | I had a method that would create and add hosts to aggregates, both in nova and placement | |
| 18:31:11 | dansmith | and the nova call I was making is additive, where the placement one is declarative | |
| 18:31:38 | dansmith | so I was putting multiple hosts in the nova aggregates, but only in the latest placement one | |
| 18:31:45 | dansmith | and the last placement one I created was "no-hosts" | |
| 18:31:48 | dansmith | so, you know.. | |
| 18:32:09 | dansmith | and the order in which I was printing debug stuff was consistent, but.. not the full story | |
| 18:33:54 | edleafe | wish I could say I | |
| 18:34:02 | mriedem | https://www.youtube.com/watch?v=AMQ8E3mTgY0 | |
| 18:34:03 | edleafe | I've never done something similar | |
| 18:34:18 | dansmith | mriedem: yeah I know, but thanks for putting into visual form | |
| 18:34:29 | mriedem | ha | |
| 18:35:17 | mriedem | ask smcginnis or jungleboyj, i was bashing my head for about 2 hours one morning over a unit test i was writing and couldn't figure out why it didn't work | |
| 18:35:33 | mriedem | can't remember what it was, but it was really dumb | |
| 18:36:17 | edleafe | cdent: ok, given the output using your trick: http://paste.openstack.org/show/706443/, how would I reference the resource provider uuid within the response_json_paths in order to compare it to the cn1uuid? | |
| 18:38:25 | cdent | edleafe: reading | |
| 18:38:29 | smcginnis | mriedem: What what? | |
| 18:40:32 | mriedem | smcginnis: last time i was at claddagh and we were upstairs, i was trying to get a gd unit test working most of the morning | |
| 18:40:41 | mriedem | maybe i was the only one that heard my swearing | |
| 18:41:08 | cdent | edleafe: you can to check that the cn1uuid is in there, which in there? | |
| 18:41:13 | smcginnis | Oh yeah, I remember now. :) | |
| 18:41:14 | cdent | s/can/want/? | |
| 18:42:33 | edleafe | cdent: dunno, just thought that checking that it's in there wasn't definitive enough. Getting allocation_requests[0].keys()[0] or something like that | |
| 18:42:35 | melwitt | I'm able to repro the EndpointNotFound problem in an old devstack I have, and what I get from ksa_adapter.get_endpoint() in both the non-list and interface list cases is: "*** EmptyCatalog: The service catalog is empty." | |
| 18:43:08 | melwitt | even though I have glance in 'openstack endpoint list' | |
| 18:44:30 | cdent | edleafe: if you know the uuid in advance, it's generally easier to check for some key down its path. you can do environ and response expansions within the left hand side of a json path thing | |
| 18:45:04 | efried | melwitt: Can you show me your openstack endpoint list for glance? | |
| 18:45:11 | efried | or 'show' would be better | |
| 18:46:43 | efried | melwitt: The ksa_adapter is going to be narrowed down based on (possibly defaulted) config options. So as Matt was saying earlier, if for example your catalog has the admin endpoint, but your conf has (possibly by defaulting) internal and public, you'll get that EmptyCatalog/EndpointNotFound when you ask the adapter for an endpoint. | |
| 18:47:08 | cdent | edleafe, sorry, I'm doing the usual too many things at once and then writing two thoughts down in one sentence | |
| 18:47:25 | melwitt | efried: okay, sec | |
| 18:47:34 | cdent | edleafe: first example after the heading: https://gabbi.readthedocs.io/en/latest/jsonpath.html#substitution | |
| 18:47:50 | cdent | (well, only example) | |
| 18:49:20 | edleafe | cdent: So given the example paste, what would I put in the left side? IOW, what is "nested.structure"? | |
| 18:50:02 | melwitt | efried: http://paste.openstack.org/show/706465/ | |
| 18:50:38 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | |
| 18:50:42 | efried | melwitt: And the glance section of the conf? | |
| 18:51:04 | melwitt | no [glance] section, all default | |
| 18:51:20 | efried | melwitt: ...and the version document from the endpoint? (curl http://127.0.0.1/image) | |
| 18:51:57 | melwitt | http://paste.openstack.org/show/706468/ | |
| 18:52:23 | cdent | edleafe: $.allocation_requests[0].allocations.["$ENVIRON['CN1']"].resources.DISK_GB: 100 | |
| 18:52:29 | cdent | I think jay did this somewhere, will find example | |
| 18:53:26 | cdent | edleafe: $.allocation_requests..allocations["$ENVIRON['SS_UUID']"].resources[DISK_GB]: [100, 100] in the allocation-candidates.yaml | |
| 18:53:37 | efried | melwitt: Yup, that ought to work. | |
| 18:54:50 | efried | melwitt: You said you were using and old stack? What version of ksa, and what commit of nova? | |
| 18:55:33 | melwitt | stepping through the code in pdb, when I get to keystoneauth1/access/service_catalog.py(362)endpoint_data_for() the self._catalog is an empty list [], even for the 'public' interface | |
| 18:55:46 | melwitt | that's where it raises exceptions.EmptyCatalog('The service catalog is empty.') | |