Page MenuHomePhabricator
Feed Advanced Search

Feb 8 2016

• jshirley created T10299: Maniphest search with slugs results in error.
Feb 8 2016, 6:03 PM · ApplicationSearch, Bug Report
• oysterFAKE added a comment to T10297: No queries available on Dashboard exception..

No cloudflare or caching.
Did restart the server after upgrade.
Issue began to happen following upgrade + restart on Saturday 30th January.

Feb 8 2016, 5:44 PM · Bug Report (Needs Information), Dashboards
chad added a comment to T10297: No queries available on Dashboard exception..

This generally points to something environmental on your install. Did you restart the webserver / apache after upgrading? Any proxies or other resource caching going on like cloudflare?

Feb 8 2016, 5:42 PM · Bug Report (Needs Information), Dashboards
• oysterFAKE added a comment to T10297: No queries available on Dashboard exception..

Can't reproduce on this install, though.

Feb 8 2016, 5:37 PM · Bug Report (Needs Information), Dashboards
• oysterFAKE added a comment to T10297: No queries available on Dashboard exception..

Ok, checking Firefox's error console and I get
Error: JX.initBehavior(map): behavior(s) not registered: dashboard-query-panel-select
Expanded to reveal:

JX.$E()                                              core.pkg.js:2
JX.initBehaviors()                                   core.pkg.js:178
.members._handleResponse/when_complete<()            core.pkg.js:235
.statics._complete()                                 core.pkg.js:193
.statics._loadJS/load_callback()                     core.pkg.js:184
Feb 8 2016, 5:35 PM · Bug Report (Needs Information), Dashboards
epriestley closed T10298: exact typeahead for archived projects low priory match as Resolved.

Yeah, it feels a bit like the "baby's first typeahead" button to me (since it didn't exist for so long, and is usually useless if you know what you're going to select) and it usually takes me a minute to remember it's there.

Feb 8 2016, 4:41 PM · Typeahead, Bug Report
cburroughs added a comment to T10298: exact typeahead for archived projects low priory match.

FWIW I don't think I had clicked on since first testing out typeahead and had come to associate it with "this is a magic typeahead box" not "click for more options".

Feb 8 2016, 4:35 PM · Typeahead, Bug Report
epriestley added a comment to T10298: exact typeahead for archived projects low priory match.

Here's a workaround:

Feb 8 2016, 4:27 PM · Typeahead, Bug Report
cburroughs created T10298: exact typeahead for archived projects low priory match.
Feb 8 2016, 3:54 PM · Typeahead, Bug Report
chad added a comment to T10297: No queries available on Dashboard exception..

I can't repro on any of these steps. Can you give us more specific details maybe that we're missing. Sounds like Javascript didn't fire or something for you. Can you reproduce on this install?

Feb 8 2016, 3:10 PM · Bug Report (Needs Information), Dashboards
tycho.tatitscheff added a comment to T10297: No queries available on Dashboard exception..

phabricator a45fe337a12f29efdf425086018f1160906f0d0f (Sun, Feb 7)
arcanist 57f6fb59d73994d90cd94143787424ce0fdbf73b (Mon, Jan 25)
phutil f43291e99d36045bc459e5133454c0d8fd8768ea (Thu, Jan 21)

Feb 8 2016, 3:03 PM · Bug Report (Needs Information), Dashboards
• oysterFAKE created T10297: No queries available on Dashboard exception..
Feb 8 2016, 1:34 AM · Bug Report (Needs Information), Dashboards

Feb 7 2016

jcowgar added a comment to T10296: SSLRead() return error -9806 on git clone.
In T10296#157683, @chad wrote:

You'll want to start debugging your curl / ssl certificates (I don't think this is a Phabricator issue).

Feb 7 2016, 4:17 PM · Bug Report (Needs Information), Diffusion
chad added a comment to T10296: SSLRead() return error -9806 on git clone.

You'll want to start debugging your curl / ssl certificates (I don't think this is a Phabricator issue).

Feb 7 2016, 4:15 PM · Bug Report (Needs Information), Diffusion
jcowgar added a comment to T10296: SSLRead() return error -9806 on git clone.

