Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-11-02
15:57:55 kmalloc I feel like it's a little too fast-and-loose with what is added to it.
15:58:15 kmalloc somewhere between alpine and debian is where I'd like to be. (Ubuntu is not the answer)
15:58:28 kmalloc debian stable*
16:46:56 kmalloc mordred: +2 on rate limit in KSA
16:47:09 kmalloc it feels like it really belongs there as much as I wish it could be higher up.
16:47:14 mordred kmalloc: woot. (pending tests, of course)
16:47:17 kmalloc and that is due to retry logic
16:47:18 kmalloc yes.
16:47:25 mordred kmalloc: yah. the retry logic is the real kicker
16:47:30 kmalloc must have tests before +A.
16:47:34 mordred kmalloc: sort of like why discovery needed to be there too
16:47:43 kmalloc functional tests are fine (doesn't need to be unit)
16:47:44 mordred since it needes to be there for keystone _anyway_ for auth to work :)
16:47:57 kmalloc it needs to be tested so we don't break
16:48:10 kmalloc unit tests of that is going to be hard(tm)
16:48:12 mordred oh - yeah. seriously
16:48:22 mordred I *think* I have an idea of how to unit test it too
16:48:55 kmalloc cool
16:49:05 kmalloc but like i said, i'm happy with that simply being functional
16:49:25 kmalloc you *might* be able to get away with unit tests utilizing freezegun
16:49:46 kmalloc and stepping forward on ticks and a very low rate (e.g. 1 per multiple seconds)
16:50:11 kmalloc concurrency could be tested via just objects that explicitly manage when they take action.
16:50:17 kmalloc but functional might be so much easier
17:00:34 Shrews mordred: comment on https://review.openstack.org/614832
17:01:53 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Add --name-lookup-one-by-one option to server list https://review.openstack.org/568345
17:49:10 dtroyer smcginnis: thanks for all that cleanup, I wish we could merge it today :)
17:50:52 dtroyer I'm thinking to release master, rocky and queens early-mid next week…
18:00:33 openstackgerrit Merged openstack/python-openstackclient master: Address issues from volume backend commands https://review.openstack.org/610161
18:03:07 cdent dtroyer: see you in berlin?
18:10:20 dtroyer cdent: yes, I land Sunday
18:10:32 cdent sweet
18:34:37 mordred Shrews: well, clearly I'm just stupid
18:36:59 smcginnis dtroyer: No worries. I'll try to keep those cleanup's rebased as conflicts come up so they are ready to go when the time is right.
18:38:16 dtroyer smcginnis: there is an osc4 feature branch that has some work in it already. I need to merge master back in to that, but after that is done we could just merge those into the feature branch
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/

Earlier   Later