Page MenuHomePhabricator
Feed All Stories

Jul 18 2017

epriestley added a revision to T10769: Read-Only Mode Errata: Unknown Object (Differential Revision).
Jul 18 2017, 3:51 PM · Clusters
epriestley closed T12916: Email Settings and Email Preferences should maybe be the same panel as Wontfix.

Let's just take another look at this during/after T10448? I think that's happening soon enough and this is minor enough that it's probably not worth fixing separately.

Jul 18 2017, 3:20 PM
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 18 2017, 2:51 PM · Phacility, Support
remusvrm updated subscribers of T12103: No active Drydock blueprint exists which can ever allocate a resource for lease, when there is one available.

Thanks @pooyajavan for your update. But in your solution, the Blueprint is cloning the repository, which I don't want to do that just for running a simple cmd.

Jul 18 2017, 2:21 PM · Bug Report
pouyana added a comment to T12103: No active Drydock blueprint exists which can ever allocate a resource for lease, when there is one available.

Hi I have found the solution, added in stackoverflow, forgot to post it here again.

Jul 18 2017, 1:54 PM · Bug Report
remusvrm added a comment to T12103: No active Drydock blueprint exists which can ever allocate a resource for lease, when there is one available.

I have exactly the same problem. Any update regarding this issue?

Jul 18 2017, 1:35 PM · Bug Report
epriestley added projects to T12927: Private Clusters: VPN Notes: Ops, Clusters.
Jul 18 2017, 11:50 AM · Clusters, Ops
epriestley created T12927: Private Clusters: VPN Notes.
Jul 18 2017, 11:49 AM · Clusters, Ops
epriestley created T12926: Document the existence of the LB `/status/` endpoint.
Jul 18 2017, 9:41 AM · Infrastructure, Setup, Documentation
epriestley updated the task description for T10769: Read-Only Mode Errata.
Jul 18 2017, 9:41 AM · Clusters
chad closed T12922: Update Bug Report docs as Resolved by committing Restricted Diffusion Commit.
Jul 18 2017, 2:21 AM · Public & Media Relations
20after4 added a comment to T12819: InnoDB FULLTEXT appears to fail catastrophically once it reaches a moderate size.

For phabricator.wikimedia.org, we decided to go with elasticsearch, partially because we already have a massive elasticsearch cluster and a lot of institutional elasticsearch knowledge / experience. My opinion is that we made the right choice. I believe this opinion is shared by most of the folks who use our phabricator on a daily basis. I've seen zero complaints about search since we made the switch, which is a huge improvement from what I saw with mysql FTS. Conclusion: Elasticsearch seems to perform well and the results are generally better (obviously this is subjective, but like I said, no complaints from users)

Jul 18 2017, 1:55 AM · Restricted Project, Search
chad added a project to T12922: Update Bug Report docs: Public & Media Relations.
Jul 18 2017, 12:51 AM · Public & Media Relations

Jul 17 2017

avivey added a revision to T12924: Applications -> Search -> Configure is fataling: Unknown Object (Differential Revision).
Jul 17 2017, 11:43 PM · Bug Report
nickhutchinson added a comment to T11364: arc confused by HTTP->HTTPS redirects.

My install hit this today: we're trying to upgrade from http -> https, but don't want to break users with the old http:// address in their .arcconfig (either because they haven't pulled master recently, are working on a different branch, are bisecting to a commit prior to the .arcconfig being updated, etc).

Jul 17 2017, 11:40 PM · Arcanist, Bug Report
chad merged T12925: Warn more loudly when diffusion setups are broken? into T12896: It isn’t obvious when repository observation breaks.
Jul 17 2017, 10:49 PM · Diffusion, Feature Request
chad merged task T12925: Warn more loudly when diffusion setups are broken? into T12896: It isn’t obvious when repository observation breaks.
Jul 17 2017, 10:49 PM · Diffusion
csilvers created T12925: Warn more loudly when diffusion setups are broken?.
Jul 17 2017, 10:32 PM · Diffusion
chad added a revision to T12922: Update Bug Report docs: Unknown Object (Differential Revision).
Jul 17 2017, 9:47 PM · Public & Media Relations
chad added a comment to T12923: Settings Forms, EditEngine instructions are visually confusing.

