On Tue, Jan 24, 2017 at 11:44 AM, Pierre-Yves Chibon <pingou@pingoured.fr> wrote:On Tue, Jan 24, 2017 at 11:30:33AM +0100, Michal Novotny wrote:
> Yes, untested changes got into production. We are sorry. I am currently
> working on a fix.
If that made it up to production, then we'll need to adjust fedmsg_meta to
support these messages as well.
fedmsg_meta should work for all our published messages (published and stored on
datagrepper).Okay, these messages have all empty body (the "msg" attribute), e.g `build.end`:{ "source_name": "datanommer", "certificate": "Does fedmsg_meta need to be adjusted even so?LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS , "i": 3, "timestamp": 1485166967.0, "msg_id": "2017-db23086c-2001-496e-a59f-0tLS0tCk1JSUVUakNDQTdlZ0F3SUJB Z0lDQVBZd0RRWUpL\ nb1pJaHZjTkFRRUZCUUF3Z2FBeEN6Q UpCZ05WQkFZVEFsVlQKTVFzd0NRWUR WUVFJRXdKT1F6RVFN\ nQTRHQTFVRUJ4TUhVbUZzWldsbmFER VhNQlVHQTFVRUNoTU9SbVZrYjNKaAp JRkJ5YjJwbFkzUXhE\ nekFOQmdOVkJBc1RCbVpsWkcxelp6R VBNQTBHQTFVRUF4TUdabVZrYlhObk1 ROHdEUVlEClZRUXBF\ nd1ptWldSdGMyY3hKakFrQmdrcWhra Uc5dzBCQ1FFV0YyRmtiV2x1UUdabFp HOXlZWEJ5YjJwbFkz\ nUXUKYjNKbk1CNFhEVEUwTURReU16R TBNamsxTVZvWERUSTBNRFF5TURFME1 qazFNVm93Z2R3eEN6\ nQUpCZ05WQkFZVApBbFZUTVFzd0NRW URWUVFJRXdKT1F6RVFNQTRHQTFVRUJ 4TUhVbUZzWldsbmFE\ nRVhNQlVHQTFVRUNoTU9SbVZrCmIzS mhJRkJ5YjJwbFkzUXhEekFOQmdOVkJ Bc1RCbVpsWkcxelp6\ nRXRNQ3NHQTFVRUF4TWtZMjl3Y2kxa mIzQnkKTFdKbExtTnNiM1ZrTG1abFp HOXlZWEJ5YjJwbFkz\ nUXViM0puTVMwd0t3WURWUVFwRXlSa mIzQnlMV052Y0hJdApZbVV1WTJ4dmR XUXVabVZrYjNKaGNI\ nSnZhbVZqZEM1dmNtY3hKakFrQmdrc WhraUc5dzBCQ1FFV0YyRmtiV2x1ClF HWmxaRzl5WVhCeWIy\ ncGxZM1F1YjNKbk1JR2ZNQTBHQ1NxR 1NJYjNEUUVCQVFVQUE0R05BRENCaVF LQmdRQ2UKREs5VFQy\ nM05BdTZPWTVGMnVVNHpMRW9Ld2k1R nRRTU5jVWV5eDdmOHJxMUZXaUxDWHB jWFhpU2tzUE1XV1NM\ nWQo5SHNoa1pvM3ZjMHFSRXVBWDNwe WRuM2VFRDA0UExrUmRlaWpvSXA5L0Y 2YlZ3MmlLMDdXRmc5\ nU2MwNlRsKzhSCld1RHNaeTQ1SVJKY XhCRTlJaHBYL0x2Y2JnQ1cvZmVHVGp 5WG1iRHd0UUlEQVFB\ nQm80SUJWekNDQVZNd0NRWUQKVlIwV EJBSXdBREF0QmdsZ2hrZ0JodmhDQVE wRUlCWWVSV0Z6ZVMx\ nU1UwRWdSMlZ1WlhKaGRHVmtJRU5sY 25ScApabWxqWVhSbE1CMEdBMVVkRGd RV0JCUm5lNTg0d3Bs\ nWGYrZVE2K25zSTZCbm5BNENaRENCM VFZRFZSMGpCSUhOCk1JSEtnQlJyUUZ yNUVnaUpXZWRaNVFY\ nMUFoMEtUbjhVQUtHQnBxU0JvekNCb 0RFTE1Ba0dBMVVFQmhNQ1ZWTXgKQ3p BSkJnTlZCQWdUQWs1\ nRE1SQXdEZ1lEVlFRSEV3ZFNZV3hsY Vdkb01SY3dGUVlEVlFRS0V3NUdaV1J 2Y21FZwpVSEp2YW1W\ namRERVBNQTBHQTFVRUN4TUdabVZrY lhObk1ROHdEUVlEVlFRREV3Wm1aV1J 0YzJjeER6QU5CZ05W\ nCkJDa1RCbVpsWkcxelp6RW1NQ1FHQ 1NxR1NJYjNEUUVKQVJZWFlXUnRhVzV BWm1Wa2IzSmhjSEp2\ nYW1WamRDNXYKY21lQ0NRRGpVQjVIV HhjZVJUQVRCZ05WSFNVRUREQUtCZ2d yQmdFRkJRY0RBakFM\ nQmdOVkhROEVCQU1DQjRBdwpEUVlKS 29aSWh2Y05BUUVGQlFBRGdZRUFVazN lbjBYUXpDQm5IUlh4\ nZDhyOHp2ZFAwVURvbEpiUysyTEl3Z 3NDClJDMnNkZ1UwNGdFblYxdFpVTjN ydEk1SzQ2MnpKT0JQ\ nOFhQd3h4eUZMN1lOYmVtWTgyTG52Y 1pHdzliMGdxTDMKdHNKbzllSFV5SXB ZMG93TlVKdzgzU1Ax\ neFJvb3NwVGJRK3BsNm9qdjVPNVpGZ 1lBUG1yckRWZ0M4a2gzRlp4Rgp0SWc 9Ci0tLS0tRU5EIENF\ nUlRJRklDQVRFLS0tLS0K\n" 10b92c466ae5" , "topic": "org.fedoraproject.prod.copr.build.end" , "source_version": "0.6.5", "signature": "HJEOhP93vZFUk3cjIzggxZqIT+IRaLpKF/t21Kn0AcQ9B1VJEe+ , "msg": {} }myerAAJMZfuXppGQqsFyzcPFx\nu+ 9p7geI5NqNxnN+diUXNlxbXN9/ VN0X3vX7U4mbc0/zLyGcbKWIn/ pcskbM5qYC2lJHLov0pMwq\nrbq/ B0N3CxBL2og0Fj8=\n" clime
Pierre
> On Tue, Jan 24, 2017 at 11:02 AM, Pierre-Yves Chibon <pingou@pingoured.fr>
> wrote:
>
> Hi all,
>
> We have recently been receiving emails about wrongly formatted fedmsg
> message
> coming from copr in stg.
> Has there been any changes made to the structure of the messages sent in
> stg?
>
> To give you an idea, this is what we received:
> Traceback (most recent call last):
> Â File "/usr/lib/python2.7/site-packages/moksha/hub/api/consumer. py",
> line 191, in _work
> Â Â self.consume(message)
> Â File "/usr/lib/python2.7/site-packages/fmn/consumer/consumer.py",
> line 89, in consume
> Â Â self.work(session, raw_msg)
> Â File "/usr/lib/python2.7/site-packages/fmn/consumer/consumer.py",
> line 105, in work
> Â Â msg['msg']['owner'] in self.ignored_copr_owners:
> KeyError: 'owner'
>
> The message being:
> {'body': {u'username': u'copr', u'certificate':
> u'LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUVUakNDQTdlZ0F3SU JBZ0lDQVBZd0RRWUpL\nb1pJaHZjTk FRRUZCUUF3Z2FBeEN6QUpCZ05WQkFZ VEFsVlQKTVFzd0NRWURWUVFJRXdKT1 F6RVFN\nQTRHQTFVRUJ4TUhVbUZzWl dsbmFERVhNQlVHQTFVRUNoTU9SbVZr YjNKaApJRkJ5YjJwbFkzUXhE\nekFO QmdOVkJBc1RCbVpsWkcxelp6RVBNQT BHQTFVRUF4TUdabVZrYlhObk1ROHdE UVlEClZRUXBF\nd1ptWldSdGMyY3hK akFrQmdrcWhraUc5dzBCQ1FFV0YyRm tiV2x1UUdabFpHOXlZWEJ5YjJwbFkz \nUXUKYjNKbk1CNFhEVEUwTURReU16 RTBNamsxTVZvWERUSTBNRFF5TURFME 1qazFNVm93Z2R3eEN6\nQUpCZ05WQk FZVApBbFZUTVFzd0NRWURWUVFJRXdK T1F6RVFNQTRHQTFVRUJ4TUhVbUZzWl dsbmFE\nRVhNQlVHQTFVRUNoTU9SbV ZrCmIzSmhJRkJ5YjJwbFkzUXhEekFO QmdOVkJBc1RCbVpsWkcxelp6\nRXRN Q3NHQTFVRUF4TWtZMjl3Y2kxamIzQn kKTFdKbExtTnNiM1ZrTG1abFpHOXlZ WEJ5YjJwbFkz\nUXViM0puTVMwd0t3 WURWUVFwRXlSamIzQnlMV052Y0hJdA pZbVV1WTJ4dmRXUXVabVZrYjNKaGNI \nSnZhbVZqZEM1dmNtY3hKakFrQmdr cWhraUc5dzBCQ1FFV0YyRmtiV2x1Cl FHWmxaRzl5WVhCeWIy\ncGxZM1F1Yj NKbk1JR2ZNQTBHQ1NxR1NJYjNEUUVC QVFVQUE0R05BRENCaVFLQmdRQ2UKRE s5VFQy\nM05BdTZPWTVGMnVVNHpMRW 9Ld2k1RnRRTU5jVWV5eDdmOHJxMUZX aUxDWHBjWFhpU2tzUE1XV1NM\nWQ> o5SHNoa1pvM3ZjMHFSRXVBWDNweWRu M2VFRDA0UExrUmRlaWpvSXA5L0Y2Yl Z3MmlLMDdXRmc5\nU2MwNlRsKzhSCl d1RHNaeTQ1SVJKYXhCRTlJaHBYL0x2 Y2JnQ1cvZmVHVGp5WG1iRHd0UUlEQV FB\nQm80SUJWekNDQVZNd0NRWUQKVl IwVEJBSXdBREF0QmdsZ2hrZ0JodmhD QVEwRUlCWWVSV0Z6ZVMx\nU1UwRWdS MlZ1WlhKaGRHVmtJRU5sY25ScApabW xqWVhSbE1CMEdBMVVkRGdRV0JCUm5l NTg0d3Bs\nWGYrZVE2K25zSTZCbm5B NENaRENCMVFZRFZSMGpCSUhOCk1JSE tnQlJyUUZyNUVnaUpXZWRaNVFY\nMU FoMEtUbjhVQUtHQnBxU0JvekNCb0RF TE1Ba0dBMVVFQmhNQ1ZWTXgKQ3pBSk JnTlZCQWdUQWs1\nRE1SQXdEZ1lEVl FRSEV3ZFNZV3hsYVdkb01SY3dGUVlE VlFRS0V3NUdaV1J2Y21FZwpVSEp2YW 1W\namRERVBNQTBHQTFVRUN4TUdabV ZrYlhObk1ROHdEUVlEVlFRREV3Wm1a V1J0YzJjeER6QU5CZ05W\nCkJDa1RC bVpsWkcxelp6RW1NQ1FHQ1NxR1NJYj NEUUVKQVJZWFlXUnRhVzVBWm1Wa2Iz SmhjSEp2\nYW1WamRDNXYKY21lQ0NR RGpVQjVIVHhjZVJUQVRCZ05WSFNVRU REQUtCZ2dyQmdFRkJRY0RBakFM\nQm dOVkhROEVCQU1DQjRBdwpEUVlKS29a SWh2Y05BUUVGQlFBRGdZRUFVazNlbj BYUXpDQm5IUlh4\nZDhyOHp2ZFAwVU RvbEpiUysyTEl3Z3NDClJDMnNkZ1Uw NGdFblYxdFpVTjNydEk1SzQ2MnpKT0 JQ\nOFhQd3h4eUZMN1lOYmVtWTgyTG 52Y1pHdzliMGdxTDMKdHNKbzllSFV5 SXBZMG93TlVKdzgzU1Ax\neFJvb3Nw VGJ> RK3BsNm9qdjVPNVpGZ1lBUG1yckRWZ 0M4a2gzRlp4Rgp0SWc9Ci0tLS0tRU5 EIENF\nUlRJRklDQVRFLS0tLS0K\n' ,
> u'i': 3, u'timestamp': 1485209011,
> u'msg_id':u'2017-dc6af56b-76c4-425d-8b66-f69295b38fe0', u'crypto':
> u'x509', u'topic':
> u'org.fedoraproject.prod.copr.build.end',u'signature':u'AJHl FzJnrrTF9XVs6b0KT4U0n3LCVtK/ cWbt+hceOeWf/0bvYn6ZMx6dUTBasu tGnA8TY998U6H6\ nzgBXPM1fyV4WCO+ekuIV7O6dpNedl fpcrnnU1LRGCPWVPSgIw0gg5QDfZ06 kQWuQvC17JxokG7Pi\ n4vIiW64ifAddRZEILxc=\n',
> u'msg': {}}, 'topic':u'org.fedoraproject.prod.copr.build.end'}
>
> Does that ring a bell to anyone?
>
> Thanks,
> Pierre
> _______________________________________________
> copr-devel mailing list -- copr-devel@lists.fedorahosted.org
> To unsubscribe send an email to copr-devel-leave@lists.fedorahosted.org
> _______________________________________________
> copr-devel mailing list -- copr-devel@lists.fedorahosted.org
> To unsubscribe send an email to copr-devel-leave@lists.fedorahosted.org
_______________________________________________
copr-devel mailing list -- copr-devel@lists.fedorahosted.org
To unsubscribe send an email to copr-devel-leave@lists.fedorahosted.org