*** bobh has joined #openstack-sdks | 01:07 | |
*** bobh has quit IRC | 01:12 | |
*** altlogbot_0 has quit IRC | 01:30 | |
*** altlogbot_1 has joined #openstack-sdks | 01:32 | |
*** jangutter has quit IRC | 01:33 | |
*** jangutter has joined #openstack-sdks | 01:33 | |
*** adriant has joined #openstack-sdks | 01:36 | |
*** bobh has joined #openstack-sdks | 02:09 | |
*** icarusfactor has quit IRC | 03:03 | |
*** whoami-rajat has joined #openstack-sdks | 03:12 | |
*** bobh has quit IRC | 03:28 | |
*** factor has joined #openstack-sdks | 04:16 | |
*** gtema has joined #openstack-sdks | 04:49 | |
*** ricolin has joined #openstack-sdks | 05:03 | |
*** Luzi has joined #openstack-sdks | 05:37 | |
openstackgerrit | Boxiang Zhu proposed openstack/python-openstackclient master: Add host and hypervisor_hostname to create servers https://review.opendev.org/669609 | 06:18 |
---|---|---|
*** gtema has quit IRC | 06:22 | |
*** gtema has joined #openstack-sdks | 06:38 | |
*** cdent has joined #openstack-sdks | 06:41 | |
*** gtema has quit IRC | 06:53 | |
*** gtema has joined #openstack-sdks | 06:55 | |
openstackgerrit | zhangqing proposed openstack/python-openstackclient master: Use cliff formattable columns in compute commands https://review.opendev.org/669304 | 06:56 |
*** gtema_ has joined #openstack-sdks | 06:58 | |
*** slaweq has joined #openstack-sdks | 07:02 | |
*** gtema has quit IRC | 07:07 | |
*** gtema has joined #openstack-sdks | 07:07 | |
*** gtema has quit IRC | 07:14 | |
*** ianw is now known as ianw_pto | 07:16 | |
*** gtema has joined #openstack-sdks | 07:16 | |
*** jpich has joined #openstack-sdks | 07:17 | |
*** ttsiouts has joined #openstack-sdks | 07:23 | |
*** tosky has joined #openstack-sdks | 07:28 | |
*** ralonsoh has joined #openstack-sdks | 07:30 | |
*** ttsiouts has quit IRC | 07:38 | |
*** gtema has quit IRC | 07:39 | |
*** gkadam has joined #openstack-sdks | 07:55 | |
*** gkadam has quit IRC | 07:55 | |
*** ttsiouts has joined #openstack-sdks | 07:57 | |
*** dtantsur|afk is now known as dtantsur | 07:57 | |
*** ttsiouts has quit IRC | 08:11 | |
*** ttsiouts has joined #openstack-sdks | 08:12 | |
*** ttsiouts_ has joined #openstack-sdks | 08:13 | |
*** ttsiouts has quit IRC | 08:17 | |
*** gtema_ has quit IRC | 08:31 | |
*** gtema has joined #openstack-sdks | 08:31 | |
openstackgerrit | Vishakha Agarwal proposed openstack/openstacksdk master: Add application credential CRUD support https://review.opendev.org/669331 | 09:20 |
openstackgerrit | Vishakha Agarwal proposed openstack/openstacksdk master: Add application credential CRUD support https://review.opendev.org/669331 | 09:42 |
*** gtema_ has joined #openstack-sdks | 09:44 | |
*** gtema has quit IRC | 09:45 | |
*** cdent has quit IRC | 10:07 | |
*** ricolin has quit IRC | 10:14 | |
*** ttsiouts_ has quit IRC | 10:16 | |
*** ttsiouts has joined #openstack-sdks | 10:17 | |
*** cdent has joined #openstack-sdks | 10:18 | |
*** ttsiouts has quit IRC | 10:21 | |
*** gtema_ has quit IRC | 10:22 | |
*** ricolin has joined #openstack-sdks | 10:22 | |
*** gtema has joined #openstack-sdks | 10:23 | |
*** ricolin has quit IRC | 10:23 | |
*** gtema has quit IRC | 10:41 | |
*** gtema has joined #openstack-sdks | 10:42 | |
*** gtema_ has joined #openstack-sdks | 10:45 | |
*** gtema has quit IRC | 10:54 | |
*** gtema has joined #openstack-sdks | 10:55 | |
*** gtema_ has quit IRC | 10:59 | |
*** ttsiouts has joined #openstack-sdks | 11:04 | |
*** gtema has quit IRC | 11:06 | |
*** gtema has joined #openstack-sdks | 11:08 | |
*** altlogbot_1 has quit IRC | 11:19 | |
*** irclogbot_0 has quit IRC | 11:19 | |
*** altlogbot_2 has joined #openstack-sdks | 11:20 | |
*** altlogbot_1 has joined #openstack-sdks | 11:20 | |
*** zbr is now known as zbr|lunch | 11:22 | |
*** altlogbot_1 has quit IRC | 11:25 | |
*** altlogbot_2 has quit IRC | 11:25 | |
*** gtema has quit IRC | 11:32 | |
*** gtema has joined #openstack-sdks | 11:32 | |
*** ricolin has joined #openstack-sdks | 11:44 | |
*** ttsiouts has quit IRC | 11:44 | |
*** ttsiouts has joined #openstack-sdks | 11:45 | |
*** odyssey4me_ has joined #openstack-sdks | 11:49 | |
*** ttsiouts has quit IRC | 11:49 | |
*** ttsiouts has joined #openstack-sdks | 11:53 | |
*** kinrui has joined #openstack-sdks | 11:54 | |
*** fyx has quit IRC | 11:55 | |
*** mordred has quit IRC | 11:55 | |
*** guilhermesp has quit IRC | 11:55 | |
*** kmalloc has quit IRC | 11:55 | |
*** TheJulia has quit IRC | 11:55 | |
*** dustinc has quit IRC | 11:55 | |
*** fungi has quit IRC | 11:55 | |
*** odyssey4me has quit IRC | 11:56 | |
*** melwitt has quit IRC | 11:56 | |
*** odyssey4me_ is now known as odyssey4me | 11:56 | |
*** zbr|lunch is now known as zbr | 11:59 | |
*** irclogbot_3 has joined #openstack-sdks | 12:00 | |
*** fyx has joined #openstack-sdks | 12:01 | |
*** mordred has joined #openstack-sdks | 12:01 | |
*** guilhermesp has joined #openstack-sdks | 12:01 | |
*** kmalloc has joined #openstack-sdks | 12:01 | |
*** TheJulia has joined #openstack-sdks | 12:01 | |
*** dustinc has joined #openstack-sdks | 12:01 | |
*** altlogbot_2 has joined #openstack-sdks | 12:03 | |
*** altlogbot_1 has joined #openstack-sdks | 12:03 | |
*** altlogbot_2 has quit IRC | 12:05 | |
*** altlogbot_1 has quit IRC | 12:05 | |
*** irclogbot_3 has quit IRC | 12:05 | |
*** altlogbot_2 has joined #openstack-sdks | 12:09 | |
*** altlogbot_2 has quit IRC | 12:11 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.opendev.org/650903 | 12:11 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Push sort_key down to server for flavor filtering https://review.opendev.org/669857 | 12:11 |
mordred | gtema: addressed a couple of review comments from dtantsur in the compute flavors patch ^^ and then added a followup. | 12:12 |
gtema | super mordred. Wanted already to remind about the whole batch, but you was faster than I had time to switch context :-) | 12:13 |
gtema | btw, I would also need attention to the "rework monitoring" change, but I have found another case where metric name is not generated properly, so will do another PS shortly | 12:14 |
mordred | cool - will look at that too | 12:17 |
gtema | thks | 12:17 |
*** kinrui is now known as fungi | 12:21 | |
gtema | mordred, do you remember about project cleanup tasks we took with you? We need to start doing that, since that is also I need in my project | 12:26 |
mordred | gtema: funny - I was *just* talking about that a little in #openstack-infra :) | 12:30 |
mordred | gtema: I probably can't get that done this week, but probably can next week | 12:30 |
gtema | no hurry, it's just a reminder, that we took that on us | 12:31 |
openstackgerrit | Merged openstack/shade master: Add Python 3 Train unit tests https://review.opendev.org/669452 | 12:43 |
mordred | gtema: ++ | 12:46 |
mgariepy | can i please have some review on : https://review.opendev.org/#/c/668017/ and https://review.opendev.org/#/c/668018/ | 12:53 |
*** altlogbot_3 has joined #openstack-sdks | 12:55 | |
Shrews | mordred: https://review.opendev.org/666905 is fyi. i haven't looked to see if latest sdk handles that yet | 12:57 |
*** altlogbot_3 has quit IRC | 12:57 | |
mordred | Shrews: it's *supposed* to - maybe we're missing a guard somewhere | 13:01 |
mordred | Shrews: oh - hrm. looking at the nodepool patch itself (not just the commit message) - nodepool is calling list_ports directly not just getting to it via create_server | 13:02 |
mordred | I think in that case we're almost *Certainly* missing a guard - we should put a if not self.has_service('network'): return [] at the top or something similar | 13:03 |
Shrews | yeah, for port cleanup iirc | 13:03 |
Shrews | yes, agreed | 13:04 |
mordred | Shrews: I'll make a patch so you can review | 13:06 |
openstackgerrit | Merged openstack/openstacksdk master: Fix typo for subnet.py This 'slacc' should be 'slaac' https://review.opendev.org/669661 | 13:06 |
Shrews | k | 13:06 |
openstackgerrit | Merged openstack/openstacksdk master: Switch cloud layer to use proxy for DNS https://review.opendev.org/665435 | 13:11 |
openstackgerrit | Merged openstack/openstacksdk master: Use Resource layer in cloud for SecurityGroups of server https://review.opendev.org/662998 | 13:13 |
*** mriedem has joined #openstack-sdks | 13:13 | |
*** goldyfruit has joined #openstack-sdks | 13:19 | |
openstackgerrit | Merged openstack/openstacksdk master: Use Resource layer for next compute methods https://review.opendev.org/663064 | 13:21 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Return empty lists for resources if neutron doesn't exist https://review.opendev.org/669869 | 13:22 |
*** gtema has quit IRC | 13:23 | |
mordred | Shrews: ^^ | 13:24 |
*** gtema has joined #openstack-sdks | 13:24 | |
*** slaweq has quit IRC | 13:43 | |
*** ttsiouts has quit IRC | 13:47 | |
*** ttsiouts has joined #openstack-sdks | 13:47 | |
*** slaweq has joined #openstack-sdks | 13:48 | |
ITD27M01 | mordred: Hello! I am in the way to catch you! Could you please leave a comment in this PR: https://github.com/ansible/ansible/pull/51401 | 13:49 |
ITD27M01 | mordred: If you have a second for this :) | 13:50 |
*** ttsiouts has quit IRC | 13:52 | |
mordred | ITD27M01: Shrews left some comments on that about needing to add self.templare.template() call which someone else also mentioned - can you either fix that or respond that you've verified it's not needed? | 13:52 |
openstackgerrit | Corey Bryant proposed openstack/openstackclient master: Add Python 3 Train unit tests https://review.opendev.org/637739 | 13:57 |
ITD27M01 | mordred: Thank you, I just wanted to make sure you knew. I am not an author and will try to contact the developer to continue working. | 13:57 |
*** Luzi has quit IRC | 13:57 | |
*** ttsiouts has joined #openstack-sdks | 13:58 | |
mordred | ah - ok, cool. maybe I'll leave that same comment on the PR | 14:00 |
mordred | (and sorry - my IRC to github user mapping is incomplete) | 14:00 |
Shrews | looks like the latest commit addresses the template thing | 14:02 |
mordred | yeah. | 14:02 |
Shrews | but not version_added | 14:02 |
mordred | the way these comments get displayed is very confusing to me | 14:03 |
Shrews | god i hate gh | 14:03 |
Shrews | yes, same | 14:03 |
Shrews | mordred: that comment from me on that PR had me questioning my sanity b/c that call is clearly there now | 14:10 |
*** whoami-rajat has quit IRC | 14:18 | |
Shrews | i almost want to send my response to your response directly to GH higher ups, but i doubt they'd care | 14:18 |
dtantsur | folks, do we bypass major version detection when endpoint_override is used? | 14:51 |
mordred | dtantsur: uhm ... | 15:05 |
mordred | dtantsur: I know that if you provide api_version _and_ endpoint_override we completely trust you | 15:05 |
dtantsur | I'm seeing ironic-inspector in bifrost accessed without /v1 | 15:05 |
dtantsur | but I don't think we provide api_version | 15:06 |
mordred | oh - yeah - if you give endpoint_override it should be the versioned endpoint | 15:06 |
mordred | endpoint_override says "use this endpoint exactly" - discovery to find a new endpoint will never happen. in some cases we might hit the discovery url to get the document so we can see microversion info | 15:07 |
mordred | but we won't look in to it to find a versioned endpoint if an unversioned endpoint was given | 15:07 |
* dtantsur contemplates whether he likes it or not | 15:08 | |
mordred | dtantsur: unfortunately, the base system has a two-layer system for finding the url to use - the keystone catalog and then service version discovery | 15:09 |
mordred | thinking about it now - it seems like having 2 different possible endpoint_override parameters would actually make more complete sense | 15:09 |
mordred | so you could say something like "catalog_endpoint_override" or "service_endpoint_override" (but with better names) | 15:10 |
mordred | dtantsur: for more context - we also have to have people give both endpoint_override and api_version sometimes just to tell us what the api version is because the discovery documents are broken on their cloud | 15:11 |
dtantsur | huh | 15:11 |
dtantsur | I think I feel a bit uneasy about putting versioned endpoints to configuration files | 15:11 |
mordred | but - maybe we could apply the heuritstic that if you provide an endpoint_override and an api_version and the endpoint you supply does not contain "v{api_version}" that we should attempt to do a discovery lookup - and if it fails just assume you gave us a versioned endpoint and that the api version you supplied is correct | 15:12 |
mordred | although I think we'd need a new option for that, since it would be a behavior change | 15:13 |
mordred | dtantsur: I agree with all of your uneasiness | 15:13 |
dtantsur | the practical question for me is whether to update openstacksdk or hack endpoint_override inside ironic | 15:15 |
*** bobh has joined #openstack-sdks | 15:15 | |
mordred | dtantsur: well - this logic is actually all in keystoneauth | 15:23 |
*** bobh has quit IRC | 15:27 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Return empty lists for resources if neutron doesn't exist https://review.opendev.org/669869 | 15:36 |
*** icarusfactor has joined #openstack-sdks | 16:04 | |
*** bobh has joined #openstack-sdks | 16:04 | |
*** factor has quit IRC | 16:06 | |
*** factor__ has joined #openstack-sdks | 16:06 | |
*** bobh has quit IRC | 16:06 | |
*** icarusfactor has quit IRC | 16:08 | |
*** altlogbot_0 has joined #openstack-sdks | 16:09 | |
*** altlogbot_0 has quit IRC | 16:13 | |
*** whoami-rajat has joined #openstack-sdks | 16:21 | |
*** jpich has quit IRC | 16:29 | |
*** factor__ has quit IRC | 16:29 | |
*** factor__ has joined #openstack-sdks | 16:30 | |
*** dtantsur is now known as dtantsur|afk | 16:34 | |
*** goldyfruit has quit IRC | 16:35 | |
*** bobh has joined #openstack-sdks | 16:38 | |
*** bobh has quit IRC | 16:38 | |
*** factor__ has quit IRC | 16:44 | |
*** ttsiouts has quit IRC | 16:48 | |
*** ttsiouts has joined #openstack-sdks | 16:48 | |
*** gtema has quit IRC | 16:52 | |
*** gtema has joined #openstack-sdks | 16:52 | |
*** ttsiouts has quit IRC | 16:53 | |
*** altlogbot_1 has joined #openstack-sdks | 17:01 | |
*** gtema has quit IRC | 17:03 | |
*** gtema has joined #openstack-sdks | 17:04 | |
*** altlogbot_1 has quit IRC | 17:05 | |
*** irclogbot_3 has joined #openstack-sdks | 17:11 | |
*** irclogbot_3 has quit IRC | 17:13 | |
*** ralonsoh has quit IRC | 17:36 | |
Shrews | mordred: gtema: fyi, looks like the openstacksdk-functional-devstack-magnum and openstacksdk-functional-devstack-masakari check jobs have been consistently failing since https://review.opendev.org/662743 | 17:54 |
Shrews | i was unable to find any earlier change with those failures (though it may exist) | 17:55 |
mordred | Shrews: oh - we should fix that then | 17:59 |
mordred | Shrews: looks like it's just failing the has_service Magnum check | 18:00 |
Shrews | i cant access the logs from this coffee shop. my vpn blocks logs.o.o for some reason | 18:00 |
mordred | maybe it's incidental and something changed in devstack | 18:00 |
Shrews | thats an odd check to fail in a magnum specific test :) | 18:01 |
mordred | Shrews: yah | 18:02 |
mordred | Shrews: also - I guess it's good that we have that check | 18:02 |
Shrews | ++ | 18:02 |
Shrews | heading back to the house. thought i could work from here just to get out of the house, but much too uncomfortable here. biab | 18:08 |
*** whoami-rajat has quit IRC | 18:30 | |
*** melwitt has joined #openstack-sdks | 18:34 | |
*** bobh has joined #openstack-sdks | 18:35 | |
*** bobh has quit IRC | 18:36 | |
*** irclogbot_3 has joined #openstack-sdks | 18:37 | |
*** irclogbot_2 has joined #openstack-sdks | 18:37 | |
*** irclogbot_2 has quit IRC | 18:39 | |
*** irclogbot_3 has quit IRC | 18:39 | |
*** factor has joined #openstack-sdks | 19:06 | |
*** goldyfruit has joined #openstack-sdks | 19:25 | |
*** ttsiouts has joined #openstack-sdks | 19:43 | |
*** Hazelesque has quit IRC | 19:49 | |
*** Hazelesque has joined #openstack-sdks | 19:49 | |
*** cdent has quit IRC | 19:57 | |
*** ttsiouts has quit IRC | 20:09 | |
*** ttsiouts has joined #openstack-sdks | 20:09 | |
*** ttsiouts has quit IRC | 20:14 | |
*** irclogbot_3 has joined #openstack-sdks | 20:25 | |
*** irclogbot_0 has joined #openstack-sdks | 20:25 | |
*** irclogbot_0 has quit IRC | 20:27 | |
*** irclogbot_3 has quit IRC | 20:27 | |
*** ttsiouts has joined #openstack-sdks | 20:53 | |
*** bobh has joined #openstack-sdks | 20:54 | |
*** bobh has quit IRC | 21:01 | |
*** ttsiouts has quit IRC | 21:04 | |
*** ttsiouts has joined #openstack-sdks | 21:04 | |
*** ttsiouts has quit IRC | 21:09 | |
*** irclogbot_3 has joined #openstack-sdks | 21:15 | |
*** irclogbot_1 has joined #openstack-sdks | 21:15 | |
*** irclogbot_3 has quit IRC | 21:19 | |
*** irclogbot_1 has quit IRC | 21:19 | |
*** goldyfruit has quit IRC | 21:20 | |
*** irclogbot_0 has joined #openstack-sdks | 21:39 | |
*** irclogbot_0 has quit IRC | 21:43 | |
*** irclogbot_2 has joined #openstack-sdks | 21:55 | |
*** irclogbot_1 has joined #openstack-sdks | 21:55 | |
*** irclogbot_2 has quit IRC | 21:59 | |
*** irclogbot_1 has quit IRC | 21:59 | |
*** tosky has quit IRC | 22:18 | |
*** goldyfruit has joined #openstack-sdks | 22:28 | |
*** altlogbot_0 has joined #openstack-sdks | 22:45 | |
*** altlogbot_0 has quit IRC | 22:49 | |
*** bobh has joined #openstack-sdks | 22:52 | |
*** slaweq has quit IRC | 22:54 | |
*** bobh has quit IRC | 22:57 | |
*** goldyfruit has quit IRC | 23:14 | |
*** bobh has joined #openstack-sdks | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!