image.png (892×978 px, 823 KB)

Jul 17 2017, 9:10 PM · Design
remusvrm added a comment to Q637: Best practice to react on commits (Answer 560).

thanks. I will ask there.

Jul 17 2017, 9:09 PM
chad added a comment to Q637: Best practice to react on commits (Answer 560).

The link is fixed.

Jul 17 2017, 9:09 PM
chad updated the answer details for Q637: Best practice to react on commits (Answer 560).
Jul 17 2017, 9:09 PM
remusvrm added a comment to Q637: Best practice to react on commits (Answer 560).

Screen Shot 2017-07-17 at 23.07.30.png (508×1 px, 47 KB)

Jul 17 2017, 9:08 PM
chad updated the answer details for Q637: Best practice to react on commits (Answer 560).
Jul 17 2017, 9:02 PM
chad added Q637: Best practice to react on commits (Answer 560).
Jul 17 2017, 9:02 PM
remusvrm asked Q637: Best practice to react on commits.
Jul 17 2017, 9:00 PM
avivey created T12924: Applications -> Search -> Configure is fataling.
Jul 17 2017, 8:47 PM · Bug Report
epriestley edited the content of Changelog.
Jul 17 2017, 2:26 PM
epriestley edited the content of 2017 Week 28 (Mid July).
Jul 17 2017, 2:24 PM
epriestley edited the content of 2017 Week 29 (Late July).
Jul 17 2017, 2:24 PM
epriestley created 2017 Week 29 (Late July).
Jul 17 2017, 2:23 PM
epriestley created 2017 Week 28 (Mid July).
Jul 17 2017, 2:21 PM
epriestley added a comment to T12904: Support Pact Errata.

Inbound mail seems to be working now.

Jul 17 2017, 2:19 PM · Phacility, Support
epriestley added a comment to T12854: Clean up resources for decommissioned `repo012`.

This is very minor, but blue-underneathie is still active even though repo012 is dead, which is causing the queue to spawn impossible backup jobs that sit there retrying forever. They should fix themselves once the instance is suspended. I'll likely pursue the remaining cleanup here shortly since we haven't seen any issues in a few weeks.

Jul 17 2017, 2:10 PM · Ops, Phacility
epriestley added a comment to T12904: Support Pact Errata.

Inbound mail later bounced. I think the issue is that we have a *.phacility.com MX record, but an admin.phacility.com A record. I can't recite all the wildcard vs nonwildcard DNS rules offhand, but I think the A record is shadowing the MX record, e.g. dig mx admin.phacility.com does not produce the right results. I've added an admin.phacilty.com MX record which might sort things out once it propagates.

Jul 17 2017, 2:07 PM · Phacility, Support
avivey created T12923: Settings Forms, EditEngine instructions are visually confusing.
Jul 17 2017, 12:25 AM · Design

Jul 16 2017

• tomekj2ee awarded T2849: Implement alternate markup engines (Markdown, reStructuredText, ...) a Dislike token.
Jul 16 2017, 8:49 PM · Remarkup
MrSchaal awarded T3569: Make Phrequent somewhat usable a Like token.
Jul 16 2017, 7:22 PM · Phrequent
siepkes backed I2 Donate to Phacility with $50.00 USD.
Jul 16 2017, 2:04 PM · Phacility
epriestley closed T12134: Develop a Nuance-based Phabricator reporting/support flow as Resolved.

The new paid support application is now in closed beta. This will become publicly available in the relatively near future, then become then channel for SAAS support after that. It may later become the channel for some types of free/community support (most likely bug reports, per above) but this probably won't happen for a while.

Jul 16 2017, 11:23 AM · Abuse
epriestley added a comment to T12922: Update Bug Report docs.

