Page MenuHomePhabricator
Feed All Stories

Dec 2 2016

epriestley added a comment to T11816: Calendar: Feedback and Suggestions.

In the general case:

Dec 2 2016, 11:20 AM · Restricted Project, Calendar
carlokok added a comment to T11816: Calendar: Feedback and Suggestions.

Getting Datetime has no timezone or viewer timezone when viewing the main calendar.(https://gist.github.com/carlokok/570d524fe2a6dd1472835c0fa67b7faf) after import. Since the import contains a quite a bit of private data, how can I narrow down which record specifically it fails on?

Dec 2 2016, 11:11 AM · Restricted Project, Calendar
epriestley updated the task description for T11939: IPv6 Support.
Dec 2 2016, 10:54 AM · Infrastructure
epriestley created T11939: IPv6 Support.
Dec 2 2016, 10:42 AM · Infrastructure
fooishbar added a comment to T11936: Editing Diffusion URIs does not update URI index.

@epriestley Thanks, that's fixed it! I'd not looked index engines at all before, so wasn't sure if I was just missing something stupid on that side.

Dec 2 2016, 9:01 AM · Diffusion, Bug Report
vanmeeuwen added a comment to T9640: Make Phabricator compatible with PHP7.

PHP 7.1.0 has been released yesterday, and among the changes;

Dec 2 2016, 7:05 AM · Prioritized, Restricted Project, Infrastructure
vanmeeuwen awarded T9640: Make Phabricator compatible with PHP7 a Like token.
Dec 2 2016, 6:48 AM · Prioritized, Restricted Project, Infrastructure
neilfitz added a comment to T11938: Running `git clone` against a Mercurial repository does not produce an error message with explicit guidance.

Absolutely makes sense - provided some more background on the problem. Looking forward to hearing your thoughts.

Dec 2 2016, 2:46 AM · Diffusion, Feature Request
neilfitz updated the task description for T11938: Running `git clone` against a Mercurial repository does not produce an error message with explicit guidance.
Dec 2 2016, 2:45 AM · Diffusion, Feature Request
nornagon awarded T10978: Modernize Audit a Like token.
Dec 2 2016, 1:51 AM · Audit
chad added a comment to T11938: Running `git clone` against a Mercurial repository does not produce an error message with explicit guidance.

Basically, if this request is just a suggestion, we'll probably pass on accepting it (see Hypotheticals). But if people are frequently having issues (we haven't seen this) please give us more details on the workflow, etc, people are tripping up on. I don't think just adding the word "Git" somewhere will necessarily solve any core problem since people are not likely to read, for instance.

Dec 2 2016, 1:31 AM · Diffusion, Feature Request
neilfitz added a comment to T11938: Running `git clone` against a Mercurial repository does not produce an error message with explicit guidance.

Ok, appreciate the feedback - I'll try to tackle it in a bit more detail.

Dec 2 2016, 1:24 AM · Diffusion, Feature Request
chad added a comment to T11938: Running `git clone` against a Mercurial repository does not produce an error message with explicit guidance.

We only take feature requests that state root problems, see Contributing Feature Requests and Describing Root Problems.

Dec 2 2016, 1:12 AM · Diffusion, Feature Request

Dec 1 2016

neilfitz created T11938: Running `git clone` against a Mercurial repository does not produce an error message with explicit guidance.
Dec 1 2016, 10:50 PM · Diffusion, Feature Request
epriestley closed T11936: Editing Diffusion URIs does not update URI index as Resolved by committing Restricted Diffusion Commit.
Dec 1 2016, 10:29 PM · Diffusion, Bug Report
epriestley added a comment to T11936: Editing Diffusion URIs does not update URI index.

I'm pretty sure this this was just a straightforward fix that escaped detection for this long because any other repository edit would rebuild the index. D16972 rebuilds the index on any repository URI edit. Let us know if you're seeing issues after that. Thanks for the report!

Dec 1 2016, 10:29 PM · Diffusion, Bug Report
epriestley added a revision to T11936: Editing Diffusion URIs does not update URI index: Unknown Object (Differential Revision).
Dec 1 2016, 10:27 PM · Diffusion, Bug Report
egg82 added a comment to T11937: GCloud SQL does not support configuring sql_mode.

