| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-08-20 | |||
| 05:48:42 | gouthamr | frickler: ack, wfm if gtema thinks so | |
| 07:02:50 | gtema | frickler, why you say gate is not fixed by that? I was locally testing your patch and it seems absolutely reasonable and working (for local unittests). The alternative is now to drop caching at all and reimplement it freshly in a different way. But on practice - it was working properly earlier | |
| 07:36:47 | frickler | gtema: according to what I tested, the "if" part in my patch is actually never triggered. so IMO it would be cleaner to drop it. the solution that I consider more correct, using arg_key = ','.join(map(str, args)), leads to unit failures, so would need more investigation | |
| 07:37:25 | frickler | gtema: I'm not saying that gate isn't fixed by that patch, it's just misleading about the actual situation | |
| 07:37:26 | gtema | but that one clearly didn't work in my test with (perhaps with older decorator version) | |
| 07:38:13 | frickler | gtema: my assumption is that the unit tests may actually be broken (or maybe the caching code) and arg_key = ','.join(map(str, args)) uncovers that | |
| 07:38:47 | gtema | no, unittest actually creates in-memory cache and verifies communication, so it should be working | |
| 07:39:30 | gtema | I am sadly still not really convinced how the cache should be reimplemented - was poking already few days without anything really useful | |
| 07:40:51 | gtema | using decorator seems to easy, but - i never thought it can be broken that easily with new release | |
| 07:41:03 | gtema | * using decorator seems so easy, but - i never thought it can be broken that easily with new release | |
| 07:45:10 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Fix key generation for caching https://review.opendev.org/c/openstack/openstacksdk/+/804530 | |
| 07:46:29 | frickler | gtema: gouthamr: I would propose to go with ^^ for now to unblock. if you want something different, feel free to replace with your own ideas | |
| 07:47:25 | gtema | ok, no objections | |
| 14:03:40 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: [WIP] Add neutron local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| 14:09:57 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: [WIP] Add neutron local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| #openstack-sdks - 2021-08-21 | |||
| 00:35:23 | jameshyde | how to properly update some field inside nova_api.request_specs database table field for certain vm? there's no such API available? | |
| #openstack-sdks - 2021-08-22 | |||
| 19:07:47 | opendevreview | Daniel Speichert proposed openstack/ansible-collections-openstack master: Added missing warn() used in cloud.openstack.quota https://review.opendev.org/c/openstack/ansible-collections-openstack/+/805534 | |
| 19:11:14 | opendevreview | Daniel Speichert proposed openstack/ansible-collections-openstack master: Added missing warn() used in cloud.openstack.quota https://review.opendev.org/c/openstack/ansible-collections-openstack/+/805534 | |
| #openstack-sdks - 2021-08-23 | |||
| 08:23:10 | opendevreview | Wenping Song proposed openstack/openstacksdk master: Add description args for device profile create https://review.opendev.org/c/openstack/openstacksdk/+/803888 | |
| 11:44:19 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 21:20:03 | opendevreview | Merged openstack/openstacksdk master: Fix key generation for caching https://review.opendev.org/c/openstack/openstacksdk/+/804530 | |
| 21:20:17 | opendevreview | Merged openstack/python-openstackclient master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/789410 | |
| #openstack-sdks - 2021-08-24 | |||
| 07:42:42 | opendevreview | Javier Peña proposed openstack/python-openstackclient stable/wallaby: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/805790 | |
| 09:21:52 | opendevreview | Javier Peña proposed openstack/python-openstackclient stable/wallaby: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/805790 | |
| 11:33:00 | opendevreview | Johannes Kulik proposed openstack/openstacksdk master: Add "security_group_ids" to Port's query parameters https://review.opendev.org/c/openstack/openstacksdk/+/804979 | |
| #openstack-sdks - 2021-08-25 | |||
| 01:28:51 | opendevreview | Wenping Song proposed openstack/openstacksdk master: Add description args for device profile create https://review.opendev.org/c/openstack/openstacksdk/+/803888 | |
| 08:41:58 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add new job osc-functional-devstack-neutron-ovs https://review.opendev.org/c/openstack/python-openstackclient/+/793143 | |
| 15:37:46 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: [WIP] Add network quota limit check https://review.opendev.org/c/openstack/python-openstackclient/+/806016 | |
| #openstack-sdks - 2021-08-26 | |||
| 08:00:41 | opendevreview | Merged openstack/python-openstackclient master: volume: Add missing 'volume list --offset' parameter https://review.opendev.org/c/openstack/python-openstackclient/+/802480 | |
| 08:02:35 | opendevreview | Merged openstack/python-openstackclient master: openstack image create: honor protection/visibility flags https://review.opendev.org/c/openstack/python-openstackclient/+/789649 | |
| 08:03:54 | opendevreview | Merged openstack/python-openstackclient master: volume: Add 'volume transfer request create --(no-)snapshots' option https://review.opendev.org/c/openstack/python-openstackclient/+/800715 | |
| 08:14:21 | opendevreview | Merged openstack/python-openstackclient stable/wallaby: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/805790 | |
| 08:14:23 | opendevreview | Merged openstack/python-openstackclient master: L3 conntrack helper: Use singular name consistently https://review.opendev.org/c/openstack/python-openstackclient/+/795222 | |
| 16:30:33 | gtema | mordred, if you have a chance - please have a look at https://review.opendev.org/c/openstack/openstacksdk/+/805851. I started reworking caching to be as deep as possible and avoids our decorator issues | |
| 16:30:58 | gtema | in next PR I plan to drop old caching in the cloud layer while still keeping it working | |
| 17:23:30 | mordred | gtema: it looks good at first glance. we should get corvus to look too - I know nodepool is the primary user of the non-dogpile caches | |
| 17:24:52 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add "check_limit" to network Quota class https://review.opendev.org/c/openstack/openstacksdk/+/806254 | |
| 17:47:22 | gtema | Sure | |
| #openstack-sdks - 2021-08-27 | |||
| 12:11:07 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: [WIP] Add neutron local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| 14:08:22 | mordred | gtema: fwiw I pinged corvus about that patch | |
| 14:08:34 | gtema | thks mordred | |
| 14:08:58 | gtema | this is definitely not the final state, but I just want to get some feedback before continuing that | |
| 14:09:08 | mordred | yah - for sure | |
| 14:09:31 | mordred | I'm excited to see that - we've talked about going in that direction for a while | |
| 14:09:43 | gtema | yah | |
| 14:10:04 | gtema | nodepool fails due to that whatever is being currently discussed in zuul, isn't it? | |
| 14:10:41 | gtema | (Unable to locate package libffi6) | |
| 15:01:38 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add network update quota "limit_check" parameter https://review.opendev.org/c/openstack/python-openstackclient/+/806016 | |
| 15:30:07 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add network update quota "limit_check" parameter https://review.opendev.org/c/openstack/python-openstackclient/+/806016 | |
| 15:54:01 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add network update quota "limit_check" parameter https://review.opendev.org/c/openstack/python-openstackclient/+/806016 | |
| #openstack-sdks - 2021-08-28 | |||
| 07:23:23 | opendevreview | LEE JAE YONG proposed openstack/python-openstackclient master: Fix typo error in listing server's column name https://review.opendev.org/c/openstack/python-openstackclient/+/806464 | |
| 07:36:56 | seongsoocho | ^ We just got a first patch in korea user group's mentorig program ! | |
| #openstack-sdks - 2021-08-29 | |||
| 12:10:25 | frigo | hi sdks! Fyi the openstack cli and rally (as installed with pip) is broken since a few hours, following a new release of importlib_metadata. I opened https://storyboard.openstack.org/#!/story/2009151 | |
| #openstack-sdks - 2021-08-30 | |||
| 10:17:27 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 10:50:32 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: [WIP] Add neutron local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| 11:13:35 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 11:49:16 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 17:07:23 | opendevreview | Ghanshyam proposed openstack/openstackclient master: [community goal] Update contributor documentation https://review.opendev.org/c/openstack/openstackclient/+/806623 | |
| 17:07:28 | opendevreview | Ghanshyam proposed openstack/python-openstackclient master: [community goal] Update contributor documentation https://review.opendev.org/c/openstack/python-openstackclient/+/806624 | |
| 17:13:17 | opendevreview | Ghanshyam proposed openstack/python-openstackclient master: [community goal] Update contributor documentation https://review.opendev.org/c/openstack/python-openstackclient/+/806624 | |
| 17:15:36 | opendevreview | Ghanshyam proposed openstack/openstackclient master: [community goal] Update contributor documentation https://review.opendev.org/c/openstack/openstackclient/+/806623 | |
| #openstack-sdks - 2021-08-31 | |||
| 13:03:38 | opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Enforce scope mutual exclusion for system https://review.opendev.org/c/openstack/keystoneauth/+/802683 | |
| 13:31:31 | opendevreview | Laurent Almeras proposed openstack/ansible-collections-openstack master: use idempotent behavior + diff https://review.opendev.org/c/openstack/ansible-collections-openstack/+/806745 | |
| 16:35:52 | opendevreview | Merged openstack/openstackclient master: [community goal] Update contributor documentation https://review.opendev.org/c/openstack/openstackclient/+/806623 | |
| 20:47:10 | opendevreview | Merged openstack/python-openstackclient master: [community goal] Update contributor documentation https://review.opendev.org/c/openstack/python-openstackclient/+/806624 | |
| #openstack-sdks - 2021-09-01 | |||
| 11:38:32 | thomasb06 | diablo_rojo_phone: gtema: stephenfin: hello. You must be sleeping by now... | |
| 11:39:38 | gtema | nope, I am full in the afternoon | |
| 11:41:42 | thomasb06 | ah, you not in the US then. Maybe diablo_rojo_phone told you about me, I'm trying to get involved in OpenStack. Today I'll try to have the sandbox up and running, but maybe later in the afternoon. | |
| 11:42:40 | gtema | I'm in Germany | |
| 11:43:17 | thomasb06 | Guten tag | |
| 11:43:23 | gtema | any way - just ask | |
| 11:43:30 | gtema | moin | |
| 11:43:42 | thomasb06 | (here it's Nice in southern France) | |
| 11:43:55 | gtema | great | |
| 11:44:50 | thomasb06 | as I didn't even try the sandbox, I wouldn't know what to ask. But if it doesn't work out, I'll tell you | |
| 11:45:05 | gtema | :) | |
| 12:18:44 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Add support for microversion 2.90 https://review.opendev.org/c/openstack/python-openstackclient/+/806921 | |
| 12:30:19 | opendevreview | Wenping Song proposed openstack/openstacksdk master: Add description args for device profile create https://review.opendev.org/c/openstack/openstacksdk/+/803888 | |
| 12:59:33 | opendevreview | Jake Yip proposed openstack/python-openstackclient master: Add trustee and trustor filters for trust list https://review.opendev.org/c/openstack/python-openstackclient/+/806927 | |
| 15:14:50 | thomasb06 | gtema: The captcha to register at Openstack isn't convinced I'm a human... | |
| 15:15:02 | thomasb06 | let me retry again | |
| 15:15:20 | gtema | :) maybe you aren't? | |
| 15:15:46 | gtema | I'm also not sure that at the end of the day I am human | |
| 15:16:39 | thomasb06 | hehe, well I feel like one at least | |
| 15:16:52 | thomasb06 | maybe there is a contact email on the site | |
| 15:17:05 | gtema | now you just need to convince others | |
| 15:17:21 | gtema | on which page you mean the captcha? | |
| 15:20:48 | thomasb06 | It's in this page, after I click the "An Individual" button: https://openinfra.dev/join/ | |
| 15:21:12 | thomasb06 | After several attempts, it still didn't work | |
| 15:22:19 | gtema | hmm, I am not even asked for captcha | |
| 15:22:46 | thomasb06 | arg, it's sounds strange already | |
| 15:23:24 | gtema | I wonder, for which step you need this now? | |
| 15:24:04 | gtema | but anyway - maybe you try different browser? | |
| 15:24:41 | thomasb06 | a different browser, good idea | |
| 15:31:51 | thomasb06 | No, the captcha still doesn't believe me... | |
| 15:32:02 | thomasb06 | maybe another browser again | |
| 15:32:06 | gtema | uhm | |
| 15:32:17 | gtema | maybe you are now blacklisted? | |
| 15:32:30 | gtema | maybe you should either try later or try different ip | |
| 15:33:19 | thomasb06 | in Vivaldi, it worked | |