Page MenuHomePhabricator
Feed All Stories

Apr 28 2016

chad added a comment to T10891: Milestones with no tasks vs. collapsed profile panel.

Closing this as it's not a bug. General feedback is fine on T10349

Apr 28 2016, 2:11 PM · Projects, Bug Report
chad closed T10891: Milestones with no tasks vs. collapsed profile panel as Wontfix.
Apr 28 2016, 2:09 PM · Projects, Bug Report
Chiel92 created T10893: repo update fails for repos with 1000s of closing branches.
Apr 28 2016, 2:06 PM · Diffusion, Bug Report
tjstum created T10892: Recommend pip instead of easy_install for Python linters.
Apr 28 2016, 1:27 PM · Lint, Arcanist, Feature Request
fooishbar added a comment to T10163: Remarkup Autocomplete Errata.
  • D15039: Two effects:
    • If you finish typing a word (e.g., type a space) and there are no results, deactivate since nothing else you could type could possibly cause matches and you're probably not fishing around since you just hit space.
Apr 28 2016, 12:52 PM · Remarkup, Typeahead
fooishbar added a comment to T10349: Projects v3 Errata.

@epriestley Maybe one to consider: you must have a default workboard column for a project; the default workboard column cannot be hidden; milestone workboard columns strictly sort after project workboard columns. The combination of these three means that I have a project which is strictly split into four milestones, but its overall workboard view always includes an empty column before each milestone column. Is it worth filing a task for this?

Apr 28 2016, 12:44 PM · Restricted Project, Projects (v3)
fooishbar created T10891: Milestones with no tasks vs. collapsed profile panel.
Apr 28 2016, 12:32 PM · Projects, Bug Report
epriestley added a revision to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API: Unknown Object (Differential Revision).
Apr 28 2016, 11:31 AM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
aik099 added a comment to T10887: [feed] Missing avatar for Herald authored feed entries.

Thanks. I'll upgrade then.

Apr 28 2016, 10:41 AM · Bug Report
epriestley closed T10889: 测试test as Spite.
Apr 28 2016, 10:17 AM · Arcanist
epriestley closed T10888: 123 as Spite.
Apr 28 2016, 10:17 AM · Bug Report
epriestley closed T10887: [feed] Missing avatar for Herald authored feed entries as Invalid.

This should be fixed in Week 17. See T8952 / D15773. Let us know if you're still seeing issues after upgrading, or can reproduce the problem on this install.

Apr 28 2016, 10:17 AM · Bug Report
fooishbar updated the task description for T10742: PhabricatorRepositoryCommitParserWorker fails on deleted commits.
Apr 28 2016, 10:02 AM · Diffusion, Bug Report
fooishbar created T10890: Less chatty Maniphest feed rendering.
Apr 28 2016, 9:13 AM · Maniphest, Feature Request
• tttpeng created T10889: 测试test.
Apr 28 2016, 8:51 AM · Arcanist
• tttpeng created T10888: 123.
Apr 28 2016, 8:47 AM · Bug Report
aik099 created T10887: [feed] Missing avatar for Herald authored feed entries.
Apr 28 2016, 8:41 AM · Bug Report
fooishbar added a comment to T9161: How can we fix "too much mail"?.

My personal, completely subjective experience is that Phabricator mail is trivial to deal with, almost all valuable, takes very little of my time, and I want to receive >95% of the mail that I receive, and that I'd barely notice if I had 10x this mail volume.

Apr 28 2016, 8:30 AM · Restricted Project, User Preferences, Mail
StingRayZA added a comment to T10885: Arcanist Windows install docs - adding in additional php extensions.

I'm not looking for support for third-party packages. What I am trying to offer is a contribution to your 'as simple as possible' instructions.

Apr 28 2016, 7:29 AM · Documentation, Bug Report

Apr 27 2016

epriestley added a comment to T7820: Add "Closed, Duplicate" option to Status Action in Maniphest.

we'll get to it when we feel like it (ok not a totally fair paraphrase, but it's one way to read it)

Apr 27 2016, 11:45 PM · Restricted Project, User Delight, Maniphest
chad added a comment to T7820: Add "Closed, Duplicate" option to Status Action in Maniphest.

