Bug ReportBugs
ActivePublic

Watchers

  • This project does not have any watchers.

Details

Description

This task was filed via the "New Bug Report" form.

Recent Activity

Yesterday

mruuska added a comment to T12835: Improve parsing of unusual "git diff --no-index --no-prefix" diffs.

I'm getting the same #1048: Column 'filename' cannot be null running just regular "arc diff". It used to work and then at some point after updates started failing.

Mon, Oct 16, 3:52 PM · Arcanist, Bug Report

Sat, Oct 7

epriestley added a comment to T13004: [Stable] bin/storage dump fails if there's a password set.

We should probably update the documentation to recommend --output and --compress, since the error behavior (e.g., in the event of a full disk) is better.

Sat, Oct 7, 8:26 PM · Bug Report
epriestley added a commit to T13004: [Stable] bin/storage dump fails if there's a password set: rP8edd04aff365: (stable) Fix "bin/storage dump" with no "--output".
Sat, Oct 7, 8:24 PM · Bug Report
epriestley added a commit to T13004: [Stable] bin/storage dump fails if there's a password set: rP4fd9d2d4bbdb: Fix "bin/storage dump" with no "--output".
Sat, Oct 7, 8:24 PM · Bug Report
borna.emami added a comment to T13004: [Stable] bin/storage dump fails if there's a password set.

Thanks for the fix on that. I have my nightly backups running again, however I had to switch from

./bin/storage dump | gzip > file.sql.gz

to

phab/bin/storage dump --output file.sql.gz --compress
Sat, Oct 7, 7:51 PM · Bug Report
epriestley added a commit to T13004: [Stable] bin/storage dump fails if there's a password set: rP0361591da0d9: (stable) Correct "bin/storage dump" command construction with passwords.
Sat, Oct 7, 12:00 PM · Bug Report
epriestley closed T13004: [Stable] bin/storage dump fails if there's a password set as Resolved by committing rP1ee7b3ab8cbb: Correct "bin/storage dump" command construction with passwords.
Sat, Oct 7, 12:00 PM · Bug Report
borna.emami added a comment to T13004: [Stable] bin/storage dump fails if there's a password set.

Thanks for the quick response. I'll try the workaround in the morning.

Sat, Oct 7, 11:38 AM · Bug Report
avivey renamed T13004: [Stable] bin/storage dump fails if there's a password set from bin/storage dump fails if there's a password set to [Stable] bin/storage dump fails if there's a password set.
Sat, Oct 7, 9:20 AM · Bug Report
avivey created T13004: [Stable] bin/storage dump fails if there's a password set.
Sat, Oct 7, 9:16 AM · Bug Report

Thu, Oct 5

epriestley added a comment to T12907: Unable to download large files through alternative file domain.

On the bin/files download side of things, we currently buffer the whole file into a string and then hit this:

Thu, Oct 5, 8:41 PM · Bug Report
epriestley added a comment to T12907: Unable to download large files through alternative file domain.

For consistency, I gave the content Cloudfront distribution a 60-second timeout as well.

Thu, Oct 5, 8:28 PM · Bug Report
epriestley added a comment to T12907: Unable to download large files through alternative file domain.

Pulling stuff from admin.phacility.com via Cloudfront, I get this instead:

Thu, Oct 5, 8:20 PM · Bug Report
epriestley added a comment to T12907: Unable to download large files through alternative file domain.

In the log above, this download should have worked:

Thu, Oct 5, 7:43 PM · Bug Report
ragnalinux added a comment to T10632: Error displaying a Differential revision.

i encountered same problem , but it happened on libphutil/src/xsprintf/xsprintf.php. I resolved this problem by setting max_nesting_level on php xdebug .

Thu, Oct 5, 9:14 AM · Bug Report

Mon, Oct 2

jkrets added a comment to T11030: Cannot clone public repository over HTTP anonymously..

Sorry to leave this hanging, but the issue was with our server configuration. We did not have the correct sudo rights confgured

Mon, Oct 2, 3:18 PM · Diffusion, Bug Report

Mon, Sep 25

epriestley closed T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs as Resolved.

