Bug ReportBugs
ArchivedPublic

Watchers

  • This project does not have any watchers.

Details

Description

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

Recent Activity

Yesterday

epriestley archived Bug Report.
Wed, Dec 13, 1:14 PM

Tue, Dec 12

epriestley merged task T9915: Audit Changes TOC fail redirecting to right subdirectory when clicking on fa-folder-open icon and produces an unhandled exception into T13030: Accessing a submodule path directly in Diffusion has less than optimal UX.
Tue, Dec 12, 8:54 PM · Restricted Project, Diffusion, Bug Report

Sat, Dec 9

aklapper added a comment to T10221: upgrading to ubuntu 15.10 breaks google authentication.

Support for Ubuntu 15.10 ended on 2016-07-28.
Did this problem also happen with any later, still supported Ubuntu version (like 16.04) or can this task be closed as obsolete?

Sat, Dec 9, 7:48 PM · Auth, Bug Report

Thu, Nov 30

epriestley closed T10233: arc error message unclear for accepted revision in "plan change" state as Resolved by committing rARCf4c80a114d36: Make "arc land" prompt on "Changes Planned" revisions more explicit.
Thu, Nov 30, 9:51 PM · Restricted Project, Bug Report
epriestley closed T12592: User awaiting approval has no image as Resolved.

This should be fixed by changes in T13024.

Thu, Nov 30, 9:43 PM · Bug Report, Auth
epriestley closed T12657: Add Authentication Factor fails silently for users without a verified email address as Resolved.

This should be fixed by changes in T13024.

Thu, Nov 30, 9:43 PM · Auth, Bug Report
epriestley merged T12570: Installs without `gd` are unable to reach the setup warning telling them to install `gd` into T10405: Phabricator should generally work properly even if "gd" is not installed.
Thu, Nov 30, 9:42 PM · Setup, Restricted Project
epriestley merged task T12570: Installs without `gd` are unable to reach the setup warning telling them to install `gd` into T10405: Phabricator should generally work properly even if "gd" is not installed.
Thu, Nov 30, 9:42 PM · Setup, Bug Report
epriestley added a commit to T10233: arc error message unclear for accepted revision in "plan change" state: rP0807b70ea167: Add an explicit warning in the Differential transaction log when users skip….
Thu, Nov 30, 7:04 PM · Restricted Project, Bug Report
epriestley added a revision to T10233: arc error message unclear for accepted revision in "plan change" state: D18808: Add an explicit warning in the Differential transaction log when users skip review.
Thu, Nov 30, 4:26 PM · Restricted Project, Bug Report
epriestley added a revision to T10233: arc error message unclear for accepted revision in "plan change" state: D18807: Make "arc land" prompt on "Changes Planned" revisions more explicit.
Thu, Nov 30, 3:53 PM · Restricted Project, Bug Report

Wed, Nov 29

epriestley added a comment to T10233: arc error message unclear for accepted revision in "plan change" state.

In PHI231, a more experienced user went through this workflow:

Wed, Nov 29, 7:24 PM · Restricted Project, Bug Report

Tue, Nov 28

lpriestley added a comment to T12544: Cannot add invitees to all future instances of a recurring event.

@epriestley what do you think of creating another column in the calendar_eventinvitee table, something like a bool futureStatus so that when we get a list of invitees back, we can check for that flag. I think this would mean a lot date comparisons (seems inefficient), and I'm not quite sure when to populate event stubs. For example, if the futureStatus flag is set to true for instance 10, and there are stubs for instances 15 and 20, when would 15 and 20 get converted to have the updated status?

Tue, Nov 28, 11:16 PM · Calendar, Bug Report

Thu, Nov 23

aik099 added a comment to T12098: Create Revision from FavoritesMenuItem doesn't work.

Obviously not a particularly important issue, but this now will populate a link in the favorites menu which leads to an exception:

Thu, Nov 23, 7:28 AM · Profile Menu Item, Differential, Bug Report
sshannin added a comment to T12098: Create Revision from FavoritesMenuItem doesn't work.

Obviously not a particularly important issue, but this now will populate a link in the favorites menu which leads to an exception:

Thu, Nov 23, 3:33 AM · Profile Menu Item, Differential, Bug Report

Tue, Nov 21

Basheer added a comment to T12741: Phabricator IP address or Mac Address level Blocking Issue..

