Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-04-12
11:50:32 openstackgerrit Jens Harbott (frickler) proposed openstack/keystoneauth master: DNM: Hard override admin interface to public https://review.openstack.org/652046
13:51:38 mordred frickler: ^^ wow, still chasing that issue?
13:57:20 frickler mordred: yes, I can't seem to find the right trigger. this is what from my understanding should fix it, but it is still failing both unit tests and running with devstack https://review.openstack.org/651790
13:57:37 frickler mordred: if you could help with digging into that, that would be great
14:17:53 openstackgerrit Ilya Shakhat proposed openstack/keystoneauth master: [WIP] Add OpenTracing instrumentation https://review.openstack.org/652080
14:18:26 openstackgerrit Ilya Shakhat proposed openstack/osc-lib master: [WIP] Add OpenTracing instrumentation https://review.openstack.org/652081
14:19:22 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add logger to Proxy object https://review.openstack.org/650901
14:19:23 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Support microversion 2.61 for nova flavors https://review.openstack.org/650902
14:19:23 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.openstack.org/650903
14:22:39 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Update baremetal to use proxy logger https://review.openstack.org/652083
14:22:59 mordred dtantsur: ^^ I did the followup baremetal logger patch
14:42:46 frickler mordred: so hard overriding any access to the admin endpoint to public instead works (mostly, I think the heat errors are either unrelated or some obscure special case) http://logs.openstack.org/46/652046/1/check/openstacksdk-functional-devstack-tips/b8a0433/
14:42:59 frickler now I wish I could find a sane way to achieve the same thing
14:46:13 mordred frickler: yeah - I'd love to know what's overriding the endpoint type to be admin
14:47:26 dtantsur thanks mordred
14:49:02 frickler mordred: the thing that mostly confuses me: when I set interface='blah' here instead of 'public', I get "endpoint blah not found" https://review.openstack.org/#/c/651790/3/keystonemiddleware/auth_token/__init__.py
14:49:34 frickler but with interface='public', why does it still try to find an admin endpoint?
14:57:48 mordred frickler: I'm stumped
14:57:54 mordred frickler: maybe kmalloc knows. he knows lots of things
14:58:52 openstackgerrit Merged openstack/openstacksdk master: Add floating IP port forwarding related methods https://review.openstack.org/650063
14:59:22 kmalloc I don't know. It very well might be a bug.
14:59:42 kmalloc It looks like it should just work.
15:02:10 kmalloc This seems like one of those default.behavior things we do. I can looks deeper post coffee...
15:02:40 kmalloc I am still too asleep and on a phone, too hard to trace code this way.
15:09:34 mordred kmalloc: I recommend coffee
15:10:13 frickler to be fair, that behaviour is at least documented, https://docs.openstack.org/keystonemiddleware/latest/middlewarearchitecture.html says "If using an auth_plugin, connection to the Identity service will be established on the admin interface as registered in the service catalog."
15:15:06 mordred yeah ... but I don't see _where_ it's doing it
15:31:54 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
15:41:18 kmalloc Am coffee-ing now
15:42:42 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Add "name" filter in "list" call when retrieving a single register https://review.openstack.org/637238
15:45:02 kmalloc You can also run the admin endpoint the exact same.as the public one in the catalog
15:45:14 kmalloc Point to the same thing. It is a silly work around for now.
17:02:57 kmalloc i don't see why we would ever automatically only use admin endpoint :P
17:07:09 mordred kmalloc: yeah. I cannot find any locations in the code where we should be setting admin if public is set
17:10:31 kmalloc I've now chased through most of KSA and most of KSM
17:10:36 kmalloc this is very weird.
17:23:02 openstackgerrit Merged openstack/openstacksdk master: Add logger to Proxy object https://review.openstack.org/650901
17:29:09 openstackgerrit Merged openstack/keystoneauth master: Update master for stable/stein https://review.openstack.org/643996
18:26:15 openstackgerrit Merged openstack/openstacksdk master: Support microversion 2.61 for nova flavors https://review.openstack.org/650902
19:11:46 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Tweak network segment range fiunction tests https://review.openstack.org/652129
19:50:56 kmalloc https://review.openstack.org/#/c/651790/4/keystonemiddleware/auth_token/_identity.py see the change here for the admin interface
19:51:01 kmalloc go go keystoneclient sucking
19:51:05 kmalloc it was KSC's default
#openstack-sdks - 2019-04-13
03:15:49 openstackgerrit Merged openstack/python-openstackclient master: Before writing object data to stdout, re-open it in binary mode https://review.openstack.org/573465
#openstack-sdks - 2019-04-15
08:59:05 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: WIP Skip attaching FloatingIP if it is already attached https://review.openstack.org/652594
09:50:10 cdent dtantsur: I kicked that thing
09:50:25 dtantsur thx!
09:59:56 openstackgerrit Merged openstack/api-sig master: Repair building guidelines https://review.openstack.org/651260
11:23:18 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Skip attaching FloatingIP if it is already attached https://review.openstack.org/652594
12:34:28 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Skip attaching FloatingIP if it is already attached https://review.openstack.org/652594
12:36:53 gtema mordred, dtantsur: would appreciate a look at https://review.openstack.org/#/c/651534/. Not a small change, but important one
12:40:13 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Try to fix the masakari CI job https://review.openstack.org/652638
12:40:19 dtantsur gtema: looking
13:19:00 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
13:35:53 brtknr Any idea when 0.28.0 will be shipped?
13:36:22 brtknr I'm still waiting for the heat tag update patch to be available in the SDK :)
13:41:30 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
14:09:34 dtantsur brtknr: now that we have Stein out, I guess mordred can request a release
14:10:05 mordred yes. I'll get that going today
14:10:27 brtknr mordred: dtantsur: excellent, thank you
14:10:55 gtema and I wanted to have https://review.openstack.org/#/c/651534 to come in first ;-(
14:17:02 mordred gtema: I took one look, but will take another in a couple of hours after the morning meetings
14:21:48 dtantsur mordred, https://review.openstack.org/#/c/652638/
14:22:47 mordred dtantsur: ah neat!
14:22:56 mordred dtantsur: sadly the bifrost job had a sad :)
14:23:47 mordred dtantsur: I hit a recheck - looks great to me though
14:57:07 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
14:57:51 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
15:28:49 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
17:06:49 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
17:09:25 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
18:19:41 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
18:22:21 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
21:56:42 openstackgerrit Merged openstack/python-openstackclient master: Tweak network segment range fiunction tests https://review.openstack.org/652129
#openstack-sdks - 2019-04-16
07:30:30 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
09:21:52 openstackgerrit LIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands https://review.openstack.org/650062
10:31:39 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Continue refactoring of the image https://review.openstack.org/651534
10:59:00 openstackgerrit Merged openstack/openstacksdk master: Try to fix the masakari CI job https://review.openstack.org/652638
11:42:43 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add image.stage methods https://review.openstack.org/652981
13:38:29 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Return None from get_server_by_id on 404 https://review.openstack.org/652995
13:41:03 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Return None from get_server_by_id on 404 https://review.openstack.org/652995
13:45:10 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
13:45:35 openstackgerrit Jeremy Houser proposed openstack/python-openstackclient master: Alter test_volume.py to ensure volume deletion https://review.openstack.org/652682
14:49:06 mordred gtema: left a review on the image patches - I think they're close
14:49:59 gtema yes, thanks. Few issues:
14:50:35 gtema auto_disk_config - I think it would be better to drop this attr from image resource and rely on the "properties" interpretation of it
14:51:03 gtema drop compute for list - ok, will do
14:52:02 gtema switch to wait_for_task - it's this fallback for _IMAGE_ERROR_396 which makes it not easy
14:52:39 mordred gtema: auto_disk_config - yeah
14:52:51 mordred gtema: or else we should add ALL of the driver-specific properties
14:53:19 gtema which we do not want to
14:53:53 mordred gtema: yeah. and unfortunately _IMAGE_ERROR_396 is a wonderful thing that rackspace does throw. like - that chunk of logic exists because of issues production nodepool sees without it
14:54:09 mordred gtema: totally. there's too many of them - it would make the image objects absurd
14:54:43 gtema morderd: so should I then drop all hw*, vmware*?
14:56:27 mordred gtema: ugh. we do have a lot of these already don't we
14:56:33 gtema yupp
14:56:46 mordred gtema: maybe it's better that we just add the rest of them? I wish they were documented somewhere that isn't a wiki page
14:56:50 mordred like, you know - in the API docs
14:57:00 mordred lemme ask rosmaita
14:57:04 gtema ok

Earlier   Later