Page MenuHomePhabricator
Feed Advanced Search

Oct 21 2014

btrahan added a revision to T6350: Diffusion spamming the feed/commits going weird: Restricted Differential Revision.
Oct 21 2014, 12:25 AM · Support Impact, Feed, Diffusion
btrahan updated subscribers of T6350: Diffusion spamming the feed/commits going weird.

so that error and this one:

Oct 21 2014, 12:23 AM · Support Impact, Feed, Diffusion
chad added a comment to T6350: Diffusion spamming the feed/commits going weird.
[2014-10-20 17:07:03] EXCEPTION: (AphrontCharacterSetQueryException) Attempting to construct a query containing characters outside of the Unicode Basic Multilingual Plane. MySQL will silently truncate this data if it is inserted into a `utf8` column. Use the `%B` conversion to escape binary strings data. at [<phutil>/src/aphront/storage/connection/mysql/AphrontBaseMySQLDatabaseConnection.php:346]
  #0 AphrontBaseMySQLDatabaseConnection::validateUTF8String(string) called at [<phutil>/src/aphront/storage/connection/mysql/AphrontMySQLiDatabaseConnection.php:10]
  #1 AphrontMySQLiDatabaseConnection::escapeUTF8String(string) called at [<phutil>/src/xsprintf/qsprintf.php:170]
  #2 xsprintf_query(AphrontMySQLiDatabaseConnection, string, integer, string, integer) called at [<phutil>/src/xsprintf/xsprintf.php:63]
  #3 xsprintf(string, AphrontMySQLiDatabaseConnection, array) called at [<phutil>/src/xsprintf/qsprintf.php:64]
  #4 qsprintf(AphrontMySQLiDatabaseConnection, string, string, string) called at [<phabricator>/src/infrastructure/storage/lisk/LiskDAO.php:1101]
  #5 LiskDAO::update() called at [<phabricator>/src/infrastructure/storage/lisk/LiskDAO.php:1041]
  #6 LiskDAO::save() called at [<phabricator>/src/applications/daemon/query/PhabricatorDaemonLogQuery.php:104]
  #7 PhabricatorDaemonLogQuery::willFilterPage(array) called at [<phabricator>/src/infrastructure/query/policy/PhabricatorPolicyAwareQuery.php:236]
  #8 PhabricatorPolicyAwareQuery::execute() called at [<phabricator>/src/applications/daemon/management/PhabricatorDaemonManagementLogWorkflow.php:42]
  #9 PhabricatorDaemonManagementLogWorkflow::execute(PhutilArgumentParser) called at [<phutil>/src/parser/argument/PhutilArgumentParser.php:394]
  #10 PhutilArgumentParser::parseWorkflowsFull(array) called at [<phutil>/src/parser/argument/PhutilArgumentParser.php:290]
  #11 PhutilArgumentParser::parseWorkflows(array) called at [<phabricator>/scripts/daemon/manage_daemons.php:23]
Oct 21 2014, 12:08 AM · Support Impact, Feed, Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.
./bin/phd log
Oct 21 2014, 12:06 AM · Support Impact, Feed, Diffusion
chad added a comment to T6350: Diffusion spamming the feed/commits going weird.

Still occurring. :(

Oct 21 2014, 12:05 AM · Support Impact, Feed, Diffusion

Oct 20 2014

btrahan triaged T6350: Diffusion spamming the feed/commits going weird as High priority.
Oct 20 2014, 11:55 PM · Support Impact, Feed, Diffusion
btrahan updated subscribers of T6355: Unable to index document PHID-CMIT-..., Incorrect key file for table './phabricator_search/search_documentfield.MYI'.

Also cc @epriestley in case he's more concerned about this generally than I am...

Oct 20 2014, 11:48 PM · Diffusion
btrahan added a comment to T6355: Unable to index document PHID-CMIT-..., Incorrect key file for table './phabricator_search/search_documentfield.MYI'.
./bin/search index PHID-CMIT-rrzk2z5yfad5i7b32bm3
Oct 20 2014, 11:48 PM · Diffusion
staticshock added a comment to T6355: Unable to index document PHID-CMIT-..., Incorrect key file for table './phabricator_search/search_documentfield.MYI'.

Ah, thanks. Is there any way to re-run the original operation that failed here from the cli?

Oct 20 2014, 11:14 PM · Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

Cool, and if its still occurring it will be top of my queue again. :)