This is now in stable, and deployed here and to the Phacility production cluster.

Mon, Sep 25, 1:40 PM · Customer Impact, Diffusion, Bug Report

Sep 15 2017

epriestley added a comment to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs.

This is now deployed on this server, we'll see how it holds up. It won't promote to stable until next week, and I'd discourage anyone from picking it up right away.

Sep 15 2017, 5:45 PM · Customer Impact, Diffusion, Bug Report
epriestley added a commit to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: rP49b718178066: Update utility "bin/repository parents" workflow to work with RefPosition.
Sep 15 2017, 5:21 PM · Customer Impact, Diffusion, Bug Report
epriestley added a commit to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: rP8982e3e52dd8: Update major RefCursor callsites to work properly with RefPosition.
Sep 15 2017, 5:21 PM · Customer Impact, Diffusion, Bug Report
epriestley added a commit to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: rP5cf62f86d728: Remove obsolete columns from RefCursor table.
Sep 15 2017, 5:21 PM · Customer Impact, Diffusion, Bug Report
epriestley added a commit to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: rP782b18e7e2ed: Migrate RefCursor data to RefPosition table.
Sep 15 2017, 5:19 PM · Customer Impact, Diffusion, Bug Report
epriestley added a commit to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: rP9d5a2b3b4f68: Add a RefPosition table to hold branch/tag positions once the RefCursor table….
Sep 15 2017, 5:19 PM · Customer Impact, Diffusion, Bug Report
epriestley added a revision to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: D18615: Update utility "bin/repository parents" workflow to work with RefPosition.
Sep 15 2017, 4:24 PM · Customer Impact, Diffusion, Bug Report
epriestley added a revision to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: D18614: Update major RefCursor callsites to work properly with RefPosition.
Sep 15 2017, 4:20 PM · Customer Impact, Diffusion, Bug Report
epriestley added a revision to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: D18613: Remove obsolete columns from RefCursor table.
Sep 15 2017, 3:36 PM · Customer Impact, Diffusion, Bug Report
epriestley added a revision to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: D18612: Migrate RefCursor data to RefPosition table.
Sep 15 2017, 3:28 PM · Customer Impact, Diffusion, Bug Report

Sep 14 2017

epriestley added a revision to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: D18602: Add a RefPosition table to hold branch/tag positions once the RefCursor table is split.
Sep 14 2017, 12:13 PM · Customer Impact, Diffusion, Bug Report
epriestley added a comment to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs.

In Mercurial, can a branch have some open heads and some closed heads?

Sep 14 2017, 12:03 PM · Customer Impact, Diffusion, Bug Report

Sep 13 2017

epriestley added a commit to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: rPbc995048c913: (stable) Work around workflow blocking error with duplicate "master" refs in….
Sep 13 2017, 11:05 PM · Customer Impact, Diffusion, Bug Report
epriestley added a commit to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: rPc310f08b7a5c: Work around workflow blocking error with duplicate "master" refs in "Land….
Sep 13 2017, 11:03 PM · Customer Impact, Diffusion, Bug Report
epriestley added a comment to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs.

See also PHI68.

Sep 13 2017, 10:55 PM · Customer Impact, Diffusion, Bug Report
epriestley added a comment to T12033: Large diffs can still repeatedly fail to insert.

Oh, yes, sorry, looked at the wrong tab.

Sep 13 2017, 10:49 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
alexmv added a comment to T12033: Large diffs can still repeatedly fail to insert.

Do you mean "follow up in PHI55"?

Sep 13 2017, 10:40 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
epriestley added a comment to T12033: Large diffs can still repeatedly fail to insert.

I'm going to presume this is resolved by D18598 -- I updated T11767 with more plans and context for smoothing over the behavior, but I think it will no longer be disastrously bad. I'll follow up in PHI36 PHI55.

Sep 13 2017, 10:34 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
epriestley added a commit to T12033: Large diffs can still repeatedly fail to insert: rP6fb3f857fb57: Stop the bleeding caused by attaching enormous patches to revision mail.
Sep 13 2017, 10:33 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
epriestley added a revision to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs: D18599: Work around workflow blocking error with duplicate "master" refs in "Land Revision".
Sep 13 2017, 7:21 PM · Customer Impact, Diffusion, Bug Report