Community and us can both use the normal task form to report bugs -- the "bug report" form just had extra instructions.

Jul 16 2017, 11:11 AM · Public & Media Relations

Jul 15 2017

chad added a comment to T12922: Update Bug Report docs.

OK I made a basic bug form for Discourse. Will update docs later today.

Jul 15 2017, 9:41 PM · Public & Media Relations
michaeljs1990 awarded T12681: Upcoming Support Product / End of Paid Prioritization a Yellow Medal token.
Jul 15 2017, 8:58 PM · Guides, Phacility
chad added a comment to T12922: Update Bug Report docs.

I had assumed the forms here would still be open for Community, at least bug reporting or us.

Jul 15 2017, 6:35 PM · Public & Media Relations
epriestley added a comment to T12922: Update Bug Report docs.

That is, the flow would be one of these:

Jul 15 2017, 5:59 PM · Public & Media Relations
epriestley added a comment to T12922: Update Bug Report docs.

Let's just do Discourse for now? I don't plan to get anything up on admin for at least a while.

Jul 15 2017, 5:53 PM · Public & Media Relations
chad created T12922: Update Bug Report docs.
Jul 15 2017, 5:50 PM · Public & Media Relations
xiaogaozi awarded T10038: Plan the mid-term pathway for unit-test/linter bindings a Love token.
Jul 15 2017, 10:29 AM · Lint, Arcanist

Jul 14 2017

amckinley created P2066 output.txt.
Jul 14 2017, 7:59 PM
amckinley created P2065 subnetter.py.
Jul 14 2017, 7:58 PM
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 14 2017, 7:04 PM · Phacility, Support
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 14 2017, 6:59 PM · Phacility, Support
epriestley added a comment to T12904: Support Pact Errata.

Using @mentions in issue comments hits a policy exception because of the slightly odd way pact policy filtering works.

Jul 14 2017, 6:40 PM · Phacility, Support
epriestley added a comment to T12904: Support Pact Errata.

This works now: PHI1

Jul 14 2017, 4:20 PM · Phacility, Support
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 14 2017, 4:07 PM · Phacility, Support
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 14 2017, 3:52 PM · Phacility, Support
epriestley added a comment to T12904: Support Pact Errata.

Test mail from the console now gets routed/accepted properly but real mail still doesn't. I'm going to write this off as possibly being a route propagation issue for a little bit and see if it fixes itself.

Jul 14 2017, 3:47 PM · Phacility, Support
epriestley added a comment to T12904: Support Pact Errata.

Email replies don't actually work since mail gets routed to the web tier (mail.phacililty.com) which can't hit admin. I think I can fix this with a mail route.

Jul 14 2017, 3:37 PM · Phacility, Support
epriestley added a comment to T12904: Support Pact Errata.

Some stuff I'm thinking of doing in the short term:

Jul 14 2017, 3:25 PM · Phacility, Support
epriestley closed T12681: Upcoming Support Product / End of Paid Prioritization as Resolved.

This is entering some kind of closed beta shortly, I'll follow up in T12904 for actual feature improvements to the Support application.

Jul 14 2017, 3:14 PM · Guides, Phacility
epriestley added a revision to T12904: Support Pact Errata: Unknown Object (Differential Revision).
Jul 14 2017, 3:13 PM · Phacility, Support
avivey closed Q636: GUI editor for Phriction as resolved.
Jul 14 2017, 3:03 PM
ivo created T12921: Link to referenced object in transaction emails.
Jul 14 2017, 1:46 PM · Mail
timor awarded T3001: Add footnote syntax to Remarkup a Love token.
Jul 14 2017, 12:35 PM · Remarkup
remusvrm added a comment to Q636: GUI editor for Phriction (Answer 558).

Thanks @avivey for your info.

Jul 14 2017, 10:32 AM
ideasman42 awarded T4213: Provide custom themes, skinning, or custom CSS a Love token.
Jul 14 2017, 10:16 AM · Design
jasonrumney added a comment to T12804: Diffusion pre-redesign UI feedback (June 2017).

