Page MenuHomePhabricator
Feed Advanced Search

Dec 31 2016

dereckson created T12053: Arcanist tries to parse phpunit output in the test plan as another test plan field.
Dec 31 2016, 3:27 AM · Arcanist, Bug Report
nipunn created T12052: Dependent revision stack UI showing endpoints incorrectly.
Dec 31 2016, 2:10 AM · Bug Report (Needs Information)

Dec 29 2016

epriestley triaged T11171: When clicking an image in the comment edit modal, the image modal gets under the edit modal as Wishlist priority.

Kicking this out of Differential because this is reproducible in every application that allows you to edit comments.

Dec 29 2016, 6:09 PM · Lightbox, Bug Report
epriestley removed a project from T11171: When clicking an image in the comment edit modal, the image modal gets under the edit modal: Differential.
Dec 29 2016, 6:04 PM · Lightbox, Bug Report
epriestley merged T7838: Link to inline comment opens in new tab, scrolls to inline comment, then scrolls to bottom of page. into T11784: Differential "stick to the comment area" and "jump to anchor" scroll behaviors interact destructively while loading content.
Dec 29 2016, 6:02 PM · Inline Comments, Differential, Bug Report
epriestley moved T11784: Differential "stick to the comment area" and "jump to anchor" scroll behaviors interact destructively while loading content from Backlog to Inline Infra on the Differential board.
Dec 29 2016, 6:01 PM · Inline Comments, Differential, Bug Report
allixsenos added a comment to T12051: "Autoclose: Disabled" could provide a stronger hint that it means "You, the user, have disabled Autoclose by adjusting a setting.".

it's a "fresh" repo, created yesterday, with no commits. I immediately
disabled those upon creation and then circled back the next day considering
it "completely new"

Dec 29 2016, 5:45 PM · Diffusion
epriestley added a comment to T12051: "Autoclose: Disabled" could provide a stronger hint that it means "You, the user, have disabled Autoclose by adjusting a setting.".

How did you disable this option in a newly created Git repository without realizing you had made the change?

Dec 29 2016, 5:39 PM · Diffusion
allixsenos added a comment to T12051: "Autoclose: Disabled" could provide a stronger hint that it means "You, the user, have disabled Autoclose by adjusting a setting.".

ah. yes. I turned those off.

Dec 29 2016, 5:35 PM · Diffusion
epriestley moved T11648: Jumping to an reply inline comment via URL anchor should highlight the comment node accurately from Backlog to Inline Infra on the Differential board.
Dec 29 2016, 4:31 PM · Inline Comments, Differential, Bug Report
epriestley moved T10323: "Action With No Effect" when marking inline comments as done from Backlog to v3 (Infrastructure) on the Differential board.
Dec 29 2016, 4:23 PM · Restricted Project, Restricted Project, Bug Report, Differential
epriestley moved T10312: Always interpret the first line of commit messages as a title, even if it begins with a recognized label like "Test Plan:" from Backlog to v3 (Infrastructure) on the Differential board.
Dec 29 2016, 4:22 PM · Restricted Project, Diffusion, Differential, Arcanist, Bug Report
epriestley moved T11344: Differential commit message parser immediately rejects objects the user can't see, but Herald can put them there from Backlog to v3 (Infrastructure) on the Differential board.
Dec 29 2016, 4:22 PM · Restricted Project, Differential, Arcanist, Bug Report
epriestley closed T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured as Resolved.

This should be resolved by T12049 + T11114, once Differential moves to the new comment area UI. The new universal haunted mode puts a spacer into the document while the panel is haunted, so if the panel is 500px tall we put a 500px empty space at the bottom of the document. This means that all content is always visible while scrolling.

Dec 29 2016, 4:16 PM · Differential, Bug Report
epriestley added a parent task for T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured: T12049: Give EditEngine universal support for "Haunted Mode".
Dec 29 2016, 4:13 PM · Differential, Bug Report
epriestley moved T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured from Backlog to v3 (Infrastructure) on the Differential board.
Dec 29 2016, 4:12 PM · Differential, Bug Report
epriestley removed a watcher for Bug Report: • Gadget0624.
Dec 29 2016, 4:11 PM
epriestley removed a watcher for Bug Report: • KalleKyllonen.
Dec 29 2016, 4:11 PM
epriestley removed a watcher for Bug Report: • multitek-inc27.
Dec 29 2016, 4:11 PM
epriestley removed a watcher for Bug Report: • naddlk.
Dec 29 2016, 4:11 PM
epriestley added a comment to T12051: "Autoclose: Disabled" could provide a stronger hint that it means "You, the user, have disabled Autoclose by adjusting a setting.".

