On Tue, Jan 24, 2017 at 11:44 AM, Pierre-Yves Chibon <[email protected]>
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":
"LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUVUakNDQTdlZ0F3SUJBZ0lDQVBZd0RRWUpL\nb1pJaHZjTkFRRUZCUUF3Z2FBeEN6QUpCZ05WQkFZVEFsVlQKTVFzd0NRWURWUVFJRXdKT1F6RVFN\nQTRHQTFVRUJ4TUhVbUZzWldsbmFERVhNQlVHQTFVRUNoTU9SbVZrYjNKaApJRkJ5YjJwbFkzUXhE\nekFOQmdOVkJBc1RCbVpsWkcxelp6RVBNQTBHQTFVRUF4TUdabVZrYlhObk1ROHdEUVlEClZRUXBF\nd1ptWldSdGMyY3hKakFrQmdrcWhraUc5dzBCQ1FFV0YyRmtiV2x1UUdabFpHOXlZWEJ5YjJwbFkz\nUXUKYjNKbk1CNFhEVEUwTURReU16RTBNamsxTVZvWERUSTBNRFF5TURFME1qazFNVm93Z2R3eEN6\nQUpCZ05WQkFZVApBbFZUTVFzd0NRWURWUVFJRXdKT1F6RVFNQTRHQTFVRUJ4TUhVbUZzWldsbmFE\nRVhNQlVHQTFVRUNoTU9SbVZrCmIzSmhJRkJ5YjJwbFkzUXhEekFOQmdOVkJBc1RCbVpsWkcxelp6\nRXRNQ3NHQTFVRUF4TWtZMjl3Y2kxamIzQnkKTFdKbExtTnNiM1ZrTG1abFpHOXlZWEJ5YjJwbFkz\nUXViM0puTVMwd0t3WURWUVFwRXlSamIzQnlMV052Y0hJdApZbVV1WTJ4dmRXUXVabVZrYjNKaGNI\nSnZhbVZqZEM1dmNtY3hKakFrQmdrcWhraUc5dzBCQ1FFV0YyRmtiV2x1ClFHWmxaRzl5WVhCeWIy\ncGxZM1F1YjNKbk1JR2ZNQTBHQ1NxR1NJYjNEUUVCQVFVQUE0R05BRENCaVFLQmdRQ2UKREs5VFQy\nM05BdTZPWTVGMnVVNHpMRW9Ld2k1RnRRTU5jVWV5eDdmOHJxMUZXaUxDWHBjWFhpU2tzUE1XV1NM\nWQo5SHNoa1pvM3ZjMHFSRXVBWDNweWRuM2VFRDA0UExrUmRlaWpvSXA5L0Y2YlZ3MmlLMDdXRmc5\nU2MwNlRsKzhSCld1RHNaeTQ1SVJKYXhCRTlJaHBYL0x2Y2JnQ1cvZmVHVGp5WG1iRHd0UUlEQVFB\nQm80SUJWekNDQVZNd0NRWUQKVlIwVEJBSXdBREF0QmdsZ2hrZ0JodmhDQVEwRUlCWWVSV0Z6ZVMx\nU1UwRWdSMlZ1WlhKaGRHVmtJRU5sY25ScApabWxqWVhSbE1CMEdBMVVkRGdRV0JCUm5lNTg0d3Bs\nWGYrZVE2K25zSTZCbm5BNENaRENCMVFZRFZSMGpCSUhOCk1JSEtnQlJyUUZyNUVnaUpXZWRaNVFY\nMUFoMEtUbjhVQUtHQnBxU0JvekNCb0RFTE1Ba0dBMVVFQmhNQ1ZWTXgKQ3pBSkJnTlZCQWdUQWs1\nRE1SQXdEZ1lEVlFRSEV3ZFNZV3hsYVdkb01SY3dGUVlEVlFRS0V3NUdaV1J2Y21FZwpVSEp2YW1W\namRERVBNQTBHQTFVRUN4TUdabVZrYlhObk1ROHdEUVlEVlFRREV3Wm1aV1J0YzJjeER6QU5CZ05W\nCkJDa1RCbVpsWkcxelp6RW1NQ1FHQ1NxR1NJYjNEUUVKQVJZWFlXUnRhVzVBWm1Wa2IzSmhjSEp2\nYW1WamRDNXYKY21lQ0NRRGpVQjVIVHhjZVJUQVRCZ05WSFNVRUREQUtCZ2dyQmdFRkJRY0RBakFM\nQmdOVkhROEVCQU1DQjRBdwpEUVlKS29aSWh2Y05BUUVGQlFBRGdZRUFVazNlbjBYUXpDQm5IUlh4\nZDhyOHp2ZFAwVURvbEpiUysyTEl3Z3NDClJDMnNkZ1UwNGdFblYxdFpVTjNydEk1SzQ2MnpKT0JQ\nOFhQd3h4eUZMN1lOYmVtWTgyTG52Y1pHdzliMGdxTDMKdHNKbzllSFV5SXBZMG93TlVKdzgzU1Ax\neFJvb3NwVGJRK3BsNm9qdjVPNVpGZ1lBUG1yckRWZ0M4a2gzRlp4Rgp0SWc9Ci0tLS0tRU5EIENF\nUlRJRklDQVRFLS0tLS0K\n",
  "i": 3,
  "timestamp": 1485166967.0,
  "msg_id": "2017-db23086c-2001-496e-a59f-10b92c466ae5",
  "topic": "org.fedoraproject.prod.copr.build.end",
  "source_version": "0.6.5",
  "signature": 
