Page MenuHomePhabricator
Feed All Stories

Feb 22 2017

willplatts created T12305: Persistent chat overlaps various "next" buttons.
Feb 22 2017, 10:59 AM · Conpherence, Design, Bug Report
willplatts created T12304: Herald fails to add auditors if the author is an auditor.
Feb 22 2017, 10:51 AM · Herald, Bug Report
20after4 added a comment to T12134: Develop a Nuance-based Phabricator reporting/support flow.

FWIW I think this is pretty genius. Especially the repro-or-it-didn't-happen aspect.

Feb 22 2017, 6:36 AM · Abuse
20after4 awarded T10612: Editing a panel causes it to duplicate in dashboard a Orange Medal token.
Feb 22 2017, 6:11 AM · Dashboards (v2), Bug Report
20after4 awarded T5698: Github-style Remarkup previews on a checked-in resource a Like token.
Feb 22 2017, 4:54 AM · User Delight, KDE, Haskell.org, Wikimedia, Restricted Project, Diffusion, Remarkup
20after4 awarded T10756: Make daemons work correctly no matter where they are or how many copies are running a Orange Medal token.
Feb 22 2017, 4:54 AM · Restricted Project, Daemons
chad added a revision to T10390: Dashboards v2 (UX): Unknown Object (Differential Revision).
Feb 22 2017, 12:34 AM · Dashboards (v2)
thoughtpolice added a comment to T12196: PHP7, particularly before 7.1.2, segfaults a whole lot while running CLI scripts.

Just as a side note: PHP 7.1.2 was released like a week ago, and I have not seen any more spurious segfaults on my instances when doing imports or anything like that on the server. The arc liberate bug remains dead and I have not seen any more faults in arc either. Builds should now be available in most PHP 7.1 repositories (in multiple distros/systems), so I suggest an upgrade for anyone who's affected.

Feb 22 2017, 12:04 AM · Bug Report

Feb 21 2017

chad updated the task description for T12270: Unbeta Badges.
Feb 21 2017, 11:15 PM · Badges (v0)
epriestley claimed T12302: Herald adding different auditors but only the last one is kept.
Feb 21 2017, 11:02 PM · Herald, Bug Report
epriestley claimed T12301: `Create Form` always leads to page creating forms for configuring forms.

Yeah I'll take it, pretty sure it's my fault.

Feb 21 2017, 11:01 PM · EditEngine, Bug Report
chad added a comment to T12301: `Create Form` always leads to page creating forms for configuring forms.

Yeah I was planning on bisecting, but sounds like you're claiming this?

Feb 21 2017, 10:56 PM · EditEngine, Bug Report
epriestley added a comment to T12301: `Create Form` always leads to page creating forms for configuring forms.

I think this is probbbbbbbably an actual bug unless I'm totally out of it -- possibly my fault from D17372.

Feb 21 2017, 10:54 PM · EditEngine, Bug Report
chad added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Feb 21 2017, 10:52 PM · Badges (v0)
ofbeaton added a comment to T10279: Allow bot users to see email addresses of users.

I realise this is a wontfix, but you kept asking for root problems.

Feb 21 2017, 9:30 PM · Feature Request
ofbeaton awarded T10279: Allow bot users to see email addresses of users a Like token.
Feb 21 2017, 9:26 PM · Feature Request
thoughtpolice awarded T2794: Implement Phage (like Hypershell) a Love token.
Feb 21 2017, 8:57 PM · Phage, Infrastructure
Mnkras added a comment to T12296: Improve Phacility repository import performance.

@epriestley The import has finished, but some commits show as importing still?

Feb 21 2017, 8:33 PM · Customer Impact, Diffusion, Ops
Mnkras created T12303: Print View in Phriction shows Persistent Chat.
Feb 21 2017, 8:26 PM · Phriction, Bug Report
jeffprouty added a comment to T4096: Generated gif-macros are not animated.

We have a hosted instance and seem to be running into the same issue. How do we get this going in this case? (Warning: we are new to phab!)

Feb 21 2017, 7:17 PM · Macros
meshulam added a comment to T7843: Marking all notifications as read - action flow tedious.

Agreed this dialog is pretty annoying.