From what I understand it's something Google is working on. I've been dealing with the limitations myself for a while and it sucks, but hopefully it'll be fixed on their end sooner rather than later.

Dec 1 2016, 10:26 PM · Infrastructure, Setup, Feature Request
epriestley renamed T11937: GCloud SQL does not support configuring sql_mode from Making Phabricator more compatible with managed SQL services to GCloud SQL does not support configuring sql_mode.
Dec 1 2016, 10:13 PM · Infrastructure, Setup, Feature Request
epriestley added a comment to T11937: GCloud SQL does not support configuring sql_mode.

We aren't interested in introducing complexity into Phabricator to work around limitations in GCloud SQL at this time. Instead, you should ask Google to fix this limitation with their service.

Dec 1 2016, 10:12 PM · Infrastructure, Setup, Feature Request
epriestley added a comment to T11936: Editing Diffusion URIs does not update URI index.

Does making any other edit to the repository (e.g., changing the description) correct the issue?

Dec 1 2016, 9:57 PM · Diffusion, Bug Report
egg82 added a comment to T11937: GCloud SQL does not support configuring sql_mode.

Specifically I'm looking at GCloud SQL, which does not yet allow you to use config options that require system-level changes. You can still use a dedicated SQL GCE instance, but using the managed SQL itself isn't yet possible without running into errors about the sql_mode.

Dec 1 2016, 9:57 PM · Infrastructure, Setup, Feature Request
epriestley added a comment to T11937: GCloud SQL does not support configuring sql_mode.

As far as I can tell, Amazon RDS allows configuration of sql_mode. Here's their announcement of support (from May 2016):

Dec 1 2016, 9:46 PM · Infrastructure, Setup, Feature Request
avivey merged T10272: Expose "branched from" information in conduit differential API into T3462: In Differential, expose "featurename (branched from master)" via Conduit.
Dec 1 2016, 9:29 PM · Differential, Arcanist
avivey merged task T10272: Expose "branched from" information in conduit differential API into T3462: In Differential, expose "featurename (branched from master)" via Conduit.
Dec 1 2016, 9:29 PM · Restricted Project, Conduit, Feature Request
avivey renamed T3462: In Differential, expose "featurename (branched from master)" via Conduit from Show "Branch: featurename (branched from master)" in Differential to Show "Branch: featurename (branched from master)" in Differential UI, conduit.
Dec 1 2016, 9:29 PM · Differential, Arcanist
farseide added a comment to T10272: Expose "branched from" information in conduit differential API.

I'm in the same situation. This would be a great addition.

Dec 1 2016, 9:23 PM · Restricted Project, Conduit, Feature Request
egg82 created T11937: GCloud SQL does not support configuring sql_mode.
Dec 1 2016, 8:33 PM · Infrastructure, Setup, Feature Request
fooishbar added a comment to T11936: Editing Diffusion URIs does not update URI index.

OK, this is interesting: having tried reproducing it on a test Phacility instance, it works as expected there. Perhaps one difference is that my local installations don't have Diffusion serving repositories itself over, e.g., SSH? These repositories do not have any URIs other than a remote Git URI which Diffusion is observing.

Dec 1 2016, 8:30 PM · Diffusion, Bug Report
fooishbar created T11936: Editing Diffusion URIs does not update URI index.
Dec 1 2016, 8:14 PM · Diffusion, Bug Report
epriestley added a revision to T11922: Upgrading: Changes to Fulltext Indexing (InnoDB Fulltext): Unknown Object (Differential Revision).
Dec 1 2016, 7:32 PM · FreeBSD, Restricted Project, Installing & Upgrading
epriestley added a comment to T11934: Make email commands ("/subscribe") work in all contexts, including Conpherence ("/me").

See also T7787 for plans to support existing !cc stuff from arc on the CLI.

Dec 1 2016, 7:31 PM · Conpherence (v4), Feature Request
epriestley added a comment to T11934: Make email commands ("/subscribe") work in all contexts, including Conpherence ("/me").

(This doesn't really follow the rules as a feature request because it's just 35 copies of "other systems implement it so you should too", but I think it's something we would probably implement anyway sooner or later so I'm fine with keeping it open.)

Dec 1 2016, 7:30 PM · Conpherence (v4), Feature Request
epriestley closed T11933: Let `PhutilTypeLexer` support PHP style qualified names as Resolved.

