*** markvoelker has quit IRC | 00:00 | |
*** tosky has quit IRC | 00:01 | |
*** bobh has quit IRC | 00:14 | |
*** bobh has joined #openstack-sdks | 00:23 | |
*** bobh has quit IRC | 00:24 | |
*** bobh has joined #openstack-sdks | 00:25 | |
*** bobh_ has joined #openstack-sdks | 00:28 | |
*** bobh has quit IRC | 00:33 | |
*** bobh_ has quit IRC | 00:54 | |
*** bobh has joined #openstack-sdks | 00:58 | |
*** bobh has quit IRC | 00:59 | |
*** bobh has joined #openstack-sdks | 01:11 | |
*** keekz has joined #openstack-sdks | 01:45 | |
*** whoami-rajat has joined #openstack-sdks | 01:53 | |
*** bobh has quit IRC | 02:01 | |
*** bobh has joined #openstack-sdks | 02:32 | |
*** bobh has quit IRC | 02:37 | |
*** ricolin has joined #openstack-sdks | 02:45 | |
*** bobh has joined #openstack-sdks | 04:47 | |
*** bobh has quit IRC | 04:51 | |
*** e0ne has joined #openstack-sdks | 04:53 | |
*** e0ne has quit IRC | 04:56 | |
*** e0ne has joined #openstack-sdks | 05:21 | |
*** e0ne has quit IRC | 05:25 | |
*** Luzi has joined #openstack-sdks | 05:37 | |
*** masayukig has quit IRC | 06:27 | |
*** kmalloc has quit IRC | 06:28 | |
*** vdrok has quit IRC | 06:28 | |
*** TheJulia has quit IRC | 06:29 | |
*** kmalloc has joined #openstack-sdks | 06:30 | |
*** johnsom has quit IRC | 06:32 | |
*** kmalloc has quit IRC | 06:40 | |
*** ralonsoh has joined #openstack-sdks | 06:43 | |
*** TheJulia has joined #openstack-sdks | 06:43 | |
*** markvoelker has joined #openstack-sdks | 06:43 | |
*** TheJulia has quit IRC | 06:47 | |
*** slaweq has joined #openstack-sdks | 06:49 | |
*** gtema has joined #openstack-sdks | 06:51 | |
*** TheJulia has joined #openstack-sdks | 06:52 | |
*** kmalloc has joined #openstack-sdks | 06:53 | |
*** johnsom has joined #openstack-sdks | 06:55 | |
*** masayukig has joined #openstack-sdks | 06:55 | |
*** vdrok has joined #openstack-sdks | 06:58 | |
*** bobh has joined #openstack-sdks | 07:04 | |
*** e0ne has joined #openstack-sdks | 07:06 | |
*** bobh has quit IRC | 07:09 | |
*** e0ne has quit IRC | 07:09 | |
*** e0ne has joined #openstack-sdks | 07:16 | |
*** tosky has joined #openstack-sdks | 07:23 | |
*** holser_ has joined #openstack-sdks | 07:27 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534 | 07:30 |
---|---|---|
*** bobh has joined #openstack-sdks | 07:37 | |
*** bobh has quit IRC | 07:41 | |
*** gtema has quit IRC | 07:45 | |
*** gtema has joined #openstack-sdks | 07:45 | |
*** e0ne has quit IRC | 07:52 | |
*** e0ne has joined #openstack-sdks | 07:58 | |
*** e0ne has quit IRC | 07:59 | |
*** jpich has joined #openstack-sdks | 08:03 | |
*** johnsom has quit IRC | 08:04 | |
*** johnsom has joined #openstack-sdks | 08:05 | |
*** masayukig has quit IRC | 08:05 | |
*** masayukig has joined #openstack-sdks | 08:05 | |
*** gkadam has joined #openstack-sdks | 08:09 | |
*** gkadam has quit IRC | 08:10 | |
*** e0ne has joined #openstack-sdks | 08:10 | |
*** ttsiouts has joined #openstack-sdks | 08:21 | |
openstackgerrit | LIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands https://review.openstack.org/650062 | 09:21 |
*** dtantsur|afk is now known as dtantsur | 09:35 | |
*** holser_ is now known as holser|luuunch | 09:53 | |
*** ttsiouts has quit IRC | 10:23 | |
*** ttsiouts has joined #openstack-sdks | 10:24 | |
*** ttsiouts has quit IRC | 10:28 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534 | 10:31 |
*** cdent has joined #openstack-sdks | 10:39 | |
*** jpich has quit IRC | 10:58 | |
openstackgerrit | Merged openstack/openstacksdk master: Try to fix the masakari CI job https://review.openstack.org/652638 | 10:59 |
*** jpich has joined #openstack-sdks | 10:59 | |
*** ttsiouts has joined #openstack-sdks | 11:02 | |
*** jpich has quit IRC | 11:12 | |
*** jpich has joined #openstack-sdks | 11:13 | |
*** jpich has quit IRC | 11:14 | |
*** holser|luuunch is now known as holser_ | 11:27 | |
*** jpich has joined #openstack-sdks | 11:30 | |
*** jpich has quit IRC | 11:30 | |
*** gtema has quit IRC | 11:40 | |
*** cdent has quit IRC | 11:42 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add image.stage methods https://review.openstack.org/652981 | 11:42 |
*** gtema has joined #openstack-sdks | 11:43 | |
*** dtantsur is now known as dtantsur|brb | 11:45 | |
*** bobh has joined #openstack-sdks | 11:48 | |
*** bobh has quit IRC | 11:53 | |
*** bobh has joined #openstack-sdks | 12:10 | |
*** bobh has quit IRC | 12:18 | |
*** cdent has joined #openstack-sdks | 12:23 | |
*** Luzi has quit IRC | 12:24 | |
*** jpich has joined #openstack-sdks | 12:25 | |
*** Luzi has joined #openstack-sdks | 12:46 | |
*** gtema has quit IRC | 12:54 | |
*** bobh has joined #openstack-sdks | 13:07 | |
*** dtantsur|brb is now known as dtantsur | 13:24 | |
*** jpich has quit IRC | 13:25 | |
*** jpich has joined #openstack-sdks | 13:26 | |
*** tobiash has quit IRC | 13:31 | |
*** tobiash has joined #openstack-sdks | 13:32 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Return None from get_server_by_id on 404 https://review.openstack.org/652995 | 13:38 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Return None from get_server_by_id on 404 https://review.openstack.org/652995 | 13:41 |
openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682 | 13:45 |
openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682 | 13:45 |
*** bobh has quit IRC | 13:50 | |
*** cdent has quit IRC | 14:01 | |
*** markvoelker has quit IRC | 14:07 | |
*** Luzi has quit IRC | 14:09 | |
*** ttsiouts has quit IRC | 14:23 | |
*** ttsiouts has joined #openstack-sdks | 14:24 | |
*** dtantsur has quit IRC | 14:27 | |
*** ttsiouts has quit IRC | 14:27 | |
*** ttsiouts has joined #openstack-sdks | 14:28 | |
*** gtema has joined #openstack-sdks | 14:29 | |
*** dtantsur has joined #openstack-sdks | 14:31 | |
*** bobh has joined #openstack-sdks | 14:39 | |
*** bobh has quit IRC | 14:41 | |
mordred | gtema: left a review on the image patches - I think they're close | 14:49 |
gtema | yes, thanks. Few issues: | 14:49 |
gtema | auto_disk_config - I think it would be better to drop this attr from image resource and rely on the "properties" interpretation of it | 14:50 |
gtema | drop compute for list - ok, will do | 14:51 |
gtema | switch to wait_for_task - it's this fallback for _IMAGE_ERROR_396 which makes it not easy | 14:52 |
mordred | gtema: auto_disk_config - yeah | 14:52 |
mordred | gtema: or else we should add ALL of the driver-specific properties | 14:52 |
gtema | which we do not want to | 14:53 |
mordred | gtema: yeah. and unfortunately _IMAGE_ERROR_396 is a wonderful thing that rackspace does throw. like - that chunk of logic exists because of issues production nodepool sees without it | 14:53 |
mordred | gtema: totally. there's too many of them - it would make the image objects absurd | 14:54 |
gtema | morderd: so should I then drop all hw*, vmware*? | 14:54 |
mordred | gtema: ugh. we do have a lot of these already don't we | 14:56 |
gtema | yupp | 14:56 |
mordred | gtema: maybe it's better that we just add the rest of them? I wish they were documented somewhere that isn't a wiki page | 14:56 |
mordred | like, you know - in the API docs | 14:56 |
mordred | lemme ask rosmaita | 14:57 |
gtema | ok | 14:57 |
mordred | gtema: https://docs.openstack.org/glance/latest/admin/useful-image-properties.html | 15:02 |
gtema | morder: ack, will go through | 15:02 |
mordred | wow. trait:trait_name is fantastic | 15:03 |
gtema | however, if on Rackspace auto_disk_config returns you "disabled" - we have a problem and need to treat all those as strings only | 15:03 |
gtema | since it should be true or false | 15:04 |
*** cdent has joined #openstack-sdks | 15:05 | |
mordred | gtema: mriedem tells me that auto_disk_config is string in the nova schema, so even the true and false values are actually "true" and "false" | 15:09 |
gtema | ok, that is the problem, that in Image it was casted to bool and therefore I had problem | 15:09 |
mordred | aha! | 15:10 |
mordred | https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py#L233 | 15:10 |
mordred | gtema: this is apparently where we can look to find out actual thing | 15:10 |
gtema | yeah, looks interesting | 15:11 |
gtema | but is also interesting "architecture" vs "hw_architecture" | 15:11 |
gtema | hw_auto_disk_config | 15:11 |
mordred | there's a legacy property map at the bottom | 15:11 |
gtema | oh, thks | 15:11 |
mordred | I wonder how hard it would be to write a gate job (or maybe a functional test that works only on devstack installs) | 15:12 |
gtema | it's huge. Do we really want all of that? | 15:12 |
mordred | that would import that nova file and do a cross-check to make sure the property lists match | 15:12 |
mordred | gtema: I don't know | 15:12 |
mordred | it's an excellent questions | 15:12 |
gtema | we will anyway put everything what we get under properties and even allow to set anything | 15:13 |
gtema | and those traits - we can't define such things as of now | 15:14 |
mordred | yeah ... so - in #openstack-nova I just said : | 15:15 |
mordred | oh - screw it - copy/paste not worth it... basically - since the various base properties can have varied data types, we might really want to do the whole list so we can make sure everything in properties is just a string type and is a user-defined property | 15:16 |
mordred | BUT | 15:16 |
mordred | maybe we can wait a bit and do that later | 15:16 |
mordred | I think what's there now is fine for now until someone gets upset | 15:17 |
gtema | so simply let it stay how it is now? | 15:17 |
gtema | auto_disk_config - will remove type=bool | 15:17 |
mordred | OH GOD. THERE ARE ENUMS | 15:18 |
mordred | gtema: yeah. let's do that for now - I think dealing with the whole thing is ... large ... and requires some real thought | 15:19 |
mordred | like - if we wanted to actually tackle it - I'd want to figure out how to keep thigns liek enum lists in sync | 15:19 |
* gtema has opened a Pandora box | 15:19 | |
mordred | gtema: you know - there's also the schema field you added support for | 15:20 |
* gtema decides to close it while it is still possible | 15:20 | |
gtema | yes, I know | 15:20 |
mordred | gtema: I wonder if it would be easier/better to just write something that grabs the schema and figures out which properties are base and which are user based on that schema | 15:20 |
gtema | you want us to make use of it? | 15:20 |
mordred | maybe? I don't know how useful it is | 15:20 |
gtema | well, I am not sure it is usefull as of now. Maybe later | 15:21 |
gtema | this would not be very efficient by default, since nearly always we would need to invoke it and/or cache schema data | 15:22 |
mordred | ++ | 15:22 |
mordred | yeah - we'd definitely have to cache schema data I think | 15:22 |
mordred | but for single-use processes like osc it woudl still be inefficient | 15:22 |
gtema | definitely | 15:22 |
mordred | so it's PROBABLY better to think about a test job that validates against the nova source code | 15:22 |
mordred | but HOLY CRAP that'll be a lot of work too due to the use of custom enum classes | 15:23 |
* mordred runs away to cry | 15:23 | |
gtema | agree | 15:23 |
*** holser_ has quit IRC | 15:25 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534 | 15:29 |
*** e0ne has quit IRC | 15:31 | |
openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add image.stage methods https://review.openstack.org/652981 | 15:43 |
*** gtema has quit IRC | 15:49 | |
*** ttsiouts has quit IRC | 15:59 | |
*** ttsiouts has joined #openstack-sdks | 15:59 | |
*** ttsiouts has quit IRC | 16:04 | |
*** jpich has quit IRC | 16:04 | |
*** dtruong has quit IRC | 16:40 | |
*** dtruong has joined #openstack-sdks | 16:41 | |
*** yolanda_ has quit IRC | 16:57 | |
*** ricolin has quit IRC | 17:25 | |
*** e0ne has joined #openstack-sdks | 17:32 | |
*** ralonsoh has quit IRC | 17:34 | |
*** dtantsur is now known as dtantsur|afk | 17:36 | |
*** e0ne has quit IRC | 17:49 | |
*** efried has joined #openstack-sdks | 18:22 | |
*** Sundar has joined #openstack-sdks | 18:22 | |
efried | mordred: This openstacksdk business is ready for primetime, yah? So like, new projects like cyborg that don't yet have a legacy commitment to python-*client and/or oslo.config for ksa can and should bootstrap their talking-to-other-services business with a clouds.yaml and production Connection constructor-ness? | 18:23 |
mordred | efried: yes | 18:30 |
mordred | efried: it would be very welcome for them to add support for their service into sdk directly | 18:30 |
efried | mordred: I was talking about cyborg wanting to communicate *at* other services, like placement and glance. | 18:30 |
mordred | ah. yes. that too | 18:31 |
Shrews | mordred: very very small nit on https://review.openstack.org/652995 but happy to merge as is too if you'd rather say "meh" | 18:31 |
efried | Sundar: ^ Given that you've just succeeded in connecting using a ksa adapter, it should be fairly trivial to translate your oslo.config-based values into a clouds.yaml. | 18:31 |
mordred | so - both things are true | 18:31 |
mordred | efried: yah. although if they _want_ to use the oslo.config stuff, we've got that helper method we put in for nova | 18:32 |
efried | mordred: Well, yeah, it's still in flight. | 18:32 |
efried | mordred: And I'm not sure there's a reason to use oslo.config stuff, is there? | 18:32 |
mordred | efried: god, have we not landed that yet? I should really finish writing tests | 18:32 |
efried | mordred: I did leave a poke there a week or two ago :) | 18:32 |
mordred | efried: not really - other than consistency with other services for operators | 18:32 |
efried | I mean, how much gerrit mail could you possibly get? | 18:33 |
mordred | efried: you think I get email from gerrit? :) | 18:33 |
efried | :P | 18:33 |
efried | I've got a guy coming up to speed and working on the ironicclient swapout for nova | 18:33 |
efried | We're getting close-ish to a point where we'll want that dep merged & released. | 18:33 |
efried | dustinc: ^ | 18:34 |
mordred | efried: cool. we'll get it merged for you then | 18:34 |
efried | mordred: Let me know if you need anything from me on that. | 18:35 |
mordred | don't think so - I think the nova depends-on patches provide good integration testing that it works - we just need to add some sdk-side testing and it should be good to go | 18:35 |
mordred | Shrews: oh piddle. can you tell I copy-pastad that? | 18:35 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Return None from get_server_by_id on 404 https://review.openstack.org/652995 | 18:36 |
mordred | Shrews: ^^ fixed | 18:36 |
mordred | thanks | 18:36 |
dtroyer | mordred: is there a known problem with Connection.add_service() currently? I am attempting some manner of support of StarlingX APIs without adding them directly (just starting with direct REST for now) and I get different failures with latest release and master… | 18:36 |
dtroyer | I need to work up a small test case, you don't want to try what I have at the moment… | 18:37 |
openstackgerrit | Jeremy Houser proposed openstack/python-openstackclient master: Exploratory Fix for Common in Compute v2 Common https://review.openstack.org/653077 | 18:37 |
dustinc | efried, mordred: 👍 | 18:37 |
*** irclogbot_2 has quit IRC | 18:39 | |
*** irclogbot_1 has joined #openstack-sdks | 18:41 | |
Shrews | mordred: +3'd | 18:42 |
dtroyer | mordred: nvm, nothing like pouring out your problems publicly to help in figuring it out… that said I am exploring the best way to approach making a derivative SDK for STX. | 18:55 |
mordred | dtroyer: cool - you may want to chat with gtema next time he's around - he's been working on that same thing for his cloud - they have some services that are non-openstack, so supporting that flow is important to him | 18:57 |
dtroyer | ok, will do. I saw his recent improvements in add_service() that explained the different behaviours. | 18:58 |
mordred | cool. also - I'm not opposed in principle to starlingx support in sdk ... but I also have spent exactly 0 seconds thinking about it, so maybe it's a bad idea for a reason I haven't thought of yet | 19:00 |
mordred | but we're all part of the same happy family, so it's not, you know, inconceivable | 19:00 |
mordred | but I'm also not going to say it _should_ be done that way if the other way seems better | 19:00 |
dtroyer | mordred: I think there is a service-types conversation to come first, that'll help decide. | 19:01 |
* mordred waves arms wildly in the air then runs into a hole to hide | 19:01 | |
mordred | cool | 19:01 |
*** bobh has joined #openstack-sdks | 19:25 | |
*** bobh has quit IRC | 19:28 | |
*** bobh has joined #openstack-sdks | 19:37 | |
*** holser_ has joined #openstack-sdks | 20:13 | |
*** ttsiouts has joined #openstack-sdks | 20:44 | |
*** cdent has quit IRC | 20:50 | |
*** bobh has quit IRC | 20:56 | |
openstackgerrit | Merged openstack/openstacksdk master: Return None from get_server_by_id on 404 https://review.openstack.org/652995 | 21:43 |
*** holser_ has quit IRC | 21:51 | |
Sundar | Are the connections in openstack-sdk low cost enough that we can create one at each call site? | 21:57 |
mordred | Sundar: well.... | 22:01 |
mordred | Sundar: they're _fairly_ low cost - but there's also a bunch of plumbing to deal with caching some various things, so I would avoid it in long-running services if you can | 22:01 |
mordred | Sundar: the Connection object itself should be thread safe - so sharing one should work properly | 22:02 |
mordred | Sundar: however - if, for whatever reason, you need to create one per call site - if you can share the Session and/or Auth objects, that will take care of most of the caching you'll care about | 22:03 |
mordred | (since service discovery gets cached on the Session) | 22:03 |
*** ttsiouts has quit IRC | 22:10 | |
*** ttsiouts has joined #openstack-sdks | 22:11 | |
*** ttsiouts has quit IRC | 22:12 | |
*** holser_ has joined #openstack-sdks | 22:36 | |
Sundar | mordred: Thanks for the detailed explanation | 22:40 |
openstackgerrit | Matt Riedemann proposed openstack/python-openstackclient master: Fix docs bug link to go to storyboard rather than launchpad https://review.openstack.org/653162 | 22:41 |
Sundar | mordred: Also, in my devstack setup, I see specific roles like devstack_admin in clouds.yaml. Can we expect more standard roles like 'admin' irl? | 22:42 |
Sundar | mordred: Also, is there a good way to avoid hardcoding the role like ``connection.Connection(cloud='devstack-admin')``? | 22:42 |
*** holser_ has quit IRC | 22:53 | |
*** whoami-rajat has quit IRC | 23:02 | |
dtroyer | Sundar: that isn't an actual role, it is a cloud user configuration that points at the devstack cloud. The configurations you will find IRL could be anything, it is just a name. with a cloud configuration behind it, that needs to be set up for your target. | 23:16 |
*** tosky has quit IRC | 23:19 | |
*** mriedem has joined #openstack-sdks | 23:20 | |
mriedem | need some design help on fixing https://storyboard.openstack.org/#!/story/2005468 | 23:20 |
mriedem | trying to fix openstack server create --config-drive in a backward compatible way | 23:20 |
mriedem | it takes a string today which is dumb b/c it's a boolean, | 23:20 |
mriedem | it should just be that specifying --config-drive means yeah, add a config drive, and omitting the option means no config drive | 23:21 |
mriedem | but i'm not sure how to deal with that as a string if it's not provided | 23:21 |
dtroyer | mriedem: it is likely possible (I personally dislike nargs but this is where it is useful) I can take a close look tomorrow… | 23:21 |
mriedem | like i guess you can specify --config-drive None? | 23:21 |
dtroyer | we have done this in one or two other places, it is possible | 23:22 |
dtroyer | and if I did that existing implementation I owe you a $COLD_BEVERAGE in DEN :) | 23:22 |
mriedem | ok i'll post what i have and we can sort it out tomorrow | 23:23 |
mriedem | dtroyer: not your fault if so b/c it's copying the busted novaclient code | 23:23 |
mriedem | which also says it can be a non-boolean string | 23:23 |
mriedem | from 2012 | 23:23 |
Sundar | dtoyer: Thanks. So, how do we get the right 'cloud name' or user configuration name for ``connection.Connection(cloud='devstack-admin')``? | 23:43 |
Sundar | dtroyer: ^ | 23:43 |
dtroyer | I don't know the exact context you are in, but that depends totally on the cloud you are calling | 23:45 |
Sundar | dtroyer: let me put it this way. When one project (say Cyborg) calls another (say Glance), is it appropriate to embed names like 'devstack-admin' in the caller code? One would tjhink not. So, where do I get the right name for the caller? | 23:47 |
dtroyer | Sundar: from the point of view of the application, it comes from the app. Most OpenStack services have a configuration value for service accounts like that, along with their DB config, etc. In that case, you probably do not want to rely on clouds.yaml for that information but create your connection from the individual pieces in your config | 23:49 |
dtroyer | where 'that case' is from a long-running service | 23:49 |
openstackgerrit | Matt Riedemann proposed openstack/python-openstackclient master: WIP: Fix openstack server create --config-drive value https://review.openstack.org/653176 | 23:50 |
mriedem | Sundar: e.g. nova's config for talking to glance https://docs.openstack.org/nova/latest/configuration/config.html#glance | 23:51 |
mriedem | uses keystoneauth1 config options for building the ksa session adapter | 23:51 |
mriedem | efried is well versed in all of that code | 23:51 |
Sundar | mriedem: I am investigating the 'new' approach, newer than ksa adapter, which is to use openstack sdk with clouds.yaml. | 23:53 |
Sundar | mriedem: efried is advocating that new approach | 23:53 |
Sundar | mriedem: Along the lines of https://review.openstack.org/#/c/643664/ | 23:56 |
Sundar | dtroyer: Got it. This should go into cyborg.conf | 23:56 |
mriedem | Sundar: ok i don't know why we're doing that, | 23:58 |
mriedem | we don't have a blueprint or anything for that in nova | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!