I installed compiled and installed git from sources on the server, v2.7.1 and that did not solve the problem:

Feb 7 2016, 1:31 PM · Bug Report (Needs Information), Diffusion
jcowgar renamed T10296: SSLRead() return error -9806 on git clone from SSLRead() return error -9806 to SSLRead() return error -9806 on git clone.
Feb 7 2016, 1:14 PM · Bug Report (Needs Information), Diffusion
jcowgar created T10296: SSLRead() return error -9806 on git clone.
Feb 7 2016, 1:13 PM · Bug Report (Needs Information), Diffusion
chad added a comment to T10292: Arcanist does not accept milestones as project tags.

These are still prototypes for now, you should have seen this when you created the Milestone:

Feb 7 2016, 3:12 AM · Projects (v3), Differential, Arcanist, Bug Report
chad merged task T10292: Arcanist does not accept milestones as project tags into T10010: Implement Subprojects.
Feb 7 2016, 3:10 AM · Projects (v3), Differential, Arcanist, Bug Report
Luke081515.2 renamed T10292: Arcanist does not accept milestones as project tags from Arcanist does not accept milestones to Arcanist does not accept milestones as project tags.
Feb 7 2016, 2:59 AM · Projects (v3), Differential, Arcanist, Bug Report
Luke081515.2 created T10292: Arcanist does not accept milestones as project tags.
Feb 7 2016, 2:58 AM · Projects (v3), Differential, Arcanist, Bug Report
epriestley moved T10168: Provide a clearer pathway forward for old projects with invalid hashtags from Future to Far Future on the Projects board.
Feb 7 2016, 12:14 AM · Projects, Bug Report
epriestley renamed T10168: Provide a clearer pathway forward for old projects with invalid hashtags from Projects with hashtags that stopped being valid on Oct 12 no longer work between Dec 27 and Jan 18 to Provide a clearer pathway forward for old projects with invalid hashtags.
Feb 7 2016, 12:14 AM · Projects, Bug Report
epriestley lowered the priority of T10168: Provide a clearer pathway forward for old projects with invalid hashtags from Normal to Wishlist.

Some day we'll provide some kind of tooling or migration approach here, but this seems stable for now.

Feb 7 2016, 12:14 AM · Projects, Bug Report

Feb 6 2016

epriestley claimed T10290: Creating first project/task is broken.
Feb 6 2016, 12:47 PM · Bug Report
epriestley closed T10290: Creating first project/task is broken as Resolved by committing Restricted Diffusion Commit.
Feb 6 2016, 12:47 PM · Bug Report
epriestley added a comment to T10290: Creating first project/task is broken.

Thanks for catching this!

Feb 6 2016, 12:47 PM · Bug Report
avivey added a comment to T10290: Creating first project/task is broken.

That fits what the code does - it looks like it gathers all good options, and that includes a "some" projects, if any.

Feb 6 2016, 7:22 AM · Bug Report
jcowgar added a comment to T10290: Creating first project/task is broken.

Turns out that it's not just if zero projects exist, but if the user attempting to create a project does not belong to any project themselves. i.e. no projects are shown.

Feb 6 2016, 7:11 AM · Bug Report
avivey added a comment to T10290: Creating first project/task is broken.

Looks like it was broken at e1c934ab22d8c0d88ec1ce0d73bc497e563b4acb (Works at 8a9f7609755af5e32c2c287c4b732d213cb680ee).

Feb 6 2016, 7:02 AM · Bug Report
avivey created T10290: Creating first project/task is broken.
Feb 6 2016, 7:00 AM · Bug Report
michaeljs1990 added a comment to T10282: Unsound test exit status creates false positives.
  • @avivey we have our own library that is loaded in when arc runs. I had an out of date version of our library which caused this to throw an error.
  • The chief complaint is that arc unit for unsound tests exits with a status of one since it's indistinguishable from arc completely failing.
  • A harbormaster build step is reading the exit status to determine if the test was a real failure or just the result of an unsound test not passing.
Feb 6 2016, 4:51 AM · Bug Report
avivey added a comment to T10282: Unsound test exit status creates false positives.