Presuming this was resolved by D16967, let me know if there was more to be done.

Dec 1 2016, 7:20 PM · libphutil, Feature Request
J5lx added a watcher for Harbormaster: J5lx.
Dec 1 2016, 6:30 PM
chad closed T11791: Lots of double posts in Conpherence as Resolved by committing Restricted Diffusion Commit.
Dec 1 2016, 6:08 PM · Conpherence (v4)
chad added a revision to T11791: Lots of double posts in Conpherence: Unknown Object (Differential Revision).
Dec 1 2016, 6:06 PM · Conpherence (v4)
chad closed T11935: Misaligned UI elements in Differential UI as Resolved by committing Restricted Diffusion Commit.
Dec 1 2016, 5:08 PM · Bug Report
chad added a revision to T11935: Misaligned UI elements in Differential UI: Unknown Object (Differential Revision).
Dec 1 2016, 5:05 PM · Bug Report
chad removed a project from T11935: Misaligned UI elements in Differential UI: Differential.
Dec 1 2016, 6:31 AM · Bug Report
jcarrillo7 created T11935: Misaligned UI elements in Differential UI.
Dec 1 2016, 6:15 AM · Bug Report
carlokok added a comment to T11816: Calendar: Feedback and Suggestions.

It makes sense in retrospect, but not from the workflow no.

Dec 1 2016, 5:32 AM · Restricted Project, Calendar
joshma added a comment to Q526: Does ELB/ALB need extra configuration to proxy git ssh?.

Thank you! I'll do that and report back.

Dec 1 2016, 2:16 AM
epriestley added a comment to Q526: Does ELB/ALB need extra configuration to proxy git ssh?.

Try putting MaxStartups 1024 in your /etc/ssh/sshd_config (or similar).

Dec 1 2016, 2:14 AM
joshma updated Q526: Does ELB/ALB need extra configuration to proxy git ssh? from to Does ELB/ALB need extra configuration to proxy git ssh?.
Dec 1 2016, 1:51 AM
joshuaspence added a project to T11933: Let `PhutilTypeLexer` support PHP style qualified names: libphutil.
Dec 1 2016, 1:36 AM · libphutil, Feature Request
20after4 awarded Blog Post: Write, Review, Merge, Publish: Phabricator Review Workflow a Orange Medal token.
Dec 1 2016, 1:29 AM
mribau_a added a comment to T10260: Harbormaster Herald Rules.

My management would like to flag Commits for Audit if they either don't have an accepted Revision (already have this setup in Herald) OR they have failed Harbormaster Builds. We're not terribly interested in notifications/emails for this. This use case seems a bit different than others discussed above.

Dec 1 2016, 1:25 AM · Haskell.org, Restricted Project, Harbormaster, Feature Request
lewellyn created T11934: Make email commands ("/subscribe") work in all contexts, including Conpherence ("/me").
Dec 1 2016, 1:24 AM · Conpherence (v4), Feature Request
wjiang added a revision to T11933: Let `PhutilTypeLexer` support PHP style qualified names: Unknown Object (Differential Revision).
Dec 1 2016, 1:06 AM · libphutil, Feature Request
wjiang updated subscribers of T11933: Let `PhutilTypeLexer` support PHP style qualified names.
Dec 1 2016, 1:02 AM · libphutil, Feature Request
wjiang created T11933: Let `PhutilTypeLexer` support PHP style qualified names.
Dec 1 2016, 1:02 AM · libphutil, Feature Request
avivey removed a project from T10260: Harbormaster Herald Rules: Restricted Project.
Dec 1 2016, 12:55 AM · Haskell.org, Restricted Project, Harbormaster, Feature Request
carwyn awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Dec 1 2016, 12:40 AM · Workboards (v3), Restricted Project

Nov 30 2016

ocrete awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Nov 30 2016, 8:34 PM · Workboards (v3), Restricted Project
epriestley updated subscribers of T11922: Upgrading: Changes to Fulltext Indexing (InnoDB Fulltext).

@csilvers, not sure if you got directed here from IRC. See also T11932. At HEAD, we'll now issue more explicit guidance about the required index rebuild.

