Page MenuHomePhabricator
Feed Advanced Search

Sep 4 2016

huangsj added a comment to T11586: Display errors/nonprintable characters more clearly when warning users about invalid data inputs.

For example, I want to import
readonly_user readonly_user@company.com readonly_user readonly_user

Sep 4 2016, 2:55 PM · Contributor Onboarding
chad added a comment to T11586: Display errors/nonprintable characters more clearly when warning users about invalid data inputs.

I am not able to reproduce any issue with these email address formats. Do you have any suggestions for how I can do things differently?

Sep 4 2016, 2:25 PM · Contributor Onboarding
chad added a comment to T11586: Display errors/nonprintable characters more clearly when warning users about invalid data inputs.

What is the exact error message you received?

Sep 4 2016, 2:15 PM · Contributor Onboarding
huangsj created T11586: Display errors/nonprintable characters more clearly when warning users about invalid data inputs.
Sep 4 2016, 7:23 AM · Contributor Onboarding
cspeckmim added a comment to T10907: Older project's icon changes back after a few days.

After upgrading and several weeks of the icons not disappearing,

cleanhouse.jpg (370×400 px, 33 KB)

Sep 4 2016, 4:15 AM · Projects, Bug Report (Needs Information)
chad added a comment to T11584: Phame commenting policy checks are weird?.

I poked a bit at this, but I think PhamePost is a special case, since it doesn't have a direct edit policy, it's edit policy is derived from PhameBlog and it's not clear to me how comment transactions get around edit policy in general reading through the code.

Sep 4 2016, 2:16 AM · Phame

Sep 3 2016

chad merged task T11585: Illegal characters confusing diffusion into T3788: Improve encoding management in Diffusion.
Sep 3 2016, 10:06 PM · Diffusion, Bug Report
chad merged T11585: Illegal characters confusing diffusion into T3788: Improve encoding management in Diffusion.
Sep 3 2016, 10:06 PM · Restricted Project, Diffusion
Luke081515.2 created T11585: Illegal characters confusing diffusion.
Sep 3 2016, 9:32 PM · Diffusion, Bug Report
chad edited the content of Changelog.
Sep 3 2016, 9:18 PM
tide added a comment to T11584: Phame commenting policy checks are weird?.

J772

Sep 3 2016, 8:08 PM · Phame
tide added a comment to T11584: Phame commenting policy checks are weird?.

pasted_file (646×1 px, 91 KB)

Sep 3 2016, 8:08 PM · Phame
chad added a comment to Blog Post: Differential.

hunter2

Sep 3 2016, 8:07 PM
chad added a comment to T11583: When initializing a new database, just apply adjustments without prompting.

thx

Sep 3 2016, 8:05 PM · NUX, Setup
epriestley created T11584: Phame commenting policy checks are weird?.
Sep 3 2016, 7:13 PM · Phame
epriestley created T11583: When initializing a new database, just apply adjustments without prompting.
Sep 3 2016, 6:52 PM · NUX, Setup
chad awarded T11506: Build smarter CLI caching for instance data in the Phacility cluster a Mountain of Wealth token.
Sep 3 2016, 6:43 PM · Phacility
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

This appears to be deployed and stable. I'm going to leave this open for 24 hours so I can get a reading out of Multimeter tomorrow, but looks like everything worked as intended.

Sep 3 2016, 6:22 PM · Phacility
epriestley added a comment to T9897: Unbeta Feedback on Phame.

Please follow the instructions in Contributing Bug Reports if you'd like to report a bug. Thanks!

Sep 3 2016, 6:08 PM · Phame
Luke081515.2 added a comment to T9897: Unbeta Feedback on Phame.

FYI, there is a report about a regression from multiple users at WMFs-Phabricator: https://phabricator.wikimedia.org/T144338

Sep 3 2016, 6:06 PM · Phame
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

(rSERVICES1fcb5cdb758211f6389729956d7c77f90edc2243)

Sep 3 2016, 5:18 PM · Phacility
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

I caught an issue with D16464 and lazy/partial parsing of the "Host:" header on the repo tier, deploying a fix for that now.

Sep 3 2016, 5:18 PM · Phacility
chad awarded Q474: Can chatrooms be configured to send message upon RETURN press? a 100 token.
Sep 3 2016, 5:12 PM · Conpherence
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

I've been fiddling with host destroy while letting things settle. Here's load on admin001 over the last week:

Sep 3 2016, 4:36 PM · Phacility
• jataved awarded Q474: Can chatrooms be configured to send message upon RETURN press? a Burninate token.
Sep 3 2016, 4:34 PM · Conpherence
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

Yeah -- I assume they wanted to know as soon as we create a badge, and can't afford to wait more than 50ms!

Sep 3 2016, 3:11 PM · Phacility
chad added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

