| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-11-02 | |||
| 18:38:59 | smcginnis | dtroyer: Yeah, that makes sense if we have a whole batch of major-version inducing changes. | |
| 18:39:30 | smcginnis | If you remember, let me know when a good point would be for that and I can rebase those over to the feature branch. | |
| 18:47:07 | dtroyer | will do. I keep threatening to do it tomorrow :) | |
| 18:48:07 | openstackgerrit | Merged openstack/python-openstackclient master: Make use of keystoneauth service-type filtering for versions https://review.openstack.org/585030 | |
| 19:32:44 | Shrews | mordred: yeah, well i didn't know if that was some sort of python trickery i wasn't aware of, or... your reason :-P | |
| 19:33:06 | mordred | Shrews: definitely my reason | |
| 19:41:25 | openstackgerrit | Merged openstack/python-openstackclient master: Add --key-name and --key-unset option for server rebuild API. https://review.openstack.org/576422 | |
| 22:28:33 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| 22:47:45 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| 23:17:35 | openstackgerrit | Merged openstack/osc-lib master: add lib-forward-testing-python3 test job https://review.openstack.org/594285 | |
| #openstack-sdks - 2018-11-03 | |||
| 14:16:30 | openstackgerrit | Merged openstack/openstacksdk master: Remove mocking workaround from adapter https://review.openstack.org/614831 | |
| #openstack-sdks - 2018-11-04 | |||
| 10:48:40 | openstackgerrit | Nguyen Hai Truong proposed openstack/openstacksdk master: [Trivial Fix] Correct spelling error of "bandwidth" https://review.openstack.org/615421 | |
| 14:29:45 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: [Trivial Fix] Correct spelling error of "bandwidth" https://review.openstack.org/615421 | |
| 14:38:29 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Shift swift segment async code out of adapter https://review.openstack.org/614832 | |
| 14:38:30 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove unused Task classes https://review.openstack.org/614833 | |
| 14:38:30 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Collect request stats with statsd https://review.openstack.org/614834 | |
| 14:38:31 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | |
| #openstack-sdks - 2018-11-05 | |||
| 06:54:24 | openstackgerrit | Andriy Shevchenko proposed openstack/python-openstackclient master: fix tox python3 overrides https://review.openstack.org/607453 | |
| 09:10:41 | openstackgerrit | Ian Wienand proposed openstack/openstacksdk master: Document "insecure" flag https://review.openstack.org/615512 | |
| 10:04:21 | openstackgerrit | Ian Wienand proposed openstack/openstacksdk master: Document "insecure" flag https://review.openstack.org/615512 | |
| 12:14:08 | openstackgerrit | yatin proposed openstack/openstacksdk master: Test python2 with py27 https://review.openstack.org/615543 | |
| 14:24:56 | Miouge | kmalloc: A few weeks back we talked about discovering the version of OpenStack etc… Apparently there is another method via dmidecode: https://fghaas.github.io/osdn2018/#/2/1 | |
| 14:25:59 | Miouge | It’s not 100% either, but it’s another datapoint I guess | |
| 19:47:35 | dtruong | mordred: wait_for_status was removed from Proxy class. This seems to be breaking senlin which was calling this function: http://logs.openstack.org/04/615604/2/check/senlin-dsvm-tempest-py27-integration/ff0eec3/logs/screen-sl-eng.txt.gz#_Nov_05_19_24_00_573158 | |
| 19:50:09 | dtruong | There is a functional tests that calls this function: https://github.com/openstack/openstacksdk/blob/master/openstack/tests/functional/orchestration/v1/test_stack.py#L58 | |
| 19:50:50 | dtruong | But I'm confused why it didn't fail after the function was removed from Proxy class. | |
| 19:51:55 | mordred | dtruong: hrm. that is confusing - and also unhappy making | |
| 19:52:19 | mordred | dtruong: we should maybe add a senlin dsvm job to openstacksdk's gate so that we can make sure not to break senlin again | |
| 19:53:22 | dtruong | There is already a senlin dvsm job but it only tests the basic cluster operations. | |
| 19:53:55 | dtruong | This error only happens when creating senlin clusters of heat stacks which is a test case in the senlin integration test suite. | |
| 20:25:00 | mordred | yeah - that's what I mean - we should run the senlin integration test suite in a job - configured with sdk installed from master | |
| 20:25:19 | mordred | that way we get the extra coverage because senlin is an sdk consumer - not only the coverage of testing using sdk to talk to senlin | |
| 22:53:21 | openstackgerrit | Merged openstack/openstacksdk master: Document "insecure" flag https://review.openstack.org/615512 | |
| 23:14:44 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| #openstack-sdks - 2018-11-06 | |||
| 00:58:06 | openstackgerrit | Duc Truong proposed openstack/openstacksdk master: Add wait functions to orchestration proxy https://review.openstack.org/615702 | |
| 01:00:54 | dtruong | mordred: ^ This change adds back the functions that senlin is calling. | |
| 02:27:10 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| 06:58:51 | openstackgerrit | Shilpa Devharakar proposed openstack/openstacksdk master: Add functional tests for masakari https://review.openstack.org/615744 | |
| 08:53:33 | openstackgerrit | Nguyen Hai Truong proposed openstack-infra/shade master: Add python 3.6 unit test job https://review.openstack.org/615783 | |
| 11:57:27 | openstackgerrit | Colleen Murphy proposed openstack/keystoneauth master: Add py36 tox environment https://review.openstack.org/615845 | |
| 12:23:55 | openstackgerrit | Sven Wegener proposed openstack/python-openstackclient master: image/v2: support multiple property filters https://review.openstack.org/615850 | |
| 13:27:11 | dtantsur | mordred: hi! how do I use baremetal_endpoint_override in clouds.yaml? a naive approach does not seem to work.. | |
| 13:35:42 | dtantsur | nvm, I'm probably being stupid | |
| 13:38:52 | mordred | dtantsur: you put it in the clouds.yaml :) | |
| 13:39:13 | dtantsur | right, I probably screwed up a thing in metalsmith | |
| 13:41:17 | mordred | it's also entirely possible there is a bug | |
| 13:41:25 | mordred | dtruong: awesome - and yay, a test failure! :) | |
| 13:48:07 | dtantsur | mordred: in case you're curious: https://review.openstack.org/#/c/615878/ | |
| 13:51:02 | mordred | I'm always curious | |
| 13:51:56 | mordred | ah - interesting | |
| 13:53:08 | mordred | dtantsur: out of more curiousity - why does _os_api.API take both a session/adapter and a Connection ... is that just hysterical raisins? | |
| 13:53:58 | mordred | ah ... because it also uses ironic_client | |
| 13:54:49 | mordred | I'm glad passing that adapter to the ironic client works! | |
| 14:06:59 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Test python2 with py27 https://review.openstack.org/615543 | |
| 14:09:54 | dtantsur | mordred: yep, it's to make ironicclient happy (I cannot find time to get rid of it) | |
| 14:10:53 | mordred | dtantsur: eventually | |
| 14:12:08 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Test python2 with py27 https://review.openstack.org/615543 | |
| 14:28:46 | dtantsur | mordred: just learned: https://github.com/openstack/openstacksdk/blob/master/openstack/service_description.py#L154 can return None, causing the following code to fail | |
| 14:30:40 | mordred | dtantsur: oh - right. poo. I think just line 161 should be if data and (data.catalog_url != data.service_url): ? | |
| 14:31:17 | mordred | hrm. no - I htink you're right - more understanding is needed | |
| 14:31:28 | dtantsur | mordred: in my case it happened when ironic was down and baremetal_endpoint_override was set | |
| 14:31:57 | mordred | ah - yeah - so in that case, the None is an indication that no service was found | |
| 14:33:04 | Shrews | such a common bug | |
| 14:33:12 | mordred | so - I think the above if data would then result in an adapter with the endpoint_override - which would then throw a network connection error on use? | |
| 14:33:30 | mordred | what a great edge case! | |
| 14:33:41 | dtantsur | \o/ | |
| 14:34:40 | mordred | dtantsur: ++ | |
| 14:34:49 | Shrews | swift actually handles this very well, too | |
| 14:35:17 | Shrews | yeah yeah, i know | |
| 14:35:27 | Shrews | i was curious | |
| 14:35:49 | Shrews | and pleasantly surprised, tbh | |
| 14:36:44 | mordred | weird | |
| 14:37:33 | Shrews | "swift" being the language, not the object store thingy (if that wasn't clear) | |
| 14:38:28 | mordred | yeah. I mean - I'm sure swift the object store would also handle this well | |
| 14:38:49 | mordred | but that's largely just becaues timburke finds all the bugs | |
| 14:53:52 | Shrews | mordred: this is unrelated to your change, yeah? http://logs.openstack.org/32/614832/3/check/osc-functional-devstack-tips/aa1ce44/job-output.txt.gz#_2018-11-04_15_03_56_197747 | |
| 14:54:12 | Shrews | i've never seen that failure before | |
| 15:58:50 | mordred | Shrews: yeah - I think that's a rando | |
| 16:01:57 | Shrews | mordred: k k | |
| 16:45:37 | mordred | Shrews: replied on https://review.openstack.org/#/c/614834 - good point about testing though - I'll go lift that from nodepool | |
| 16:46:58 | Shrews | mordred: gotcha. can live w/o the depends | |
| 16:47:10 | mordred | Shrews: I could also totes go the other way :) | |
| 16:49:08 | Shrews | mordred: let's not add it until we *need* to | |
| 16:49:15 | mordred | ++ | |
| 17:27:43 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| 20:18:23 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| 21:21:11 | hongbin | hi, i am traiging this bug: https://bugs.launchpad.net/zun/+bug/1802001 , and wondering why the user is trying to consume zun-api via openstacksdk, does openstacksdk supports zun? | |
| 21:21:11 | openstack | Launchpad bug 1802001 in Zun "please add documentation and example for python-zunclient API" [Undecided,New] | |
| 21:35:10 | mordred | hongbin: we don't REALLY support it - you can get a configured REST client for any openstack service in openstacksdk | |
| 21:35:43 | mordred | hongbin: so conn.application_container.get('/containers') should work and return a requests.response.Response | |
| 21:36:05 | mordred | hongbin: but we do not currently have any support for zun in the object layer | |
| 21:36:41 | mordred | hongbin: of course, support for zun is totaly _welcome_ :) | |
| 21:36:55 | hongbin | mordred: i see, then to be exact, what is the difference between the confiugred REST client and the support in object layer? | |
| 21:37:31 | hongbin | any pointer to a docs would be enough :) | |
| 21:38:30 | mordred | hongbin: the wrapper layer makes higher level objects - so for example, for neutron, there is http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/network/v2/_proxy.py | |
| 21:39:09 | mordred | hongbin: which provides things liek "create_network()" and "networks()" ... and then each resource that is managed gets a Resouce object | |
| 21:39:18 | mordred | like http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/network/v2/network.py | |
| 21:39:39 | hongbin | i see | |
| 21:40:11 | mordred | https://docs.openstack.org/openstacksdk/latest/user/guides/network.html has some examples of using that | |
| 21:40:42 | hongbin | mordred: everything is clear now, thank for the information | |