Since the History is now dumbed down and separated from the Technical details on the Graph view, it might be more useful to filter out merge commits from the History view. Perhaps they can be replaced by a small "Merged to <branch>" link at the bottom of the box for the last non-merge commit that was part of the merge.

Jul 14 2017, 6:11 AM · Diffusion, Design
epriestley added a comment to T12847: A Pathway Towards Private Clusters.

Ah, that makes sense. Everything else sounds entirely reasonable to me on the "VPC per customer" plan.

Jul 14 2017, 12:42 AM · Plans, Ops, Phacility
amckinley added a comment to T12847: A Pathway Towards Private Clusters.

bastion host per subnet

Is "subnet" a typo or am I thinking of the wrong subnets? Not 4 bastions per customer, right? Don't the bastions have to be in the customer VPCs?

Jul 14 2017, 12:40 AM · Plans, Ops, Phacility
epriestley added a comment to T12847: A Pathway Towards Private Clusters.

bastion host per subnet

Jul 14 2017, 12:33 AM · Plans, Ops, Phacility
amckinley added a comment to T12847: A Pathway Towards Private Clusters.

So having done a bunch more reading/thinking about private cluster isolation, I'm starting to lean towards "one VPC per customer per region" instead of "one subnet per customer per region".

Jul 14 2017, 12:18 AM · Plans, Ops, Phacility

Jul 13 2017

epriestley added a revision to T12681: Upcoming Support Product / End of Paid Prioritization: Unknown Object (Differential Revision).
Jul 13 2017, 10:29 PM · Guides, Phacility
avivey added Q636: GUI editor for Phriction (Answer 558).
Jul 13 2017, 10:19 PM
epriestley added a comment to T12920: Retracting requested changes can leave revision in "Needs Revision" state.

I think we end up more consistent overall if "Resign" and "Accept" have the same behavior in this case.

Jul 13 2017, 8:37 PM · Restricted Project, Differential, Bug Report
jboning added a comment to T12920: Retracting requested changes can leave revision in "Needs Revision" state.

That sounds less surprising to me, in the sense that it's not an explicit reversal of having requested changes. I can imagine arguments in both directions for the resign case.

Jul 13 2017, 8:32 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T12920: Retracting requested changes can leave revision in "Needs Revision" state.

From reading the code, resigning (instead of accepting) probably also leaves the revision in "Needs Revision".

Jul 13 2017, 8:29 PM · Restricted Project, Differential, Bug Report
amckinley created Image Macro "howneatisthat".
Jul 13 2017, 7:34 PM
20after4 added a comment to T12896: It isn’t obvious when repository observation breaks.

We"solved" the problem with a small python daemon that pings repository_statusmessage.epoch every minute and calculates the delta. When these deltas surpass Phabricator's max delta (21,600 sec) we page the oncall. For repositories that are mission critical, we use a smaller value. It obviously isn't a perfect solution, but it gets the job done and it's been extremely stable.

Jul 13 2017, 7:34 PM · Diffusion, Feature Request
20after4 added a comment to T12909: Calendar ignores email preferences if a user is invited via a project.

I've seen complaints about this on phabricator.wikimedia.org as well, so I can confirm that this is an issue.

Jul 13 2017, 7:29 PM · Calendar, Bug Report
chad added a comment to T12920: Retracting requested changes can leave revision in "Needs Revision" state.

O.O

Jul 13 2017, 7:20 PM · Restricted Project, Differential, Bug Report
jboning created T12920: Retracting requested changes can leave revision in "Needs Revision" state.
Jul 13 2017, 6:57 PM · Restricted Project, Differential, Bug Report
amckinley added a comment to T12847: A Pathway Towards Private Clusters.

AWS doesn't have a single device which can both listen for TCP on 22 (only ELB) and terminate SSL for websockets (only ALB).