Oct 20 2014, 10:40 PM · Support Impact, Feed, Diffusion
michaeloa added a comment to T6350: Diffusion spamming the feed/commits going weird.

Will check it ASAP in the morning.

Oct 20 2014, 10:40 PM · Support Impact, Feed, Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

Alrighty folks impacted by this - could you please update your Phabricator install, including re-starting daemons, and let us know if the issue is resolved or you're still having problems?

Oct 20 2014, 10:35 PM · Support Impact, Feed, Diffusion
btrahan added a revision to T6350: Diffusion spamming the feed/commits going weird: Restricted Differential Revision.
Oct 20 2014, 10:31 PM · Support Impact, Feed, Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

okay, I kind of get how come it happens at least. that's nice.

Oct 20 2014, 10:25 PM · Support Impact, Feed, Diffusion
chad added a comment to T6350: Diffusion spamming the feed/commits going weird.

pasted_file (651×1 px, 414 KB)

Oct 20 2014, 10:21 PM · Support Impact, Feed, Diffusion
chad added a comment to T6350: Diffusion spamming the feed/commits going weird.

I get this tracking Phabricator on my home install.

Oct 20 2014, 10:17 PM · Support Impact, Feed, Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

...still working on reproducing this... if anyone has a public repo I can clone that reproduces that issue this would be helpful.

Oct 20 2014, 10:17 PM · Support Impact, Feed, Diffusion
btrahan closed T6355: Unable to index document PHID-CMIT-..., Incorrect key file for table './phabricator_search/search_documentfield.MYI' as Invalid.

This is a mysql configuration issue / scaling issue. See something like

Oct 20 2014, 9:30 PM · Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

@chad - what remote are you tracking that reproduces this?

Oct 20 2014, 7:38 PM · Support Impact, Feed, Diffusion
staticshock created T6355: Unable to index document PHID-CMIT-..., Incorrect key file for table './phabricator_search/search_documentfield.MYI'.
Oct 20 2014, 7:37 PM · Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

Ah okay, I got the link to load. Its going to take me a few hours to figure this out so I'd stop importing those repositories for now if its driving you batty.

Oct 20 2014, 3:54 PM · Support Impact, Feed, Diffusion
btrahan added a comment to T6350: Diffusion spamming the feed/commits going weird.

the phab.met.no link doesn't resolve for me - should it?

Oct 20 2014, 3:48 PM · Support Impact, Feed, Diffusion
chad assigned T6350: Diffusion spamming the feed/commits going weird to btrahan.

I see this on my local install tracking a remote as well.

Oct 20 2014, 2:08 PM · Support Impact, Feed, Diffusion
michaeloa added a comment to T6350: Diffusion spamming the feed/commits going weird.

Re: the last point. I'm also seeing this for existing employees at the institute, who have submitted commits using e-mails different from the ones that they are registered with on the site. I haven't noticed this for anyone else - the problem seems to be limited to commiters who are not registered on the site.

Oct 20 2014, 12:10 PM · Support Impact, Feed, Diffusion
michaeloa created T6350: Diffusion spamming the feed/commits going weird.
Oct 20 2014, 8:15 AM · Support Impact, Feed, Diffusion

Oct 19 2014

chad merged task T6349: "Authored" and "Commited" is truncated on the Commit Page into T6334: Layout issue when viewing a commit.
Oct 19 2014, 10:20 PM · Diffusion
chad updated subscribers of T6334: Layout issue when viewing a commit.
Oct 19 2014, 10:20 PM · Badge Awarded, Design, Diffusion
fchovich added a comment to T6349: "Authored" and "Commited" is truncated on the Commit Page.