Nov 30 2016, 7:24 PM · FreeBSD, Restricted Project, Installing & Upgrading
epriestley triaged T11932: Manual Activity: Rebuild Search Index as Normal priority.
Nov 30 2016, 7:13 PM · Search, Guides
epriestley added a comment to T11816: Calendar: Feedback and Suggestions.

Hmm, I'm guessing this wasn't clear from the workflow either, then?

Nov 30 2016, 6:48 PM · Restricted Project, Calendar
epriestley added a revision to T11922: Upgrading: Changes to Fulltext Indexing (InnoDB Fulltext): Unknown Object (Differential Revision).
Nov 30 2016, 6:47 PM · FreeBSD, Restricted Project, Installing & Upgrading
carlokok added a comment to T11816: Calendar: Feedback and Suggestions.

My apologies. I read over that bit and presumed it meant edit the calendar items

Nov 30 2016, 6:37 PM · Restricted Project, Calendar
epriestley created U33 Manual Activity: Rebuild Search Index.
Nov 30 2016, 6:18 PM
epriestley created T11932: Manual Activity: Rebuild Search Index.
Nov 30 2016, 6:18 PM · Search, Guides
epriestley added a comment to T11816: Calendar: Feedback and Suggestions.

Why did you allow everyone everyone edit the event?

Nov 30 2016, 5:15 PM · Restricted Project, Calendar
carlokok added a comment to T11816: Calendar: Feedback and Suggestions.

Everyone can see from which url a calendar event came.

This isn't expected. Can you give me reproduction steps for this?

Nov 30 2016, 4:41 PM · Restricted Project, Calendar
epriestley added a comment to T11816: Calendar: Feedback and Suggestions.

Everyone can see from which url a calendar event came.

Nov 30 2016, 4:33 PM · Restricted Project, Calendar
epriestley closed T11931: Conduit API of Projects return their parents projects info as Invalid.

Please read Contributing Feature Requests carefully before filing a feature request.

Nov 30 2016, 4:31 PM · Feature Request
chad updated subscribers of T11931: Conduit API of Projects return their parents projects info.
Nov 30 2016, 3:20 PM · Feature Request
carlokok added a comment to T11816: Calendar: Feedback and Suggestions.

My only concern with the current ics importing is that the url for importing from google, owncloud/nextcloud usually contains secrets other users shouldn't see, like in the case of owncloud: un/pw. Everyone can see from which url a calendar event came.

Nov 30 2016, 3:06 PM · Restricted Project, Calendar
• abdulrahman17 updated subscribers of T11931: Conduit API of Projects return their parents projects info.
Nov 30 2016, 2:58 PM · Feature Request
• abdulrahman17 created T11931: Conduit API of Projects return their parents projects info.
Nov 30 2016, 2:57 PM · Feature Request
cbierstedt closed Q525: How to properly migrate hosted repositories? as resolved.
Nov 30 2016, 1:06 PM
cbierstedt updated the answer details for Q525: How to properly migrate hosted repositories? (Answer 483).
Nov 30 2016, 1:06 PM
cbierstedt added Q525: How to properly migrate hosted repositories? (Answer 483).
Nov 30 2016, 1:05 PM
niteen added a comment to T5508: Add branch name parameter to Harbormaster "Make HTTP Request" build step.

did anyone fix this issue?

Nov 30 2016, 10:58 AM · Restricted Project, Harbormaster
cbierstedt updated Q525: How to properly migrate hosted repositories? from to How to properly migrate hosted repositories?.
Nov 30 2016, 10:53 AM
dvcrn added a comment to T10612: Editing a panel causes it to duplicate in dashboard.

Same problem here. A way around this problem is to not re-use panels: Creating a completely new panel is adding it only once, however picking an existing panel is adding it twice.

Nov 30 2016, 7:17 AM · Dashboards (v2), Bug Report

Nov 29 2016

hach-que added a comment to T11925: Set niceity levels of background processes: git diff sometimes appears to thrash disk I/O and CPU from background worker.

Separately I managed to grab this output from top at an impacted time, but this looks to be a different issue now since the CPU time is being reported against the taskmasters themselves:

Nov 29 2016, 11:34 PM · Feature Request
epriestley edited the content of Phacility Cluster Devices.
Nov 29 2016, 9:55 PM
joshuaspence added a comment to T11930: Users don't realise that they must link their account.