Feb 21 2017, 6:36 PM · Notifications
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Feb 21 2017, 4:50 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Feb 21 2017, 4:48 PM · Daemons, Ops, Phacility
epriestley added a comment to T12251: Add author information when creating a build in Buildkite.

No.

Feb 21 2017, 4:36 PM · Harbormaster, Feature Request
ivo updated subscribers of T12251: Add author information when creating a build in Buildkite.

@epriestley Would you guys be willing to accept a patch that implements this?

Feb 21 2017, 4:31 PM · Harbormaster, Feature Request
chad added a comment to T12301: `Create Form` always leads to page creating forms for configuring forms.

I could be wrong

Feb 21 2017, 4:20 PM · EditEngine, Bug Report
dtf added a comment to T12301: `Create Form` always leads to page creating forms for configuring forms.

Also, note that in this install, Create Form links to /transactions/editengine/maniphest.task/edit/nocreate/, not to /transactions/editengine/transactions.editengine.config/edit/nocreate/. So, to the correct place, but "forbidden" one.
And there is simply no any other button to push.

Feb 21 2017, 4:03 PM · EditEngine, Bug Report
nicolast added a comment to T12302: Herald adding different auditors but only the last one is kept.

This also seems related: https://secure.phabricator.com/D17262

Feb 21 2017, 3:59 PM · Herald, Bug Report
nicolast created T12302: Herald adding different auditors but only the last one is kept.
Feb 21 2017, 3:56 PM · Herald, Bug Report
dtf added a comment to T12301: `Create Form` always leads to page creating forms for configuring forms.

Yep. How do I create the form I need then?

Feb 21 2017, 3:32 PM · EditEngine, Bug Report
chad added a comment to T12301: `Create Form` always leads to page creating forms for configuring forms.

This isn't a bug, but is confusing.

Feb 21 2017, 3:25 PM · EditEngine, Bug Report
dtf created T12301: `Create Form` always leads to page creating forms for configuring forms.
Feb 21 2017, 3:14 PM · EditEngine, Bug Report
AlexL777 updated Q571: How can I include any JS code in Phabricator from to How can I include any JS code in Phabricator.
Feb 21 2017, 1:18 PM
OCram added a comment to T12294: File Extension Changes while Downloading.

Removed apache settings:

<LocationMatch "\.(?i:os)$">
    ForceType application/octet-stream
    Header set Content-Disposition attachment
</LocationMatch>
Feb 21 2017, 11:15 AM · Files, Bug Report
OCram updated the task description for T12294: File Extension Changes while Downloading.
Feb 21 2017, 11:06 AM · Files, Bug Report
OCram updated the task description for T12294: File Extension Changes while Downloading.
Feb 21 2017, 11:02 AM · Files, Bug Report
cspeckmim added a comment to T11708: Daemons erroneously saying "Diff does not exist".

I don't believe we've seen this pop up again. I can't be certain because right now all the log files are empty, but that means the errors went away?

Feb 21 2017, 5:26 AM · Daemons, Differential, Bug Report
hach-que updated the task description for T12300: git push --mirror --force does not work on Phacility with repo that has refs/pull.
Feb 21 2017, 5:07 AM · Diffusion, Git
hach-que created T12300: git push --mirror --force does not work on Phacility with repo that has refs/pull.
Feb 21 2017, 5:03 AM · Diffusion, Git
epriestley moved T6615: Worker task table has some remaining awkward keys from Availability to vNext on the Daemons board.
Feb 21 2017, 12:37 AM · Infrastructure, Daemons
epriestley closed T10756: Make daemons work correctly no matter where they are or how many copies are running, a subtask of T10751: Make Phabricator Highly Available, as Resolved.
Feb 21 2017, 12:35 AM · Restricted Project, Infrastructure, Prioritized
epriestley closed T10756: Make daemons work correctly no matter where they are or how many copies are running as Resolved.

I believe this is effectively resolved now -- we're running all the daemons on multiple hosts on secure. There's remaining followup work (T6768, T6615, T10753, T10754) but I think the core is in reasonable shape.

Feb 21 2017, 12:35 AM · Restricted Project, Daemons
epriestley added a comment to T12115: ApplicationTransactionPublishWorker can retry indefinitely, sending email as a side effect, if a transaction takes more than 2 hours to publish.

T11708 has almost nothing to do with this, but the fix for this will rewrite the code that's running into issues and probably moot them.

Feb 21 2017, 12:31 AM · Restricted Project, Daemons, Differential, Transactions
epriestley merged task T11708: Daemons erroneously saying "Diff does not exist" into T12115: ApplicationTransactionPublishWorker can retry indefinitely, sending email as a side effect, if a transaction takes more than 2 hours to publish.
Feb 21 2017, 12:30 AM · Daemons, Differential, Bug Report
epriestley merged T11708: Daemons erroneously saying "Diff does not exist" into T12115: ApplicationTransactionPublishWorker can retry indefinitely, sending email as a side effect, if a transaction takes more than 2 hours to publish.
Feb 21 2017, 12:30 AM · Restricted Project, Daemons, Differential, Transactions
epriestley added a comment to T11708: Daemons erroneously saying "Diff does not exist".

I'm just going to merge this into T12115, which isn't really related, but will rewrite this code and probably "fix" this, since a reproduction case seems elusive.

Feb 21 2017, 12:30 AM · Daemons, Differential, Bug Report
epriestley moved T5401: Record total time in queue for tasks from Backlog to vNext on the Daemons board.
Feb 21 2017, 12:27 AM · Daemons
epriestley closed T8809: Support more complex human-usable weekly clocks in trigger infrastructure as Wontfix.

This could probably be built with RRULEs now, but we don't currently have use cases / plans around a general-purpose cron-like tool.

Feb 21 2017, 12:27 AM · Daemons
epriestley closed T3554: Implement a `bin/worker kill <criteria>` for batch-flushing tasks from the queue as Resolved.

Use cases seem to be fairly well covered now.

Feb 21 2017, 12:25 AM · Restricted Project, Daemons
epriestley closed T5334: View a graph showing worker load over time as Resolved.

This is 2+ years old and autoscaling probably covers it now. T5401 is probably a more tangible attack on this.

Feb 21 2017, 12:23 AM · Restricted Project, Facts, Daemons
epriestley closed T7484: Lock is not locked by this process as Wontfix.

This got a little more work when clustering was written but it's essentially a log of two-year-old ghost sightings at this point and not actionable.

Feb 21 2017, 12:21 AM · Restricted Project, Daemons
epriestley added a comment to T8871: Indexing a task with 2,000 comments required a lot of RAM in mid-2015.

I guess this one can live for now since that's reproducible/actionable.

Feb 21 2017, 12:19 AM · Search, Daemons
epriestley renamed T8871: Indexing a task with 2,000 comments required a lot of RAM in mid-2015 from Phabricator daemons autoscale themselves to OOM to Indexing a task with 2,000 comments required a lot of RAM in mid-2015.
Feb 21 2017, 12:19 AM · Search, Daemons
epriestley closed T9603: PHD daemons regularly stopping/starting as Resolved.

Probably a dupe of T11708? I'm just going to kill this one since it's old as dirt.

Feb 21 2017, 12:16 AM · Daemons
epriestley closed T10148: diffusion.rawdiffquery can not encode/transport all diffs (e.g. to non-utf8 files) as Resolved.

I believe this was likely resolved by D16460. This (binary transport over Conduit) is still a problem in general, and T5955 is the general remedy, but diffusion.rawdiffquery specifically now returns data via Files.

Feb 21 2017, 12:15 AM · Conduit, Daemons
epriestley closed T9855: DifferentialChangesSinceLastUpdateField may fail policy checks and prevent mail from sending as Resolved.

I think it is likely that this was resolved by D17123. If it wasn't, we don't have a reproduction case anyway so we can't move forward. I'm going to call this one dead until more information turns up.

Feb 21 2017, 12:13 AM · Daemons, Differential
epriestley closed T10811: `phd-daemon` crash after automatic restart following config change as Resolved.

We were never able to reproduce this convincingly and as far as we know the reporting install no longer uses Phabricator, so I'm going to close this out.

