Wednesday, 2019-04-03

openstackgerritayoung proposed openstack/keystone master: Replace UUID with id_generator for Federated users  https://review.openstack.org/60516900:06
openstackgerritMerged openstack/keystone master: DRY: Remove redundant policies from policy.v3cloudsample.json  https://review.openstack.org/62258900:16
cmurphyokay now https://review.openstack.org/649344 is ready kmalloc lbragstad00:17
*** jamesmcarthur has quit IRC00:24
*** raildo has quit IRC00:28
*** raildo has joined #openstack-keystone00:35
lbragstadcmurphy done - pending you're cool with a self-approval00:35
lbragstadrc2 is so close i can taste it00:35
*** jamesmcarthur has joined #openstack-keystone00:55
*** jamesmcarthur has quit IRC00:55
*** jamesmcarthur has joined #openstack-keystone00:56
*** lbragstad_ has joined #openstack-keystone01:10
*** ChanServ sets mode: +o lbragstad_01:10
*** lbragstad has quit IRC01:12
*** phasespace has quit IRC01:12
*** kklimonda has quit IRC01:12
*** spotz has quit IRC01:12
*** kklimonda has joined #openstack-keystone01:12
*** erus has quit IRC01:40
*** erus has joined #openstack-keystone01:41
*** shyamb has joined #openstack-keystone01:55
*** whoami-rajat has joined #openstack-keystone01:57
*** raildo has quit IRC02:06
*** spsurya has joined #openstack-keystone02:12
*** jamesmcarthur has quit IRC02:31
*** jamesmcarthur has joined #openstack-keystone03:01
*** lbragstad_ is now known as lbragstad03:14
*** shyamb has quit IRC03:26
*** shyamb has joined #openstack-keystone03:41
*** shyamb has quit IRC03:46
*** whoami-rajat has quit IRC04:57
*** shyamb has joined #openstack-keystone05:11
*** shyamb has quit IRC05:16
*** shyamb has joined #openstack-keystone05:26
*** lbragstad has quit IRC05:28
*** vishakha has joined #openstack-keystone05:42
*** shyamb has quit IRC06:05
*** shyamb has joined #openstack-keystone06:16
*** shyamb has quit IRC06:18
*** shyamb has joined #openstack-keystone06:18
*** pcaruana has joined #openstack-keystone06:36
*** pcaruana has quit IRC06:38
*** pcaruana has joined #openstack-keystone06:38
*** phasespace has joined #openstack-keystone06:52
*** tkajinam has quit IRC06:58
*** tkajinam has joined #openstack-keystone07:03
*** shyamb has quit IRC07:05
*** shyamb has joined #openstack-keystone07:05
*** whoami-rajat has joined #openstack-keystone07:13
*** awalende has joined #openstack-keystone07:13
*** bbobrov has quit IRC07:17
*** bbobrov has joined #openstack-keystone07:17
*** xek has joined #openstack-keystone07:19
*** jistr is now known as jistr|afk07:22
*** shyamb has quit IRC08:26
*** tkajinam has quit IRC08:45
*** shyamb has joined #openstack-keystone09:06
*** xek has quit IRC09:20
*** xek has joined #openstack-keystone09:21
*** xek has quit IRC09:22
*** whoami-rajat has quit IRC09:37
*** markvoelker has joined #openstack-keystone09:55
*** shyamb has quit IRC09:56
*** shyamb has joined #openstack-keystone10:00
*** erus has quit IRC10:11
*** erus has joined #openstack-keystone10:11
*** shyamb has quit IRC10:23
*** shyamb has joined #openstack-keystone10:23
*** markvoelker has quit IRC10:27
*** Dinesh_Bhor has quit IRC10:34
*** shyamb has quit IRC10:46
*** mvkr has quit IRC11:01
*** jistr|afk is now known as jistr11:02
*** hoonetorg has joined #openstack-keystone11:16
*** markvoelker has joined #openstack-keystone11:24
*** erus has quit IRC11:24
*** erus has joined #openstack-keystone11:25
*** erus has quit IRC11:30
*** erus has joined #openstack-keystone11:31
*** rcernin has quit IRC11:38
*** raildo has joined #openstack-keystone11:38
cmurphyhttps://review.openstack.org/64958911:45
*** yan0s has joined #openstack-keystone11:52
*** mvkr has joined #openstack-keystone11:54
*** markvoelker has quit IRC11:57
*** shyamb has joined #openstack-keystone12:02
*** odyssey4me has quit IRC12:12
*** whoami-rajat has joined #openstack-keystone12:19
*** odyssey4me has joined #openstack-keystone12:26
*** jamesmcarthur has quit IRC12:26
*** jamesmcarthur has joined #openstack-keystone12:27
*** shyamb has quit IRC12:33
*** shyamb has joined #openstack-keystone12:34
*** jamesmcarthur has quit IRC12:36
*** jamesmcarthur_ has joined #openstack-keystone12:43
*** lbragstad has joined #openstack-keystone12:56
*** ChanServ sets mode: +o lbragstad12:56
*** jmlowe has quit IRC12:57
*** shyamb has quit IRC13:10
openstackgerritLance Bragstad proposed openstack/oslo.policy master: Clarify policy_file configuration option help text  https://review.openstack.org/64960613:12
*** vishakha has quit IRC13:21
*** dklyle has quit IRC13:30
*** dklyle has joined #openstack-keystone13:31
*** jmlowe has joined #openstack-keystone13:49
*** awalende has quit IRC14:03
*** awalende has joined #openstack-keystone14:03
*** awalende_ has joined #openstack-keystone14:07
*** awalende has quit IRC14:08
*** awalende_ has quit IRC14:11
*** dklyle has quit IRC14:15
gagehugoo/14:32
cmurphy\o14:32
lbragstado/14:34
* lbragstad isn't quite sure what the deal is with the rc2 release patch14:34
cmurphysphinx 2.0 release14:35
cmurphythey're working on it14:35
*** awalende has joined #openstack-keystone14:35
*** awalende has quit IRC14:39
*** dklyle has joined #openstack-keystone14:41
lbragstadah14:41
*** erus has quit IRC15:08
*** erus has joined #openstack-keystone15:09
*** phasespace has quit IRC15:12
*** erus has quit IRC15:24
*** erus has joined #openstack-keystone15:25
*** mvkr has quit IRC15:28
bnemeckmalloc: Did we need https://review.openstack.org/#/c/644774/ for stein?15:32
bnemecI guess that's probably backportable so if it misses the final release it's not the end of the world.15:33
kmallocbnemec: backport it after s releases15:33
kmallocway easier at this point15:33
bnemeckmalloc: Sounds good, thanks.15:33
*** erus has quit IRC15:37
*** erus has joined #openstack-keystone15:37
*** shyamb has joined #openstack-keystone15:44
*** erus has quit IRC15:44
*** erus has joined #openstack-keystone15:45
*** jamesmcarthur_ has quit IRC15:53
*** phasespace has joined #openstack-keystone15:58
*** yan0s has quit IRC16:09
*** jamesmcarthur has joined #openstack-keystone16:12
*** jamesmcarthur has quit IRC16:15
*** jamesmcarthur has joined #openstack-keystone16:23
knikollao/16:32
*** erus has quit IRC16:32
*** erus has joined #openstack-keystone16:33
*** spsurya has quit IRC16:46
*** jamesmcarthur has quit IRC16:52
*** shyamb has quit IRC16:53
*** jamesmcarthur has joined #openstack-keystone16:56
openstackgerritMerged openstack/oslo.policy master: Clarify policy_file configuration option help text  https://review.openstack.org/64960617:17
ayoungcmurphy, I've found a couple other places where the Federation tests have failed.17:21
ayoungI suspect that the CADF code is being overly sensitive when creating a notification17:21
cmurphyayoung: have a link?17:25
ayoungAND...the log is gone here is the code that fails in keystone http://git.openstack.org/cgit/openstack/keystone/tree/keystone/notifications.py#n72317:27
*** spsurya has joined #openstack-keystone17:28
ayoungIt calls https://git.openstack.org/cgit/openstack/pycadf/tree/pycadf/credential.py#n8717:29
ayoung # FederatedCredential.user17:29
ayoung        setattr(self, FED_CRED_KEYNAME_USER, user)17:29
ayoungSo username, not ID17:29
ayoungAh, but that is a key, bet it is a whole dictionary17:29
ayoungAnd here is the exception raised https://git.openstack.org/cgit/openstack/pycadf/tree/pycadf/cadftype.py#n6617:30
ayoungself.func(value)  is the lambda17:31
ayoungWhere is stevemar when you need him?  He did all the cadf work IIRC17:34
*** jamesmcarthur has quit IRC17:35
ayoungcmurphy, looks like the lambda is here https://git.openstack.org/cgit/openstack/pycadf/tree/pycadf/credential.py#n3917:37
ayoungso, it is not coming across as a string type?17:37
cmurphyreally strange17:44
ayoungcmurphy, got a paste for you...1 sec17:45
cmurphyRBAC: auth_context: {'token': <TokenModel (audit_id=egOx1wPnSwezUpnI57-XYA, audit_chain_id=['egOx1wPnSwezUpnI57-XYA']) at 0x7f2839645080>, 'domain_id': None, 'trust_id': None, 'trustor_id': None, 'trustee_id': None, 'domain_name': None, 'group_ids': [], 'user_id': '18fafa79e50f4d06b04364624730f378', 'user_domain_id': 'default', 'system_scope': None, 'project_id':17:45
cmurphy'485249bc0c6e4f788494f2952f7e5d6f', 'project_domain_id': 'default', 'roles': ['member', 'reader', 'admin'], 'is_admin_project': True, 'service_user_id': None, 'service_user_domain_id': None, 'service_project_id': None, 'service_project_domain_id': None, 'service_roles': []} {{(pid=15674) fill_context /opt/stack/keystone/keystone/server/flask/request_processing/middleware/auth_context.py:472}}17:45
cmurphyuser_id looks like it should be fine there17:45
ayounghttp://paste.openstack.org/show/748826/17:45
ayounghttps://docs.python.org/3/library/hashlib.html17:48
ayounghash.hexdigest()17:48
ayoung    Like digest() except the digest is returned as a string object of double length, containing only hexadecimal digits. This may be used to exchange the value safely in email or other non-binary environments.17:48
ayounghttps://git.openstack.org/cgit/openstack/pycadf/tree/pycadf/credential.py#n71   that is where the lambda is set, and it is still just looking for a string17:50
ayoungI'm tempted to put in some logging into the patch and see if we can get a smoking gun.17:52
ayoungMaybe the hash is returning None for some reason17:52
*** jmlowe has quit IRC17:52
*** jmlowe has joined #openstack-keystone17:55
*** erus has quit IRC18:00
*** erus has joined #openstack-keystone18:01
cmurphyayoung: what version of python3 did you use for http://paste.openstack.org/show/748826/ ? i wonder if hexdigest() returns something that's not quite a string on python3.618:07
ayoung37...native to Fedora.  Let me try 3.618:07
cmurphyseems like older versions of py3 had a slightly different behavior https://stackoverflow.com/questions/3583265/compare-result-from-hexdigest-to-a-string18:08
ayoungpy36 seems to produce the same thing18:10
*** lbragstad has quit IRC18:11
cmurphyhmm18:12
*** lbragstad has joined #openstack-keystone18:12
*** ChanServ sets mode: +o lbragstad18:12
*** jamesmcarthur has joined #openstack-keystone18:17
*** jamesmcarthur has quit IRC18:21
*** samueldmq has joined #openstack-keystone18:25
*** erus has quit IRC18:25
*** erus has joined #openstack-keystone18:26
*** jamesmcarthur has joined #openstack-keystone18:31
*** jmlowe has quit IRC18:57
ayoungcmurphy, what do you want me to do about the change?19:05
ayoungI can add a test that ensures the id generator value is a string19:06
ayoungOr maybe it should that the value generated that way passes pyCadf?19:06
cmurphyayoung: are there other changes where this bug is appearing? if so then we can go ahead with your change and file a bug for the issue, if not then something in your patch needs to be fixed19:09
cmurphyi haven't seen this bug appear anywhere else so far19:10
ayounghow do we confirm it is a bug?19:12
cmurphyi've been trying to trigger it with https://review.openstack.org/64945319:14
*** erus has quit IRC19:14
cmurphyof course zuul is not super speedy this week19:14
*** erus has joined #openstack-keystone19:15
cmurphyi'll deploy your change locally and see if i can trigger it19:15
*** erus has quit IRC19:20
*** erus has joined #openstack-keystone19:21
*** jmlowe has joined #openstack-keystone19:48
*** jamesmcarthur has quit IRC19:55
*** awalende has joined #openstack-keystone20:20
*** whoami-rajat has quit IRC20:29
*** jamesmcarthur has joined #openstack-keystone20:33
*** jamesmcarthur has quit IRC20:39
mnaserhi friends.  how much stuff in rc2 were ~really~ critical vs rc1 (aka, is there some sort of 'list of bugs we caught in rc2' ?)20:40
mnaserjust to know if we need to quickly leap onto rc220:40
*** jamesmcarthur has joined #openstack-keystone20:41
lbragstadi don't think there was anything super critical20:42
lbragstadhttps://launchpad.net/keystone/+milestone/stein-rc2 are the specific bugs that were included in rc220:42
lbragstadlot of scope types work20:43
lbragstadsome performance stuff20:43
*** rcernin has joined #openstack-keystone20:44
*** spsurya has quit IRC20:59
cmurphyayoung: well i can't reproduce it at all21:05
cmurphytotally clueless21:05
*** pcaruana has quit IRC21:06
openstackgerritayoung proposed openstack/keystone master: Replace UUID with id_generator for Federated users  https://review.openstack.org/60516921:13
*** rcernin has quit IRC21:42
*** raildo has quit IRC21:45
openstackgerritayoung proposed openstack/keystone master: Test case for id Generator  https://review.openstack.org/64974721:47
ayoungcmurphy, ^^  that is the start for testing.21:48
*** awalende has quit IRC21:59
*** jamesmcarthur has quit IRC22:14
*** jamesmcarthur has joined #openstack-keystone22:32
*** jamesmcarthur has quit IRC22:32
*** lbragstad has quit IRC22:49
*** lbragstad has joined #openstack-keystone22:50
*** ChanServ sets mode: +o lbragstad22:50
*** tkajinam has joined #openstack-keystone23:04
*** erus has quit IRC23:10
*** erus has joined #openstack-keystone23:11
*** ayoung has quit IRC23:18

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!