Hi Epriestley,

Tue, Nov 21, 2:59 PM · Bug Report
Basheer renamed T12741: Phabricator IP address or Mac Address level Blocking Issue. from Unhandled Exception ("HTTPFutureCURLResponseStatus") to Phabricator IP address or Mac Address level Blocking Issue..
Tue, Nov 21, 2:58 PM · Bug Report
mcorteel updated subscribers of T12785: Arc unit command fails with PHPUnit 6.

@epriestley I tried submitting a patch, but I get a Permission denied (publickey) which results in Unable to push changes to the staging area..
I added my public key and created a config rule to use it for requests to secure.phabricator.com, I'm not sure what else to do.

Tue, Nov 21, 1:37 PM · Arcanist, Bug Report
mcorteel added a comment to T12785: Arc unit command fails with PHPUnit 6.

So, to recap:

  • PhpunitTestEngine::run must be updated to use --log-junit which will output XML instead of JSON
  • ArcanistPhpunitTestResultParser::parseTestResults must be updated to parse this XML file.

This seems simple enough, I will try to make it work.

Tue, Nov 21, 11:28 AM · Arcanist, Bug Report

Nov 2 2017

epriestley added a commit to T12689: Mail is still received after resigning from a revision: rP03d059dd26e2: Don't include resigned reviewers in the Differential "To" list.
Nov 2 2017, 12:22 AM · Mail, Differential, Bug Report

Nov 1 2017

epriestley added a revision to T12689: Mail is still received after resigning from a revision: D18758: Don't include resigned reviewers in the Differential "To" list.
Nov 1 2017, 6:12 PM · Mail, Differential, Bug Report
epriestley updated the task description for T12689: Mail is still received after resigning from a revision.
Nov 1 2017, 6:10 PM · Mail, Differential, Bug Report
epriestley merged T7704: Rejecting then accepting a revision with blocking reviewers doesn't correctly change state into T12920: Retracting requested changes can leave revision in "Needs Revision" state.
Nov 1 2017, 6:05 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T12920: Retracting requested changes can leave revision in "Needs Revision" state.

Pretty sure T7704 is the same as this.

Nov 1 2017, 6:05 PM · Restricted Project, Differential, Bug Report

Oct 31 2017

epriestley added a comment to T10109: Differental does not trigger Herald for mundane updates.

Ah, thanks. I think your guess is right. I'll make a note in T2543, should be a one-line fix I think.

Oct 31 2017, 1:51 PM · Differential, Restricted Project, Herald, Bug Report
avivey added a comment to T10109: Differental does not trigger Herald for mundane updates.

From https://discourse.phabricator-community.org/t/644 - it appears that HM now triggers a build on revisions when they are closed via landing, which fails because the build expects a Staging ref. I don't have the setup to test any of it, but the latest revision landed here doesn't show the behavior, so either f7f3dd5b2084 hasn't rolled in here yet, or that user is doing something funny.

Oct 31 2017, 11:09 AM · Differential, Restricted Project, Herald, Bug Report

Oct 27 2017

epriestley closed T10109: Differental does not trigger Herald for mundane updates as Resolved by committing rPf7f3dd5b2084: Don't run Herald build and mail rules when they don't make sense.
Oct 27 2017, 3:44 PM · Differential, Restricted Project, Herald, Bug Report

Oct 24 2017

epriestley added a revision to T10109: Differental does not trigger Herald for mundane updates: D18731: Don't run Herald build and mail rules when they don't make sense.
Oct 24 2017, 9:21 PM · Differential, Restricted Project, Herald, Bug Report

Oct 16 2017

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.

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

Oct 7 2017

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.

Oct 7 2017, 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".
Oct 7 2017, 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".
Oct 7 2017, 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
Oct 7 2017, 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.
Oct 7 2017, 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.
Oct 7 2017, 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.

Oct 7 2017, 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.
Oct 7 2017, 9:20 AM · Bug Report
avivey created T13004: [Stable] bin/storage dump fails if there's a password set.
Oct 7 2017, 9:16 AM · Bug Report

Oct 5 2017

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:

Oct 5 2017, 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.

Oct 5 2017, 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:

Oct 5 2017, 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:

Oct 5 2017, 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 .

Oct 5 2017, 9:14 AM · Bug Report

Oct 2 2017

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

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

Sep 25 2017

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.

Sep 25 2017, 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