Yes, you are right, I've cloned the repo yesterday, but I hadn't updated it yet.
My mistake, sorry. Do I change the state to Resolved or Invalid in this case?

Oct 19 2014, 9:24 PM · Diffusion
chad added a comment to T6349: "Authored" and "Commited" is truncated on the Commit Page.

Is your instance up to date? This should be resolved .

Oct 19 2014, 9:16 PM · Diffusion
fchovich created T6349: "Authored" and "Commited" is truncated on the Commit Page.
Oct 19 2014, 9:05 PM · Diffusion

Oct 17 2014

shrinidhirao added a comment to T5029: Fill in missing context for low-context diffs against tracked repositories.

Was about to create a new task for same thing for SVN working copies.

Oct 17 2014, 9:06 AM · KDE, Restricted Project, Diffusion, Differential
emil.styrke added a comment to T5029: Fill in missing context for low-context diffs against tracked repositories.

Another +1 from me. Would it make sense to just ask the submitter for a revision to use as the base?

Oct 17 2014, 8:08 AM · KDE, Restricted Project, Diffusion, Differential
• xuyulong added a comment to T4030: Phabricator can't process commits which delete files from a relocated directory.

Now has a solution to this problem?

Oct 17 2014, 6:58 AM · Diffusion, Subversion
btrahan closed T6331: Closing a revision via a commit generates superfluous "mention" in Task as Resolved.

Closed by commit rPd9dd098e405b.

Oct 17 2014, 12:08 AM · Transactions, Diffusion

Oct 16 2014

btrahan added a revision to T6331: Closing a revision via a commit generates superfluous "mention" in Task: Restricted Differential Revision.
Oct 16 2014, 11:49 PM · Transactions, Diffusion
chad triaged T6331: Closing a revision via a commit generates superfluous "mention" in Task as Normal priority.
Oct 16 2014, 11:15 PM · Transactions, Diffusion
richardvanvelzen renamed T6331: Closing a revision via a commit generates superfluous "mention" in Task from Closing a revision via a commit generates superfluous "mention" to Closing a revision via a commit generates superfluous "mention" in Task.
Oct 16 2014, 10:07 PM · Transactions, Diffusion
chad closed T6334: Layout issue when viewing a commit as Resolved.

Closed by commit rPd9ad4ab385c2.

Oct 16 2014, 8:12 PM · Badge Awarded, Design, Diffusion
chad added a revision to T6334: Layout issue when viewing a commit: Restricted Differential Revision.
Oct 16 2014, 8:03 PM · Badge Awarded, Design, Diffusion
chad added a comment to T6334: Layout issue when viewing a commit.

Yes, it's a known issue.

Oct 16 2014, 6:19 PM · Badge Awarded, Design, Diffusion
staticshock added a comment to T6334: Layout issue when viewing a commit.

Seeing the same thing on this instance:

away-status-trimmed.png (191×470 px, 26 KB)

Oct 16 2014, 6:10 PM · Badge Awarded, Design, Diffusion
• levi.blackstone closed T6333: Host key verification failed on GitHub repo using deploy key as Resolved.

That did the trick. Thanks!

Oct 16 2014, 5:23 PM · Diffusion
btrahan claimed T6331: Closing a revision via a commit generates superfluous "mention" in Task.

Ah, okay. Thanks! I'll get a patch out later today pr early tomorrow.

Oct 16 2014, 5:08 PM · Transactions, Diffusion
chad renamed T6334: Layout issue when viewing a commit from Layout issue when viewing a commit makes me female to Layout issue when viewing a commit.
Oct 16 2014, 4:42 PM · Badge Awarded, Design, Diffusion
chad claimed T6334: Layout issue when viewing a commit.
Oct 16 2014, 4:41 PM · Badge Awarded, Design, Diffusion
GMTA created T6334: Layout issue when viewing a commit.
Oct 16 2014, 4:30 PM · Badge Awarded, Design, Diffusion
richardvanvelzen added a comment to T6331: Closing a revision via a commit generates superfluous "mention" in Task.

