#1984 Oddity in fedmsg when assignee is changed
Closed: Fixed 5 years ago Opened 7 years ago by tibbs.

I reassigned packaging-commitee#680 to from orion to myself, but fedmsg sent me the following in email:

tibbs assigned ticket packaging-committee#680 to orion
        https://pagure.io/packaging-committee/issue/680

I went through datagrepper and found:

{
    "arguments": {
        "categories": [],
        "contains": [],
        "delta": 18000.0,
        "end": 1487974126.0,
        "grouped": false,
        "meta": [],
        "not_categories": [],
        "not_packages": [],
        "not_topics": [],
        "not_users": [],
        "order": "desc",
        "packages": [],
        "page": 1,
        "rows_per_page": 25,
        "start": 1487956126.0,
        "topics": [
            "io.pagure.prod.pagure.issue.assigned.added"
        ],
        "users": [
            "tibbs"
        ]
    },
    "count": 1,
    "pages": 1,
    "raw_messages": [
        {
            "certificate": "LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUVTRENDQTdHZ0F3SUJBZ0lDQWZzd0RRWUpL\nb1pJaHZjTkFRRUZCUUF3Z2FBeEN6QUpCZ05WQkFZVEFsVlQKTVFzd0NRWURWUVFJRXdKT1F6RVFN\nQTRHQTFVRUJ4TUhVbUZzWldsbmFERVhNQlVHQTFVRUNoTU9SbVZrYjNKaApJRkJ5YjJwbFkzUXhE\nekFOQmdOVkJBc1RCbVpsWkcxelp6RVBNQTBHQTFVRUF4TUdabVZrYlhObk1ROHdEUVlEClZRUXBF\nd1ptWldSdGMyY3hKakFrQmdrcWhraUc5dzBCQ1FFV0YyRmtiV2x1UUdabFpHOXlZWEJ5YjJwbFkz\nUXUKYjNKbk1CNFhEVEUxTURVeE9ERTNNamMxTTFvWERUSTFNRFV4TlRFM01qYzFNMW93Z2RZeEN6\nQUpCZ05WQkFZVApBbFZUTVFzd0NRWURWUVFJRXdKT1F6RVFNQTRHQTFVRUJ4TUhVbUZzWldsbmFE\nRVhNQlVHQTFVRUNoTU9SbVZrCmIzSmhJRkJ5YjJwbFkzUXhEekFOQmdOVkJBc1RCbVpsWkcxelp6\nRXFNQ2dHQTFVRUF4TWhjR0ZuZFhKbExYQmgKWjNWeVpUQXhMbVpsWkc5eVlYQnliMnBsWTNRdWIz\nSm5NU293S0FZRFZRUXBFeUZ3WVdkMWNtVXRjR0ZuZFhKbApNREV1Wm1Wa2IzSmhjSEp2YW1WamRD\nNXZjbWN4SmpBa0Jna3Foa2lHOXcwQkNRRVdGMkZrYldsdVFHWmxaRzl5CllYQnliMnBsWTNRdWIz\nSm5NSUdmTUEwR0NTcUdTSWIzRFFFQkFRVUFBNEdOQURDQmlRS0JnUUN5RXZjRndxWUkKUXg2OWpo\nSEJYekExVlpoTFgySkRVUkdROGlLbXFXUFBHaVVEWHhDcURpa1M5YnN5VFBuN2ZTaWY1Uk5HUkZF\nZwpKQmlqRitaM09RMVh5Q1JrY1kxM0VLUDJpbVVEZW0xcnB3Umg3b2gyMlBZaDU4SHFLOWsvNGdk\nZVpXVGM2QmFjCk1RckZpMlFhbWVqL1ZpRW1KdVd2KzgyaitnN2RzV1ZNTVFJREFRQUJvNElCVnpD\nQ0FWTXdDUVlEVlIwVEJBSXcKQURBdEJnbGdoa2dCaHZoQ0FRMEVJQlllUldGemVTMVNVMEVnUjJW\ndVpYSmhkR1ZrSUVObGNuUnBabWxqWVhSbApNQjBHQTFVZERnUVdCQlFyZ09qazgyV05zdW1wbE1T\nbXJ1d3RVeTVIVERDQjFRWURWUjBqQklITk1JSEtnQlJyClFGcjVFZ2lKV2VkWjVRWDFBaDBLVG44\nVUFLR0JwcVNCb3pDQm9ERUxNQWtHQTFVRUJoTUNWVk14Q3pBSkJnTlYKQkFnVEFrNURNUkF3RGdZ\nRFZRUUhFd2RTWVd4bGFXZG9NUmN3RlFZRFZRUUtFdzVHWldSdmNtRWdVSEp2YW1WagpkREVQTUEw\nR0ExVUVDeE1HWm1Wa2JYTm5NUTh3RFFZRFZRUURFd1ptWldSdGMyY3hEekFOQmdOVkJDa1RCbVps\nClpHMXpaekVtTUNRR0NTcUdTSWIzRFFFSkFSWVhZV1J0YVc1QVptVmtiM0poY0hKdmFtVmpkQzV2\nY21lQ0NRRGoKVUI1SFR4Y2VSVEFUQmdOVkhTVUVEREFLQmdnckJnRUZCUWNEQWpBTEJnTlZIUThF\nQkFNQ0I0QXdEUVlKS29aSQpodmNOQVFFRkJRQURnWUVBSjhUVGdMMHJhZ3lCdlo5RlBhdis2ZE5B\nZzEza0poQnYrWS9xeU9rT0RadHFwOWovCkdkOHU1emQwVkFwdXlIMnFBYzg1cUcyaEVsY0ZqUytx\nOU95YmtkQnlwQzF5cklzRSt2akZTRHZqalQxMjV6d3EKS2FNV1UwSVgyTnp2d2l4cWxYN1pxc2cz\nMHIwNzgvUHNPTGRLTlo5VWR0QXJGUHZ2b3hQRHpsTTM4aUE9Ci0tLS0tRU5EIENFUlRJRklDQVRF\nLS0tLS0K\n",
            "i": 1,
            "msg": {
                "agent": "tibbs",
                "issue": {
                    "assignee": {
                        "fullname": "Orion Poplawski",
                        "name": "orion"
                    },
                    "blocks": [],
                    "close_status": null,
                    "closed_at": null,
                    "comments": [
                        {
                            "comment": "No objection from me, though install from /dev/null is perhaps a bit abstruse for the entirety of the audience.  Why isn't touch sufficient?  If it's about the permissions, touch and then chmod would probably be less confusing to someone who isn't trying to avoid using two commands for some reason.\r\n\r\n(This makes me think once again that it would be nice if we were guaranteed a specific default umask in rpm scriptlets.)",
                            "date_created": "1487228211",
                            "edited_on": null,
                            "editor": null,
                            "id": 146007,
                            "notification": false,
                            "parent": null,
                            "user": {
                                "fullname": "Jason ティビツ",
                                "name": "tibbs"
                            }
                        },
                        {
                            "comment": "**Metadata Update from @tibbs**:\n- Issue assigned to orion",
                            "date_created": "1487316481",
                            "edited_on": null,
                            "editor": null,
                            "id": 146008,
                            "notification": true,
                            "parent": null,
                            "user": {
                                "fullname": "Jason ティビツ",
                                "name": "tibbs"
                            }
                        },
                        {
                            "comment": "**Metadata Update from @tibbs**:\n- Issue close_status updated to: None",
                            "date_created": "1487350416",
                            "edited_on": null,
                            "editor": null,
                            "id": 172188,
                            "notification": true,
                            "parent": null,
                            "user": {
                                "fullname": "Jason ティビツ",
                                "name": "tibbs"
                            }
                        },
                        {
                            "comment": "**Metadata Update from @tibbs**:\n- Issue tagged with: meeting",
                            "date_created": "1487350446",
                            "edited_on": null,
                            "editor": null,
                            "id": 172189,
                            "notification": true,
                            "parent": null,
                            "user": {
                                "fullname": "Jason ティビツ",
                                "name": "tibbs"
                            }
                        },
                        {
                            "comment": "We discussed this at this weeks meeting (http://meetbot.fedoraproject.org/fedora-meeting-1/2017-02-23/fpc.2017-02-23-17.00.txt):\r\n\r\n* 680 tmpfiles.d minor cleanup  (geppetto, 17:10:49)\r\n  * LINK:\r\n    https://fedoraproject.org/w/index.php?title=Packaging%3ATmpfiles.d&diff=current&oldid=458504\r\n    (orionp, 17:17:57)\r\n  * ACTION: tmpfiles.d minor cleanup (+1:6, 0:1, -1:0)  (geppetto,\r\n    17:21:54)",
                            "date_created": "1487910121",
                            "edited_on": null,
                            "editor": null,
                            "id": 220036,
                            "notification": false,
                            "parent": null,
                            "user": {
                                "fullname": "James Antill",
                                "name": "james"
                            }
                        },
                        {
                            "comment": "**Metadata Update from @james**:\n- Issue **un**tagged with: meeting\n- Issue tagged with: writeup",
                            "date_created": "1487910146",
                            "edited_on": null,
                            "editor": null,
                            "id": 220037,
                            "notification": true,
                            "parent": null,
                            "user": {
                                "fullname": "James Antill",
                                "name": "james"
                            }
                        }
                    ],
                    "content": "I made this minor change:\r\n\r\nhttps://fedoraproject.org/w/index.php?title=Packaging%3ATmpfiles.d&diff=486249&oldid=458504\r\n\r\nand just wanted to make sure folks thought is was okay.  Also looks like there may be some other outstanding tmpfiles.d stuff to look at - #670 and #629",
                    "custom_fields": [],
                    "date_created": "1487203876",
                    "depends": [],
                    "id": 680,
                    "last_updated": "1487957397",
                    "milestone": null,
                    "priority": null,
                    "private": false,
                    "status": "Open",
                    "tags": [
                        "writeup"
                    ],
                    "title": "tmpfiles.d minor cleanup",
                    "user": {
                        "fullname": "Orion Poplawski",
                        "name": "orion"
                    }
                },
                "project": {
                    "close_status": [
                        "accepted",
                        "rejected",
                        "nothingtodo",
                        "invalid",
                        "fixed",
                        "wontfix",
                        "duplicate",
                        "temporaryexception",
                        "permanentexception",
                        "exceptionexpired"
                    ],
                    "custom_keys": [
                        [
                            "blocking",
                            "text"
                        ],
                        [
                            "blockedby",
                            "text"
                        ]
                    ],
                    "date_created": "1479233266",
                    "description": "The Fedora Packaging Committee",
                    "id": 1388,
                    "milestones": {},
                    "name": "packaging-committee",
                    "namespace": null,
                    "parent": null,
                    "priorities": {},
                    "settings": {
                        "Enforce_signed-off_commits_in_pull-request": false,
                        "Minimum_score_to_merge_pull-request": -1,
                        "Only_assignee_can_merge_pull-request": false,
                        "Web-hooks": null,
                        "always_merge": false,
                        "fedmsg_notifications": true,
                        "issue_tracker": true,
                        "issues_default_to_private": false,
                        "project_documentation": false,
                        "pull_requests": true
                    },
                    "tags": [
                        "fedora",
                        "guidelines",
                        "packaging",
                        "committee",
                        "rpm",
                        "spec",
                        "specfiles"
                    ],
                    "user": {
                        "fullname": "Jason ティビツ",
                        "name": "tibbs"
                    }
                }
            },
            "msg_id": "2017-01ae9344-6aae-4377-9a62-f1d1b4d02c9c",
            "signature": "YCFFd6dvGhbKIajJIqxTKjz4Wuw23E3DrIBmaJWuuVXaD8csrNby58NoU5+Eaht5eudBndhDk83F\nOJbYApQwjOa9EkKy8K/RdCqJ04KWJzzSPcLmpZPJfANIf+sVdGpJGWSe0wl2gKrG8Rgj24ulB5Wa\nrpHcPAzBy27Ut7Tmu5M=\n",
            "source_name": "datanommer",
            "source_version": "0.6.5",
            "timestamp": 1487957398.0,
            "topic": "io.pagure.prod.pagure.issue.assigned.added"
        }
    ],
    "total": 1
}

