Page MenuHomePhabricator
Feed Advanced Search

Feb 22 2017

chad closed T12305: Persistent chat overlaps various "next" buttons as Resolved by committing Restricted Diffusion Commit.
Feb 22 2017, 4:34 PM · Conpherence, Design, Bug Report
cspeckmim added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

In my scenario the external companies would not see eachother's tasks (separate Spaces), nor other internal items (default Space). However the developers within my company would need to see things from across multiple Spaces.

Feb 22 2017, 4:30 PM · Dashboards, Profile Menu Item, Bug Report
chad added a revision to T12305: Persistent chat overlaps various "next" buttons: Unknown Object (Differential Revision).
Feb 22 2017, 4:29 PM · Conpherence, Design, Bug Report
chad added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

Yeah, but I still want Spaces to be "hard walls" around content. If it's just some minor dashboard permissions, it might be overkill (ie, they both need to see same tasks).

Feb 22 2017, 4:12 PM · Dashboards, Profile Menu Item, Bug Report
cspeckmim added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

Do these dashboards contain sensitive information?

For the use-case I was trying to outline in T9501, this would be the case. We work with a handful of external companies and use Phabricator for organizing and communicating work. We also use Phabricator for our code review. Each project with external company (People, Projects, Maniphest) is organized into separate Spaces so that they are fully isolated, and we have a default space for internal use for Diffusion/Differential. Ideally when a user from Company A logs in they would land on a dashboard that shows only the Projects/Maniphest for them, while Company B user logs in and only shows the Projects/Maniphest for them.

Feb 22 2017, 3:42 PM · Dashboards, Profile Menu Item, Bug Report
dtf added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.
In T12308#212972, @chad wrote:

Do these dashboards contain sensitive information?

Feb 22 2017, 3:26 PM · Dashboards, Profile Menu Item, Bug Report
chad added a project to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user: Profile Menu Item.
Feb 22 2017, 3:24 PM · Dashboards, Profile Menu Item, Bug Report
chad added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

Basically we're in feature request territory, and we'll want to understand the root problem installing all these dashboards solves before deciding on what to build.

Feb 22 2017, 3:24 PM · Dashboards, Profile Menu Item, Bug Report
chad added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

Do these dashboards contain sensitive information?

Feb 22 2017, 3:22 PM · Dashboards, Profile Menu Item, Bug Report
dtf added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.
In T12308#212968, @chad wrote:

What is your expectation here that we do?

Feb 22 2017, 3:19 PM · Dashboards, Profile Menu Item, Bug Report
chad added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

I think this is still the best way we have to let you if the dashboard visibility is over-restrictive or mis-installed. I'm not sure magically disappearing it solves anything. It might help to know more about what you are trying to do (was this on purpose, or a mistake) in order to try to better inform admins when they order dashboards like this.

Feb 22 2017, 3:17 PM · Dashboards, Profile Menu Item, Bug Report
chad added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

What is your expectation here that we do?

Feb 22 2017, 3:06 PM · Dashboards, Profile Menu Item, Bug Report
cspeckmim added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

Probably very similar or even dupe of T9501

Feb 22 2017, 2:28 PM · Dashboards, Profile Menu Item, Bug Report
dtf created T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.
Feb 22 2017, 1:44 PM · Dashboards, Profile Menu Item, Bug Report
epriestley created T12307: Users unable to add new credit cards to Autopay?.
Feb 22 2017, 12:41 PM · Phortune, Phacility
epriestley added a subtask for T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile: T7954: Make it easier to upload XHProf profiles.
Feb 22 2017, 12:16 PM · XHProf, Ops, Phacility
epriestley added a parent task for T7954: Make it easier to upload XHProf profiles: T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.
Feb 22 2017, 12:16 PM · Infrastructure, XHProf
epriestley moved T12306: Failing svn connection brings daemon down from Backlog to vNext on the Daemons board.
Feb 22 2017, 12:15 PM · Daemons, Bug Report
epriestley added a project to T12306: Failing svn connection brings daemon down: Daemons.

It is expected that taskmasters will exit and restart after an unexpected failure, but this should not trigger setup warnings. I'll see if I can reproduce this.

Feb 22 2017, 12:14 PM · Daemons, Bug Report
pmatos updated the task description for T12306: Failing svn connection brings daemon down.
Feb 22 2017, 12:08 PM · Daemons, Bug Report
pmatos updated the task description for T12306: Failing svn connection brings daemon down.
Feb 22 2017, 12:06 PM · Daemons, Bug Report
pmatos created T12306: Failing svn connection brings daemon down.
Feb 22 2017, 12:05 PM · Daemons, Bug Report
epriestley claimed T12304: Herald fails to add auditors if the author is an auditor.

This might be the same as T12302, but I'll see what's going on.

Feb 22 2017, 12:05 PM · Herald, Bug Report
epriestley added projects to T12305: Persistent chat overlaps various "next" buttons: Design, Conpherence.

Persistent chat was originally a separate column (so it didn't overlap with anything on the page) but was brought into the main page recently.

Feb 22 2017, 12:04 PM · Conpherence, Design, Bug Report
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