Sep 12 2017

epriestley added a revision to T12033: Large diffs can still repeatedly fail to insert: D18598: Stop the bleeding caused by attaching enormous patches to revision mail.
Sep 12 2017, 10:43 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
epriestley closed T12443: Applying fulltext limits first causes missing results as Resolved.

This is resolved by the Ferret engine, which can execute all parts of the query logic in MySQL.

Sep 12 2017, 3:09 PM · Restricted Project, Search, Bug Report

Sep 6 2017

epriestley closed T12118: Differential email does not contain reviewers/subscribers information anymore as Resolved by committing rPe91d72fefbf9: Un-hide the "X added reviewers: ..." transactions in revision creation mail.
Sep 6 2017, 5:23 PM · Restricted Project, Differential, Bug Report
epriestley added a revision to T12118: Differential email does not contain reviewers/subscribers information anymore: D18542: Un-hide the "X added reviewers: ..." transactions in revision creation mail.
Sep 6 2017, 4:45 PM · Restricted Project, Differential, Bug Report

Sep 1 2017

epriestley added a parent task for T12443: Applying fulltext limits first causes missing results: T12974: Upgrading: "Ferret" Fulltext Engine.
Sep 1 2017, 5:06 PM · Restricted Project, Search, Bug Report

Aug 28 2017

epriestley added a revision to T12443: Applying fulltext limits first causes missing results: D18484: Build a prototype fulltext engine ("Ferret") using only basic MySQL primitives.
Aug 28 2017, 9:35 PM · Restricted Project, Search, Bug Report

Aug 24 2017

jkrets added a comment to T11030: Cannot clone public repository over HTTP anonymously..

I'm not sure it's a bug. The error message was surprising; HTTP/1.0 500 Error 1: sudo: a password is required. I see a matching error message in Q261. The solution there seems to be allowing the webserver process owner to have sudo rights. I now found there are docs on the subject diffusion_hosting#configuring-sudo.

Aug 24 2017, 11:59 AM · Diffusion, Bug Report

Aug 23 2017

Herald updated subscribers of T12779: File anchor links can load scrolled to the wrong file.
Aug 23 2017, 9:27 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T11030: Cannot clone public repository over HTTP anonymously..

If you believe you've found a bug in Phabricator, please follow the instructions in Contributing Bug Reports to file a report.

Aug 23 2017, 6:01 PM · Diffusion, Bug Report
jkrets added a comment to T11030: Cannot clone public repository over HTTP anonymously..

This might be useful debugging information:

>> export GIT_CURL_VERBOSE=1
>> git clone http://phabricator.internal/diffusion/TRACTORAPI/tractor-api.git
Cloning into 'tractor-api'...
* Couldn't find host phabricator.internal in the .netrc file; using defaults
* About to connect() to phabricator.internal port 80 (#0)
*   Trying 10.11.45.21... * Connected to phabricator.internal (10.11.45.21) port 80 (#0)
> GET /diffusion/TRACTORAPI/tractor-api.git/info/refs?service=git-upload-pack HTTP/1.1
User-Agent: git/2.14.1
Host: phabricator.internal
Accept: */*
Accept-Encoding: gzip
Accept-Language: en-US, *;q=0.9
Pragma: no-cache
Aug 23 2017, 4:58 PM · Diffusion, Bug Report
remusvrm added a comment to T11030: Cannot clone public repository over HTTP anonymously..

This doesn't work again.

Aug 23 2017, 4:52 PM · Diffusion, Bug Report

Aug 22 2017

chad added a comment to T12871: Setting a restricted dashboard as the topmost item in the global menu can lock some users out.

Somewhat related, if you have a disabled "Home" as your top item, we still show that regardless of the active dashboard below it.

Aug 22 2017, 10:23 PM · Policy, Profile Menu Item, Dashboards, Bug Report

Aug 17 2017

chad added a project to T12967: Mercurical branches don't display tracking status properly: Bug Report.
Aug 17 2017, 7:32 PM · Bug Report, Diffusion (v3)