That didn't seem to be the case. It might have been because we only enforced Google authentication around one year ago, before that users could use a password for authentication. I can check later today, but perhaps the users that ended up with duplicate accounts never setup Google Auth.

Nov 29 2016, 9:42 PM · NUX, Auth
epriestley added a comment to T11930: Users don't realise that they must link their account.

If the only way to register is Google, it should be impossible to create more than one account per Google account.

Nov 29 2016, 9:37 PM · NUX, Auth
epriestley added a comment to T11930: Users don't realise that they must link their account.

Why is registration disabled?

Nov 29 2016, 9:36 PM · NUX, Auth
joshuaspence added a project to T11930: Users don't realise that they must link their account: NUX.
Nov 29 2016, 9:35 PM · NUX, Auth
joshuaspence added a comment to T11930: Users don't realise that they must link their account.

To provide some additional context, we originally allowed users to register their own accounts, but we eventually disabled this because we ended up with around 30 duplicate accounts (users with more than one account). I think that this happened because users had listed an alias address in their Phabricator settings, but Google was using a canonical email address to authenticate, which didn't map to any existing user accounts (we have a lot of users that have aliases set up. For example, abraham@mycompany.com would be an alias for the canonical alincoln@mycompany.com address).

Nov 29 2016, 9:35 PM · NUX, Auth
joshuaspence created T11930: Users don't realise that they must link their account.
Nov 29 2016, 9:32 PM · NUX, Auth
chad updated the task description for T3612: Lightbox v2.
Nov 29 2016, 8:44 PM · Lightbox, Restricted Project, Files, Pholio
epriestley added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

This one is fine, I'm still going to fiddle with "in" too.

Nov 29 2016, 8:36 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
aklapper added a comment to T8510: Typeahead project proposals in Maniphest advanced search do not always include exact matches.

In https://phabricator.wikimedia.org/T76732#2803813 ksmith brings up that

In the search bar in the toolbar at the top of the screen, searching for "Maps" brings up Maps as the third option. Searching for "Discovery" brings up Discovery as the second option.
When editing a task and entering tags, autocomplete does bring up Maps and Discovery as the first items.

but that could be a separate (unprioritized) task I guess.

Nov 29 2016, 8:35 PM · Typeahead, Prioritized, Wikimedia, Infrastructure, Search
epriestley closed T11929: Merge in duplicates doesn't contain latest tasks as Resolved by committing Restricted Diffusion Commit.
Nov 29 2016, 8:15 PM · Bug Report, Maniphest
rsolomon added a comment to T9262: Community and Upstream interests for a flowtype linter.

I know this is more than a year old but...

Nov 29 2016, 7:36 PM · Lint
chad added a revision to T3612: Lightbox v2: Unknown Object (Differential Revision).
Nov 29 2016, 7:01 PM · Lightbox, Restricted Project, Files, Pholio
epriestley added a revision to T11929: Merge in duplicates doesn't contain latest tasks: Unknown Object (Differential Revision).
Nov 29 2016, 6:40 PM · Bug Report, Maniphest
faulconbridge edited P2020 create repo via conduit.
Nov 29 2016, 5:53 PM
faulconbridge edited P2020 create repo via conduit.
Nov 29 2016, 5:35 PM
faulconbridge edited P2020 create repo via conduit.
Nov 29 2016, 5:31 PM
faulconbridge created P2020 create repo via conduit.
Nov 29 2016, 5:30 PM
chad added a comment to T11928: Risk of losing work when creating a new document while editing a Phriction page .

(Also, I can't reproduce this.)

Nov 29 2016, 5:24 PM · Phriction, Bug Report
chad added a revision to T3612: Lightbox v2: Unknown Object (Differential Revision).
Nov 29 2016, 5:21 PM · Lightbox, Restricted Project, Files, Pholio
MarcDufresne awarded T10612: Editing a panel causes it to duplicate in dashboard a Like token.
Nov 29 2016, 4:08 PM · Dashboards (v2), Bug Report
chad added a comment to T11928: Risk of losing work when creating a new document while editing a Phriction page .

Generally there shouldn't bee a "New <object>" on edit object pages. I think this goes away with EditEngine.

Nov 29 2016, 3:26 PM · Phriction, Bug Report
chad created T11929: Merge in duplicates doesn't contain latest tasks.
Nov 29 2016, 3:22 PM · Bug Report, Maniphest