| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-07 | |||
| 13:08:09 | mordred | tosky: tripleo-common===8.6.3 is in the upper-constraints file for stable/queens | |
| 13:11:28 | mordred | tosky: ah- the check plugins job does not respect upper-constraints | |
| 13:13:10 | tosky | ah | |
| 13:13:29 | tosky | something had to be wrong | |
| 13:19:24 | Shrews | mordred: https://review.openstack.org/588656 must have been fun | |
| 13:19:43 | mordred | Shrews: so much fun | |
| 13:19:59 | mordred | Shrews: but if you think that one was fun, you should meet my friend https://review.openstack.org/589250 | |
| 13:20:58 | Shrews | mordred: u have weird friends | |
| 13:23:24 | openstackgerrit | Monty Taylor proposed openstack/openstackclient master: Update openstackclient-check-plugins to honor constraints https://review.openstack.org/589465 | |
| 13:23:27 | mordred | tosky: ^^ | |
| 13:25:53 | tosky | I miss some of the logic of the joke, but that's fine :D | |
| 13:26:28 | tosky | oh, pbrx is a real thing | |
| 13:29:06 | tosky | what is the difference between pip freeze and pbr freeze? | |
| 13:33:35 | openstackgerrit | Merged openstack/os-service-types master: Add missing release notes https://review.openstack.org/589458 | |
| 13:38:50 | tosky | something did not work | |
| 13:53:45 | openstackgerrit | Monty Taylor proposed openstack/openstackclient master: Update openstackclient-check-plugins to honor constraints https://review.openstack.org/589465 | |
| 13:56:34 | mordred | tosky: pbr freeze emits some extra info about git shas | |
| 13:57:11 | mordred | tosky: I figured in this case it might be a nice addition so we can check things | |
| 13:58:34 | tosky | ack | |
| 14:05:15 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |
| 14:05:40 | mordred | Shrews: I *think* that ^^ is going to pass this time | |
| 14:05:59 | mordred | Shrews: I hope you'll enjoy the change in openstack/tests/functional/image/v2/test_image.py | |
| 14:06:26 | Shrews | mordred: i'm still digesting 589250 now that i have coffee | |
| 14:06:32 | Shrews | mordred: something doesn't seem right | |
| 14:07:06 | Shrews | mordred: you never set the alias_flag attr afaict | |
| 14:07:16 | mordred | yes - that's correct | |
| 14:07:38 | Shrews | mordred: and the set and immediately delete the seen_flag | |
| 14:07:39 | mordred | the alias_flag is the thing we need to check to see if it's already been set | |
| 14:07:53 | mordred | well, the getattr may not seem like a call into a recursion stack ... but it is | |
| 14:08:38 | mordred | so setting the flag before the getattr is a way for the check in the getattr call to know it shouldn't itself make a subsequent getattr call | |
| 14:09:29 | mordred | Shrews: http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/object_store/v1/container.py#n44 and http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/object_store/v1/container.py#n51 | |
| 14:09:40 | mordred | Shrews: are the example case that make this whole thing go batty | |
| 14:10:19 | mordred | count is aliased to object_count and object_count is aliased to count - if _neither_ are set, without some sort of something, the infinite recursion happens | |
| 14:10:51 | mordred | Shrews: (also, this took me literally the entire work day yesterday to wrap my head around, so if you're even partially following it on a first coffee, you're doing great) | |
| 14:12:34 | Shrews | mordred: ok, follow the alias thing now. but: setattr(instance, seen_flag, True); value = ; delattr(instance, seen_flag) | |
| 14:12:46 | Shrews | don't get the set-then-delete | |
| 14:13:42 | Shrews | oh! | |
| 14:13:47 | Shrews | ok, coffee just kicked in | |
| 14:13:52 | mordred | \o/ | |
| 14:13:53 | Shrews | mordred: duh | |
| 14:14:21 | mordred | Shrews: should we add a comment there to help future-us understand what the heck is going on? | |
| 14:14:51 | Shrews | meh, i've been away for a week so i'm just slow. up to you | |
| 14:15:15 | mordred | writing comments is hard | |
| 14:15:40 | Shrews | yah | |
| 14:16:14 | mordred | adriant: if you get a sec when your around, could you check out https://review.openstack.org/#/q/topic:resource-dict - and most importantly make sure it doesn't break you? | |
| 14:16:43 | mordred | it's one of the last 2 breaking API changes on the list before a 1.0 | |
| 14:17:52 | Shrews | mordred: it totally not apparent (until you pasted those container lines) how that test was testing recursion. maybe a comment in that test? | |
| 14:18:13 | mordred | Shrews: ++ | |
| 14:25:31 | mordred | Shrews: ok in that case, I'm adding a comment to the method too | |
| 14:25:55 | Shrews | k. sorry | |
| 14:27:04 | mordred | no - it's a good point - the chances we understand this in a months' time are pretty low | |
| 14:27:46 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix to_dict recursion issues with circular aliases https://review.openstack.org/589250 | |
| 14:27:46 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename Resource get and update to not clash with dict https://review.openstack.org/589193 | |
| 14:27:47 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add computed attribute type and location to base resource https://review.openstack.org/589194 | |
| 14:27:47 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove special handling of stacks https://review.openstack.org/589195 | |
| 14:27:48 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |
| 14:27:58 | mordred | Shrews: kk. updated | |
| 14:45:28 | openstackgerrit | Merged openstack/python-openstackclient master: Fix broken gate jobs https://review.openstack.org/587005 | |
| 15:17:21 | mordred | \o | |
| 15:17:23 | mordred | gah | |
| 15:17:25 | mordred | \o/ | |
| 15:17:28 | mordred | is what I meant | |
| 15:25:57 | openstackgerrit | Monty Taylor proposed openstack/openstackclient master: Update openstackclient-check-plugins to honor constraints https://review.openstack.org/589465 | |
| 17:19:42 | mordred | Shrews: yay https://review.openstack.org/#/c/589250/ is green! :) | |
| 17:20:28 | Shrews | mordred: +Ad | |
| 17:47:29 | PagliaccisCloud | anyone got a +2 to spare? https://review.openstack.org/#/c/584484/ | |
| 18:30:37 | mordred | PagliaccisCloud: I got AJaeger to leave a comment - but it's an update to a translated string which should go through the translations process. the base string seems to have already been fixed in the codebase. however, translation string updates are only proposed when there has been a change in the translations | |
| 18:44:40 | frickler | mordred: dtroyer: most of the rechecks I did seem to have passed, the others look like genuine issues to me. would be great if you could put https://review.openstack.org/516701 onto your review list | |
| 19:46:40 | openstackgerrit | Merged openstack/openstacksdk master: Fix to_dict recursion issues with circular aliases https://review.openstack.org/589250 | |
| 19:46:41 | openstackgerrit | Merged openstack/python-openstackclient master: Fix missing trailing spaces in network help messages https://review.openstack.org/588616 | |
| 20:13:56 | openstackgerrit | Monty Taylor proposed openstack/openstackclient master: Update openstackclient-check-plugins to honor constraints https://review.openstack.org/589465 | |
| 20:15:27 | mordred | dtroyer, frickler: ^^ that should fix theissues tosky was seeing with openstackclient-check-plugins on stable/queens (and the one in the stack before it, about stestr, is needed in that repo anyway) | |
| 20:16:16 | mordred | dhellmann: ^^ you might find that patch interesting as well - it is a patch thatuses the siblings logic outside of a tox context | |
| 20:55:44 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: DNM Testing pep8 line comments patch https://review.openstack.org/589635 | |
| 21:17:40 | mordred | Shrews: easy review if you have a sec: https://review.openstack.org/#/c/589212/ | |
| 22:10:42 | openstackgerrit | Merged openstack/python-openstackclient master: Add dns-domain support to Network object https://review.openstack.org/516701 | |
| 23:05:53 | openstackgerrit | Merged openstack/openstacksdk master: Stop calling get_all_types when service-type is None https://review.openstack.org/589212 | |
| 23:54:48 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Stop using the -consumer devstack jobs https://review.openstack.org/589675 | |
| 23:56:49 | mordred | Shrews: ^^ I think openstacksdk is sufficiently integrated with devstack that we shoudl do that | |
| #openstack-sdks - 2018-08-08 | |||
| 01:26:35 | adriant | mordred: will look! | |
| 01:27:21 | adriant | mordred: sorry I haven't had much time to look at anything SDK related. Been deep within billing related stuff for the last while. | |
| 09:14:50 | openstackgerrit | yanpuqing proposed openstack/python-openstackclient master: Add error message when network resources use "set" "unset" command https://review.openstack.org/579515 | |
| 11:09:20 | openstackgerrit | Yushiro FURUKAWA proposed openstack/osc-lib master: Revert "Fixes a bug with issubclass() being called on an instance" https://review.openstack.org/586617 | |
| 12:58:05 | Shrews | mordred: any idea what these failures are about? http://logs.openstack.org/75/589675/1/check/osc-functional-devstack-tips/cc9ccf7/testr_results.html.gz | |
| 13:00:11 | mordred | Shrews: I do not - osc's gate tests were broken until yesterday | |
| 13:21:31 | mordred | Shrews: got a sec for this one? https://review.openstack.org/#/c/589675/ should reduce some node churn | |
| 13:22:23 | Shrews | mordred: well, the tests I asked about are for that one | |
| 13:24:32 | mordred | ah - yeah | |
| 14:33:06 | amotoki | dtroyer: could you look at https://review.openstack.org/#/c/586617/ ? it is a revert in osc-lib which breaks neutronclient OSC plugin | |
| 14:38:37 | dmsimard | mordred: if it's any consolation, after doing even a bit of AWS I find the OpenStack APIs and Ansible modules for OpenStack MUCH better (and especially more consistent) | |
| 14:39:30 | dmsimard | The AWS APIs and modules are very inconsistent and unpredictable, it's almost like if they had been developed by entirely different companies | |
| 14:39:34 | dmsimard | so... good job ? :D | |
| 14:39:48 | dtroyer | amotoki: +2+W | |
| 14:40:10 | mordred | dmsimard: yay! thanks - that's great to hear | |
| 14:40:34 | amotoki | dtroyer: thanks. | |
| 14:43:13 | openstackgerrit | Merged openstack/osc-lib master: Remove PyPI downloads https://review.openstack.org/589369 | |
| 14:46:42 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename Resource get and update to not clash with dict https://review.openstack.org/589193 | |
| 14:46:42 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add computed attribute type and location to base resource https://review.openstack.org/589194 | |
| 14:46:43 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove special handling of stacks https://review.openstack.org/589195 | |
| 14:46:43 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |