| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-07-20 | |||
| 15:53:13 | corvus | mordred: perfect, thanks! | |
| 15:53:49 | corvus | (i wanted to set the max poolsize for the connection pool, since i'm multithreaded; this will allow more connection reuse) | |
| 15:53:50 | mordred | corvus: out of curiosity - are you wanting to use it to make a rest call against one of the openstack endpoints? | |
| 15:54:02 | mordred | ah! | |
| 15:54:04 | corvus | sess.mount('https://', adapter) | |
| 15:54:04 | corvus | adapter = requests.adapters.HTTPAdapter(pool_maxsize=100) | |
| 15:54:04 | corvus | sess = cloud.config.get_session() | |
| 15:54:23 | mordred | neat. that should totally work | |
| 15:54:28 | corvus | it does! | |
| 15:54:42 | mordred | corvus: do you think that's an option we should expose in openstack.config ? | |
| 15:54:45 | corvus | no more warnings about the pool being full | |
| 15:54:48 | mordred | (not today - just in general) | |
| 15:56:00 | corvus | mordred: perhaps -- might be a nice friendly thing -- there's some subtlety about how connection pooling works for multiple hosts that i skimmed over because i'm only talking to one host. would probably need to read up a bit more on that before implementing something. | |
| 15:56:19 | mordred | corvus: ++ | |
| 15:57:07 | corvus | aiui from my quick reading, there's only one pool by default, so if you access a second host, it drops the pool for the first host. but in addition to pool_maxsize (max connections in pool), there's also pool_connections -- number of pools | |
| 15:57:36 | corvus | probably the thing to do would be to expose both of those. but like i said, i skipped some details so i'm not 100% sure | |
| 15:58:14 | mordred | yeah - especially since for openstack it's extrememly common for each service to have its own host - and for sdk to use the same session for all of them | |
| 15:58:31 | corvus | fwiw, changing the pool size did not greatly impact the total amount of time spent uploading objects to swift. but it still makes me feel better. :) | |
| 15:59:19 | mordred | corvus: we could maybe get fancy and look in the catalog, count the number of distinct hosts contained in it and use that to set a default for pool_connections | |
| 15:59:41 | corvus | mordred: may want more for the multi-cloud case... | |
| 15:59:53 | corvus | oh, this is a per-cloud session anyway | |
| 15:59:57 | mordred | corvus: multi-cloud case would ... yeah | |
| 15:59:58 | corvus | so yeah, your idea could work | |
| 16:00:10 | corvus | unless redirects are involved? | |
| 16:00:19 | corvus | (heh, just double it! :) | |
| 16:00:34 | mordred | well, if they are involved, we can still expose the setting :) | |
| 17:56:32 | PagliaccisCloud | hey guys i'm trying to fix a typo in the help menu on python-openstackclient. to create a bug, do i make a new story on storyboard? | |
| 18:25:25 | dtroyer | PagliaccisCloud: yes | |
| 18:33:49 | PagliaccisCloud | cool, thanks dtroyer. sending to gerrit now | |
| 18:37:46 | openstackgerrit | Treva Nichole Williams proposed openstack/python-openstackclient master: Fix typo in "floating ip set" properties https://review.openstack.org/584484 | |
| 21:23:24 | openstackgerrit | Toure Dunnon proposed openstack-infra/shade master: python-shade expose MTU setting. https://review.openstack.org/578861 | |
| #openstack-sdks - 2018-07-21 | |||
| 02:06:33 | corvus | for dealing with connection info in yaml... | |
| 02:06:33 | corvus | mordred, Shrews: for the zuul swift thing, i'd like to define a secret in zuul where the data is exactly the contents of a cloud from clouds.yaml. is it possible to hand that dictionary directly to something in openstacksdk or os-client-config and get a connection object? it seems like i ought to be able to avoid writing out a clouds.yaml for this case, and i don't want to invent another format | |
| 02:08:14 | corvus | (if not, i'll probably just write it to a secure tempfile and set OS_CLIENT_CONFIG) | |
| 02:29:07 | openstackgerrit | James E. Blair proposed openstack/openstacksdk master: Docs: Include CloudRegion class https://review.openstack.org/584552 | |
| 02:29:21 | openstackgerrit | James E. Blair proposed openstack/openstacksdk master: Docs: Fix references in connection page https://review.openstack.org/584553 | |
| 02:29:34 | openstackgerrit | James E. Blair proposed openstack/openstacksdk master: Docs: Remove duplicate content in connection page https://review.openstack.org/584554 | |
| 13:15:48 | mordred | corvus: yes - you can use it to create a CloudRegion object | |
| 13:20:20 | mordred | corvus: or, rather, you can do it directly with Connection ... openstack.connection.Connection(**config_dict) | |
| 13:23:52 | openstackgerrit | Merged openstack/openstacksdk master: Docs: Include CloudRegion class https://review.openstack.org/584552 | |
| 13:23:53 | openstackgerrit | Merged openstack/openstacksdk master: Docs: Remove duplicate content in connection page https://review.openstack.org/584554 | |
| #openstack-sdks - 2018-07-23 | |||
| 09:11:22 | openstackgerrit | Josephine Seifert proposed openstack/python-openstackclient master: [WIP] osc-included image signing (using openstacksdk) https://review.openstack.org/580086 | |
| 13:41:32 | openstackgerrit | Merged openstack/os-service-types master: Add release note link in README https://review.openstack.org/578439 | |
| 13:50:35 | Luzi | hi osc and sdk: looking through the latest patches I have seen, that since Jul 19 every time zuul checks "osc-functional-devstack" it fails | |
| 13:50:49 | Luzi | with the same failure in every patch | |
| 13:52:14 | Luzi | any idea, why this is happening? it is the "test_volume_snapshot" that fails with "Invalid input for field/attribute size. Value: None. None is not of type 'integer', 'string' (HTTP 400)" | |
| 13:59:47 | Luzi | this is the earliest patch, that has this error: https://review.openstack.org/#/c/582334/ | |
| 14:04:17 | Luzi | mordred, dtroyer this might be interesting for you - this might be a bigger bug somewhere | |
| 14:08:31 | pasm | hi guys, I found and fixed a bug in osc-lib (https://github.com/psmiraglia/osc-lib) | |
| 14:08:51 | pasm | how to submit a PR? | |
| 14:09:02 | dtroyer | Luzi: we have had sporadic problems with volume tests over the years, this feels like a real bug somewhere. It is possible that cinderclient has a change in it that we need to account for, or isn't backward-compatible. I'll make some time to look at it after lunch… | |
| 14:09:15 | pasm | (hope this is the right place to ask) | |
| 14:09:33 | Luzi | dtroyer: thank you :) | |
| 14:09:36 | dtroyer | pasm: it is! | |
| 14:09:40 | pasm | :-) | |
| 14:09:47 | dtroyer | and that looks oddly familiar... | |
| 14:10:50 | dtroyer | pasm: I think it was laready fined here: https://review.openstack.org/#/c/581886/ | |
| 14:11:47 | pasm | dtroyer: it is incopmlete | |
| 14:11:52 | dtroyer | and that was released last week as 1.11.0 | |
| 14:11:55 | dtroyer | ah, ok | |
| 14:11:59 | dtroyer | grrr... | |
| 14:12:03 | pasm | dtroyer: there is another occurrence of issubclass() | |
| 14:12:23 | dtroyer | so are you set up to contribute to OpenStack via Gerrit (all the accounts, etc?) | |
| 14:12:50 | pasm | no, totally new | |
| 14:13:59 | pasm | can you link me a guide about your review workflow? | |
| 14:14:27 | dtroyer | https://docs.openstack.org/contributors/common/setup-gerrit.html is the specifics of Gerrit | |
| 14:14:41 | dtroyer | much of that doc should be useful | |
| 14:16:02 | dtroyer | pasm: if you''d like to create a bug you can do it in https://storyboard.openstack.org/#!/project/974 (it would be osc-lib's first in storyboard :) | |
| 14:18:10 | pasm | I'll try with Gerrit | |
| 14:19:27 | pasm | dtroyer: If I understood well, by setting up git-review I only have to push my change. Is it correct? | |
| 14:20:55 | dtroyer | pasm: the command would be 'git review' rather than push but that's basically what it does, plus some other Gerrit details | |
| 14:21:29 | dtroyer | the first time you run git review in a repo it sets itself up | |
| 14:21:38 | pasm | dtroyer: ok, I'll try... | |
| 14:32:00 | openstackgerrit | Ryan Brady proposed openstack-infra/shade master: Add support for static routes https://review.openstack.org/580661 | |
| 14:32:10 | pasm | dtroyer: I should be ready to run "git review", could you confirm me some steps? | |
| 14:32:21 | dtroyer | pasm: ok | |
| 14:32:56 | pasm | dtroyer: 1) clone from https://git.openstack.org/openstack/osc-lib | |
| 14:33:44 | pasm | dtroyer: 2) do changes (withot commit) | |
| 14:34:03 | pasm | dtroyer: 3) git review -s | |
| 14:34:12 | pasm | dtroyer: 4) git commit | |
| 14:34:20 | pasm | dtroyer: is it correct? | |
| 14:34:55 | dtroyer | I'd swap 2 and 3 to make sure you have everything happy before you have uncommitted changes lying about but it works either way | |
| 14:36:03 | dtroyer | and then git review to submit to gerrit | |
| 14:37:09 | pasm | dtroyer: after "git commit"? | |
| 14:37:11 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Add command to show all service versions https://review.openstack.org/584905 | |
| 14:37:20 | dtroyer | pasm: yes | |
| 14:37:21 | mordred | dtroyer: ^^ there's that command I was mentioning on friday | |
| 14:38:42 | dtroyer | mordred: nice! | |
| 14:42:38 | openstackgerrit | Paolo Smiraglia proposed openstack/osc-lib master: Change issubclass() to isinstance() https://review.openstack.org/584920 | |
| 14:45:16 | pasm | dtroyer: more easy than expected... thanks again! | |
| 14:45:35 | dtroyer | pasm: \o/ thank you! | |
| 14:46:35 | pasm | dtroyer: and now? someone will took in charge my review request? | |
| 14:47:57 | dtroyer | pasm: yes. we are in a freeze on osc-lib so it may be a bit before it merges, unless we decide that a py37 issue is critical :) | |
| 14:49:05 | pasm | dtroyer: ok, I asked just for personal knowledge | |
| 14:50:04 | mordred | lgtm | |
| 14:50:46 | pasm | dtroyer: back to work now... ciao! | |
| 15:15:57 | mordred | dtroyer: I'm looking at the volume snapshot ting ... | |
| 15:17:18 | mordred | ugh. it seems like size is a required value now, even though for "create volume from snapshot" the volume size will be the size of the snapshot | |
| 15:20:56 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Pass volume snapshot size to volume create https://review.openstack.org/584932 | |
| 15:21:06 | mordred | dtroyer: I *think* that will take care of it | |