| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-10-10 | |||
| 13:00:51 | mordred | get_flavor_extra_specs: false | |
| 13:01:10 | mordred | to clouds.yaml (top level) it will turn off the shade behavior (in case that winds up being useful to you) | |
| 13:01:26 | mordred | umbSublime: not at the moment (re REST) - but I actually have a patch up to add that ability | |
| 13:01:57 | mordred | umbSublime: in part of the patch series aimed at laying the plumbing groundwork for integrating shade and sdk with each other | |
| 13:02:05 | umbSublime | awesome, then I guess I'll work-around my issue with shade for now and switch to using the REST call when it's ready. Thanks for the help | |
| 13:02:06 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstacksdk master: Avoid default mutable values in arguments https://review.openstack.org/510870 | |
| 13:02:40 | mordred | umbSublime: sure thing! hopefully in the not-too-distant-future we'll have answers for both side of this :) | |
| 13:03:15 | umbSublime | mordred, maybe I can do a "hack" to get the token from the connection object and just use requests module for my custom api call. Would that be possible ? | |
| 13:05:22 | mordred | umbSublime: I think instead just try using the session property on the Connection | |
| 13:05:40 | mordred | umbSublime: one sec ... | |
| 13:05:59 | umbSublime | ohh right, that's actually what i was refering too. Thanks again | |
| 13:12:30 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Avoid default mutable values in arguments https://review.openstack.org/510876 | |
| 15:10:25 | openstackgerrit | Stephen Finucane proposed openstack/cliff master: Generate demoapp CLI refernece https://review.openstack.org/483134 | |
| 15:10:26 | openstackgerrit | Stephen Finucane proposed openstack/cliff master: doc: Cleanup of demoapp doc https://review.openstack.org/510802 | |
| 15:24:53 | openstackgerrit | Stephen Finucane proposed openstack/cliff master: sphinxext: Warn if namespace or command pattern invalid https://review.openstack.org/510913 | |
| 16:39:29 | efried | mordred cdent (sdague) Should we be adding a block-storage endpoint in devstack, since that be the official service type for cinder? | |
| 16:40:26 | cdent | efried: yes, it would be great if what’s in devstack matched sta | |
| 16:40:46 | cdent | but it’s likely/possible that could break some stuff | |
| 16:40:50 | efried | cdent Okay. I'm going to be poking in the dark a little bit here, but I'll shoot you the review once I've got it. | |
| 16:40:56 | efried | cdent Oh, I wasn't going to remove volumev3 | |
| 16:40:57 | cdent | ✔ | |
| 16:40:58 | efried | yet | |
| 16:41:45 | mordred | efried: yah- definitely adding a block-storage owuld be great | |
| 16:42:06 | mordred | efried: obvoiusly if you do, add the unversioned endpoint as block-storage, not the versoined :) | |
| 16:42:33 | efried | mordred Cool, on it. (Currently trying to figure out whether I can duplicate the service name, or if not, what I should call it) | |
| 16:44:23 | mordred | efried: service name carries no meaning - so it's fine to duplicate it | |
| 16:44:28 | efried | sweet | |
| 16:46:30 | cdent | <pic of service> this is not a service | |
| 16:50:43 | efried | cdent mordred This look right? https://review.openstack.org/510939 | |
| 16:52:13 | cdent | efried: superficially glancing, yes | |
| 16:52:27 | cdent | it’ll be a great shiny day when we can get rid of these rest of those | |
| 16:53:48 | efried | cdent I got here from test failures on https://review.openstack.org/#/c/508345/ which I'm pretty sure are happening because there's no 'block-storage' entry in the catalog. | |
| 16:54:31 | efried | though I thought ksa was supposed to try the aliases, maybe? | |
| 16:54:32 | cdent | isn’t it supposed to fall back to aliases? | |
| 16:54:35 | efried | I'm confusing myself. | |
| 16:54:36 | cdent | jinx | |
| 16:54:39 | cdent | me too | |
| 16:54:57 | efried | and cinder is way messed up | |
| 17:13:29 | mordred | cdent, efried: alias fallback is the last missing thing we haven't yet added to ksa | |
| 17:13:39 | efried | ah | |
| 17:13:50 | mordred | cdent, efried: the patch is here: https://review.openstack.org/#/c/484341/ and it's on my list to finish as soon as zuul v3 rollout is done | |
| 17:14:06 | cdent | k3w! | |
| 18:47:32 | briancurtin | umbSublime: if you just want the token you can call https://developer.openstack.org/sdks/python/openstacksdk/users/connection.html#openstack.connection.Connection.authorize | |
| 18:48:17 | briancurtin | that’s used for more explicitly authorizing (if necessary, it’s not the usual way you auth), and that will return the auth token | |
| 22:10:48 | openstackgerrit | Jackie Truong proposed openstack/python-openstackclient master: Add trusted_image_certificates to server https://review.openstack.org/501926 | |
| 23:20:07 | openstackgerrit | Merged openstack/cliff master: Alias exit to cmd2's quit command to exit interactive shell https://review.openstack.org/491217 | |
| #openstack-sdks - 2017-10-11 | |||
| 04:25:51 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Split request logging into four different loggers https://review.openstack.org/505764 | |
| 06:51:53 | ldl | https://stackoverflow.com/questions/46681591/my-project-and-admin-panels-resource-can-not-be-retrieve-but-identity-resource | |
| 06:52:15 | ldl | Hi, friends, do you have encountered this issue? | |
| 08:14:16 | umbSublime | thanks briancurtin | |
| 08:50:21 | slaweq_ | hello briancurtin | |
| 08:50:41 | slaweq_ | briancurtin: some time ago You told me that You will check if You can release new version of OpenStack SDK | |
| 08:50:50 | slaweq_ | did You check this already? | |
| 08:50:59 | slaweq_ | do You know if it's possible to release it? | |
| 09:47:29 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Convert 'server' commands to use autoprogram https://review.openstack.org/458452 | |
| 13:43:22 | kuzko | hello, I am using shade to try to create my instances, but I cannot seem to boot from a volume only, image seems to be a requirement | |
| 13:44:13 | kuzko | I am getting TypeError: create_server() missing 1 required positional argument: 'image' | |
| 13:51:32 | mordred | kuzko: one sec ... lemme check something | |
| 13:53:00 | mordred | kuzko: OH FOR THE LOVE OF ... wow. yah. that's a bug, I'm sorry. pass image=None and it should work - and I'll fix it in the code | |
| 13:53:31 | kuzko | ok, I've seen that placing image=None works.. but, now my volume isn't bootable ^^ | |
| 13:54:00 | mordred | oh. well, that's less good - can you paste me your whole create_server call? | |
| 13:54:23 | mordred | this is basic feature so I'm *extra* unpleased with it not working for you | |
| 13:54:43 | kuzko | ok, I create a volume beforehand, maybe there is a switch to set it bootable I did not see.. hang on i'm pastebining it | |
| 13:55:13 | kuzko | don't sweat it, there is a workaround so that's not biggie... | |
| 13:56:25 | kuzko | https://pastebin.com/FrySUpSP there you go mordred | |
| 13:57:25 | kuzko | that's the correct one, remove image=None and that's what I had beforehand | |
| 14:01:15 | mordred | kuzko: sweet - thanks. that's SUPPOSED to create a bootable volume (the create_volume(image= ..) | |
| 14:02:21 | mordred | aha | |
| 14:02:41 | mordred | kuzko: there's a way to toggle the bootable flag on a volume but we don't expose it - I'll add that for you too | |
| 14:05:01 | kuzko | well that's nice, I used pip to fech the libray, should I update using pip or git clone? | |
| 14:05:15 | kuzko | *library... | |
| 14:10:01 | mordred | kuzko: I'm overdue for a release - so I'll try to get one cut tomorrow | |
| 14:24:02 | umbSublime | mordred, if you can add that feature for custom api call like we discussed yesterday in the next release (if it's ready that is) it would be awesome !! | |
| 14:25:46 | mordred | umbSublime: yes - I fully intend to get that in to the next release | |
| 14:26:18 | umbSublime | awesome!! | |
| 14:26:21 | mordred | umbSublime: well - next sdk release - not next shade release | |
| 14:26:45 | umbSublime | ohh right you guys were talking about shade, my bad :) | |
| 14:27:15 | mordred | umbSublime: well - hopefully that confusion will not persist for TOO much longer :) | |
| 14:27:21 | umbSublime | hehe | |
| 14:27:25 | mordred | but that certaily won't get done by tomorrow :) | |
| 14:27:41 | umbSublime | no worries, I have my work-around working :) | |
| 14:37:34 | kuzko | is shade supposed to become the SDK at a later date? ^^ | |
| 14:38:34 | mordred | kuzko: the current proposed plan is that we merge the shade code into the sdk tree, then make the shade library a thin backwards-compat shim that wraps and uses the equivilent code in the sdk | |
| 14:38:59 | mordred | kuzko: so it's still totally safe to use shade today - and I do not intend to break it, well, ever | |
| 14:39:25 | kuzko | nice, because I think we might use it for our operations going forward | |
| 14:40:04 | kuzko | simplicity allow for quick newbs onboarding and teaching | |
| 14:46:21 | mordred | ++ | |
| 14:47:38 | mordred | kuzko: the overall intent is to have all three types of access co-exist using the same Connection object - the object-oriented API that sdk currently has - the more procedural business-logic 'easy' api of shade - and fallback/passthrough layer for just making direct REST calls if you need something that doesnt' have a higher-level wrapper | |
| 14:48:42 | mordred | so you can do conn.list_servers() ... conn.compute.server.servers() and conn.compute.get('/servers') depending on your need | |
| 14:49:04 | kuzko | oh right, that would be sweet | |
| 14:49:57 | umbSublime | indeed ^^ | |
| 14:50:04 | mordred | that way for times like today if you're like 'hi, I need to set bootable flag' I can say "crap, well, for now just do conn.block_storage.post('/volumes/{id}/action'.format(volume['id']), {'os-set_bootable': True}) | |
| 14:50:43 | mordred | I've got most of that working locally but not passing tests yet | |
| 14:53:29 | mordred | (which is mostly just due to mocking issues in the tests - not actual issues) | |
| 14:56:37 | mordred | kuzko: so your workaroud is just to create the volume using cloud.create_volume(image=imageMA, name='ATEL-Modele-20170926',size=50) and that did properly wind up with a bootable volume yeah? | |
| 14:59:53 | mordred | kuzko: did you specifically want to do that in two steps? asking because if your goal is 'create a volume from this image and then boot a server from it' - you can totally do that all just in the create_server call | |
| 15:00:53 | kuzko | it's in two step because I want to reuse the volume to upload it at a later date, it's part of an image update script... and no, it's not doing a proper bootable volume... | |
| 15:01:17 | mordred | gotcha. | |
| 15:01:18 | kuzko | *at a later date ... in the script... | |
| 15:01:21 | mordred | yah | |