I don't fully understand the issue description:

Feb 6 2016, 4:10 AM · Bug Report

Feb 5 2016

epriestley closed T10287: Clicking on a Milestone's workboard link 404's as Resolved by committing Restricted Diffusion Commit.
Feb 5 2016, 8:26 PM · Projects (v3), Bug Report
epriestley added a revision to T10287: Clicking on a Milestone's workboard link 404's: Restricted Differential Revision.
Feb 5 2016, 8:25 PM · Projects (v3), Bug Report
chad created T10287: Clicking on a Milestone's workboard link 404's.
Feb 5 2016, 7:56 PM · Projects (v3), Bug Report
chad created T10285: Can't typeahead a Project Milestone.
Feb 5 2016, 7:42 PM · Projects (v3), Bug Report
• brahn added a comment to T10281: in workboard, open and closed tickets are now too visually similar.

Looks great, thanks for the quick turn-around!

Feb 5 2016, 5:57 PM · Bug Report, Workboards
chad closed T10281: in workboard, open and closed tickets are now too visually similar as Resolved by committing Restricted Diffusion Commit.
Feb 5 2016, 4:05 PM · Bug Report, Workboards
• naddlk added a watcher for Bug Report: • naddlk.
Feb 5 2016, 1:39 PM
michaeljs1990 updated the task description for T10282: Unsound test exit status creates false positives.
Feb 5 2016, 6:35 AM · Bug Report
michaeljs1990 renamed T10282: Unsound test exit status creates false positives from Unsound test exit status creates false negatives to Unsound test exit status creates false positives.
Feb 5 2016, 6:33 AM · Bug Report
michaeljs1990 renamed T10282: Unsound test exit status creates false positives from Unsound test exit status creates false positives to Unsound test exit status creates false negatives.
Feb 5 2016, 6:32 AM · Bug Report
michaeljs1990 updated subscribers of T10282: Unsound test exit status creates false positives.
Feb 5 2016, 6:31 AM · Bug Report
michaeljs1990 created T10282: Unsound test exit status creates false positives.
Feb 5 2016, 6:31 AM · Bug Report
chad added a revision to T10281: in workboard, open and closed tickets are now too visually similar: Restricted Differential Revision.
Feb 5 2016, 5:55 AM · Bug Report, Workboards
chad added a comment to T10281: in workboard, open and closed tickets are now too visually similar.

Hows this:

Feb 5 2016, 5:52 AM · Bug Report, Workboards
chad added a project to T10281: in workboard, open and closed tickets are now too visually similar: Bug Report.
Feb 5 2016, 5:38 AM · Bug Report, Workboards
epriestley closed T10280: large file still stored in mysql when "mysql-engine.max-size=5mb" as Wontfix.

The behavior is not inconsistent with the documentation, and the documentation does not suggest the configuration you selected.

Feb 5 2016, 3:51 AM · Bug Report
epriestley added a comment to T10274: Did arc land --hold change behavior?.

I was referring to T9973

Feb 5 2016, 3:45 AM · Bug Report
ppggff created T10280: large file still stored in mysql when "mysql-engine.max-size=5mb".
Feb 5 2016, 3:28 AM · Bug Report
anton.vladimirov added a comment to T10274: Did arc land --hold change behavior?.

Where's the other request for this?

I was referring to T9973

The motivation was this scenario:

You can land onto a remote branch with a local copy of the branch which isn't really that branch but is just some other branch which you've named the same thing as the other branch.

We can't do that if we actually write onto Release1. The original task or tasks are linked from T9657. The new behavior generally remedies a large range of scenarios where users were confused by arc land writing to Release1 when they did not expect it to.

I think that makes sense for arc land, and as odd a workflow as I find it, I understand the need to support it, but arc land --hold implies modification of local repository, no? Previously arc land --hold was a legitimate workflow that put the user's repository in a very predictable and easy to understand state. Now the end result feels like some process got interrupted in the middle and is really not useful for anything other than pushing. It even kind of looks like a repository that is left in the middle of a failed rebase, with detached heads and unmerged branches. It also leaves garbage branches behind that the user now needs to go clean up manually, and since the branches do not appear merged into trunk, it needs to be a force delete, which is always a little bit of a scary operation, especially for people not experienced with git.
>