Unfortunately we have 1,500~ish tasks between 2 developers and we do our best to build things that we think we have the largest impact (which tends to mean small quality of life stuff doesn't happen as often). We'd love to bring more people on board to help improve Phabricator faster, but that means being evil and chasing money first to make that happen.

Apr 27 2016, 11:29 PM · Restricted Project, User Delight, Maniphest
dgoldstein added a comment to T7820: Add "Closed, Duplicate" option to Status Action in Maniphest.

I guess I was hoping for a timeline. Or an explanation of why it's hard
and not done yet. That page seems to me to say "we'll get to it when we
feel like it" (ok not a totally fair paraphrase, but it's one way to read
it). Waiting a year for a minor feature that would drastically improve one
of my workflows just makes it feel like you guys aren't being very
responsive to feedback about how the UI could be made better. And I have
plenty of other complaints about the ui - just haven't bothered filing most
of them.

Apr 27 2016, 11:26 PM · Restricted Project, User Delight, Maniphest
epriestley added a comment to T7820: Add "Closed, Duplicate" option to Status Action in Maniphest.

Perhaps helpful for context is T6027, which was open for longer than this (about 18 months).

Apr 27 2016, 11:13 PM · Restricted Project, User Delight, Maniphest
epriestley added a comment to T7820: Add "Closed, Duplicate" option to Status Action in Maniphest.

What isn't helpful about it? I believe it has a complete answer to your question, and a great deal of detail explaining why that's the answer. How could it be more helpful? Did I misunderstand what you're asking?

Apr 27 2016, 11:05 PM · Restricted Project, User Delight, Maniphest
dgoldstein added a comment to T7820: Add "Closed, Duplicate" option to Status Action in Maniphest.

that was not a particularly helpful response

Apr 27 2016, 11:00 PM · Restricted Project, User Delight, Maniphest
epriestley closed T7177: Peculiar Authorization Issue in cloned repositories as Invalid.

I suspect this isn't a bug -- diffs have repository associations, and you must be able to see a repository to see associated diffs. You must be able to see the active diff to see a revision. So andy caused the active diff to belong to a repository that nuno can not see, which prevents nuno from seeing the revision.

Apr 27 2016, 10:58 PM
epriestley updated the task description for T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 10:51 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley added a revision to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API: Unknown Object (Differential Revision).
Apr 27 2016, 10:47 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley merged T9679: Maybe rename "Import an Existing External Repository" in New Repo Wizard into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 10:46 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley merged task T9679: Maybe rename "Import an Existing External Repository" in New Repo Wizard into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 10:46 PM · Diffusion
epriestley added a comment to T9679: Maybe rename "Import an Existing External Repository" in New Repo Wizard.

I'm going to merge this into T10748, which is now more clearly the way forward.

Apr 27 2016, 10:46 PM · Diffusion
epriestley added a comment to T7820: Add "Closed, Duplicate" option to Status Action in Maniphest.

See Planning for information on timelines and progress.

Apr 27 2016, 10:38 PM · Restricted Project, User Delight, Maniphest
dgoldstein added a comment to T7820: Add "Closed, Duplicate" option to Status Action in Maniphest.

Any updates here? This task has been open for over a year now and it's something I wish for approximately weekly.

Apr 27 2016, 10:33 PM · Restricted Project, User Delight, Maniphest
epriestley edited this Badge.
Apr 27 2016, 10:04 PM
epriestley added a member for Community: jparise.
Apr 27 2016, 10:04 PM
epriestley added a revision to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API: Unknown Object (Differential Revision).
Apr 27 2016, 5:52 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley added a revision to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API: Unknown Object (Differential Revision).
Apr 27 2016, 5:34 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
jamchamb added a comment to T7432: Unable to start Aphlict server with a passphrase protected SSL cert/key.

Would it be possible to add something similar to Apache's SSLPassPhraseDialog exec:/path/to/program directive?

Apr 27 2016, 5:06 PM
epriestley merged T9604: Creating an imported repository via Conduit results in filesystem error into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 1:58 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley merged task T9604: Creating an imported repository via Conduit results in filesystem error into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 1:58 PM · Restricted Project, Phacility Support, Diffusion, Conduit
epriestley added a comment to T9604: Creating an imported repository via Conduit results in filesystem error.

I'm going to merge this into T10748, since that's now clearly the way forward.