I can't immediately reproduce this.

Dec 29 2016, 4:09 PM · Diffusion
epriestley moved T12033: Large diffs can still repeatedly fail to insert from Backlog to Future Work on the Differential board.
Dec 29 2016, 3:40 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
allixsenos updated the task description for T12051: "Autoclose: Disabled" could provide a stronger hint that it means "You, the user, have disabled Autoclose by adjusting a setting.".
Dec 29 2016, 11:34 AM · Diffusion
allixsenos created T12051: "Autoclose: Disabled" could provide a stronger hint that it means "You, the user, have disabled Autoclose by adjusting a setting.".
Dec 29 2016, 11:34 AM · Diffusion

Dec 26 2016

epriestley closed T12045: Commandeered revisions do not show original author as "Authored By" as Wontfix.

Although this wording is a little unclear, commandeering makes you the author. A more precise wording would be something like "The current author for this revision is X. It was created on Y." but that's clunky and only smooths out this particular case.

Dec 26 2016, 4:08 PM · Bug Report
plonk created T12045: Commandeered revisions do not show original author as "Authored By".
Dec 26 2016, 3:31 PM · Bug Report

Dec 24 2016

chad added a comment to T12044: Upgrading phabricator resulted in excess tables/table data.

I have no idea how to reproduce this error. I'd guess something along the lines of take a new version of Phabricator and apply an old upgrade script, but that's really just a guess.

Dec 24 2016, 4:43 PM · Bug Report (Needs Information)
eFrane updated the task description for T12044: Upgrading phabricator resulted in excess tables/table data.
Dec 24 2016, 10:51 AM · Bug Report (Needs Information)
eFrane created T12044: Upgrading phabricator resulted in excess tables/table data.
Dec 24 2016, 10:41 AM · Bug Report (Needs Information)

Dec 23 2016

epriestley closed T12039: Emails from Diffusion contain space or new line character in link making them not clickable as Resolved.

After D17103, at HEAD of master:

Dec 23 2016, 7:57 PM · Bug Report
epriestley closed T12040: Make version reporting more clear when running a fork, a subtask of T12039: Emails from Diffusion contain space or new line character in link making them not clickable, as Resolved.
Dec 23 2016, 7:42 PM · Bug Report

Dec 22 2016

jessjohnson added a comment to T12039: Emails from Diffusion contain space or new line character in link making them not clickable.

Thanks I made the smtp-encoding change. We're just about ready to update to a vanilla install. If it doesn't fix it and I can provide reproducible steps, I'll update this bug report.

Dec 22 2016, 5:31 PM · Bug Report
epriestley added a comment to T12039: Emails from Diffusion contain space or new line character in link making them not clickable.

Hmm, I don't see that one in the upstream either. Maybe your origin is also a fork?

Dec 22 2016, 5:26 PM · Bug Report
jessjohnson added a comment to T12039: Emails from Diffusion contain space or new line character in link making them not clickable.

We're running a fork and sorry, I know it's a little behind stable. We're working on getting back to a vanilla install to make updates easier.

$ git merge-base HEAD origin/master
33bf2a79def2e03c2248730afa080ead489942be
Dec 22 2016, 4:32 PM · Bug Report
epriestley added a comment to T12039: Emails from Diffusion contain space or new line character in link making them not clickable.

What does this show, when run in your local working copy?

Dec 22 2016, 4:27 PM · Bug Report
jessjohnson created T12039: Emails from Diffusion contain space or new line character in link making them not clickable.
Dec 22 2016, 4:22 PM · Bug Report
epriestley edited projects for T12033: Large diffs can still repeatedly fail to insert, added: Bug Report, Differential, Diffusion, Mail; removed Bug Report (Needs Information).
Dec 22 2016, 2:01 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
sreschke80 added a comment to T10608: Arc diff failing to generate diff with svn 1.9.3.

My workaround for arcanist:

Dec 22 2016, 10:30 AM · Arcanist, Bug Report

Dec 21 2016