That's not correct. We are routing our websocket traffic for Aphlict through an ELB that has been setup using SSL instead of TCP as the protocol. In Terraform, it looks like this:

Jul 13 2017, 6:51 PM · Plans, Ops, Phacility
epriestley added a comment to T12847: A Pathway Towards Private Clusters.

It looks like the price difference is pretty large -- $45/month for a m4.large reserved instance vs $80/month for a m4.large dedicated -- plus ~$1,500/month/region for having at least one dedicated instance. It would be easy to start using dedicated hosts in the future, right? Just an issue of converting existing hosts if we run into issues?

Jul 13 2017, 5:27 PM · Plans, Ops, Phacility
amckinley added a comment to T12847: A Pathway Towards Private Clusters.

We should also make a decision about whether or not we want to use "dedicated" AWS instances: https://aws.amazon.com/ec2/purchasing-options/dedicated-instances/

Jul 13 2017, 4:31 PM · Plans, Ops, Phacility
remusvrm asked Q636: GUI editor for Phriction.
Jul 13 2017, 1:44 PM
arielyang edited the content of Community Resources.
Jul 13 2017, 2:49 AM

Jul 12 2017

joshuaspence added a comment to T12856: Evaluate various "infrastructure-as-code" products.

Another correction, Puppet doesn't //need/ a dedicated server. You can run Puppet in a standalone (agent) mode, if you are willing to somehow get your Puppet manifests onto each host that you wish to provision.

Jul 12 2017, 10:27 PM · Ops, Phacility
amckinley added a comment to T12856: Evaluate various "infrastructure-as-code" products.

Terraform Review
Pros:

  • Written in Python, which everyone knows is superior to Ruby in every way

Huh? Terraform is written in Golang, not Python.

Jul 12 2017, 10:24 PM · Ops, Phacility
joshuaspence added a comment to T12856: Evaluate various "infrastructure-as-code" products.

Terraform Review
Pros:

  • Written in Python, which everyone knows is superior to Ruby in every way
Jul 12 2017, 10:22 PM · Ops, Phacility
joshuaspence created T12919: Healthcheck endpoint for alternate file domain.
Jul 12 2017, 9:58 PM · Files
epriestley moved T7789: Support Git Large File Storage from Backlog to Future on the Customer Impact board.
Jul 12 2017, 9:57 PM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a project to T7789: Support Git Large File Storage: Customer Impact.

A customer is hitting an issue where pushes to staging areas do not push LFS objects. I'd guess this is an LFS upstream problem based on a hazy recollection of events here.

Jul 12 2017, 9:56 PM · Customer Impact, Restricted Project, Files, Diffusion
joshuaspence edited the content of Community Resources.
Jul 12 2017, 9:55 PM
joshuaspence added a comment to T12856: Evaluate various "infrastructure-as-code" products.

FWIW, I am actively working on a Puppet module for Phabricator, see https://forge.puppet.com/joshuaspence/phabricator.

Jul 12 2017, 9:54 PM · Ops, Phacility
joshuaspence added a comment to T12847: A Pathway Towards Private Clusters.

AWS doesn't have a single device which can both listen for TCP on 22 (only ELB) and terminate SSL for websockets (only ALB).

Jul 12 2017, 9:52 PM · Plans, Ops, Phacility
epriestley added a revision to T11989: Wage an endless war against HP Printer Fax Support: Unknown Object (Differential Revision).
Jul 12 2017, 8:10 PM · Abuse
epriestley added a revision to T12681: Upcoming Support Product / End of Paid Prioritization: Unknown Object (Differential Revision).
Jul 12 2017, 7:10 PM · Guides, Phacility
avivey added a comment to T12891: Phacility does not currently offer instances in the Sydney region.

Also, have you tried looking around the git community? I was always under the impression that SSH should be faster than HTTPS cloning, or possibly using the same protocol.

Jul 12 2017, 7:09 PM · Feature Request
chad created U38 changelog-signup.
Jul 12 2017, 6:29 PM