Apr 27 2016, 1:57 PM · Restricted Project, Phacility Support, Diffusion, Conduit
epriestley merged T6848: It should be possible to view a repository's policies without necessarily being able to edit it into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 1:54 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley merged task T6848: It should be possible to view a repository's policies without necessarily being able to edit it into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 1:54 PM · Diffusion (v3)
epriestley added a comment to T6848: It should be possible to view a repository's policies without necessarily being able to edit it.

I'm going to merge this into T10748 because that's clearly the pathway forward now. You can view the new UI at /diffusion/X/manage/, although it mostly doesn't work yet and there are no links to it in the UI.

Apr 27 2016, 1:53 PM · Diffusion (v3)
epriestley added a comment to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.

T10751 and T4292 are reaching a steadier state now and have some indirect/soft dependencies on this (for example, the new panel for monitoring the cluster status of a particular repository is part of this new UI) so I expect to drive this forward at least somewhat in the next few days. I'm not sure how ambitious I'll feel about meddling with the URI code before the release cut, but there's a good chunk of basic work remaining before that becomes an issue.

Apr 27 2016, 1:49 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley added a revision to T10843: Don't write Celerity resources which aren't defined in the local map to cache: Unknown Object (Differential Revision).
Apr 27 2016, 1:46 PM · Infrastructure
nubunto added a member for Almanac: nubunto.
Apr 27 2016, 1:44 PM
epriestley added a comment to T10885: Arcanist Windows install docs - adding in additional php extensions.

See also Contributing Bug Reports:

Apr 27 2016, 1:39 PM · Documentation, Bug Report
epriestley closed T10885: Arcanist Windows install docs - adding in additional php extensions as Wontfix.

The instructions are correct as written, and are aimed at being as simple as possible for users who may not be familiar with PHP. We can not possibly include instructions for every imaginable installer, see also T4200.

Apr 27 2016, 1:37 PM · Documentation, Bug Report
StingRayZA added a comment to T10885: Arcanist Windows install docs - adding in additional php extensions.

I didn't use the windows-binary installer, I installed php into my cygwin install using the cygwin installer. (It's still on windows, but through another process).

Apr 27 2016, 1:32 PM · Documentation, Bug Report
epriestley added a comment to T10885: Arcanist Windows install docs - adding in additional php extensions.

In particular, the install instructions tell you to:

Apr 27 2016, 1:09 PM · Documentation, Bug Report
epriestley added a comment to T10885: Arcanist Windows install docs - adding in additional php extensions.

How did you install PHP, exactly?

Apr 27 2016, 1:08 PM · Documentation, Bug Report
epriestley renamed T10886: Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge from Inline comments in Pholio are not visible anymore to Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge.
Apr 27 2016, 1:06 PM · Pholio, Bug Report
sascha-egerer updated the task description for T10886: Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge.
Apr 27 2016, 12:46 PM · Pholio, Bug Report
sascha-egerer created T10886: Unpublished, unauthored inline comments in Pholio are incorrectly counted on the image badge.
Apr 27 2016, 12:46 PM · Pholio, Bug Report
StingRayZA created T10885: Arcanist Windows install docs - adding in additional php extensions.
Apr 27 2016, 12:27 PM · Documentation, Bug Report
• ogolubovic closed Q379: notification.servers as resolved.
Apr 27 2016, 11:54 AM
epriestley added a comment to T10884: Sort repository, database and notification services better (by network distance).

Yeah, measuring latency and using it as a signal is tricky. If there's a long disruption and Oregon measures faster than N. California for "a while", we probably legitimately want to use it. If N. California completely goes down, we almost certainly want to keep things alive and send writes to Mars, even if Mars is at a huge network distance, since "slow write" is probably almost always better than "complete failure" (although maybe Mars bandwith is so much more expensive that we don't want to fail over to it, or the size of the link is small enough that it simply can't accommodate the failover traffic).

Apr 27 2016, 11:47 AM · Clusters
jayvdb added a comment to T6051: workboard drag+drop doesn't work on android (both chrome+firefox broken).