Feb 21 2017, 12:12 AM · Daemons, Bug Report
epriestley closed T10811: `phd-daemon` crash after automatic restart following config change, a subtask of T10756: Make daemons work correctly no matter where they are or how many copies are running, as Resolved.
Feb 21 2017, 12:12 AM · Restricted Project, Daemons
epriestley moved T12298: Allow daemon pools to autoscale down to 0 processes from Backlog to vNext on the Daemons board.
Feb 21 2017, 12:10 AM · Daemons, Ops, Phacility
epriestley moved T10867: Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect from Backlog to vNext on the Daemons board.
Feb 21 2017, 12:10 AM · Daemons, Feature Request
epriestley moved T11037: `bin/phd status` is misleading about local / remote daemons from Backlog to vNext on the Daemons board.
Feb 21 2017, 12:10 AM · Daemons, Restricted Project
epriestley added a comment to T12298: Allow daemon pools to autoscale down to 0 processes.

I believe I have the first part of this (restructuring the code into a more sensible Overseer > Pool > Daemon sort of thing) working, but it could use more testing. I'm going to see if we have anything else in Daemons that I can fix while I'm here to help me kick the tires a bit.

Feb 21 2017, 12:09 AM · Daemons, Ops, Phacility

Feb 20 2017

bcooksley added a project to T5698: Github-style Remarkup previews on a checked-in resource: KDE.
Feb 20 2017, 10:01 PM · User Delight, KDE, Haskell.org, Wikimedia, Restricted Project, Diffusion, Remarkup
epriestley updated the task description for T12298: Allow daemon pools to autoscale down to 0 processes.
Feb 20 2017, 9:42 PM · Daemons, Ops, Phacility
cspeckmim added a comment to T12299: Linking Maniphest Tasks to external Bugtrackers.

You can make your own extension to Remarkup to allow auto-linking. I have one setup which I based off of P1129, P1209 which regex matches on Task-xxxx and uses the xxxx to link to our system. That works well and is sufficient for what we need.

Feb 20 2017, 5:29 PM · Feature Request
epriestley closed T12299: Linking Maniphest Tasks to external Bugtrackers as Invalid.

This request doesn't describe a root problem, just suggests we do about a thousand hours of work integrating with remote APIs. See Describing Root Problems.

Feb 20 2017, 5:16 PM · Feature Request
friesenschrauber created T12299: Linking Maniphest Tasks to external Bugtrackers.
Feb 20 2017, 5:02 PM · Feature Request
epriestley added a comment to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.

Some possible approaches here:

Feb 20 2017, 4:34 PM · XHProf, Ops, Phacility
epriestley updated the task description for T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.
Feb 20 2017, 4:33 PM · XHProf, Ops, Phacility
epriestley created T12298: Allow daemon pools to autoscale down to 0 processes.
Feb 20 2017, 4:30 PM · Daemons, Ops, Phacility
epriestley created T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.
Feb 20 2017, 4:19 PM · XHProf, Ops, Phacility
epriestley added a subtask for T12101: Phabricator PHP 7 Compatibility: T9805: XHProf will not build on PHP7.
Feb 20 2017, 4:17 PM · Infrastructure, Guides
epriestley added a parent task for T9805: XHProf will not build on PHP7: T12101: Phabricator PHP 7 Compatibility.
Feb 20 2017, 4:16 PM · XHProf
epriestley moved T7954: Make it easier to upload XHProf profiles from Backlog to vNext on the XHProf board.
Feb 20 2017, 4:13 PM · Infrastructure, XHProf
epriestley added a parent task for T12217: Reduce the hardware cost of Phacility free instances: T12218: Reduce the operational cost of a larger Phacility cluster.
Feb 20 2017, 4:12 PM · Ops, Phacility
epriestley added a subtask for T12218: Reduce the operational cost of a larger Phacility cluster: T12217: Reduce the hardware cost of Phacility free instances.
Feb 20 2017, 4:12 PM · Ops, Phacility
epriestley added a comment to T12296: Improve Phacility repository import performance.

There's also a bunch of the usual free-tier shenanigans on this host -- free instances with 1 user importing multiple large repositories -- but that's to be expected.

