Page MenuHomePhabricator
Feed All Stories

Apr 22 2016

salvian added a comment to T7789: Support Git Large File Storage.
  1. It's not configured
  2. Yes
  3. I'm not sure, these are my install versions (stable branch) :
    • phabricator f75b1cf562c0c3646324864851d693ef1069a068 (Mon, Apr 18)
    • arcanist 01b6fe8bb0239a4bce03f58d6288a3a52ad83a91 (Sat, Apr 16)
    • phutil b8c65df2a910a7028d39bec602d181428b6ce01f (Sat, Apr 16)

(edit, it has rP37b93f4)

Apr 22 2016, 11:13 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a comment to T7789: Support Git Large File Storage.
  • Is security.alternate-file-domain configured on your install?
  • When you say "redirect to the correct file", do you mean you get taken to the file detail page /F1234?
  • Also, do you have rP37b93f4 locally (April 7)? (You can grab the hashes in Config → Versions if you aren't immediately sure and I can figure it out from there.)
Apr 22 2016, 11:08 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a revision to T4292: Implement repository replication: Unknown Object (Differential Revision).
Apr 22 2016, 11:03 AM · Restricted Project, Diffusion
salvian added a comment to T7789: Support Git Large File Storage.

Ah, is the auth token (first [shortened]) same for every user? It is in my case

  1. If I visit the URI while being logged in to the app, the browser redirects to the correct file, the download link works
  2. The download URI is the same as the logged URI
  3. Visiting the URI with private window returns normal 404 page
Apr 22 2016, 11:03 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a comment to T7789: Support Git Large File Storage.

You shouldn't need to configure diffusion.allow-http-auth -- the auth token is basically the first [shortened] in the URI, so that appears to be working properly.

Apr 22 2016, 10:51 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley triaged T10852: Updating a very large diff continues to warn against updating very large diffs as Wishlist priority.
Apr 22 2016, 10:44 AM · Restricted Project, Feature Request, Arcanist
epriestley added a comment to T10852: Updating a very large diff continues to warn against updating very large diffs.

I think we generate this warning long before attaching the diff to a revision, so it isn't trivial to know where to look for a "you already confirmed this prompt once for the current revision" flag.

Apr 22 2016, 10:44 AM · Restricted Project, Feature Request, Arcanist
epriestley closed T10848: Notification mails containing old links after changing phabricator.base-uri as Resolved.
Apr 22 2016, 10:25 AM · Bug Report
• damio updated the question details for Q376: How to work with multiple branch to land.
Apr 22 2016, 10:23 AM · Differential, Arcanist
salvian added a comment to T7789: Support Git Large File Storage.

I'm using S3 as Files storage backend, phabricator runs behind nginx, and the diffusion is hosted over SSH (diffusion.allow-http-auth=false, HTTP is not configured anyway). I'm having an issue:

  • pushing and pulling small files (<2MB) tracked as LFS -> works flawlessly
  • pushing 11x 1 GB files works, but can't do git clone / git lfs fetch (got the same issue as @benjumanji, the lfs_trace log is similar, 404 errors when getting the lfs-* files)
  • pushing a 11GB file, doesn't work. forgot the error message, will add it later if needed
Apr 22 2016, 9:03 AM · Customer Impact, Restricted Project, Files, Diffusion
vinzent added a comment to T10848: Notification mails containing old links after changing phabricator.base-uri.

please close this. i think not restarting phd is my fault.

Apr 22 2016, 7:00 AM · Bug Report
vinzent renamed T10848: Notification mails containing old links after changing phabricator.base-uri from Mail links containing hostname instead of phabricator.base-uri to Notification mails containing old links after changing phabricator.base-uri.
Apr 22 2016, 6:52 AM · Bug Report
vinzent updated the task description for T10848: Notification mails containing old links after changing phabricator.base-uri.
Apr 22 2016, 6:49 AM · Bug Report
vinzent added a comment to T10848: Notification mails containing old links after changing phabricator.base-uri.

reproduced it on a fresh vm.

Apr 22 2016, 6:46 AM · Bug Report
vinzent updated the task description for T10848: Notification mails containing old links after changing phabricator.base-uri.
Apr 22 2016, 6:45 AM · Bug Report
salvian awarded T7789: Support Git Large File Storage a Like token.
Apr 22 2016, 4:47 AM · Customer Impact, Restricted Project, Files, Diffusion
Jacksgong added a comment to T5132: Document the special syntax you can use in commit messages to cause effects.

Very Useful!

Apr 22 2016, 4:14 AM · Guides, Diffusion
eadler updated the task description for T5132: Document the special syntax you can use in commit messages to cause effects.
Apr 22 2016, 4:12 AM · Guides, Diffusion
Jacksgong awarded T5132: Document the special syntax you can use in commit messages to cause effects a Like token.
Apr 22 2016, 4:11 AM · Guides, Diffusion
chad edited projects for T10852: Updating a very large diff continues to warn against updating very large diffs, added: Feature Request; removed Bug Report.
Apr 22 2016, 4:03 AM · Restricted Project, Feature Request, Arcanist
cspeckmim updated the task description for T10852: Updating a very large diff continues to warn against updating very large diffs.
Apr 22 2016, 2:57 AM · Restricted Project, Feature Request, Arcanist
cspeckmim renamed T10852: Updating a very large diff continues to warn against updating very large diffs from Updating a very large diff continues to warn against creating very large diffs to Updating a very large diff continues to warn against updating very large diffs.
Apr 22 2016, 2:56 AM · Restricted Project, Feature Request, Arcanist
cspeckmim created T10852: Updating a very large diff continues to warn against updating very large diffs.
Apr 22 2016, 2:50 AM · Restricted Project, Feature Request, Arcanist
cspeckmim added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

@fcoelho - Would you be able to test your log change with pushing to a mercurial repository not hosted by Phabricator, to compare with the log from your previous comment?

Apr 22 2016, 2:24 AM · Mercurial, Bug Report
cspeckmim awarded T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable a Like token.
Apr 22 2016, 2:02 AM · Ops, Phacility
chad updated the task description for T8909: Provide ability to hide all inline or done comments.
Apr 22 2016, 1:07 AM · Inline Comments, Prioritized, Restricted Project, Restricted Project, Differential, FreeBSD
svendjo added Q360: Exclude directory in .arcconfig? (Answer 366).
Apr 22 2016, 1:05 AM
svendjo added a comment to Q360: Exclude directory in .arcconfig?.
Apr 22 2016, 1:05 AM
epriestley closed T10695: OOM killer nuked `mysql` on `secure.phabricator.com` after apache2 spawned 75+ workers as Resolved.

We appear to be stable here now:

Apr 22 2016, 12:28 AM · Phacility, Ops
epriestley added a comment to T10849: Support query timeouts via `max_statement_time` or MYSLQI_ASYNC.

This is also maybe a good time to fix all the bad old mysql vs mysqli stuff, mostly:

Apr 22 2016, 12:25 AM · Infrastructure, Phacility

Apr 21 2016

angie added a project to T8909: Provide ability to hide all inline or done comments: Restricted Project.
Apr 21 2016, 11:37 PM · Inline Comments, Prioritized, Restricted Project, Restricted Project, Differential, FreeBSD
epriestley added a comment to T10848: Notification mails containing old links after changing phabricator.base-uri.

I can't reproduce this. For example, this host does not reproduce the issue:

Apr 21 2016, 11:21 PM · Bug Report
epriestley added a comment to T10849: Support query timeouts via `max_statement_time` or MYSLQI_ASYNC.

D15781 is promising and it looks like we can do this purely on the client if it has relatively old PHP, without requiring relatively new MySQL and special configuration on the server.

Apr 21 2016, 11:18 PM · Infrastructure, Phacility
epriestley added a revision to T10849: Support query timeouts via `max_statement_time` or MYSLQI_ASYNC: Unknown Object (Differential Revision).
Apr 21 2016, 11:18 PM · Infrastructure, Phacility
angie moved T5031: Draft Comment Persists After Submission from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:22 PM · Restricted Project, Support Impact, Mobile
angie moved T9222: Arc upgrade returns success message when it failed from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:22 PM · Restricted Project, Arcanist
angie moved T9696: /paste/ fails to load from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:22 PM · Paste, Restricted Project
angie moved T9132: Build an ApplicationEditor abstraction from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:22 PM · Restricted Project, ApplicationEditor, Transactions
angie moved T6226: Conduit-created task always results in [Reopened] tag in the email subject from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:22 PM · Restricted Project, Maniphest, Conduit
angie moved T9605: Running prod daemons fails on test files from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:22 PM · Daemons, Restricted Project
angie moved T3855: `arc land` should work when the current branch/bookmark and the target branch/bookmark are the same from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:22 PM · Restricted Project, Arcanist
angie moved T8610: No threaded comments --hard to follow in line comments when two bugs exist on one line from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:20 PM · Restricted Project
angie moved T10009: phabricator and maniphest should recognize links to themselves from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:19 PM · Maniphest, Restricted Project, Feature Request
angie moved T10088: ability to wontfix a maniphest task via an email command from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 21 2016, 10:19 PM · Maniphest, Restricted Project, Feature Request
Taskle added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

Also, the workaround now is to do a search so it's not horribly broken, it's just not as great/fast/efficient for us as it used to be.

Apr 21 2016, 9:44 PM · Projects (v3), Feature Request
Taskle added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

@chad yeah it's been a very long time, a year sounds plausible. I keep meaning to share this with you guys but I had been putting it off for a very long time as it hasn't been as important as other items in my world. One of the challenges with these sorts of things is that even though people in the field may be hitting the issue, they're usually trying to do something and in their workflow and don't bother sharing it... it was only when I asked around that people were like, "yeah, I miss how it used to be a long time ago..." etc.

Apr 21 2016, 9:43 PM · Projects (v3), Feature Request
chad added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

Yeah, looks like Tasks were removed from ProjectProfileController in January of 2015. rP3bc54c204

Apr 21 2016, 9:41 PM · Projects (v3), Feature Request
chad added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

I honestly can't even find where tasks was the default on project home... How old was your instance? I checked back 1 year and it was a side-nav icon then, so it's been over a year since we moved tasks off the project home page.

Apr 21 2016, 9:37 PM · Projects (v3), Feature Request
chad added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

It isn't a common use case, actually. If it were we'd be very aware of it given the significant length of time it's already been at HEAD.

Apr 21 2016, 9:25 PM · Projects (v3), Feature Request
epriestley added a comment to T10751: Make Phabricator Highly Available.

Mixed progress. Repository clustering has been stable in production since the last update, and appears to actually work, which is good. We've hit a few minor snags (like a missing sync-before-read in diffusion.querycommits) but all pretty much expected stuff and nothing troubling.

Apr 21 2016, 9:06 PM · Restricted Project, Infrastructure, Prioritized
Taskle added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

@chad I see - my gut is that this is a common use case and that most teams will not take the time/energy to do this customization (or realize it's possible) but that they'd value having an easy way to see tasks as the primary use case of clicking a project name. Just my 2 cents.

Apr 21 2016, 8:50 PM · Projects (v3), Feature Request
vinzent added a watcher for Bug Report: vinzent.
Apr 21 2016, 8:47 PM
chad added a comment to T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable.

Sorry to hear that @otto.bretz, but glad you're still happy with the service overall. I think this is the longest unplanned downtime we've had since launch and it's definitely not acceptable to us.

Apr 21 2016, 8:46 PM · Ops, Phacility
chad added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

We'd prefer a customizable solution vs a hard wired one, if that's what you mean, as that benefits the most scenarios.

Apr 21 2016, 8:43 PM · Projects (v3), Feature Request
Taskle added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

@chad thanks - so sounds like you don't think this would be beneficial to everyone to have?

Apr 21 2016, 8:35 PM · Projects (v3), Feature Request
epriestley added a comment to T8952: Hide "Herald added subscribers..." stories from feed completely.

Like just don't show it at all? That seems reasonable to me, although 0.00000000001 is very funny and we would be losing a truly great joke at such a young age.

Apr 21 2016, 8:11 PM · Feed, Herald
chad added a comment to T8952: Hide "Herald added subscribers..." stories from feed completely.

maybe herald adding a subscriber is truly 0 then as a feed story.

Apr 21 2016, 8:10 PM · Feed, Herald
chad added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

T10308#158036 has a custom panel you can add to your install to shortcut and make adding this locally to projects easier.

Apr 21 2016, 8:01 PM · Projects (v3), Feature Request
chad added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

Long term you can expect Projects will pick up features from Dashboards. We'll let you customize the nav, customize the page, and customize the landing page. You will be able to recreate the old experience if you so choose.

Apr 21 2016, 7:58 PM · Projects (v3), Feature Request
Taskle added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

I appreciate that we can customize this, however this solution is not feasible for us as we have dozens of projects and add new ones regularly.

Apr 21 2016, 7:51 PM · Projects (v3), Feature Request
epriestley added a comment to T8952: Hide "Herald added subscribers..." stories from feed completely.

That's because it was deliberate self-subscribe which is so completely boring that we normally wouldn't even show it. I think it's arguably correct to show the Herald story? Maybe?

Apr 21 2016, 7:50 PM · Feed, Herald
chad added a comment to T8952: Hide "Herald added subscribers..." stories from feed completely.

This is an emergency.

Apr 21 2016, 7:39 PM · Feed, Herald
chad added a comment to T8952: Hide "Herald added subscribers..." stories from feed completely.

Deliberate Subscriber + Herald Subscriber = Herald Subscriber Story.

Apr 21 2016, 7:38 PM · Feed, Herald
chad added a comment to T10851: Please bring back task lists on project pages.

See T10308#158014 for specific instructions.

Apr 21 2016, 7:29 PM · Feature Request
chad merged T10851: Please bring back task lists on project pages into T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.
Apr 21 2016, 7:28 PM · Projects (v3), Feature Request
chad merged task T10851: Please bring back task lists on project pages into T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.
Apr 21 2016, 7:28 PM · Feature Request
chad added a comment to T10851: Please bring back task lists on project pages.

You can customize your navigation to point to open tasks on any Project. For now, you just have to manually add each link to each project.

Apr 21 2016, 7:27 PM · Feature Request
Taskle created T10851: Please bring back task lists on project pages.
Apr 21 2016, 7:25 PM · Feature Request
otto.bretz added a comment to T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable.

Though this downtime came at an unfortunate time for us (we just migrated from bitbucket and in the middle of the workday here in Sweden) I must say that you have dealt with it in a phrofessional way. Thanks.

Apr 21 2016, 7:20 PM · Ops, Phacility
epriestley added a comment to T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable.

Thanks, I think that should be fixed now. Not clear if it was related to everything else and I can't immediately come up with a way that it would have been -- I'll keep an eye on it. We've made some changes to the Aphlict server recently and it might have some issues that still need to be worked out, so it could be purely coincidental.

Apr 21 2016, 7:15 PM · Ops, Phacility
chad closed T6803: Diffusion: sudo rules documentation refinement as Wontfix.
Apr 21 2016, 7:13 PM · Diffusion, Documentation
vinzent added a comment to T6803: Diffusion: sudo rules documentation refinement.

so would you mind closing the task? I can't do it myself.

Apr 21 2016, 7:12 PM · Diffusion, Documentation
chad added a comment to T6803: Diffusion: sudo rules documentation refinement.

I have no idea if we plan to add it or not. See Planning for how we prioritize time.

Apr 21 2016, 7:11 PM · Diffusion, Documentation
otto.bretz added a comment to T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable.

Not sure if its related but I get a message regarding "unresolved setup issue" on our instance[1] now:

Apr 21 2016, 7:09 PM · Ops, Phacility
eadler added a project to T5029: Fill in missing context for low-context diffs against tracked repositories: Restricted Project.
Apr 21 2016, 7:07 PM · KDE, Restricted Project, Diffusion, Differential
vinzent added a comment to T6803: Diffusion: sudo rules documentation refinement.

If you don't plan to add it: just reject the task - no problem. I just wanted to let you guys no that there is a slightly more secure way of doing what you provide to all the novice users out there.

Apr 21 2016, 7:01 PM · Diffusion, Documentation
epriestley added a comment to T4292: Implement repository replication.

The diffusion.querycommits method needs to sync-before-read (at least, if bypassCache is provided?) but currently does not. This can lead to tasks failing on the daemon on an un-synchronized node. Things self-heal, but it would be nice to prevent this.

Apr 21 2016, 6:59 PM · Restricted Project, Diffusion
epriestley closed T10843: Don't write Celerity resources which aren't defined in the local map to cache, a subtask of T10751: Make Phabricator Highly Available, as Resolved.
Apr 21 2016, 6:57 PM · Restricted Project, Infrastructure, Prioritized
epriestley closed T10843: Don't write Celerity resources which aren't defined in the local map to cache as Resolved by committing Restricted Diffusion Commit.
Apr 21 2016, 6:57 PM · Infrastructure
epriestley added a comment to T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable.

Some additional response steps:

Apr 21 2016, 6:53 PM · Ops, Phacility
vinzent added a comment to T10848: Notification mails containing old links after changing phabricator.base-uri.

i'm offsite until next thursday. will update the latest version then and recheck. the version used is about 2 weeks old and from stable branch.

Apr 21 2016, 6:53 PM · Bug Report
vinzent updated the task description for T10848: Notification mails containing old links after changing phabricator.base-uri.
Apr 21 2016, 6:51 PM · Bug Report
avivey added a parent task for T10850: Search revisions by "Repository Projects": T9287: Working with many repositories.
Apr 21 2016, 6:44 PM · Restricted Project, Restricted Project, Differential, Feature Request
avivey added a subtask for T9287: Working with many repositories: T10850: Search revisions by "Repository Projects".
Apr 21 2016, 6:44 PM · Restricted Project, Restricted Project, Diffusion
avivey created T10850: Search revisions by "Repository Projects".
Apr 21 2016, 6:39 PM · Restricted Project, Restricted Project, Differential, Feature Request
epriestley claimed T10849: Support query timeouts via `max_statement_time` or MYSLQI_ASYNC.
Apr 21 2016, 6:21 PM · Infrastructure, Phacility
epriestley created T10849: Support query timeouts via `max_statement_time` or MYSLQI_ASYNC.
Apr 21 2016, 6:21 PM · Infrastructure, Phacility
lpriestley closed T8897: "Attending" in Calendar list view shows all users, not just attending users as Resolved by committing Restricted Diffusion Commit.
Apr 21 2016, 6:11 PM · Calendar
cguenther awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Apr 21 2016, 6:01 PM · Workboards (v3), Restricted Project
avivey added a comment to T10848: Notification mails containing old links after changing phabricator.base-uri.

It should use the base-uri; However, the early-return in that function makes sense, in the case of an absolute URI.

Apr 21 2016, 5:35 PM · Bug Report
epriestley edited the content of Adding Hardware.
Apr 21 2016, 5:28 PM
lpriestley added a revision to T8897: "Attending" in Calendar list view shows all users, not just attending users: Restricted Differential Revision.
Apr 21 2016, 5:21 PM · Calendar
chad added a comment to T6803: Diffusion: sudo rules documentation refinement.

Accepting contributions isn't free to us. We bear the cost of testing, accepting, documenting, and providing lifetime support. See https://secure.phabricator.com/book/phabcontrib/article/contributing_code/ for more details.

Apr 21 2016, 3:44 PM · Diffusion, Documentation
epriestley updated the task description for T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable.
Apr 21 2016, 3:22 PM · Ops, Phacility
epriestley updated the task description for T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable.
Apr 21 2016, 3:13 PM · Ops, Phacility
epriestley added a comment to T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable.

From here on, this analysis discusses some private instance data. I'm generally very hesitant to disclose information like this, but the data is non-identifying and the nature of the data is important to understanding the nature of the failure, so I think disclosure is reasonable in this case. I've limited the disclosure to only include information important in understanding the technical aspects of the issue.

Apr 21 2016, 3:04 PM · Ops, Phacility
epriestley renamed T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable from 40GB Phacility instance caused a series of cascading failures which left web services unreachable to 30GB Phacility instance caused a series of cascading failures which left web services unreachable.
Apr 21 2016, 2:38 PM · Ops, Phacility
vinzent added a comment to T6803: Diffusion: sudo rules documentation refinement.

this still applies 2 years after sending a diff. is there missing something?

Apr 21 2016, 2:31 PM · Diffusion, Documentation
epriestley renamed T10847: 30GB Phacility instance caused a series of cascading failures which left web services unreachable from Phacility web services were unreachable to 40GB Phacility instance caused a series of cascading failures which left web services unreachable.
Apr 21 2016, 2:26 PM · Ops, Phacility
vinzent created T10848: Notification mails containing old links after changing phabricator.base-uri.
Apr 21 2016, 2:23 PM · Bug Report