https://phabricator.wikimedia.org/T129049#2242513 was quite easy to do using iceweasel on debian on android 10" tablet. It involved moving the task from Backlog to the last column which wasnt visible on the screen. This might be more difficult with larger workboards, but there are definitively scenarios where this is painless on a tablet, and is a natural UI for the action on that type of device.
I wonder if it is any more difficult on high res smart phones in landscape mode- I suspect it might work quite well also.

Apr 27 2016, 11:42 AM · Android, Wikimedia
hach-que added a comment to T10884: Sort repository, database and notification services better (by network distance).

I mean none of these things are show-stoppers in the example above, but you can imagine how bad things would be over a link between the Sydney data centre and N. California if it was happening all the time.

Apr 27 2016, 11:41 AM · Clusters
hach-que added a comment to T10884: Sort repository, database and notification services better (by network distance).

There's probably almost certainly a correlation there that link speed increases with lower latency because of proximity? But having an SSH proxy accidentally pick a link that's say in Oregon instead of North California because at the time of sampling some local network issues were being had that made the local copy appear worse would be a reasonably bad situation for a few reasons:

Apr 27 2016, 11:40 AM · Clusters
epriestley added a comment to T10884: Sort repository, database and notification services better (by network distance).

Ah, link speed vs latency is interesting.

Apr 27 2016, 11:36 AM · Clusters
epriestley added a comment to T10884: Sort repository, database and notification services better (by network distance).

Is "use measured latency" probably always the correct rule in your (hypothetical) cluster setup? Or do you have any cases offhand where it isn't the desired result, and you really want to send traffic to a more distant node on the network?

Apr 27 2016, 11:33 AM · Clusters
hach-que added a comment to T10884: Sort repository, database and notification services better (by network distance).

Gut feeling is that network latency almost certainly isn't the only reasonable concern here.

Apr 27 2016, 11:31 AM · Clusters
hach-que added a comment to T10884: Sort repository, database and notification services better (by network distance).

I don't currently expect anyone to be deploying multi-region clusters in the first iteration of real clustering

Apr 27 2016, 11:26 AM · Clusters
epriestley added a comment to T10884: Sort repository, database and notification services better (by network distance).

I don't currently expect anyone to be deploying multi-region clusters in the first iteration of real clustering, so this isn't high on my list of things to fix in this round.

Apr 27 2016, 11:23 AM · Clusters
epriestley added a comment to T10883: Allow repository cluster nodes to be read-only.

See also T10884 for related work: ranking similar nodes more intelligently.

Apr 27 2016, 11:17 AM · Restricted Project, Diffusion, Clusters
epriestley added a comment to T10884: Sort repository, database and notification services better (by network distance).

See also T10883 for related work: making some nodes read-only.

Apr 27 2016, 11:17 AM · Clusters
epriestley created T10884: Sort repository, database and notification services better (by network distance).
Apr 27 2016, 11:16 AM · Clusters
epriestley created T10883: Allow repository cluster nodes to be read-only.
Apr 27 2016, 11:13 AM · Restricted Project, Diffusion, Clusters
epriestley added Q379: notification.servers (Answer 370).
Apr 27 2016, 10:45 AM
• ogolubovic updated Q379: notification.servers from to notification.servers.
Apr 27 2016, 10:35 AM
epriestley closed T10882: test as Spite.
Apr 27 2016, 10:03 AM · Bug Report
• lenny.dou created T10882: test.
Apr 27 2016, 9:22 AM · Bug Report
chad added a comment to T10878: workboard horizontal scroll wonky on ie11.

http://phacility.com can still up a clean test instance in under a minute.

Apr 27 2016, 3:06 AM · Workboards, Windows, Bug Report (Needs Information)
ofbeaton added a comment to T10878: workboard horizontal scroll wonky on ie11.

Should note that with the test video I posted, I was able to replicate with a brand new workboard with 5 extra columns (6 total) with zero tasks on them, so the vertical scroll bar has no impact on it.

Apr 27 2016, 2:35 AM · Workboards, Windows, Bug Report (Needs Information)

Apr 26 2016

epriestley added a revision to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API: Unknown Object (Differential Revision).
Apr 26 2016, 11:11 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley added a revision to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API: Unknown Object (Differential Revision).
Apr 26 2016, 10:55 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
minyoung added a comment to T10881: git diff parsing fails with --submodule=log.