Oh, I notice I forgot to mention it was about a task :/ My apologies for
that.

Oct 16 2014, 4:05 PM · Transactions, Diffusion
btrahan added a comment to T6331: Closing a revision via a commit generates superfluous "mention" in Task.

(This didn't reproduce on this install just now. See D10712 and corresponding commit rPac83ff706023e266907ed625954ef9b988daf014 )

Oct 16 2014, 3:38 PM · Transactions, Diffusion
epriestley added a comment to T6333: Host key verification failed on GitHub repo using deploy key.

You have a very old version of Phabricator. Try updating.

Oct 16 2014, 3:14 PM · Diffusion
btrahan added a comment to T6331: Closing a revision via a commit generates superfluous "mention" in Task.

Can you show me a raw commit message involved in these shenanigans?

Oct 16 2014, 3:10 PM · Transactions, Diffusion
• levi.blackstone created T6333: Host key verification failed on GitHub repo using deploy key.
Oct 16 2014, 2:58 PM · Diffusion
richardvanvelzen added a comment to T6331: Closing a revision via a commit generates superfluous "mention" in Task.

I've now seen the same issue over several commits, leading me to believe
that it's not a fluke.

Oct 16 2014, 2:52 PM · Transactions, Diffusion
btrahan added a comment to T6331: Closing a revision via a commit generates superfluous "mention" in Task.

I am not reproducing this in my test environment.

Oct 16 2014, 2:46 PM · Transactions, Diffusion
chad added a comment to T6331: Closing a revision via a commit generates superfluous "mention" in Task.

Nice catch.

Oct 16 2014, 2:19 PM · Transactions, Diffusion
chad updated subscribers of T6331: Closing a revision via a commit generates superfluous "mention" in Task.
Oct 16 2014, 2:17 PM · Transactions, Diffusion
richardvanvelzen updated the task description for T6331: Closing a revision via a commit generates superfluous "mention" in Task.
Oct 16 2014, 1:12 PM · Transactions, Diffusion
richardvanvelzen created T6331: Closing a revision via a commit generates superfluous "mention" in Task.
Oct 16 2014, 1:12 PM · Transactions, Diffusion

Oct 15 2014

avivey added a comment to T6304: Git hosting not working as documented.

Well, it should have worked there...

Oct 15 2014, 9:01 PM · Documentation, Daemons, Diffusion
btrahan closed T4896: Move Audit to ApplicationTransactions, a subtask of T929: Allow two arbitrary revisions to be compared in Diffusion, as Resolved.
Oct 15 2014, 8:14 PM · Diffusion (v3), Restricted Project
btrahan closed T6293: Unable to mention differential revisions from commits as Resolved.

Closed by commit rP3dd92a78ac2a.

Oct 15 2014, 8:14 PM · Transactions, Diffusion, Differential
btrahan claimed T6293: Unable to mention differential revisions from commits.
Oct 15 2014, 8:14 PM · Transactions, Diffusion, Differential
btrahan added a revision to T6293: Unable to mention differential revisions from commits: Restricted Differential Revision.
Oct 15 2014, 8:05 PM · Transactions, Diffusion, Differential
• danbst added a comment to T6315: Unhandled EXCEPTION: (AphrontCharacterSetQueryException) when running "scripts/repository/reparse.php".

Seems like I've found a problem. There are filenames in CP1251 encoding, but Phabricator tries to parse them as UTF-8

Oct 15 2014, 3:59 PM · Diffusion
epriestley merged task T6315: Unhandled EXCEPTION: (AphrontCharacterSetQueryException) when running "scripts/repository/reparse.php" into T1191: Comments are truncated at first non-base-plane character.
Oct 15 2014, 1:11 PM · Diffusion
• danbst created T6315: Unhandled EXCEPTION: (AphrontCharacterSetQueryException) when running "scripts/repository/reparse.php".
Oct 15 2014, 1:10 PM · Diffusion

Oct 14 2014

lucaro created T6306: Add a "members of associated projects" visibility setting to repositories.
Oct 14 2014, 3:11 PM · Projects, Diffusion
chad raised the priority of T6304: Git hosting not working as documented from Normal to Needs Triage.
Oct 14 2014, 2:11 PM · Documentation, Daemons, Diffusion
Judge added a project to T6304: Git hosting not working as documented: Documentation.
Oct 14 2014, 1:51 PM · Documentation, Daemons, Diffusion
Judge updated the task description for T6304: Git hosting not working as documented.
Oct 14 2014, 1:50 PM · Documentation, Daemons, Diffusion
Judge created T6304: Git hosting not working as documented.
Oct 14 2014, 1:50 PM · Documentation, Daemons, Diffusion

Oct 13 2014

epriestley added a comment to T5646: Viewing a non-existent repository in Diffusion throws an unhandled exception.

I dig the bigger picture plan.

Oct 13 2014, 10:02 PM · Diffusion
• danbst added a comment to T6118: Support for Mercurial extensions in hosted repositories.

This is a very low priority because there isn't much interest in any specific extension.

Our project is a game with lots of graphics. As we plan to move from SVN to Mercurial, it is a limitation that Phab does not support Largefiles (which is a must in our situation). Though I'll try a patch from T5189 as temporal solution.

Oct 13 2014, 7:23 PM · Mercurial, Diffusion
epriestley added a comment to T4369: Phabricator HTTP repository hosting has fairly severe scalability limits.

We plan to resolve this eventually but it's not a priority. SSH does not have these scalability limits and is the recommended workaround for large repositories.

Oct 13 2014, 3:56 PM · Diffusion (v3), Restricted Project
• danbst added a comment to T4369: Phabricator HTTP repository hosting has fairly severe scalability limits.

We've hit the

Oct 13 2014, 1:34 PM · Diffusion (v3), Restricted Project
nkovacs added a comment to T6107: "Reply to inline comment" shortcut broken in Diffusion.

The same error happens in differential, when you have a renamed file. You get the message "This file was changed only by adding or removing whitespace. Show File Contents", and clicking Show File Contents throws this error (in core.pkg.js line 333).

Oct 13 2014, 9:27 AM · Support Impact, Diffusion

Oct 12 2014

epriestley added a comment to T6293: Unable to mention differential revisions from commits.

Specifically, the commit message summary is never processed as a transaction, so user/object/file mentions don't activate. This also means @user doesn't CC, and {Fxxx} doesn't adjust permissions.

Oct 12 2014, 10:14 PM · Transactions, Diffusion, Differential
epriestley added a comment to T6293: Unable to mention differential revisions from commits.

We added mentions recently. The merge was correct. See next transaction for an example.

Oct 12 2014, 10:12 PM · Transactions, Diffusion, Differential
joshuaspence added a comment to T6293: Unable to mention differential revisions from commits.

Ref is for maniphest tasks... or does it work with differential revisions as well?

Oct 12 2014, 10:07 PM · Transactions, Diffusion, Differential
chad reopened T6293: Unable to mention differential revisions from commits as "Open".
Oct 12 2014, 10:06 PM · Transactions, Diffusion, Differential
chad added a comment to T6293: Unable to mention differential revisions from commits.

Ref currently does that job, I'm just wondering if we should make Ref action the default anytime an object is referenced anywhere (instead of via a comment mention).

Oct 12 2014, 10:05 PM · Transactions, Diffusion, Differential
joshuaspence added a comment to T6293: Unable to mention differential revisions from commits.

Hmm... maybe not a mention, but I feel that there should be some sort of relationship.

Oct 12 2014, 10:01 PM · Transactions, Diffusion, Differential
chad updated subscribers of T6293: Unable to mention differential revisions from commits.

Maybe @epriestley can weigh in too.

Oct 12 2014, 10:00 PM · Transactions, Diffusion, Differential
chad added a comment to T6293: Unable to mention differential revisions from commits.

Oh, maybe I am misunderstanding your description. I do not expect an object in a message to count as a "Mention", only a person actually mentioning it via a comment box.

Oct 12 2014, 9:59 PM · Transactions, Diffusion, Differential
chad merged task T6293: Unable to mention differential revisions from commits into T4896: Move Audit to ApplicationTransactions.
Oct 12 2014, 9:55 PM · Transactions, Diffusion, Differential
joshuaspence renamed T6293: Unable to mention differential revisions from commits from Unable to mention differentials from commits to Unable to mention differential revisions from commits.
Oct 12 2014, 9:47 PM · Transactions, Diffusion, Differential
joshuaspence created T6293: Unable to mention differential revisions from commits.
Oct 12 2014, 9:43 PM · Transactions, Diffusion, Differential
joshuaspence triaged T5644: Add controls to prevent `pygmentize` from saturating system resources as High priority.
Oct 12 2014, 9:31 PM · Differential, Diffusion

Oct 9 2014

chad closed T6278: Possible to limit revision import on remote svn repository? as Resolved.
Oct 9 2014, 2:16 PM · Diffusion
chad added a comment to T6278: Possible to limit revision import on remote svn repository?.

This feature to my knowledge does not exist.

Oct 9 2014, 2:15 PM · Diffusion
epriestley moved T6008: Editing files and contributing changes via web from In Progress to Details on the Wikimedia board.
Oct 9 2014, 11:52 AM · Restricted Project, Wikimedia, Diffusion, Differential
• andrew.grant created T6278: Possible to limit revision import on remote svn repository?.
Oct 9 2014, 10:00 AM · Diffusion
Unknown Object (User) moved T6008: Editing files and contributing changes via web from Details to In Progress on the Wikimedia board.
Oct 9 2014, 8:47 AM · Restricted Project, Wikimedia, Diffusion, Differential

Oct 8 2014

epriestley added a comment to T5132: Document the special syntax you can use in commit messages to cause effects.
Oct 8 2014, 4:09 PM · Guides, Diffusion
dserafin added a comment to T2999: When we detect a file as binary, explain why.

The first time you commit a good (UTF-8) file it will still list as binary because the old side of the diff is binary. The second good commit will result in a good new and old version that can be text diff'ed. Also, pure ASCII is not needed, UTF-8 is fine so you should be able to use whatever characters you need for your native language.

Oct 8 2014, 1:16 PM · Arcanist, Diffusion

Oct 7 2014

chad triaged T6131: Mirror Status on repository edit view as Low priority.
Oct 7 2014, 10:22 PM · Diffusion

Oct 6 2014

altendky added a comment to T2999: When we detect a file as binary, explain why.

The first time you commit a good (UTF-8) file it will still list as binary because the old side of the diff is binary. The second good commit will result in a good new and old version that can be text diff'ed. Also, pure ASCII is not needed, UTF-8 is fine so you should be able to use whatever characters you need for your native language.

Oct 6 2014, 6:13 PM · Arcanist, Diffusion
tgrosinger added a comment to T4330: Associate Git commit to Maniphest task.

Ah, I think I know what is causing this issue then. It says that auto-close is not enabled until the import is complete. For some reason my repo has been stuck at 99.98% finished for several days.

Oct 6 2014, 3:42 PM · Maniphest, Diffusion
dserafin added a comment to T2999: When we detect a file as binary, explain why.

Same problem with our assembly files (working directory on Windows7 disk folder): is not possible to view them in Audit because they're seen as "binary" (while they are visible as diffs in Differential probably because unix diff we use fixes encoding). Verifying their nature with cygwin 'file' tool they are seen as:

Oct 6 2014, 2:48 PM · Arcanist, Diffusion

Oct 3 2014

chad added a comment to T4330: Associate Git commit to Maniphest task.

My bad, missed that document!

Oct 3 2014, 10:43 PM · Maniphest, Diffusion
epriestley added a comment to T4330: Associate Git commit to Maniphest task.

It's expected to work with commits.

Oct 3 2014, 10:39 PM · Maniphest, Diffusion