Page MenuHomePhabricator
Feed All Stories

Nov 21 2016

faulconbridge added a comment to T11902: Periods in repository shortnames make the repo unreachable.

(From our own install's perspective) I don't think the would make anyone at all sad. It seemed to be a pretty random decision by one of our newer users while he was playing around with things.

Nov 21 2016, 11:22 PM · Bug Report
epriestley added a comment to T9979: Build support for ngram indexes for substring searches (e.g., file, paste, package, task titles).

Partly, just haven't gotten there yet.

Nov 21 2016, 11:18 PM · Owners, Files, ApplicationSearch
epriestley added a comment to T11905: Search shouldn't use word boundaries.

(This is T6740, not T9979.)

Nov 21 2016, 11:16 PM · Bug Report
epriestley closed T11906: BACKTRACE error 2f48cc as Invalid.
Nov 21 2016, 11:12 PM
eadler added a comment to T11903: Please provide diff ids as an attachment in differential.search.

Performance concern. At least one use case requires us to scour *all diffs and all diff ids*. Also a desire to avoid having deprecated calls in our codebase.

Nov 21 2016, 11:09 PM · Restricted Project, Conduit, Feature Request
epriestley added a comment to T11903: Please provide diff ids as an attachment in differential.search.

(Is this a performance concern or do you just want like 6 fewer lines of code to maintain?)

Nov 21 2016, 10:59 PM · Restricted Project, Conduit, Feature Request
epriestley closed T11901: Thirdparty thephpleague/oauth2-client support as Resolved.

Feel free to add it to Community Resources, you should be able to edit that page.

Nov 21 2016, 10:59 PM · Feature Request
• smun claimed T11906: BACKTRACE error 2f48cc.
Nov 21 2016, 10:58 PM
epriestley added a comment to T11902: Periods in repository shortnames make the repo unreachable.

This is because we now recognize shortname.git in some cases, and it's ambiguous if quack.git is "the repository with shortname quack.git", or "the repository with shortname quack, addressed as a Git repository".

Nov 21 2016, 10:57 PM · Bug Report
epriestley closed T11904: Obtaining Phabricator version should not require administrative permissions as Wontfix.

WMF has extensions and modifications to Phabricator so you shouldn't report anything you experience on that install to us directly. Report it locally, make sure it isn't because of an extension, then bring it upstream. The versions we could show you would not be useful anyway because they'd be hashes of local commits which do not appear in the upstream, so we would just close your report as invalid and say "test with unforked code before submitting a report" anyway.

Nov 21 2016, 10:53 PM · Feature Request
chad added a comment to T9979: Build support for ngram indexes for substring searches (e.g., file, paste, package, task titles).

Looking through the code, looks like Maniphest doesn't use ngrams yet? Any reason?

Nov 21 2016, 10:46 PM · Owners, Files, ApplicationSearch
chad added a comment to T11905: Search shouldn't use word boundaries.

I think this might be T9979 though.

Nov 21 2016, 10:03 PM · Bug Report
chad added a comment to T11905: Search shouldn't use word boundaries.

Spin up a test instance on Phacility, build a reproduction case and update the ticket. We don't have any insights into how the server you mentioned is configured of administered. We can with Phacility instances if you can't install something locally.

Nov 21 2016, 10:00 PM · Bug Report
niedzielski added a comment to T11905: Search shouldn't use word boundaries.

@chad, thanks but isn't this an upstream bug? The example I gave downstream were of convenience. Do you need me to find an example upstream?

Nov 21 2016, 9:57 PM · Bug Report
chad added a comment to T11905: Search shouldn't use word boundaries.

Please contact your administrator if you want to file a bug against Phabricator. I believe there is an area where you can file bugs for us to look into in the downstream.

Nov 21 2016, 9:52 PM · Bug Report
chad closed T11905: Search shouldn't use word boundaries as Invalid.
Nov 21 2016, 9:49 PM · Bug Report
avivey added a comment to T11904: Obtaining Phabricator version should not require administrative permissions.

Basically anything that would be accepted as a Bug would require you to have admin access to some install before you have enough information to submit it, so this is more-or-less a moot point.

Nov 21 2016, 9:47 PM · Feature Request
chad triaged T11904: Obtaining Phabricator version should not require administrative permissions as Wishlist priority.
Nov 21 2016, 9:47 PM · Feature Request
chad added a comment to T11904: Obtaining Phabricator version should not require administrative permissions.

I'd say 75% or more bug reports we get require administrative access to resolve. You can spin up a Phacility instance for free.

Nov 21 2016, 9:47 PM · Feature Request
chad edited projects for T11904: Obtaining Phabricator version should not require administrative permissions, added: Feature Request; removed Bug Report.
Nov 21 2016, 9:46 PM · Feature Request
niedzielski created T11905: Search shouldn't use word boundaries.
Nov 21 2016, 9:44 PM · Bug Report
niedzielski created T11904: Obtaining Phabricator version should not require administrative permissions.
Nov 21 2016, 9:40 PM · Feature Request
jcox added a comment to T11902: Periods in repository shortnames make the repo unreachable.

Confirmed this on my local. It seems to be due to D16851, specifically this line: https://secure.phabricator.com/source/phabricator/browse/master/src/applications/diffusion/application/PhabricatorDiffusionApplication.php$104.

Nov 21 2016, 8:02 PM · Bug Report
eadler moved T4953: If arc diff is run twice on exact same commit, will upload new diff to differential from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Nov 21 2016, 7:25 PM · Restricted Project, Arcanist, Wikimedia
eadler moved T11691: arc diff with untracked directory complains about every single file in that directory from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Nov 21 2016, 7:25 PM · FreeBSD, Restricted Project, Arcanist, Bug Report
eadler moved T11903: Please provide diff ids as an attachment in differential.search from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Nov 21 2016, 7:25 PM · Restricted Project, Conduit, Feature Request
eadler added a project to T11903: Please provide diff ids as an attachment in differential.search: Restricted Project.
Nov 21 2016, 7:24 PM · Restricted Project, Conduit, Feature Request
eadler created T11903: Please provide diff ids as an attachment in differential.search.
Nov 21 2016, 7:24 PM · Restricted Project, Conduit, Feature Request
faulconbridge created T11902: Periods in repository shortnames make the repo unreachable.
Nov 21 2016, 7:06 PM · Bug Report
chad removed a watcher for Safari: chad.
Nov 21 2016, 6:57 PM
chad added a watcher for Safari: chad.
Nov 21 2016, 6:56 PM
chad closed T3165: Provide Search in Conpherence as Resolved.
Nov 21 2016, 6:33 PM · Conpherence (v4)
ofbeaton updated the task description for T11901: Thirdparty thephpleague/oauth2-client support.
Nov 21 2016, 6:03 PM · Feature Request
ofbeaton created T11901: Thirdparty thephpleague/oauth2-client support.
Nov 21 2016, 6:03 PM · Feature Request
mbm added a comment to T11865: "Aleo" does not include cyrillic characters.

The Croatian letter "č" is also missing, but it's the only Croatian diacritic that's not present in the distributed version of Aleo, resulting in this:

Nov 21 2016, 5:40 PM · FreeBSD, Feature Request
d.maznekov added a comment to T11899: Ability to add Room per Project.

Why? We set up email integration before, but each task needs to have specific projects so that didn't work and now I'm stuck doing the entry by hand.
This is in the realm of reasonable, and likely easy to solve with custom inbound addresses and Herald rules, or with a small extension to Herald. We might try to improve the documentation to make the feature easier to discover or understand.

Nov 21 2016, 5:32 PM · Feature Request
epriestley added a comment to T11817: Configuration issue MySQL max allowed packet does not mention preferred value.

I've marked D16905 as resolving this: we now recommend a minimum value (33554432).

Nov 21 2016, 5:13 PM · Setup, Bug Report
epriestley added a revision to T11817: Configuration issue MySQL max allowed packet does not mention preferred value: Unknown Object (Differential Revision).
Nov 21 2016, 5:10 PM · Setup, Bug Report
epriestley added a revision to T10759: Run PhabricatorDatabase/MySQLSetupCheck against all configured replicas: Unknown Object (Differential Revision).
Nov 21 2016, 5:10 PM · Restricted Project
ioeric added a comment to T11900: Create bot reviewers that do code analysis and post findings as inline comments..

Thank you so much for the tips! I'll check the resources and try them out.

Nov 21 2016, 5:05 PM · Differential, Feature Request
epriestley added a comment to T11900: Create bot reviewers that do code analysis and post findings as inline comments..

See also T10246#163438 for Drydock, and the Harbormaster / Drydock documentation.

Nov 21 2016, 5:00 PM · Differential, Feature Request
epriestley added a comment to T11900: Create bot reviewers that do code analysis and post findings as inline comments..

Here's the general shape of how to do this. Since we encourage synchronous pre-review lint to fill this role instead of asynchronous build-like processes, support is much rougher than support for lint.

Nov 21 2016, 4:59 PM · Differential, Feature Request
ioeric updated subscribers of T11900: Create bot reviewers that do code analysis and post findings as inline comments..
Nov 21 2016, 4:40 PM · Differential, Feature Request
ioeric added a comment to T11900: Create bot reviewers that do code analysis and post findings as inline comments..

Have you looked at the existing lint support already?

The major advantage of lint over a bot is that it happens before code goes for review, so that patches generated by beautifiers like clang-tidy are incorporated locally. Reviewers never need to see unbeautified code or deal with inlines about formatting/style issues.

Nov 21 2016, 4:40 PM · Differential, Feature Request
ioeric added a comment to T11900: Create bot reviewers that do code analysis and post findings as inline comments..

(Sorry I clicked submit too early)

Nov 21 2016, 4:33 PM · Differential, Feature Request
epriestley added a comment to T11900: Create bot reviewers that do code analysis and post findings as inline comments..

Have you looked at the existing lint support already?

Nov 21 2016, 4:30 PM · Differential, Feature Request
ioeric created T11900: Create bot reviewers that do code analysis and post findings as inline comments..
Nov 21 2016, 4:26 PM · Differential, Feature Request
epriestley added a parent task for T11672: Evaluate persistent connections from HTTP contexts: T11044: Support partitioning application databases across multiple database hosts.
Nov 21 2016, 4:01 PM · Restricted Project, Performance, Clusters, Database
epriestley added a subtask for T11044: Support partitioning application databases across multiple database hosts: T11672: Evaluate persistent connections from HTTP contexts.
Nov 21 2016, 4:01 PM · Clusters, Prioritized
epriestley added a revision to T10759: Run PhabricatorDatabase/MySQLSetupCheck against all configured replicas: Unknown Object (Differential Revision).
Nov 21 2016, 4:00 PM · Restricted Project
epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.

We have at least one tricky issue remaining: when applying storage upgrades, we currently apply them like this:

Nov 21 2016, 3:48 PM · Clusters, Prioritized
epriestley added a revision to T10759: Run PhabricatorDatabase/MySQLSetupCheck against all configured replicas: Unknown Object (Differential Revision).
Nov 21 2016, 2:51 PM · Restricted Project
epriestley added a comment to T11894: Calendar "away icons" do not automatically appear.

Although it's very possible that this is a bug, this state also has a 72-hour internal cache which wasn't written correctly in older versions of Phabricator.

Nov 21 2016, 2:34 PM · Calendar, Bug Report
d.maznekov added a comment to T11899: Ability to add Room per Project.

Ok, whatever you say.

Nov 21 2016, 2:31 PM · Feature Request
epriestley closed T11899: Ability to add Room per Project as Invalid.

See T11898 for the previous iteration on this.

Nov 21 2016, 2:29 PM · Feature Request
stevex awarded T11898: Room per project a Like token.
Nov 21 2016, 2:10 PM · Feature Request
d.maznekov created T11899: Ability to add Room per Project.
Nov 21 2016, 2:09 PM · Feature Request
epriestley closed T11898: Room per project as Invalid.

Please read Contributing Feature Requests carefully before filing a feature request.

Nov 21 2016, 1:58 PM · Feature Request
d.maznekov created T11898: Room per project.
Nov 21 2016, 1:55 PM · Feature Request
epriestley added a revision to T10759: Run PhabricatorDatabase/MySQLSetupCheck against all configured replicas: Unknown Object (Differential Revision).
Nov 21 2016, 1:39 PM · Restricted Project
epriestley added a comment to T11897: [Phriction] Create new page does not respect given path.

Can you show me the <VirtualHost> entry you've added for Phabricator in your Apache config? I suspect you may have omitted the B flag in the RewriteRule.

Nov 21 2016, 1:04 PM · Bug Report (Needs Information)
epriestley closed T11896: Issue with tracking time as Invalid.

Phrequent is a prototype, and we don't accept bug reports for prototypes. See User Guide: Prototype Applications for discussion.

Nov 21 2016, 1:01 PM · Bug Report
gx72xk created P2017 how to kill the mail in queued?.
Nov 21 2016, 12:52 PM · Mail
terirem created T11897: [Phriction] Create new page does not respect given path.
Nov 21 2016, 9:34 AM · Bug Report (Needs Information)
• andrewww2w12w12 created T11896: Issue with tracking time .
Nov 21 2016, 8:33 AM · Bug Report

Nov 20 2016

• ryzhas updated the task description for T11891: Incorrect tasks status in email notification.
Nov 20 2016, 8:35 PM · Bug Report (Needs Information)
rphl awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Nov 20 2016, 4:27 PM · Workboards (v3), Restricted Project
epriestley closed T11895: bug report as Invalid.
Nov 20 2016, 2:17 PM · Bug Report
• sontx created T11895: bug report.
Nov 20 2016, 9:43 AM · Bug Report
• hd350 added a watcher for Localization: • hd350.
Nov 20 2016, 5:21 AM

Nov 19 2016

cspeckmim added a comment to T11892: Aphlict does not work using SSL and node 6.9.1.

I tried the openssl connection you suggested and it still bombed.

Nov 19 2016, 11:05 PM · Aphlict, Bug Report
epriestley added a comment to T11044: Support partitioning application databases across multiple database hosts.

General state of the world here:

Nov 19 2016, 10:11 PM · Clusters, Prioritized
dmgarcia added a comment to Q520: How to automatic move maniphests on workboard of other project after a workboard movement (Answer 479).

Thanks to point out @chad I will take a look closer to it.

Nov 19 2016, 10:01 PM
Luke081515.2 created T11894: Calendar "away icons" do not automatically appear.
Nov 19 2016, 9:38 PM · Calendar, Bug Report
chad added a comment to T11892: Aphlict does not work using SSL and node 6.9.1.

What about Node.js v0.10 and v0.12?

If you're still currently using Node.js v0.10 or v0.12, it is time to begin the transition to v4 or v6. Both v0.10 and v0.12 are considered to be in Maintenance mode currently and will fall off our support plan completely later this year.

Nov 19 2016, 9:24 PM · Aphlict, Bug Report
epriestley added a comment to T11892: Aphlict does not work using SSL and node 6.9.1.
$ nodejs --version
v0.10.25
Nov 19 2016, 9:23 PM · Aphlict, Bug Report
benjumanji added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

Great thanks!

Nov 19 2016, 6:13 PM · Bug Report
chad closed T11891: Incorrect tasks status in email notification as Invalid.

Closing for now, happy to reopen with reproduction steps we can follow.

Nov 19 2016, 4:43 PM · Bug Report (Needs Information)
epriestley closed T11893: storage upgrade claims it lacks permissions, but is running as root mysql user as Resolved.

This should now be fixed in HEAD of master and stable.

Nov 19 2016, 4:43 PM · Bug Report
epriestley added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

Cool. D16901 should fix this properly. Thanks for the report!

Nov 19 2016, 4:38 PM · Bug Report
epriestley added a revision to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user: Unknown Object (Differential Revision).
Nov 19 2016, 4:38 PM · Bug Report
benjumanji added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

Fixed and I'm back up and running, thanks!

Nov 19 2016, 4:19 PM · Bug Report
benjumanji added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

I can confirm that SHOW TABLES IN phabricator_xhpast works just fine as root mysql user, and dropping the credentials from the upgrade command yields exactly the same results as running under root. So that seems like it all ties up. I can work around this any number of ways and I will do just that.

Nov 19 2016, 3:55 PM · Bug Report
epriestley added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

Yeah, looks like some credential shenanigans are afoot. I instrumented the user we're actually connecting as and am getting a mixture of what I passed with -u ("trunk") and the default user ("root").

Nov 19 2016, 3:49 PM · Bug Report
epriestley added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

Maybe also: do you have cluster.databases configured?

Nov 19 2016, 3:44 PM · Bug Report
epriestley added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

If you run this from mysql directly, as root, does it work?

Nov 19 2016, 3:42 PM · Bug Report
benjumanji changed the visibility for F1931085: trace.txt.
Nov 19 2016, 12:32 PM
benjumanji updated the task description for T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.
Nov 19 2016, 12:30 PM · Bug Report
benjumanji updated the task description for T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.
Nov 19 2016, 12:29 PM · Bug Report
benjumanji created T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.
Nov 19 2016, 12:29 PM · Bug Report
cspeckmim updated the task description for T11892: Aphlict does not work using SSL and node 6.9.1.
Nov 19 2016, 3:14 AM · Aphlict, Bug Report
cspeckmim updated the task description for T11892: Aphlict does not work using SSL and node 6.9.1.
Nov 19 2016, 2:59 AM · Aphlict, Bug Report
cspeckmim updated the task description for T11892: Aphlict does not work using SSL and node 6.9.1.
Nov 19 2016, 2:52 AM · Aphlict, Bug Report
cspeckmim updated the task description for T11892: Aphlict does not work using SSL and node 6.9.1.
Nov 19 2016, 2:39 AM · Aphlict, Bug Report
cspeckmim updated the task description for T11892: Aphlict does not work using SSL and node 6.9.1.
Nov 19 2016, 2:20 AM · Aphlict, Bug Report
cspeckmim created T11892: Aphlict does not work using SSL and node 6.9.1.
Nov 19 2016, 2:20 AM · Aphlict, Bug Report

Nov 18 2016

epriestley edited the content of Changelog.
Nov 18 2016, 11:42 PM
epriestley created 2016 Week 47 (Late November).
Nov 18 2016, 11:42 PM
epriestley changed the visibility for D5324: Parsing rcsdiff -u.
Nov 18 2016, 10:48 PM
• aHa added a comment to T11816: Calendar: Feedback and Suggestions.

The Event Hovercards are not really useful: E990 The Mouse-over dialog shows no date and time for example...

Nov 18 2016, 10:23 PM · Restricted Project, Calendar