chad merged T12036: hg push fails (remote: Exception: Unknown Mercurial command '448!) into T10382: Transaction abort when pushing via SSH to a Mercurial repository.
Dec 21 2016, 6:05 PM · Mercurial, Bug Report
chad merged task T12036: hg push fails (remote: Exception: Unknown Mercurial command '448!) into T10382: Transaction abort when pushing via SSH to a Mercurial repository.
Dec 21 2016, 6:05 PM · Bug Report
epriestley closed T12037: reboot after SD card is removed as Invalid.
Dec 21 2016, 2:19 PM · Bug Report
• Tools_Deployment created T12037: reboot after SD card is removed.
Dec 21 2016, 9:55 AM · Bug Report
marco-m added a comment to T12036: hg push fails (remote: Exception: Unknown Mercurial command '448!).

I had a look at other bug reports. It looks like I am not the only one having this kind of problems. So it seems that hg >= 4.x is not supported yet, and actually not even the whole 3.x is supported.

Dec 21 2016, 9:28 AM · Bug Report
marco-m renamed T12036: hg push fails (remote: Exception: Unknown Mercurial command '448!) from hg push fails (remote: abort: stream ended unexpectedly) to hg push fails (remote: Exception: Unknown Mercurial command '448!).
Dec 21 2016, 9:23 AM · Bug Report
marco-m created T12036: hg push fails (remote: Exception: Unknown Mercurial command '448!).
Dec 21 2016, 9:20 AM · Bug Report
cspeckmim added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

@gabe - That's the same issue, I believe. You should be able to push using Mercurial 3.4. I've been meaning to do some testing to try and fix this over the winter break but a day is no longer what it used to be.

Dec 21 2016, 5:46 AM · Mercurial, Bug Report
gabe added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

Not sure if this is the same issue, this is on a new phacility hosted phabricator instance, when attempting to push to a new mercurial repo:

Dec 21 2016, 3:50 AM · Mercurial, Bug Report

Dec 20 2016

epriestley closed T12035: Diffusion URI without trailing slash redirects to login page as Resolved by committing Restricted Diffusion Commit.
Dec 20 2016, 3:48 PM · Diffusion, Bug Report
epriestley added a comment to T12035: Diffusion URI without trailing slash redirects to login page.

Thanks, I think D17097 should fix this. It isn't intended, and is just a curious artifact of some other behaviors.

Dec 20 2016, 1:54 PM · Diffusion, Bug Report
epriestley claimed T12035: Diffusion URI without trailing slash redirects to login page.
Dec 20 2016, 1:54 PM · Diffusion, Bug Report
epriestley added a revision to T12035: Diffusion URI without trailing slash redirects to login page: Restricted Differential Revision.
Dec 20 2016, 1:43 PM · Diffusion, Bug Report
aubort updated the task description for T12035: Diffusion URI without trailing slash redirects to login page.
Dec 20 2016, 12:56 PM · Diffusion, Bug Report
epriestley added a comment to T12033: Large diffs can still repeatedly fail to insert.

I strongly suspect this report has enough information to let us replicate the issue locally, so you don't need to take any further action yet. I'll follow up if I have difficulty replicating it and we can figure out what differs between my synthetic test case and your real environment.

Dec 20 2016, 12:42 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
epriestley claimed T12033: Large diffs can still repeatedly fail to insert.
Dec 20 2016, 12:40 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
dereckson added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Reference in PHP 7.1 migration guide: http://php.net/manual/en/migration71.new-features.php#migration71.new-features.asynchronous-signal-handling

Dec 20 2016, 12:20 PM · Daemons, Bug Report
aubort created T12035: Diffusion URI without trailing slash redirects to login page.
Dec 20 2016, 10:04 AM · Diffusion, Bug Report
alexmv added a comment to T12033: Large diffs can still repeatedly fail to insert.

It will likely be easiest to replicate again after upgrading to Week 52 stable this Friday, since I can't use precise same diff, and don't know the precise set of replication criteria yet.

Dec 20 2016, 1:52 AM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
avivey added a comment to T12033: Large diffs can still repeatedly fail to insert.

You can always create a test instance in Phacility, and upload the diff there - those are free and temporary - see Reproduction/Resources
There's also rGITTEST, although maybe it's not so good for a super huge diff...

Dec 20 2016, 1:23 AM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report

Dec 19 2016

gregprice added a comment to T12033: Large diffs can still repeatedly fail to insert.

We weren't running quite HEAD then, but we were running a version that was our patches on top of something about a week old -- see the end of the description of T11743, which was the first report in that chain. Our version today is a couple of months old, so I see why that'd feel different.

Dec 19 2016, 11:35 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
alexmv added a comment to T12033: Large diffs can still repeatedly fail to insert.

Hm; we weren't running HEAD when we reported T11748. We have an upgrade planned to latest-stable in a week or so -- will that be sufficient to re-test with? Or is there a repository that we can apply a 4,800-file diff to on Phacility?

Dec 19 2016, 11:16 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
chad added a comment to T12033: Large diffs can still repeatedly fail to insert.

Please update to HEAD before filing a bug report. We can't take reports against old versions of Phabricator. You can use Phacility if you'd like to test an up to date installation.

Dec 19 2016, 11:11 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
alexmv created T12033: Large diffs can still repeatedly fail to insert.
Dec 19 2016, 10:54 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
epriestley closed T12031: PhabricatorProjectWorkboardProfilePanel still defined in legacy database upgrade code as Resolved by committing Restricted Diffusion Commit.
Dec 19 2016, 8:53 PM · Installing & Upgrading, Bug Report
dereckson updated the task description for T12031: PhabricatorProjectWorkboardProfilePanel still defined in legacy database upgrade code.
Dec 19 2016, 8:47 PM · Installing & Upgrading, Bug Report
epriestley added a comment to T12031: PhabricatorProjectWorkboardProfilePanel still defined in legacy database upgrade code.

(2) and (3) are correct.

Dec 19 2016, 6:22 PM · Installing & Upgrading, Bug Report
dereckson added a revision to T12031: PhabricatorProjectWorkboardProfilePanel still defined in legacy database upgrade code: Restricted Differential Revision.
Dec 19 2016, 6:04 PM · Installing & Upgrading, Bug Report
dereckson created T12031: PhabricatorProjectWorkboardProfilePanel still defined in legacy database upgrade code.
Dec 19 2016, 6:03 PM · Installing & Upgrading, Bug Report

Dec 18 2016

avivey added a comment to T11992: daemon hits policy exception repeatedly.

I've been playing with it for maybe half an hour, and couldn't reproduce:

  • Made revision public
  • stopped the daemons
  • mention a user
  • updated the revision to a new diff
  • made the revision not-visible to that user (visible only to the acting user)
  • start daemons and let them finish the queue
Dec 18 2016, 12:44 AM · Daemons, Policy, Bug Report

Dec 17 2016

• sshannin added a comment to T11992: daemon hits policy exception repeatedly.

Thanks for the advice. That yields the trace below. My understanding is that it was a diff with policy set to subscribers which had an author plus a single subscriber. That subscriber was added by herald upon the diff's creation, so there's certainly grounds for something going on there. I had trouble divining the exact issue, but tinkering with the permissions for the object ended up letting the emails go through.

Dec 17 2016, 9:48 PM · Daemons, Policy, Bug Report
avivey added a comment to T11992: daemon hits policy exception repeatedly.

We used to have a tool to show a task's data, but it must have gotten lost in some refactoring.

Dec 17 2016, 8:13 PM · Daemons, Policy, Bug Report
avivey added a comment to T11992: daemon hits policy exception repeatedly.
  • In Daemons, find the task ID in the leftmost column of "Leased Tasks".
  • Run bin/worker execute --id <id> --trace to execute the task in the foreground.

That will give you a more detailed error information which you can paste here.

Dec 17 2016, 8:04 PM · Daemons, Policy, Bug Report
• sshannin added a comment to T11992: daemon hits policy exception repeatedly.

Just updated to head and am seeing the same issues.

Dec 17 2016, 6:19 PM · Daemons, Policy, Bug Report
gim913 added a comment to T11833: "parent"/"child" relationship for Revisions is confusing.

I know it's closed, but here are few proposals (not sure if better in any way):

  • base
  • derived / derivative
Dec 17 2016, 11:51 AM · Differential, Bug Report

Dec 16 2016

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

cat is slow to appear

Dec 16 2016, 7:27 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.

I see cat :)

Dec 16 2016, 7:22 PM · Customer Impact, Diffusion, Bug Report
chad added a comment to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs.

ok now cat is in feed

Dec 16 2016, 7:22 PM · Customer Impact, Diffusion, Bug Report
chad added a comment to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs.

hmm, cat no show in feed

Dec 16 2016, 7:07 PM · Customer Impact, Diffusion, Bug Report
chad added a comment to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs.

frowncat

Dec 16 2016, 7:07 PM · Customer Impact, Diffusion, Bug Report
avivey added a comment to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs.

It's doing it again now...

Dec 16 2016, 6:35 PM · Customer Impact, Diffusion, Bug Report
netroby updated the task description for T12021: svn import error . can we just trust local ssl certificate?.
Dec 16 2016, 5:39 AM · Bug Report

Dec 15 2016

epriestley closed T11660: Diff creation fails when submitting too long file paths as Resolved by committing Restricted Diffusion Commit.
Dec 15 2016, 7:35 PM · Differential, Bug Report
epriestley added a revision to T11660: Diff creation fails when submitting too long file paths: Restricted Differential Revision.
Dec 15 2016, 7:33 PM · Differential, Bug Report
chad added a comment to T12023: Harbormaster linter report for UI and API have some confusing names.

It's currently unclear to me if this is a bug report or feature request. The code appears to be working as intended, and adding the description to the property table would be an added feature.

Dec 15 2016, 5:22 PM · Bug Report (Needs Information), Harbormaster
chad renamed T12023: Harbormaster linter report for UI and API have some confusing names from Harbormaster linter report for buildable renders linter name in description column to Harbormaster linter report for buildable renders linter name in message column.
Dec 15 2016, 5:17 PM · Bug Report (Needs Information), Harbormaster
chad added a comment to T12023: Harbormaster linter report for UI and API have some confusing names.

Please follow all guidelines in Contributing Bug Reports. These are required on all bug reports.

Dec 15 2016, 5:16 PM · Bug Report (Needs Information), Harbormaster
cinek created T12023: Harbormaster linter report for UI and API have some confusing names.
Dec 15 2016, 5:05 PM · Bug Report (Needs Information), Harbormaster
epriestley closed T12021: svn import error . can we just trust local ssl certificate? as Invalid.

This isn't a valid bug report (for example, it is missing reproduction instructions). Please see Contributing Bug Reports for help writing bug reports we can accept upstream.

Dec 15 2016, 12:25 PM · Bug Report
netroby updated the task description for T12021: svn import error . can we just trust local ssl certificate?.
Dec 15 2016, 9:12 AM · Bug Report
netroby created T12021: svn import error . can we just trust local ssl certificate?.
Dec 15 2016, 9:11 AM · Bug Report
netroby added a comment to T12014: Failed to create diff by web .

Tested. master branch working . thanks.

Dec 15 2016, 1:35 AM · Bug Report

Dec 14 2016

epriestley closed T12020: Broken migration: phabricator:20140521.projectslug.2.mig.php as Resolved by committing Restricted Diffusion Commit.
Dec 14 2016, 10:14 PM · Bug Report
epriestley added a revision to T12020: Broken migration: phabricator:20140521.projectslug.2.mig.php: Restricted Differential Revision.
Dec 14 2016, 10:10 PM · Bug Report
epriestley added a revision to T12020: Broken migration: phabricator:20140521.projectslug.2.mig.php: Restricted Differential Revision.
Dec 14 2016, 10:02 PM · Bug Report
avivey added a comment to T12020: Broken migration: phabricator:20140521.projectslug.2.mig.php.

(the revision number is in ` because I got an exception trying to attach the mention, I guess)

Dec 14 2016, 9:27 PM · Bug Report
avivey created T12020: Broken migration: phabricator:20140521.projectslug.2.mig.php.
Dec 14 2016, 9:27 PM · Bug Report
epriestley closed T12015: Unable to resolve method 'describeAutomaticCapability'. as Resolved by committing Restricted Diffusion Commit.
Dec 14 2016, 7:23 PM · Diviner, Bug Report
epriestley added a revision to T12015: Unable to resolve method 'describeAutomaticCapability'.: Restricted Differential Revision.
Dec 14 2016, 7:04 PM · Diviner, Bug Report
epriestley added a comment to T12015: Unable to resolve method 'describeAutomaticCapability'..

I think the major intent was being able to edit "View Policy" so you could create private documentation, but I don't recall if we got that far.

Dec 14 2016, 7:00 PM · Diviner, Bug Report
chad added a comment to T12015: Unable to resolve method 'describeAutomaticCapability'..

You could tag a project I think, but not sure what that actually did.

Dec 14 2016, 6:18 PM · Diviner, Bug Report
avivey added a project to T12015: Unable to resolve method 'describeAutomaticCapability'.: Diviner.

Basically nobody is using Diviner, and I don't think the "Edit Book" page actually does anything useful.

Dec 14 2016, 6:00 PM · Diviner, Bug Report