See also T8227.

I'm very much with you on that one. That's why I'm suggesting assuming a certain intent behind using arc land --hold and Doing The Right Thing(TM).

You can use arc alias to define a new command which runs arc land --hold --onto X && git branch -f X HEAD or similar.

Yeah, but that requires us to specify the branch to land onto, which is something arc already knows from .arcconfig.

Feb 5 2016, 12:36 AM · Bug Report

Feb 4 2016

chad added a comment to T10277: Error Processing Mail (No Receivers).

T7477 is tracking this, it's reasonable to support, but not very common overall.

Feb 4 2016, 9:43 PM · Maniphest, Bug Report
jonah214 added a comment to T10277: Error Processing Mail (No Receivers).

External.

Feb 4 2016, 9:42 PM · Maniphest, Bug Report
chad added a comment to T10277: Error Processing Mail (No Receivers).

Is the mailing list a Phabricator mailing list or external?

Feb 4 2016, 9:41 PM · Maniphest, Bug Report
jonah214 added a comment to T10277: Error Processing Mail (No Receivers).

Because I wasn't replying directly to the task email, I was replying to a colleague's response in the thread. I did not change any recipients from the default when hitting reply all. The sequence of emails went like this:

Feb 4 2016, 9:33 PM · Maniphest, Bug Report
chad merged task T10277: Error Processing Mail (No Receivers) into T7477: Handle inbound email with phabricator address in the CC.
Feb 4 2016, 9:29 PM · Maniphest, Bug Report
chad added a comment to T10277: Error Processing Mail (No Receivers).

Why did you CC the task instead of leaving it in the TO field?

Feb 4 2016, 9:28 PM · Maniphest, Bug Report
jonah214 created T10277: Error Processing Mail (No Receivers).
Feb 4 2016, 9:20 PM · Maniphest, Bug Report
epriestley closed T10275: People profile display error when prototypes are disabled as Resolved by committing Restricted Diffusion Commit.
Feb 4 2016, 3:17 PM · Badges, People, Bug Report
epriestley merged task T10276: Aphlict option parsing for empty values into T8982: Aphlict wont LISTEN to 22280 if notification.client-uri is switch to /ws (nginx).
Feb 4 2016, 3:17 PM · Bug Report
epriestley added a revision to T10275: People profile display error when prototypes are disabled: Restricted Differential Revision.
Feb 4 2016, 3:15 PM · Badges, People, Bug Report
epriestley claimed T10275: People profile display error when prototypes are disabled.
Feb 4 2016, 3:15 PM · Badges, People, Bug Report
• pmoreau added a comment to T10276: Aphlict option parsing for empty values.

Isn't this a duplicate of T8982 (and of T7097 by extension)? The fix you propose has already been submitted, though it was later abandonned (see D13745).

Feb 4 2016, 2:35 PM · Bug Report
• kanarip created T10276: Aphlict option parsing for empty values.
Feb 4 2016, 11:44 AM · Bug Report
clemvangelis created T10275: People profile display error when prototypes are disabled.
Feb 4 2016, 10:53 AM · Badges, People, Bug Report
epriestley added a comment to T10274: Did arc land --hold change behavior?.

Where's the other request for this?

Feb 4 2016, 1:38 AM · Bug Report
anton.vladimirov added a comment to T10274: Did arc land --hold change behavior?.

Yes, that will work.

Feb 4 2016, 1:31 AM · Bug Report
epriestley changed the status of T10274: Did arc land --hold change behavior? from Invalid to Resolved.
Feb 4 2016, 1:29 AM · Bug Report
epriestley added a comment to T10274: Did arc land --hold change behavior?.

In your first example, you can likely:

Feb 4 2016, 1:28 AM · Bug Report
anton.vladimirov added a comment to T10274: Did arc land --hold change behavior?.

thanks

Feb 4 2016, 1:25 AM · Bug Report
epriestley added a comment to T10274: Did arc land --hold change behavior?.

T9657 is probably the relevant guidance. There is no way to get the old behavior back.