Feb 20 2017, 4:10 PM · Customer Impact, Diffusion, Ops
epriestley created T12296: Improve Phacility repository import performance.
Feb 20 2017, 4:09 PM · Customer Impact, Diffusion, Ops
ivo created T12295: Allow commenting on imported calendar events.
Feb 20 2017, 1:37 PM · Calendar, Feature Request
epriestley added a comment to T12134: Develop a Nuance-based Phabricator reporting/support flow.

Here's an attempt to manually classify recent feature requests. I'm not sure how much value we're really getting out of this channel except from users who are already part of Community:

Feb 20 2017, 12:28 PM · Abuse
epriestley closed T12239: Search for deleted or renamed files in Diffusion as Invalid.

Unfortunately, we haven't received enough information on this to move forward. See Contributing Feature Requests and Contributing Bug Reports if you'd like to try filing a new request.

Feb 20 2017, 12:17 PM · Feature Request
OCram created T12294: File Extension Changes while Downloading.
Feb 20 2017, 11:13 AM · Files, Bug Report
sergio.castro added a comment to T12222: Add custom fields to the "Task Action..." menu.

Can I make a question?

Feb 20 2017, 9:42 AM · Maniphest, Feature Request
rsmarples added a comment to T12292: Diffusion commit details references HEAD -> master broken.
  1. Install Phabricator at the version noted above.
  2. Import a GIT repository via observing a URI.
  3. Note the reference HEAD -> master link broken on latest commit.
Feb 20 2017, 9:30 AM · Bug Report (Needs Information)
chad closed T12293: test1 as Spite.
Feb 20 2017, 3:39 AM · Sandbox
• AmazingTrans moved T12293: test1 from test to Backlog on the Sandbox board.
Feb 20 2017, 3:26 AM · Sandbox
• AmazingTrans moved T12293: test1 from Backlog to test on the Sandbox board.
Feb 20 2017, 3:26 AM · Sandbox
• AmazingTrans moved T12293: test1 from test to Backlog on the Sandbox board.
Feb 20 2017, 3:26 AM · Sandbox
• AmazingTrans created T12293: test1.
Feb 20 2017, 3:26 AM · Sandbox
• Losy added a member for Android: • Losy.
Feb 20 2017, 1:20 AM
• Losy added a watcher for Android: • Losy.
Feb 20 2017, 1:20 AM
• Losy added a watcher for Design: • Losy.
Feb 20 2017, 1:17 AM
• Losy added a watcher for Clusters: • Losy.
Feb 20 2017, 1:14 AM
• Losy added a watcher for Bug Report: • Losy.
Feb 20 2017, 1:12 AM

Feb 19 2017

chad added a comment to T12292: Diffusion commit details references HEAD -> master broken.

Please provide reproduction steps, we don't accept bug reports that ask us to go look at their server.

Feb 19 2017, 11:14 PM · Bug Report (Needs Information)
rsmarples created T12292: Diffusion commit details references HEAD -> master broken.
Feb 19 2017, 10:18 PM · Bug Report (Needs Information)
yuanchen.zhu added a comment to T5028: Commit message text formatting in remote git repository (SSH) is not preserved.

Hi I recently started experimenting with phabricator with my team and stumbled on this issue.

Feb 19 2017, 7:26 PM · Diffusion
epriestley added a comment to T12291: Diffusion doesn't display file moves/renames in an easily consumable/reviewable way.

The in-diff issue is described in T11482.

Feb 19 2017, 7:10 PM · Diffusion, Feature Request
Mnkras added a comment to T12291: Diffusion doesn't display file moves/renames in an easily consumable/reviewable way.

We once rendered these renames like this:

new/path/to/file.c
  Moved From: old/path/to/file.c

Personally I like that, I think a compromise to make it easier to consume, would be to try and group the moves together and showing the relation, for example
now:

a ->
b ->
a <-
b <-

to

a ->
a <-
b ->
b <-
Feb 19 2017, 7:06 PM · Diffusion, Feature Request
epriestley added a comment to T12291: Diffusion doesn't display file moves/renames in an easily consumable/reviewable way.

I'm not sure I understand what changes you're suggesting.

Feb 19 2017, 6:43 PM · Diffusion, Feature Request