Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-07-24
08:29:50 nightmare_unreal Thanks
12:05:26 dtantsur seeing something cryptic in bifrost, maybe someone has an idea
12:05:29 dtantsur "The baremetal service for : exists but does not have any supported versions."
12:05:41 dtantsur however, I don't see anyone hitting the API in ironic-api logs
12:06:04 dtantsur where could it go sideways? the patch I'm working on is related to authentication
12:06:11 dtantsur mordred: if you're around ^^
12:49:26 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: DNM nothing to see here https://review.opendev.org/742870
13:13:53 elmiko dtantsur: could it be a ghost?
13:14:00 elmiko XD
13:14:04 dtantsur also quite possible :)
13:14:07 elmiko lol
13:16:34 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: openstack CLI migration list support https://review.opendev.org/742210
13:16:50 dtantsur or maybe computers were a mistake?
13:20:24 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: DNM nothing to see here https://review.opendev.org/742870
13:56:37 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Add id and enabled param in ListIdentityProvider parser https://review.opendev.org/742886
13:58:22 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: DNM nothing to see here https://review.opendev.org/742870
14:25:22 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: DNM nothing to see here https://review.opendev.org/742870
15:48:07 openstackgerrit Dmitry Tantsur proposed openstack/ansible-collections-openstack master: Do not require ironic_url if cloud is provided https://review.opendev.org/742932
#openstack-sdks - 2020-07-25
14:01:51 openstackgerrit Merged openstack/openstacksdk master: Use unittest.mock instead of mock https://review.opendev.org/734455
#openstack-sdks - 2020-07-26
06:14:16 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add periodic jobs for collections https://review.opendev.org/742346
06:33:21 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Temporarly disable check-import https://review.opendev.org/743054
07:05:48 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Temporarly disable check-import https://review.opendev.org/743054
07:39:36 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Temporarly disable check-import https://review.opendev.org/743054
08:49:32 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Temporarly disable check-import https://review.opendev.org/743054
11:09:53 openstackgerrit Merged openstack/ansible-collections-openstack master: Temporarly disable check-import https://review.opendev.org/743054
11:21:57 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Do not require ironic_url if cloud is provided https://review.opendev.org/742932
11:22:07 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add periodic jobs for collections https://review.opendev.org/742346
11:22:25 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add support for setting the Flavor when creating a load balancer https://review.opendev.org/740948
12:32:50 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add periodic jobs for collections https://review.opendev.org/742346
15:04:52 openstackgerrit Merged openstack/ansible-collections-openstack master: Add periodic jobs for collections https://review.opendev.org/742346
16:41:20 openstackgerrit Merged openstack/ansible-collections-openstack master: Add support for setting the Flavor when creating a load balancer https://review.opendev.org/740948
#openstack-sdks - 2020-07-27
00:03:13 openstackgerrit Ghanshyam Mann proposed openstack/keystoneauth master: DNM: testing gate on ubuntu focal https://review.opendev.org/743118
09:37:44 openstackgerrit Dmitry Tantsur proposed openstack/ansible-collections-openstack master: Add non-voting bifrost jobs https://review.opendev.org/743176
09:40:24 openstackgerrit Dmitry Tantsur proposed openstack/ansible-collections-openstack master: DNM try breaking ironic https://review.opendev.org/743177
10:05:12 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: DNM nothing to see here https://review.opendev.org/742870
10:42:29 dtantsur hey folks, config.get_endpoint seems to misbehave in the SDK :(
10:42:42 dtantsur see the patch above, it fails on bifrost on the assertion error
10:45:43 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: DNM nothing to see here https://review.opendev.org/742870
10:47:05 openstackgerrit Dmitry Tantsur proposed openstack/ansible-collections-openstack master: Add non-voting bifrost jobs https://review.opendev.org/743176
11:06:19 dtantsur 'auth': {'username': '********', 'password': '********', 'endpoint': '********'}
11:06:29 dtantsur for god's sake, what on earth is replacing endpoint with stars? Oo
11:08:19 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: DNM nothing to see here https://review.opendev.org/742870
12:23:57 dtantsur mordred: hey! so, we use get_api_major_version in our service discovery, which does not work without endpoint_override, right?\
12:24:02 dtantsur which sounds.. wrong to me
12:29:26 mordred that sounds wrong to me too
12:30:06 mordred oh - I think it's possible that we set endpoint_override as part of proxy creation
12:30:30 mordred like - iirc, we do a round of discovery to find the endpoint and then create an adapter with that endpoint set via endpoint_override
12:30:43 mordred maybe
12:42:20 dtantsur well, I'm trying to use http_basic auth and get_api_major_version is broken in two palces
12:42:47 dtantsur first, get_endpoint_data always returns None if endpoint_override is None (ignoring self.endpoint)
12:42:54 dtantsur I have no clue how it works for no-auth now, it should not
12:43:13 mordred hrm
12:43:29 dtantsur seconds, this condition is hit for ironic https://opendev.org/openstack/keystoneauth/src/branch/master/keystoneauth1/discover.py#L1115-L1119
12:43:51 dtantsur again, no clue why it works for no-auth, but here it ends up with EndpointData populated with None
12:44:00 dtantsur because we don't run any discovery whatsoever
12:44:45 mordred well - you need to run some discovery to get microversion info - even with an endpoint_override
12:45:05 dtantsur even to get major version info
12:45:39 dtantsur killing the code in the 2nd link with fire and fixing http_basic.py to implement get_endpoint_data allows me to progress
12:45:46 mordred yeah. because otherwise how would it know
12:45:47 dtantsur but I wonder what I'm missing since it works for no-auth..
12:46:06 mordred it's a good question
12:47:06 dtantsur it HAS to hit all the same problems. I don't understand why it does not.
13:16:43 mordred dtantsur: I agree with you - I cannot see any difference between the two
13:19:08 KeithMnemonic is there anyone here he wolud be kind enough to do a few review on openstackclient that passed a while back but are stuck waiting on reviews? thank you https://review.opendev.org/#/q/project:openstack/python-openstackclient+owner:%22Keith+Berger+%253Ckeith.berger%2540suse.com%253E%22+status:open
13:19:08 dtantsur :(
13:19:37 dtantsur I'm close to giving up on this, honestly.. but that would be a big loss if we cannot use authentication in bifrost without keystone
13:21:09 mordred dtantsur: there's a couple of unit tests we have for noauth but not for httpbasic - let me see what adding similar ones for basic doe
13:21:17 dtantsur +++
13:22:01 mordred dtantsur: oh - when you get back ...
13:22:10 mordred no, nevermind
13:26:50 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Add unit test for httpbasicauth and discovery https://review.opendev.org/743227
13:27:13 mordred dtantsur: when you get back, can you point me to an example anywhere of this not working? unit test for the auth plugin calls get_api_major_version just fine on an http basic auth plugin
13:27:13 mordred dtantsur: remote: https://review.opendev.org/743227 Add unit test for httpbasicauth and discovery
13:35:49 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: openstack CLI migration list support https://review.opendev.org/742210
13:36:39 nightmare_unreal hello can someone review this patch and also leave a response to my comment regarding it. https://review.opendev.org/#/c/742210/
15:07:37 dtantsur mordred: if you have a spare VM, I can show you how to reproduce the bifrost failure
15:07:48 dtantsur I don't have a simple reproducer (which is another reason why I'm so frustrated)
15:12:25 dtantsur mordred: may I take over https://review.opendev.org/#/c/743227 and add a fix things there that might be a partial fix for my situation?
15:21:07 openstackgerrit Dmitry Tantsur proposed openstack/keystoneauth master: Fix get_endpoint_data for non-keystone plugins https://review.opendev.org/743227
15:21:20 dtantsur mordred: ^^^
15:26:49 mordred dtantsur: neat
15:27:21 mordred dtantsur: oh - that looks good
15:27:30 mordred dtantsur: still no clue why it works for one and not the other :)
15:28:03 mordred dtantsur: I had a thought - is it possible that ironic is http-auth protecting the discovery endpoit and ksa is just not sending auth when it talks to it?
15:28:33 mordred so ksa is trying to do discovery on the endpoint and failing?
15:28:49 mordred (which would explain why noauth works)
15:28:57 dtantsur mordred: checked that already
15:29:04 mordred nod
15:29:05 dtantsur curl works correctly
15:29:23 dtantsur (we had this problem with ironic-inspector, but we seem to be past it )
16:04:37 dtantsur AttributeError: 'EntryPoint' object has no attribute 'module_name'
16:04:46 dtantsur hasn't it been fixed? I see it in keystoneauth on `tox -edocs`
16:06:43 mordred I thougth so?
16:07:00 mordred I thought it was a version something something
16:07:58 mordred dtantsur: I agree - I see that same issue
16:08:10 dtantsur yep :(
16:08:17 mordred dtantsur: maybe we fixed it in sdk but not ksa
16:12:14 mordred dtantsur: fix coming
16:12:45 dtantsur nice! if you also guess why this bloody discovery does not work, you'll be my saviour today :)
16:13:18 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fix docs builds for modern stevedors https://review.opendev.org/743274

Earlier   Later