Load wasn't dropping as much as I expected, but that seems to be because someone started making 20 requests / second to list badges available on admin in the last 24 hours. Shutting that down now.

Sep 3 2016, 2:31 PM · Phacility
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

rCOREddab1d4e shut down the user who was really excited about badges. Looks like they started about 15 hours ago.

Sep 3 2016, 2:17 PM · Phacility
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

Load wasn't dropping as much as I expected, but that seems to be because someone started making 20 requests / second to list badges available on admin in the last 24 hours. Shutting that down now.

Sep 3 2016, 2:02 PM · Phacility
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

I've deployed repo001 and the disk cache appears to be holding. I've checked a bunch of operations and logs without catching anything that looks broken, so I'm going to roll the rest of repo, then db, then web, then look for load effects on admin.

Sep 3 2016, 1:19 PM · Phacility
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

The instance management cache seems OK, new cost is 11 queries in 80ms. The wall-time isn't directly comparable but both numbers being smaller is good.

Sep 3 2016, 12:51 PM · Phacility
nemobis added a comment to T6296: Offer "Do not notify me of changes when the task's subscribers change unless I am reporter of the task" setting.

(Self-reminder of the downstream report: https://phabricator.wikimedia.org/T624#2606686 .)

Sep 3 2016, 12:43 PM · Wikimedia, Maniphest
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

I've deployed all the auxiliary tiers, admin is up next.

Sep 3 2016, 12:43 PM · Phacility
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

I'm starting this now.

Sep 3 2016, 12:31 PM · Phacility
kebe added a comment to T9640: Make Phabricator compatible with PHP7.

Is worth to mention that PHP 7.1 RC1 is available. Does it resolve our problems with comparability?

Sep 3 2016, 9:34 AM · Prioritized, Restricted Project, Infrastructure
• sheny added a comment to T11575: Make PhutilDocblockParser always return arrays when parsing "@doc-tags".
Sep 3 2016, 7:05 AM · Bug Report
• sheny added a comment to T11575: Make PhutilDocblockParser always return arrays when parsing "@doc-tags".
Sep 3 2016, 7:04 AM · Bug Report
chad merged task T11581: Add alternative (grid) layout for projects listing into T4011: Improve Projects list view / search result layout.
Sep 3 2016, 4:50 AM · Feature Request
chad merged T11581: Add alternative (grid) layout for projects listing into T4011: Improve Projects list view / search result layout.
Sep 3 2016, 4:50 AM · Design, Projects (v3)
ablekh added a comment to T11582: Consider adding hierarchical visual management of tasks within a project.

No problem and you're welcome.

Sep 3 2016, 4:46 AM · Feature Request
chad closed T11582: Consider adding hierarchical visual management of tasks within a project as Invalid.

We don't take Design/UX suggestions as feature requests, unfortunately. Thanks for the suggestion, we'll keep it in mind.

Sep 3 2016, 4:45 AM · Feature Request
ablekh created T11582: Consider adding hierarchical visual management of tasks within a project.
Sep 3 2016, 4:12 AM · Feature Request
ablekh created T11581: Add alternative (grid) layout for projects listing.
Sep 3 2016, 4:00 AM · Feature Request
cspeckmim added a comment to T11580: Tasks created from workboards frequently have to be edited to add properties common to all tasks on the workboard.

I tried to rephrase slightly to clarify the root cause rather than looking for a solution.

Sep 3 2016, 1:22 AM · Workboards, EditEngine, Projects, Feature Request
epriestley added a comment to T11506: Build smarter CLI caching for instance data in the Phacility cluster.

I'm planning to deploy this tomorrow morning. It also looks like the change associated with D16262 (small cache improvement) got a little lost in the switch to bin/host release and never actually deployed, so that's going out too.

Sep 3 2016, 1:22 AM · Phacility
cspeckmim renamed T11580: Tasks created from workboards frequently have to be edited to add properties common to all tasks on the workboard from Allow workboards to have a default create task form to Tasks created from workboards frequently have to be edited to add properties common to all tasks on the workboard.
Sep 3 2016, 1:22 AM · Workboards, EditEngine, Projects, Feature Request
epriestley edited the content of Changelog.
Sep 3 2016, 1:11 AM
epriestley created 2016 Week 36 (Very Early September).
Sep 3 2016, 1:10 AM
cspeckmim created T11580: Tasks created from workboards frequently have to be edited to add properties common to all tasks on the workboard.
Sep 3 2016, 12:41 AM · Workboards, EditEngine, Projects, Feature Request
caov297 updated Q485: PhabricatorFileDataController class handRequest doesn't have valid viewer/user for CDN file request from to PhabricatorFileDataController class handRequest doesn't have valid viewer/user for CDN file request.
Sep 3 2016, 12:19 AM

Sep 2 2016

epriestley closed T11481: Allow customization of forms for Workboards as Wontfix.
  • T6064 already covers creating tasks with one click instead of two.
  • T10946 covers using EditEngine templates on workboards, I guess. That task is pretty iffy, too. This could probably use a separate, more focused task -- @cspeckmim, if you want to file one describing your use case in a little more detail I can merge T10946 into it if I don't have better ideas for you. (I thought I saw one more of these somewhere while dredging through the backlog to populate Contributor Onboarding but I wasn't immediately able to find it.)
  • We don't currently plan to implement a distinct inline quick edit mode like Trello.
