[06:46:58] 10Phabricator: Say “Story Point” in singular - https://phabricator.wikimedia.org/T142285#2530080 (10Nnemo) [06:57:58] 10Phabricator: The search forgets the author given - https://phabricator.wikimedia.org/T142286#2530098 (10Nnemo) [12:12:47] 10Phabricator, 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 07Zuul: Create a duplicate zuul type interface in phabricator - https://phabricator.wikimedia.org/T142296#2530384 (10Paladox) [12:13:26] 10Phabricator, 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 07Zuul: Create a zuul type interface in phabricator - https://phabricator.wikimedia.org/T142296#2530396 (10Paladox) [12:53:41] 10Phabricator, 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 07Zuul: Create a zuul type interface in phabricator - https://phabricator.wikimedia.org/T142296#2530410 (10Aklapper) How is this different from T85123 and why do you propose to keep Zuul instead of trying Harbormaster? [13:52:20] 10Phabricator, 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 07Zuul: Create a zuul type interface in phabricator - https://phabricator.wikimedia.org/T142296#2530424 (10Paladox) @aklapper the reason we should use Zuul over Harbormaster is because Zuul you get a nice layout and ca... [15:04:45] 10Phabricator, 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 07Zuul: Create a zuul type interface in phabricator - https://phabricator.wikimedia.org/T142296#2530442 (10Aklapper) As I asked before: How is this task different from T85123 ? [15:10:11] 10Phabricator, 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 07Zuul: Create a zuul type interface in phabricator - https://phabricator.wikimedia.org/T142296#2530443 (10Luke081515) [15:14:28] 10Phabricator, 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 07Zuul: Create a zuul type interface in phabricator - https://phabricator.wikimedia.org/T142296#2530460 (10Paladox) Because that task discuss a integration plan, but looking at the description it doesn't discuss Zuul.... [16:27:29] 10Phabricator, 06Release-Engineering-Team, 13Patch-For-Review: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#2530489 (10Paladox) [16:27:32] 10Phabricator, 10Continuous-Integration-Config, 10Continuous-Integration-Infrastructure, 07Zuul: Create a zuul type interface in phabricator - https://phabricator.wikimedia.org/T142296#2530491 (10Paladox) [16:30:47] 10Phabricator, 06Release-Engineering-Team, 13Patch-For-Review: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#2530493 (10Paladox) [16:31:29] 10Phabricator, 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review, 07Zuul: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#939268 (10Paladox) [16:34:20] 10Phabricator, 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review, 07Zuul: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#2530495 (10Aklapper) [16:34:28] 10Phabricator, 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#939268 (10Aklapper) [16:35:27] 10Phabricator, 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#939268 (10Aklapper) @Paladox: Reverting latest edits as Zuul might be //one// outcome but... [16:38:28] 10Phabricator, 10Continuous-Integration-Infrastructure, 06Release-Engineering-Team, 13Patch-For-Review: Create a continuous integration plan for Wikimedia Phabricator patches - https://phabricator.wikimedia.org/T85123#2530498 (10Paladox) Oh sorry. Here's my personal opinion We should develop a Zuul like i... [19:44:53] qchris_ hi i wonder if its-phabricator needs updating to support phabricator new conduit [19:45:17] since i am thinking gerritbot on phabricator wikimedia used the old token and when i do it, it fails [19:46:40] Not sure I understand. its-phabricator currently can use Conduit, as it is able to add comments. So I think it should be fine. [20:59:02] qchris_ but i keep getting token errors [20:59:07] even though they are correct [20:59:30] unless they are wrong but, i doint know where to get the token that the plugin needs [20:59:31] ? [20:59:36] But since it's working in production, I assume there is a different issue. [20:59:55] I think I added how to get the token in the docs. Let me check [21:01:08] Yup, it is in config-connectivity.md [21:01:20] Oh [21:01:23] But I think you can also get it through Phabricator's web UI [21:01:27] i did do it that way though [21:01:57] Not sure I can help you with that. [21:02:11] oh [21:03:11] qchris_ it is now a token though [21:03:33] i doint think it is a certificate since it calls its self the api token when doing arc install-certificate [21:03:51] root@gerrit-test:/var/www/html/review_site# arc install-certificate https://phab-01.wmflabs.org/ [21:03:51] PHP Warning: Module 'pcntl' already loaded in Unknown on line 0 [21:03:51] CONNECT Connecting to "https://phab-01.wmflabs.org/api/"... [21:03:51] LOGIN TO PHABRICATOR [21:03:51] Open this page in your browser and login to Phabricator if necessary: [21:03:53] https://phab-01.wmflabs.org/conduit/login/ [21:03:55] Then paste the API Token on that page below. [21:03:57] Paste API Token from that page: [21:04:49] is the token the plugin wants [21:04:52] begin with [21:04:53] cli- [21:09:45] paladox: It's been way to long since I last did it. I have no clue. But as I said: I'll look at the its-phabricator thing this weekend. [21:09:57] ok thanks [21:10:01] All I recall, is written in the docs. [21:11:30] Oh [22:42:17] qchris__away yay i got it working [22:42:34] \o/ [22:42:41] What was the missing part? [22:42:41] the docs deffitly need updating in its-phabricator since your not given the certificate [22:42:49] that that way [22:42:53] you need to go to [22:43:56] https://phab-01.wmflabs.org/conduit/method/conduit.getcertificate/ [22:44:05] and paste your token in from [22:44:19] https://phab-01.wmflabs.org/conduit/token/ [22:44:23] qchris__away ^^ [22:44:35] that will get you the certificate [22:45:15] Now that you mention it ... doesn't `install-certificate` also ask you some kind of information from the web UI? [22:45:20] Maybe that was the token? [22:45:26] Nope [22:45:30] it gets you the api token [22:45:36] which is totatally different [22:45:43] Ok. [22:45:46] to what we want to get the certificate [22:45:50] they used to do that [22:45:58] but seems was updated to only do the api token now [22:46:06] The docs were up-to-date when I wrote them. But I guess the maintainers would love to get patches :-) [22:46:14] Maybe yep [22:46:47] `install-certificate` only gets you the token? ... That also sounds like a misnomer on the Phabriactor side then :-) [22:46:59] Yep [22:47:09] But you can get your cetificate by doing the above [22:47:17] changing the url to your phab install [22:47:25] Glad you found that. [22:47:35] Yep, thanks took me a while though [22:47:38] to figure that [22:47:38] out [22:47:54] they doint make it easy, expecially finding out that they changed the whole system [22:48:28] qchris__away one thing though can we do Task: and Bug: [22:48:39] i am not sure if those are hardcoded in the plugin? [22:49:04] If we want that, we can do that. `Bug` is not hardcoded in the plugin. [22:49:17] Oh, how do we do that please? [22:49:33] Oh wait maybe i found it [22:49:39] Cool. [22:49:42] [commentlink "bugzilla"] [22:49:43] match = "\\b[bB][uU][gG]\\:?\\s+#?(\\d+)\\b" [22:49:43] link = https://bugzilla.wikimedia.org/$1 [22:50:08] Hw do we do Task in there [22:50:13] making task optional [22:50:14] please? [22:50:14] That different. [22:50:18] oh [22:50:32] If you want its-phabricator to act on tasks, look at the action configs. [22:50:38] Oh [22:50:41] Those define when to trigger. [22:50:46] Oh thanks [22:51:21] Oh thankyou [22:51:23] yay found it [22:51:28] * paladox tests Task: [22:54:58] thanks [22:55:07] \o/ [22:55:10] * paladox uploads the patch to gerrit wikimedia :) [22:55:15] one more bug fixed now [22:55:28] I have been testing here http://gerrit-test.wmflabs.org/gerrit/#/c/16/ [23:00:06] I wonder for this [23:00:07] match = "\\bT(\\d+)(#\\d+)?\\b(?!\")" [23:00:07] link = https://phab-01.wmflabs.org/T$1 [23:00:18] how can i support #1 comments in the full phabricator link [23:00:22] qchris__away ^^ [23:00:29] http://gerrit-test.wmflabs.org/gerrit/#/c/16/ [23:00:39] since it works partially but dosent work fully? [23:08:01] paladox: You're asking me for a bug fix. I do not know. Also I haven't looked. Please stop pinging me for every problem that you want to solve. [23:08:12] Ok [23:08:15] sorry [23:08:29] I do not mean to be rude, so do not get me wrong. [23:08:44] But we've had similar situation on-wiki. Then on Phabricator. ;-) [23:09:02] Ok