Which seems to be the only related message I received, and came at exactly the same time, but it doesn't appear to reflect the change that I made. The actual code generating the email seems to be good: https://github.com/fedora-infra/fedmsg_meta_fedora_infrastructure/blob/develop/fedmsg_meta_fedora_infrastructure/pagure.py#L232-L238

        elif 'pagure.issue.assigned.added' in msg['topic']:
            issueid = msg['msg']['issue']['id']
            assignee = msg['msg']['issue']['assignee']['name']
            tmpl = self._(
                '{user} assigned ticket {project}#{id} to {assignee}')
            return tmpl.format(user=user, project=project, id=issueid, assignee=assignee)

That's pretty much exhausted my comprehension. Hope it helps.


Metadata Update from @pingou:
- Issue tagged with: bug

7 years ago

I think this is the same problem reported in #1984.

Metadata Update from @pingou:
- Issue assigned to cverna

5 years ago

Metadata Update from @pingou:
- Issue assigned to pingou (was: cverna)

5 years ago

It looks like this has been fixed:

16:45:59    +fm-pagure | pagure.issue.assigned.added -- pingou assigned ticket pagure#1984 to cverna https://pagure.io/pagure/issue/1984  
16:46:15    +fm-pagure | pagure.issue.assigned.added -- pingou assigned ticket pagure#1984 to pingou https://pagure.io/pagure/issue/1984  

Let's close as such and re-open if it happens not to be.

Thanks for your report @tibbs :)

Metadata Update from @pingou:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

5 years ago

Login to comment on this ticket.

Metadata