Sep 2 2016, 10:38 PM · Workboards, Feature Request
epriestley closed T11466: Build status on Differential search results as Invalid.

See Contributing Feature Requests and Describing Root Problems.

Sep 2 2016, 10:27 PM · Differential, Feature Request
epriestley closed T11530: Conduit query for Differential revisions mentioning a project as Wontfix.

Broadly, this isn't a use case we intend to support specifically. See T5873, T10363, T11330, T11563#192717 for some related discussion.

Sep 2 2016, 10:26 PM · Herald, Conduit, Differential, Feature Request
epriestley closed T11542: Make remarkup checklists interactive as Invalid.

See Contributing Feature Requests and Describing Root Problems. The context and discussion in T3945 and T9652 may be helpful in formulating a problem description.

Sep 2 2016, 10:25 PM · Remarkup, Feature Request
epriestley closed T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline" as Invalid.

No, it isn't a bug.

Sep 2 2016, 10:23 PM · Feature Request
epriestley closed T11425: Add "Locate File" functionality to branch view in Diffusion as Invalid.

See Contributing Feature Requests and Describing Root Problems.

Sep 2 2016, 10:22 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

:) It's not a very important bug, but isn't it still a bug that the button shows up?

Sep 2 2016, 9:55 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

So the button already does what you want, but instead of clicking it you filed a bug report?

Sep 2 2016, 9:54 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

Got it.

Sep 2 2016, 9:51 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

(Or how our behavior differs materially from the behavior of git diff, except that you have to click one extra time on "Show All Files Inline".)

Sep 2 2016, 9:43 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

It's because more than 100 total paths were affected by any kind of edit.

Sep 2 2016, 9:40 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

I was expecting to see exactly the view in your screenshot -- changed files expanded inline, and moved files described as such.

Sep 2 2016, 9:33 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

What are you expecting to see there instead, exactly?

Sep 2 2016, 9:30 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

I wonder why I had a different behavior:

Sep 2 2016, 9:28 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

If I make a trivial change to one, I get this:

Sep 2 2016, 9:28 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

Okay. If I diff this:

Sep 2 2016, 9:26 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

I suspect that diff.renames has the same behavior from recent versions, just that it is enabled by default.

Sep 2 2016, 9:23 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".
$ git diff HEAD^..HEAD | head
diff --git a/app/client/config/BaseStyles.js b/app-moved/client/config/BaseStyles.js
similarity index 100%
rename from app/client/config/BaseStyles.js
rename to app-moved/client/config/BaseStyles.js
diff --git a/app/client/config/DateTimeFormats.json b/app-moved/client/config/DateTimeFormats.json
similarity index 100%
rename from app/client/config/DateTimeFormats.json
rename to app-moved/client/config/DateTimeFormats.json
diff --git a/app/client/config/categories.json b/app-moved/client/config/categories.json
similarity index 100%
Sep 2 2016, 9:22 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

The behavior appears to have changed in Git 2.9.0, which was released in mid June of this year:

Sep 2 2016, 9:19 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

Wow, that's strange. Is it possible that you have a different setting for diff.renames?

Sep 2 2016, 9:11 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

I can't reproduce that:

Sep 2 2016, 9:07 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

Yes, I'm not claiming that git can somehow not show actual changes, just that it doesn't show file renames. This is what I was doing:

Sep 2 2016, 9:05 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

I can't reproduce that either:

Sep 2 2016, 8:55 PM · Feature Request
jasonaburton added a comment to T11579: DB column missing default value or should be nullable....

Thanks anyways!

Sep 2 2016, 8:53 PM · Bug Report (Needs Information)
epriestley closed T11579: DB column missing default value or should be nullable... as Invalid.

We can't move forward then, sorry.

Sep 2 2016, 8:51 PM · Bug Report (Needs Information)
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

I was thinking of what you see when you then do git add, git commit, and git diff HEAD..HEAD^. I might be using the wrong terminology perhaps

Sep 2 2016, 8:51 PM · Feature Request
jasonaburton added a comment to T11579: DB column missing default value or should be nullable....

No sorry, I don't have logging set up on that server.

Sep 2 2016, 8:48 PM · Bug Report (Needs Information)
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