Feb 4 2016, 1:25 AM · Bug Report
anton.vladimirov added a comment to T10274: Did arc land --hold change behavior?.

and you beat me to it.

Feb 4 2016, 1:25 AM · Bug Report
anton.vladimirov added a comment to T10274: Did arc land --hold change behavior?.

Sorry, I accidentally submitted in the middle of typing :)

Feb 4 2016, 1:24 AM · Bug Report
anton.vladimirov updated the task description for T10274: Did arc land --hold change behavior?.
Feb 4 2016, 1:23 AM · Bug Report
epriestley closed T10274: Did arc land --hold change behavior? as Invalid.

This is not a valid Phabricator bug report. Please see Contributing Bug Reports.

Feb 4 2016, 1:23 AM · Bug Report
anton.vladimirov created T10274: Did arc land --hold change behavior?.
Feb 4 2016, 1:15 AM · Bug Report

Feb 3 2016

perezd added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

It ok no feel bad

Feb 3 2016, 10:53 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

Thanks for the report @perezd and sorry for my bad testing!

Feb 3 2016, 10:52 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

I feel dumb because in my testing I set the field to literal null and not null.

Feb 3 2016, 10:48 PM · Diffusion, Bug Report
epriestley added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

and also the culprit! hoo hoo hoo!!

Feb 3 2016, 10:47 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

@epriestley is the hero.

Feb 3 2016, 10:46 PM · Diffusion, Bug Report
epriestley closed T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception. as Resolved by committing Restricted Diffusion Commit.
Feb 3 2016, 10:46 PM · Diffusion, Bug Report
epriestley added a revision to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception.: Restricted Differential Revision.
Feb 3 2016, 10:45 PM · Diffusion, Bug Report
perezd added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

mine looks identical to yours (app settings)

Feb 3 2016, 10:25 PM · Diffusion, Bug Report
perezd added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

PHP 5.5.9-1ubuntu4.14 (cli) (built: Oct 28 2015 01:34:46)
Ubuntu LTS

Feb 3 2016, 10:23 PM · Diffusion, Bug Report
chad updated subscribers of T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

(also maybe helpful, what PHP/OS you are running)

Feb 3 2016, 10:21 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

/applications/view/PhabricatorFilesApplication/

Feb 3 2016, 10:20 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

Can you look and see what your limit is set to?

Feb 3 2016, 10:19 PM · Diffusion, Bug Report
perezd added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

These files were well under 1MB FWIW.

Feb 3 2016, 10:18 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

b51a859636ac was recent, so maybe it did break something.

Feb 3 2016, 9:58 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

It's anything over 1MB.

Feb 3 2016, 9:56 PM · Diffusion, Bug Report
avivey added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

I haven't tried to reproduce this, but the error about "store large files" seems out-of-place if the files are small.

Feb 3 2016, 9:56 PM · Diffusion, Bug Report
perezd added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

I had never setup anything for file storage in the past. It was never configured.
I've restarted php/apache many times, including immediately after I updated it as the documentation states to do.

Feb 3 2016, 9:52 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

If you just updated Phabricator, how long has it been and did you restart PHP/Apache?

Feb 3 2016, 9:49 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

set that field to null, and you can repro it.

Feb 3 2016, 9:40 PM · Diffusion, Bug Report
perezd added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

set that field to null, and you can repro it. I did not get this error prior, and I should not be getting an "upload" error as I try to simply view a source file in my git repo via diffusion.

Feb 3 2016, 9:33 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

https://secure.phabricator.com/book/phabricator/article/configuring_file_storage/ explains file storage in Phabricator.

Feb 3 2016, 9:32 PM · Diffusion, Bug Report
chad added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

Maybe I'm missing something obvious, You say if you set the local-disk path the error goes away. So what is the bug you are reporting, specifically, and how can we reproduce it.

Feb 3 2016, 9:29 PM · Diffusion, Bug Report
perezd added a comment to T10273: Viewing any file (regardless of size) in Diffusion throws Unhandled Exception..

Yes?

Feb 3 2016, 9:28 PM · Diffusion, Bug Report