{ "id": "155860", "key": "ALOY-1374", "fields": { "issuetype": { "id": "1", "description": "A problem which impairs or prevents the functions of the product.", "name": "Bug", "subtask": false }, "project": { "id": "11113", "key": "ALOY", "name": "Alloy", "projectCategory": { "id": "10400", "description": "Tools for developing applications", "name": "Tooling" } }, "fixVersions": [], "resolution": { "id": "7", "description": "", "name": "Invalid" }, "resolutiondate": "2018-03-07T19:05:38.000+0000", "created": "2016-03-21T09:35:00.000+0000", "priority": { "name": "None", "id": "6" }, "labels": [], "versions": [], "issuelinks": [ { "id": "51136", "type": { "id": "10122", "name": "Gantt: start-finish", "inward": "is triggered by", "outward": "is triggering" }, "inwardIssue": { "id": "154359", "key": "CLA-1", "fields": { "summary": "Users report to be unable to login via GitHub", "status": { "description": "A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.", "name": "Resolved", "id": "5", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "priority": { "name": "High", "id": "2" }, "issuetype": { "id": "1", "description": "A problem which impairs or prevents the functions of the product.", "name": "Bug", "subtask": false } } } } ], "assignee": { "name": "batman", "key": "batman", "displayName": "Bruce Wayne", "active": true, "timeZone": "America/New_York" }, "updated": "2018-03-07T19:05:38.000+0000", "status": { "description": "The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.", "name": "Closed", "id": "6", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "components": [ { "id": "13604", "name": "Tooling", "description": "Items related to Alloy tooling and workflow" } ], "description": "It seems the Alloy repository only has a check for Travis, not for the CLA:\r\nhttps://github.com/appcelerator/alloy/pull/770#partial-pull-merging\r\n\r\nWe should add the CLA check", "attachment": [ { "id": "58683", "filename": "Screen Shot 2016-03-22 at 11.19.00 AM.png", "author": { "name": "mgoff", "key": "mgoff", "displayName": "Michael Goff", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2016-03-22T18:20:51.000+0000", "size": 252492, "mimeType": "image/png" }, { "id": "58684", "filename": "Screen Shot 2016-03-22 at 19.48.03.png", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2016-03-22T18:48:17.000+0000", "size": 81321, "mimeType": "image/png" } ], "flagged": false, "summary": "GitHub Alloy repository is missing CLA check", "creator": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "subtasks": [], "reporter": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "environment": null, "comment": { "comments": [ { "id": "380408", "author": { "name": "mgoff", "key": "mgoff", "displayName": "Michael Goff", "active": true, "timeZone": "America/Los_Angeles" }, "body": "[~fokkezb] I see a webbook for CLA in the Alloy repo:\r\n!Screen Shot 2016-03-22 at 11.19.00 AM.png|thumbnail!", "updateAuthor": { "name": "mgoff", "key": "mgoff", "displayName": "Michael Goff", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2016-03-22T18:20:05.000+0000", "updated": "2016-03-22T18:21:17.000+0000" }, { "id": "380412", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "body": "But I don't see the check on PRs:\r\n\r\n!Screen Shot 2016-03-22 at 19.48.03.png|thumbnail!", "updateAuthor": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2016-03-22T18:48:32.000+0000", "updated": "2016-03-22T18:48:44.000+0000" }, { "id": "380413", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "body": "Maybe the hook needs to be updated? I think the new CLA server has a different path. Maybe check what the titanium_mobile repository has?", "updateAuthor": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2016-03-22T18:49:48.000+0000", "updated": "2016-03-22T18:49:48.000+0000" }, { "id": "380430", "author": { "name": "mgoff", "key": "mgoff", "displayName": "Michael Goff", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Moved this to an Alloy ticket since DevOps is not involved with setting up github hooks for projects.", "updateAuthor": { "name": "mgoff", "key": "mgoff", "displayName": "Michael Goff", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2016-03-22T20:15:11.000+0000", "updated": "2016-03-22T20:15:11.000+0000" }, { "id": "380482", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "body": "[~ingo] [~fmiao] could you look into the CLA hook for Alloy? Probably needs a change for the new CLA server that we also did for titanium_mobile", "updateAuthor": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2016-03-23T08:13:39.000+0000", "updated": "2016-03-23T08:13:39.000+0000" }, { "id": "380515", "author": { "name": "ingo", "key": "ingo", "displayName": "Ingo Muschenetz", "active": true, "timeZone": "America/Los_Angeles" }, "body": "[~fokkezb] No idea. See https://github.com/appcelerator/alloy/settings/hooks/4034859. It's registered and working apparently, though does not show up in PRs.", "updateAuthor": { "name": "ingo", "key": "ingo", "displayName": "Ingo Muschenetz", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2016-03-23T13:52:38.000+0000", "updated": "2016-03-23T13:52:38.000+0000" }, { "id": "380516", "author": { "name": "ingo", "key": "ingo", "displayName": "Ingo Muschenetz", "active": true, "timeZone": "America/Los_Angeles" }, "body": "[~fokkezb] also made you an admin of the Alloy repo so you can check for yourself.", "updateAuthor": { "name": "ingo", "key": "ingo", "displayName": "Ingo Muschenetz", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2016-03-23T13:53:22.000+0000", "updated": "2016-03-23T13:53:22.000+0000" }, { "id": "380521", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "body": "[~ingo] could you make me admin for {{titanium_mobile}} as well then? No way to compare now.", "updateAuthor": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2016-03-23T15:09:37.000+0000", "updated": "2016-03-23T15:09:37.000+0000" }, { "id": "380525", "author": { "name": "ingo", "key": "ingo", "displayName": "Ingo Muschenetz", "active": true, "timeZone": "America/Los_Angeles" }, "body": "[~fokkezb] Done.", "updateAuthor": { "name": "ingo", "key": "ingo", "displayName": "Ingo Muschenetz", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2016-03-23T15:30:23.000+0000", "updated": "2016-03-23T15:30:23.000+0000" }, { "id": "380527", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "body": "Thanks. Actually, I see PRs on {{titanium_mobile}} are randomly missing checks as well:\r\n\r\n* https://github.com/appcelerator/titanium_mobile/pull/7722 has CLA + CI\r\n* https://github.com/appcelerator/titanium_mobile/pull/7875 has CI only\r\n* https://github.com/appcelerator/titanium_mobile/pull/5788 has CLA only\r\n\r\nFor Alloy it seems random as well:\r\n\r\n* https://github.com/appcelerator/alloy/pull/708 has CLA + CI\r\n* https://github.com/appcelerator/alloy/pull/664 has CLA + 2 x CI\r\n* https://github.com/appcelerator/alloy/pull/627 has CI only\r\n\r\nAll PRs in the last few weeks have only CI for both repos, so it does seem to be related to CLA-1\r\n\r\nI can see in the logs from the CLA hook a month ago it response is:\r\n\r\n{code}\r\nOK passed pull_request\r\n{code}\r\n\r\nWhile more recently it is:\r\n\r\n{code}\r\n{\"success\":true,\"request-id\":\"0070d897-95f7-4ee3-a67a-332b52bbb8cb\",\"key\":\"clauser\",\"clauser\":\"OK!\"}\r\n{code}\r\n\r\nSo that seems to be the issue. I will reopen CLA-1", "updateAuthor": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2016-03-23T15:49:24.000+0000", "updated": "2016-03-23T15:49:24.000+0000" }, { "id": "385994", "author": { "name": "hknoechel", "key": "hansknoechel", "displayName": "Hans Knöchel", "active": true, "timeZone": "Europe/Berlin" }, "body": "There a no checks on recent titanium_mobile PR's as well. Would welcome to get that back.", "updateAuthor": { "name": "hknoechel", "key": "hansknoechel", "displayName": "Hans Knöchel", "active": true, "timeZone": "Europe/Berlin" }, "created": "2016-05-18T04:57:42.000+0000", "updated": "2016-05-18T04:57:42.000+0000" } ], "maxResults": 11, "total": 11, "startAt": 0 } } }