I marked it as a bug because git diff will return something that's pretty small in this scenario.

Sep 2 2016, 8:46 PM · Feature Request
epriestley added a comment to T11579: DB column missing default value or should be nullable....

Does your webserver error log have a long-form version of that error with a stack trace in it?

Sep 2 2016, 8:44 PM · Bug Report (Needs Information)
jasonaburton added a comment to T11579: DB column missing default value or should be nullable....

I'm honestly not sure. I followed the normal upgrading instructions as always:

Sep 2 2016, 8:35 PM · Bug Report (Needs Information)
epriestley edited projects for T11579: DB column missing default value or should be nullable..., added: Bug Report (Needs Information); removed Bug Report.

I can't reproduce this. arc diff is working fire for me and many other users, and 5 different users have created 7 revisions against this install so far today alone (D16482 through D16489).

Sep 2 2016, 8:27 PM · Bug Report (Needs Information)
jasonaburton created T11579: DB column missing default value or should be nullable....
Sep 2 2016, 8:22 PM · Bug Report (Needs Information)
chad added a comment to T11132: New Phabricator NUX.

You can today, just extend PhabricatorGuideModule and drop it in src/extensions. It should just show up.

Sep 2 2016, 8:11 PM · Design, Phacility, NUX
Luke081515.2 added a comment to T11132: New Phabricator NUX.

Maybe it might be a good idea to allow people to create own Guides? Could be helpful for recuring setup stuff for newusers for example, like we got it at our instance.

Sep 2 2016, 8:07 PM · Design, Phacility, NUX
venky edited projects for T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline", added: Feature Request; removed Bug Report.
Sep 2 2016, 6:32 PM · Feature Request
venky added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

I can make it a feature request. I marked it as a bug because git diff will return something that's pretty small in this scenario.

Sep 2 2016, 6:32 PM · Feature Request
epriestley added a comment to T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".

Can you walk me through how this is a bug?

Sep 2 2016, 6:21 PM · Feature Request
chad added a comment to T11574: Consider making "Page updated, reload" notification more prominent.

All notifications currently show up in the same place, so I think the consistency here is better than something new. I'd be more apt to try new treatments in general vs. moving it. We'll probably need to revisit "something on the page changes" UI if we ever build realtime workboards.

Sep 2 2016, 6:16 PM · Feature Request
venky added a comment to T11574: Consider making "Page updated, reload" notification more prominent.

We'll take this into consideration and keep an eye open for other feedback,

Sep 2 2016, 6:13 PM · Feature Request
venky updated subscribers of T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".
Sep 2 2016, 5:57 PM · Feature Request
venky created T11578: Moving many files and changing a few leads to "This diff is very large and affects 111 files. You may load each file individually or Show All Files Inline".
Sep 2 2016, 5:57 PM · Feature Request
avivey added a comment to T10635: Loading differential revision slow when lots of unit test messages exist.

For people who suffer from this: Note that D16483 might have actually made things a little worse, by searching the DB for old messages (Not loading them though); I tested with 1M messages on an SSD drive, and I didn't see any changes, but YMMV.

Sep 2 2016, 5:35 PM · Restricted Project, Haskell.org, Harbormaster, Performance
avivey added a comment to Q481: base URI & Allowed URI change issues (Answer 451).

The shell is probably eating away the ". Try surrounding the whole argument with ' to stop it.

Sep 2 2016, 5:03 PM
epriestley added a comment to T11577: Initializing databases for the first time while already in cluster mode fails.

I believe this should be fixed in HEAD of master. It should promote to stable in about 24 hours.

Sep 2 2016, 3:23 PM · Clusters, Bug Report
epriestley closed T11577: Initializing databases for the first time while already in cluster mode fails as Resolved by committing Restricted Diffusion Commit.
Sep 2 2016, 3:23 PM · Clusters, Bug Report
epriestley added a revision to T11577: Initializing databases for the first time while already in cluster mode fails: Unknown Object (Differential Revision).
Sep 2 2016, 3:08 PM · Clusters, Bug Report
epriestley added a comment to T11577: Initializing databases for the first time while already in cluster mode fails.

A bad half-fix for this is here:

Sep 2 2016, 3:01 PM · Clusters, Bug Report
epriestley renamed T11577: Initializing databases for the first time while already in cluster mode fails from Create db schema in cluster mode to Initializing databases for the first time while already in cluster mode fails.
Sep 2 2016, 2:56 PM · Clusters, Bug Report
epriestley added a comment to T11577: Initializing databases for the first time while already in cluster mode fails.

I can reproduce this. Fixing it will take a bit of work.

Sep 2 2016, 2:56 PM · Clusters, Bug Report
aubort created T11577: Initializing databases for the first time while already in cluster mode fails.
Sep 2 2016, 2:34 PM · Clusters, Bug Report