Page MenuHomePhabricator
Feed All Stories

Feb 26 2017

epriestley updated subscribers of T12319: Investigate an Owners-related performance issue on revision detail pages.
Feb 26 2017, 3:29 PM · Owners, Differential, Performance
epriestley created T12319: Investigate an Owners-related performance issue on revision detail pages.
Feb 26 2017, 3:29 PM · Owners, Differential, Performance
epriestley added a comment to T12318: Loading Symbols in Phacility (Conduit).

I think this (and lint/coverage) should move to Conduit, yeah.

Feb 26 2017, 3:26 PM · Customer Impact, Symbols, Feature Request
avivey created T12318: Loading Symbols in Phacility (Conduit).
Feb 26 2017, 3:13 PM · Customer Impact, Symbols, Feature Request
gabe awarded T9948: Modernize "arc land" for Mercurial: bookmark-to-branch, branch-to-self, multiple heads a Like token.
Feb 26 2017, 4:55 AM · Mercurial, Arcanist
Mnkras added a comment to T12296: Improve Phacility repository import performance.

Cool, now I don't feel so bad since I know you only recently added the option 😄

Feb 26 2017, 4:44 AM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

(That said, my expectation is that behaviors around unreachable commits have not changed -- D17279 added the "Unreachable" option to solve this problem.)

Feb 26 2017, 4:43 AM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

Likely somewhere in T9430 / T10978, which rewrote a big chunk of the audit code.

Feb 26 2017, 4:42 AM · Customer Impact, Diffusion, Ops
Mnkras added a comment to T12296: Improve Phacility repository import performance.

That works, Im curious when this changed, as I didn't need to filter since the last few weeks.

Feb 26 2017, 4:37 AM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

If you don't want to see unreachable commits, use a query which hides them:

Feb 26 2017, 4:32 AM · Customer Impact, Diffusion, Ops
Mnkras added a comment to T12296: Improve Phacility repository import performance.

The reason I think something isn't working correctly is in my dashboard, I list recent commits, and since I updated my install a few weeks ago my commit list looks like this a lot:

pasted_file (1×1 px, 470 KB)

Feb 26 2017, 4:29 AM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

It is expected that some deleted commits will import and others will not.

Feb 26 2017, 4:05 AM · Customer Impact, Diffusion, Ops
epriestley closed T12317: PhabricatorTriggerDaemon shows as "Waiting" but there are no errors as Resolved.

See T12298#213181.

Feb 26 2017, 4:01 AM · Bug Report
Mnkras created T12317: PhabricatorTriggerDaemon shows as "Waiting" but there are no errors.
Feb 26 2017, 3:35 AM · Bug Report
CodeMouse92 awarded T10350: Scripts to migrate old "points" fields and move projects beneath other projects a Mountain of Wealth token.
Feb 26 2017, 1:49 AM · Projects (v3)
CodeMouse92 added a comment to T10350: Scripts to migrate old "points" fields and move projects beneath other projects.

Thanks @epriestley and @rfreebern - your scripts worked perfectly! Between them both (and tweaking that one "hasSubprojects" field in the database myself), I was able to move an entire project tree. Yay!

Feb 26 2017, 1:49 AM · Projects (v3)
Mnkras added a comment to T12296: Improve Phacility repository import performance.

@epriestley all other commits that have the message "This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref." have imported, so I don't think the reachable code is working correctly, as until within the last few weeks I wasn't seeing these "This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref." commits showing up, but now I see them all the time.

Feb 26 2017, 1:30 AM · Customer Impact, Diffusion, Ops
swisspol added a comment to T10817: Allow more characters for built-in `filename` linter.

For the record, you can work around the problem as such:

{
  "linters": {
    "filename": {
      "type": "filename",
      "exclude": ["(\\+)", "(\\@)"]
    }
}
Feb 26 2017, 1:29 AM · Lint, Feature Request

Feb 25 2017

greenhatman awarded T5051: Burn down charts for Milestones a Like token.
Feb 25 2017, 11:46 PM · Restricted Project, Wikimedia, Projects
epriestley added a comment to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.

Total cost is about 1.6s. Of this:

Feb 25 2017, 4:09 PM · XHProf, Ops, Phacility
epriestley added a comment to T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile.

This is still a little bit of a pain but far more reasonable now, here's an actual profile:

Feb 25 2017, 4:03 PM · XHProf, Ops, Phacility
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I cycled the rest of the tier and things look better now.

Feb 25 2017, 4:00 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

After manually deploying that to repo001, I'm now seeing a consistent Taskmaster/PullLocal count. I'm redeploying the rest of the repo hosts.

Feb 25 2017, 3:05 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I think the issue is that the --reserve flag for reserving system memory now applies to even the first daemon in a pool, because all daemons are now part of pools. Previously, it only applied only to the 2nd..Nth daemon.

Feb 25 2017, 3:02 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

Some taskmasters failed to start on repo001 and perhaps elsewhere.

Feb 25 2017, 2:46 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I suspended and destroyed the largest instance on the shard, a free instance with one user and no logins for >30 days (the user appears to have imported a huge repository and then never logged back in). We still have backups, but this instance was way over reasonable resource use anyway. The volume is now 78% full.

Feb 25 2017, 2:45 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I pruned some very old backups (backup pruning is currently very conservative in several cases). The volume is now 72% full. This could easily be reduced much further by allowing bin/host prune to remove backups older than 90 days for suspended/terminated instances that have been out of service for 90 days, but backup utilization is generally relatively stable so I'm not concerned about this.

Feb 25 2017, 2:26 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I synchronized services for this shard and destroyed old instances. The volume is now 68% full.

Feb 25 2017, 2:13 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I synchronized services for this shard and destroyed old instances. The volume is now 44% full.

Feb 25 2017, 1:44 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

This was a bunch of repository file data in /tmp. Each file was unique, so this doesn't look like any sort of buggy loop or anything. There's some actual bug here but since there's a long list of other issues here I just "fixed" this by clearing /tmp.

Feb 25 2017, 1:08 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

Some reserved instances ended on Feb 19th; I should make sure reserved instances match production instances.

Feb 25 2017, 1:02 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

Shards 017 through 020 are now in production.

Feb 25 2017, 12:57 PM · Phacility, Ops
storrgie added a comment to T5505: Plan Legalpad v2.

We would be prepared to step in for prioritization on this.

Feb 25 2017, 12:49 PM · Restricted Project, Restricted Project
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

The re-launched hosts seem OK, I'm waiting for DNS to update and then redeploying them.

Feb 25 2017, 12:10 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

The hosts launched into an "Auto-Assign Public IPv4" subnet but didn't get IPv4 addresses. Restarting them didn't help, and there's no apparent way to assign public addresses after the fact. I'm going to throw them away and try again.

Feb 25 2017, 12:06 PM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.

I'm going to put more hosts into production immediately before pursuing other issues.

Feb 25 2017, 11:59 AM · Phacility, Ops
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.
  • Some reserved instances ended on Feb 19th; I should make sure reserved instances match production instances.
Feb 25 2017, 11:29 AM · Phacility, Ops
epriestley edited the content of Phacility Cluster Devices.
Feb 25 2017, 11:21 AM
epriestley added a comment to T12316: Phacility Deploy Log 2017 Week 8.
  • The tier as a whole is 88% full.
Feb 25 2017, 11:12 AM · Phacility, Ops
epriestley added a parent task for T12316: Phacility Deploy Log 2017 Week 8: T12217: Reduce the hardware cost of Phacility free instances.
Feb 25 2017, 11:11 AM · Phacility, Ops
epriestley added a subtask for T12217: Reduce the hardware cost of Phacility free instances: T12316: Phacility Deploy Log 2017 Week 8.
Feb 25 2017, 11:11 AM · Ops, Phacility
epriestley created T12316: Phacility Deploy Log 2017 Week 8.
Feb 25 2017, 11:10 AM · Phacility, Ops
• jignesh created D17414: Test.
Feb 25 2017, 5:45 AM
epriestley edited the content of Changelog.
Feb 25 2017, 2:18 AM
epriestley created 2017 Week 8 (Very Late February).
Feb 25 2017, 2:17 AM
• tomekj2ee awarded T4863: Allow Workboard Cards to be customized for display a Like token.
Feb 25 2017, 1:42 AM · Projects, Workboards, Wikimedia
epriestley updated subscribers of T12314: Support formal task subtypes (like "bug" vs "feature").

So here's a tentative plan for subtyping:

Feb 25 2017, 1:09 AM · EditEngine, Maniphest, Custom Fields, Feature Request
epriestley added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

Custom Forms are the general tool we want to use here, but they currently have some limitations for this use case. As a starting point, here's what they can do today, some of the strengths, and some of the weaknesses.

Feb 25 2017, 12:49 AM · EditEngine, Maniphest, Custom Fields, Feature Request
chad closed T12315: Arcanist setup documentation is circular as Resolved by committing Restricted Diffusion Commit.
Feb 25 2017, 12:37 AM · Bug Report
chad added a revision to T12315: Arcanist setup documentation is circular: Unknown Object (Differential Revision).
Feb 25 2017, 12:35 AM · Bug Report
kfsone added a comment to T12315: Arcanist setup documentation is circular.

@epriestley Yep - like what you're doing there :)

Feb 25 2017, 12:35 AM · Bug Report
kfsone added a comment to T12315: Arcanist setup documentation is circular.

So I don't forget:

Feb 25 2017, 12:34 AM · Bug Report
epriestley added a comment to T12315: Arcanist setup documentation is circular.

Maybe the "Install Arcanist" header on Quick Start could be like:

Feb 25 2017, 12:29 AM · Bug Report
kfsone added a comment to T12315: Arcanist setup documentation is circular.

Oh, no, it's just to do with the ordering of the links vs the annotations: if you're totally new to phabricator, you're going to get tripped on the "quick start" link which says "install stuff" and in the middle links you back two pages...

Feb 25 2017, 12:28 AM · Bug Report
chad added a comment to T12315: Arcanist setup documentation is circular.

I'm assuming he wants us to duplicate the installation information onto Win/Mac pages or only link to those pages after installing arcanist.

Feb 25 2017, 12:16 AM · Bug Report
epriestley added a comment to T12315: Arcanist setup documentation is circular.

That is, this passage:

Feb 25 2017, 12:15 AM · Bug Report
epriestley added a comment to T12315: Arcanist setup documentation is circular.

There's a short version and a long version. Each version links to the other version, saying "here's a different version of these instructions, if you'd prefer that one". How could we make this more clear?

Feb 25 2017, 12:13 AM · Bug Report
chad added a comment to T12315: Arcanist setup documentation is circular.

Thanks! I guess nobody ever noticed that before!

Feb 25 2017, 12:13 AM · Bug Report

Feb 24 2017

kfsone created T12315: Arcanist setup documentation is circular.
Feb 24 2017, 11:52 PM · Bug Report
Firehed added a comment to T11741: Quickstart can fail to initialize databases if MyISAM is not available (currently, only in Google Cloud).

Just a note for the above - the initial DB seed script will need to be updated accordingly as well. Right now, trying to perform a clean install on a 2nd-gen Google Cloud SQL instance (which has the above limitation) simply errors out when it reaches one of the tables coded as MyISAM.

Feb 24 2017, 11:06 PM
chad updated the task description for T12270: Unbeta Badges.
Feb 24 2017, 11:01 PM · Badges (v0)
chad added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Feb 24 2017, 11:00 PM · Badges (v0)
jmeador added a comment to T12002: Set header for CURL query for the Notification server status to HTTP:Upgrade = websocket.

@pouyana Here's a workaround for you. In your notification.servers Phabricator configuration, add path: "/ws/" to the client server. Then, modify your apache configuration like so:

Feb 24 2017, 10:53 PM · Bug Report
avivey added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

I don't see how to configure/use that

Neither can I - I may have imagined it being possible to "lock" the edit form of a task to the one that was used to create them...

Feb 24 2017, 10:31 PM · EditEngine, Maniphest, Custom Fields, Feature Request
chad renamed T12314: Support formal task subtypes (like "bug" vs "feature") from Multilevel custom fields. to Multilevel custom fields.
Feb 24 2017, 10:31 PM · EditEngine, Maniphest, Custom Fields, Feature Request
cos added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

I think it's fine to disallow changing from one type to another - can that be enforced? And sure, I think the custom form approach would work but I don't see how to configure/use that - can you point me in the right direction? I presume this implies that the underlying fields are the union of those exposed by all of the forms which would be fine.

Feb 24 2017, 10:23 PM · EditEngine, Maniphest, Custom Fields, Feature Request
epriestley added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

This is a followup to a private discussion elsewhere, I'll flesh it out a bit when I have a chance.

Feb 24 2017, 10:20 PM · EditEngine, Maniphest, Custom Fields, Feature Request
avivey added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

Can a task mutate between "feature" and "risk"? If not, can you use a different Custom Form for each type, which will hide the unused fields?

Feb 24 2017, 10:16 PM · EditEngine, Maniphest, Custom Fields, Feature Request
cos created T12314: Support formal task subtypes (like "bug" vs "feature").
Feb 24 2017, 9:59 PM · EditEngine, Maniphest, Custom Fields, Feature Request
chad updated the task description for T12270: Unbeta Badges.
Feb 24 2017, 9:53 PM · Badges (v0)
chad added revisions to T12270: Unbeta Badges: Unknown Object (Differential Revision), Unknown Object (Differential Revision).
Feb 24 2017, 9:51 PM · Badges (v0)
chad changed the visibility for T2378: Document how to build `javelinsymbols` and allow its location to be specified rather than read from $PATH.
Feb 24 2017, 9:47 PM · Javelin
chad added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Feb 24 2017, 8:57 PM · Badges (v0)
20after4 added a comment to T12296: Improve Phacility repository import performance.

The commit cache should be a fairly large performance win.

Feb 24 2017, 7:22 PM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12298: Allow daemon pools to autoscale down to 0 processes.

Hibernating daemons currently show as "Waiting" in the Daemon console, but I'm not going to worry about that for now.

Feb 24 2017, 6:49 PM · Daemons, Ops, Phacility
epriestley added a comment to T12296: Improve Phacility repository import performance.

Ah, okay. This is the relevant part:

Feb 24 2017, 6:46 PM · Customer Impact, Diffusion, Ops
Mnkras added a comment to T12296: Improve Phacility repository import performance.

@epriestley
https://test-3uoifmrptebb.phacility.com/R2:80233a27d7b72bb03710d358e12a531ce46808a9

Feb 24 2017, 6:25 PM · Customer Impact, Diffusion, Ops
epriestley added a comment to E1394: Dentist.

good luck w/ teeth 😁

Feb 24 2017, 6:07 PM
epriestley added a revision to T12296: Improve Phacility repository import performance: Unknown Object (Differential Revision).
Feb 24 2017, 6:05 PM · Customer Impact, Diffusion, Ops
epriestley added a comment to T12296: Improve Phacility repository import performance.

Starting with repository update:

Feb 24 2017, 5:56 PM · Customer Impact, Diffusion, Ops
chad added a comment to E1394: Dentist.

so excite. much radiation.

Feb 24 2017, 5:54 PM
chad created E1394: Dentist.
Feb 24 2017, 5:44 PM
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Feb 24 2017, 5:14 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Feb 24 2017, 5:06 PM · Daemons, Ops, Phacility
allixsenos updated the task description for T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
Feb 24 2017, 2:41 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Broadly, there is nothing Phabricator could have done differently to anticipate or prevent this issue.

Feb 24 2017, 2:31 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

We could also provide some kind of bin/auth burn-everything-to-the-ground to automate these steps, but I worry that it would almost never be tested (I think it's very difficult to test comprehensively in an automated way) or run, and there's a good chance it might be forgotten about when authentication changes occur.

Feb 24 2017, 2:24 PM · Guides, Security
epriestley updated the task description for T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
Feb 24 2017, 2:12 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Ah, you're right. It looks like request data may also have leaked. I'll edit my earlier posts to reflect that. Briefly, the change is: passwords and VCS passwords are potentially at risk.

Feb 24 2017, 2:12 PM · Guides, Security
allixsenos added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Has cloudflare explicitly said that only responses were leaked, and never
requests? I recall "POST data" being explicitly called out as showing up in
the leaks, but I can't remember if this was in an official write up or
media coverage.

Feb 24 2017, 1:43 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").
NOTE: If Aphlict is served from the same domain as Phabricator (e.g., just on a different port) with Cloudflare as an intermediary, none of the rest of this comment applies: browsers could have sent credentials with the request and Cloudflare could have cached them.
Feb 24 2017, 12:50 PM · Guides, Security
epriestley added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

To cycle passwords, you can do this:

Feb 24 2017, 12:43 PM · Guides, Security
allixsenos added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

Only our websocket host is behind CF proxies, so I invalidated all auth sessions by changing the security.hmac-key. I wanted to also invalidate all Conduit tokens just as a fire drill but couldn't found a method of doing this.

Feb 24 2017, 10:56 AM · Guides, Security
20after4 added a comment to T12296: Improve Phacility repository import performance.

My impression is that there is a lot of room for optimization in the RepositoryPullLocalDaemon. I've wanted to tackle that for a while because it causes a fair delay between when a commit goes into git and when it actually shows up in Diffusion.

Feb 24 2017, 10:45 AM · Customer Impact, Diffusion, Ops
voltcode added a comment to T12310: Phd out of memory error.

I found that svn repo import was stuck on a very large branch import. I marked the repo as imported, suddenly the herald rules and notifications started to work properly (which was my other problem that I wanted to solve with my instance).

Feb 24 2017, 8:17 AM · Bug Report (Needs Information)
celia added a comment to T4382: Pholio inline comment "not submitted yet" message: add title to explain the message.

Hello I have a question that:
if there are several users written in comments in Pholio Mocks part, how can I see other person's comments?

Feb 24 2017, 6:48 AM · Pholio
celia added a project to T4382: Pholio inline comment "not submitted yet" message: add title to explain the message: Pholio.
Feb 24 2017, 6:46 AM · Pholio
celia removed a project from T4382: Pholio inline comment "not submitted yet" message: add title to explain the message: Pholio.
Feb 24 2017, 6:46 AM · Pholio
revi added a comment to T12313: Cloudflare leaked all HTTPS traffic on the internet into public caches ("Cloudbleed").

If you have configured your entire install behind Cloudflare (we do not encourage this in the documentation), you are at greater risk, as anything served by Phabricator could have been leaked. If you believe you are affected and would like help rotating sessions and invalidating credentials, let us know.

Feb 24 2017, 6:03 AM · Guides, Security
chad added a comment to T7190: Project templates/Copying tasks.

Events stuff isn't on any roadmap, so it also could be years before it was modified.

Feb 24 2017, 4:41 AM · Projects, Maniphest