"HJEOhP93vZFUk3cjIzggxZqIT+IRaLpKF/t21Kn0AcQ9B1VJEe+myerAAJMZfuXppGQqsFyzcPFx\nu+9p7geI5NqNxnN+diUXNlxbXN9/VN0X3vX7U4mbc0/zLyGcbKWIn/pcskbM5qYC2lJHLov0pMwq\nrbq/B0N3CxBL2og0Fj8=\n",
  "msg": {}
}

Does fedmsg_meta need to be adjusted even so?
clime


>
> Pierre
>
>
> >    On Tue, Jan 24, 2017 at 11:02 AM, Pierre-Yves Chibon <
> [email protected]>
> >    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'LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUVUakNDQTdlZ0F3SUJB
> Z0lDQVBZd0RRWUpL\nb1pJaHZjTkFRRUZCUUF3Z2FBeEN6Q
> UpCZ05WQkFZVEFsVlQKTVFzd0NRWURWUVFJRXdKT1F6RVFN\
> nQTRHQTFVRUJ4TUhVbUZzWldsbmFERVhNQlVHQTFVRUNoTU9SbVZrYjNKaAp
> JRkJ5YjJwbFkzUXhE\nekFOQmdOVkJBc1RCbVpsWkcxelp6R
> VBNQTBHQTFVRUF4TUdabVZrYlhObk1ROHdEUVlEClZRUXBF\
> nd1ptWldSdGMyY3hKakFrQmdrcWhraUc5dzBCQ1FFV0YyRmtiV2x1UUdabFp
> HOXlZWEJ5YjJwbFkz\nUXUKYjNKbk1CNFhEVEUwTURReU16R
> TBNamsxTVZvWERUSTBNRFF5TURFME1qazFNVm93Z2R3eEN6\
> nQUpCZ05WQkFZVApBbFZUTVFzd0NRWURWUVFJRXdKT1F6RVFNQTRHQTFVRUJ
> 4TUhVbUZzWldsbmFE\nRVhNQlVHQTFVRUNoTU9SbVZrCmIzS
> mhJRkJ5YjJwbFkzUXhEekFOQmdOVkJBc1RCbVpsWkcxelp6\
> nRXRNQ3NHQTFVRUF4TWtZMjl3Y2kxamIzQnkKTFdKbExtTnNiM1ZrTG1abFp
> HOXlZWEJ5YjJwbFkz\nUXViM0puTVMwd0t3WURWUVFwRXlSa
> mIzQnlMV052Y0hJdApZbVV1WTJ4dmRXUXVabVZrYjNKaGNI\
> nSnZhbVZqZEM1dmNtY3hKakFrQmdrcWhraUc5dzBCQ1FFV0YyRmtiV2x1ClF
> HWmxaRzl5WVhCeWIy\ncGxZM1F1YjNKbk1JR2ZNQTBHQ1NxR
> 1NJYjNEUUVCQVFVQUE0R05BRENCaVFLQmdRQ2UKREs5VFQy\
> nM05BdTZPWTVGMnVVNHpMRW9Ld2k1RnRRTU5jVWV5eDdmOHJxMUZXaUxDWHBjWFhpU2tzUE1XV1NM\nWQ>
> o5SHNoa1pvM3ZjMHFSRXVBWDNweWRuM2VFRDA0UExrUmRlaWpvSXA5L0Y2Yl
> Z3MmlLMDdXRmc5\nU2MwNlRsKzhSCld1RHNaeTQ1SVJKY
> XhCRTlJaHBYL0x2Y2JnQ1cvZmVHVGp5WG1iRHd0UUlEQVFB\
> nQm80SUJWekNDQVZNd0NRWUQKVlIwVEJBSXdBREF0QmdsZ2hrZ0JodmhDQVE
> wRUlCWWVSV0Z6ZVMx\nU1UwRWdSMlZ1WlhKaGRHVmtJRU5sY
> 25ScApabWxqWVhSbE1CMEdBMVVkRGdRV0JCUm5lNTg0d3Bs\
> nWGYrZVE2K25zSTZCbm5BNENaRENCMVFZRFZSMGpCSUhOCk1JSEtnQlJyUUZ
> yNUVnaUpXZWRaNVFY\nMUFoMEtUbjhVQUtHQnBxU0JvekNCb
> 0RFTE1Ba0dBMVVFQmhNQ1ZWTXgKQ3pBSkJnTlZCQWdUQWs1\
> nRE1SQXdEZ1lEVlFRSEV3ZFNZV3hsYVdkb01SY3dGUVlEVlFRS0V3NUdaV1J
> 2Y21FZwpVSEp2YW1W\namRERVBNQTBHQTFVRUN4TUdabVZrY
> lhObk1ROHdEUVlEVlFRREV3Wm1aV1J0YzJjeER6QU5CZ05W\
> nCkJDa1RCbVpsWkcxelp6RW1NQ1FHQ1NxR1NJYjNEUUVKQVJZWFlXUnRhVzV
> BWm1Wa2IzSmhjSEp2\nYW1WamRDNXYKY21lQ0NRRGpVQjVIV
> HhjZVJUQVRCZ05WSFNVRUREQUtCZ2dyQmdFRkJRY0RBakFM\
> nQmdOVkhROEVCQU1DQjRBdwpEUVlKS29aSWh2Y05BUUVGQlFBRGdZRUFVazN
> lbjBYUXpDQm5IUlh4\nZDhyOHp2ZFAwVURvbEpiUysyTEl3Z
> 3NDClJDMnNkZ1UwNGdFblYxdFpVTjNydEk1SzQ2MnpKT0JQ\
> nOFhQd3h4eUZMN1lOYmVtWTgyTG52Y1pHdzliMGdxTDMKdHNKbzllSFV5SXB
> ZMG93TlVKdzgzU1Ax\neFJvb3NwVGJ> RK3BsNm9qdjVPNVpGZ1lBUG1yckRWZ
> 0M4a2gzRlp4Rgp0SWc9Ci0tLS0tRU5EIENF\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'
> AJHlFzJnrrTF9XVs6b0KT4U0n3LCVtK/cWbt+hceOeWf/
> 0bvYn6ZMx6dUTBasutGnA8TY998U6H6\nzgBXPM1fyV4WCO+
> ekuIV7O6dpNedlfpcrnnU1LRGCPWVPSgIw0gg5QDfZ06kQWuQvC17JxokG7P
> i\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 -- [email protected]
> >      To unsubscribe send an email to copr-devel-leave@lists.
> fedorahosted.org
>
> > _______________________________________________
> > copr-devel mailing list -- [email protected]
> > To unsubscribe send an email to [email protected]
> _______________________________________________
> copr-devel mailing list -- [email protected]
> To unsubscribe send an email to [email protected]
>
_______________________________________________
copr-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to