Tested with:
arcanist: rARCa2ab38df78a9d111459fab8d0d63e74d811edf29
libphutil: rPHU4b44d03ba60c8a67f70a91cb11765b8b61f73598

Apr 26 2016, 10:30 PM · Contributor Onboarding, Arcanist, Bug Report
jparise awarded T5055: Distribution mechanism for arc extensions a Mountain of Wealth token.
Apr 26 2016, 9:50 PM · Arcanist
chad added a comment to T10881: git diff parsing fails with --submodule=log.

What version arcanist are you running?

Apr 26 2016, 9:29 PM · Contributor Onboarding, Arcanist, Bug Report
minyoung added a comment to T10881: git diff parsing fails with --submodule=log.

Knowing what the problem is, the workaround is obvious (remove the relevant git config option).

Apr 26 2016, 9:25 PM · Contributor Onboarding, Arcanist, Bug Report
minyoung created T10881: git diff parsing fails with --submodule=log.
Apr 26 2016, 9:21 PM · Contributor Onboarding, Arcanist, Bug Report
theguild added a comment to T10879: Daily TODO list.

I am involved in few projects and I have several issues for each project.

Apr 26 2016, 6:51 PM · Feature Request
robertkraig awarded T10778: ${NAMESPACE}_search is missing an edge and edgedata table a Like token.
Apr 26 2016, 6:34 PM · ApplicationEditor, Bug Report
epriestley added a comment to T10880: Cannot persist changes of visibility of Form Template.

Thanks -- T10778 had instructions which I was more successful with. This should be fixed in HEAD.

Apr 26 2016, 6:31 PM · Bug Report
robertkraig added a comment to T10880: Cannot persist changes of visibility of Form Template.

@epriestley I updated the description in this ticket.

Apr 26 2016, 6:30 PM · Bug Report
robertkraig updated the task description for T10880: Cannot persist changes of visibility of Form Template.
Apr 26 2016, 6:30 PM · Bug Report
epriestley closed T10778: ${NAMESPACE}_search is missing an edge and edgedata table as Resolved by committing Restricted Diffusion Commit.
Apr 26 2016, 6:26 PM · ApplicationEditor, Bug Report
epriestley added a revision to T10778: ${NAMESPACE}_search is missing an edge and edgedata table: Unknown Object (Differential Revision).
Apr 26 2016, 6:22 PM · ApplicationEditor, Bug Report
epriestley claimed T10778: ${NAMESPACE}_search is missing an edge and edgedata table.
Apr 26 2016, 6:16 PM · ApplicationEditor, Bug Report
chad merged T10879: Daily TODO list into T5793: Add support for personal workboards.
Apr 26 2016, 6:15 PM · User Delight, Wikimedia, Workboards
chad merged task T10879: Daily TODO list into T5793: Add support for personal workboards.
Apr 26 2016, 6:15 PM · Feature Request
epriestley added a comment to T10778: ${NAMESPACE}_search is missing an edge and edgedata table.

Oh, thanks. I can reproduce this variant of things. I thought I remembered seeing this...

Apr 26 2016, 6:15 PM · ApplicationEditor, Bug Report
chad merged task T10880: Cannot persist changes of visibility of Form Template into T10778: ${NAMESPACE}_search is missing an edge and edgedata table.
Apr 26 2016, 6:14 PM · Bug Report
chad merged T10880: Cannot persist changes of visibility of Form Template into T10778: ${NAMESPACE}_search is missing an edge and edgedata table.
Apr 26 2016, 6:14 PM · ApplicationEditor, Bug Report
epriestley added a comment to T10880: Cannot persist changes of visibility of Form Template.

Do you have a stack trace in your webserver error log? (Often /var/log/apache2/error.log or similar.) I can't immediately reproduce this myself, but that might help narrow down the cause.

Apr 26 2016, 6:14 PM · Bug Report
robertkraig created T10880: Cannot persist changes of visibility of Form Template.
Apr 26 2016, 6:11 PM · Bug Report
avivey added a subtask for T9287: Working with many repositories: T10874: Add repository or project to email subject.
Apr 26 2016, 5:58 PM · Restricted Project, Restricted Project, Diffusion
avivey added a parent task for T10874: Add repository or project to email subject: T9287: Working with many repositories.
Apr 26 2016, 5:58 PM · Restricted Project, Mail, Feature Request