Page MenuHomePhabricator
Feed Advanced Search

Jul 13 2017

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
chad created U37 Discourse.
Jul 12 2017, 6:21 PM
chad added a comment to T12792: Consider an option to disable all popup notifications.

What if they're on the page, is that notification annoying? I love click that little guy.

Jul 12 2017, 6:13 PM · Feature Request, Customer Impact, Notifications
jmeador added a comment to T12792: Consider an option to disable all popup notifications.

+1, we've also had a handful of users request this as a per-user preference.

Jul 12 2017, 6:11 PM · Feature Request, Customer Impact, Notifications
chad added a comment to T12916: Email Settings and Email Preferences should maybe be the same panel.

Ahhh, ok, these were all in one place. Yeah it's hard to find. I wanted to turn off self-actions, but "Preference" link sends me to a place that doesn't have that link so it seemed a point of user confusion.

Jul 12 2017, 5:48 PM
chad added a comment to T12916: Email Settings and Email Preferences should maybe be the same panel.

Erm, sorry Email Delivery.

Jul 12 2017, 5:45 PM
epriestley created T12917: Move domain name registration and SSL to AWS.
Jul 12 2017, 5:25 PM · Ops, Phacility
epriestley added a comment to T12916: Email Settings and Email Preferences should maybe be the same panel.

I don't see an "Email Settings" panel, am I missing it or did you mean a different panel?

Jul 12 2017, 5:15 PM
chad created T12916: Email Settings and Email Preferences should maybe be the same panel.
Jul 12 2017, 4:48 PM
chad closed T12915: SVN repositories give `Unhandled Exception ("RuntimeException")` in Diffusion as Resolved by committing Restricted Diffusion Commit.
Jul 12 2017, 2:05 PM · Subversion, Diffusion, Bug Report
epriestley added a revision to T12681: Upcoming Support Product / End of Paid Prioritization: Unknown Object (Differential Revision).
Jul 12 2017, 1:57 PM · Guides, Phacility
saga.karlsson awarded T12588: create subtask side menu to allow creation of subtasks of different types to the parent task a Love token.
Jul 12 2017, 1:30 PM · Feature Request
saga.karlsson added a comment to T12588: create subtask side menu to allow creation of subtasks of different types to the parent task.

Or just have a subtask-array configurable, going with the inline approach mentioned above:

Jul 12 2017, 1:29 PM · Feature Request
saga.karlsson added a comment to T12588: create subtask side menu to allow creation of subtasks of different types to the parent task.

Also, it would be great if you can choose an optional "default subtask subtype" where you configure the subtypes. Something like this:

Jul 12 2017, 1:26 PM · Feature Request
keithzg added a comment to T12915: SVN repositories give `Unhandled Exception ("RuntimeException")` in Diffusion.

Solves things for my instances, cheers!

Jul 12 2017, 9:25 AM · Subversion, Diffusion, Bug Report
saga.karlsson added a comment to T4207: Implement (tree) subtasks, as distinct from (graph) task dependencies.

Does this also include the functionality to specify what subtype to be used when you click "Create subtask", or is this just the visual element?

Jul 12 2017, 7:38 AM · Maniphest
chad added a revision to T12915: SVN repositories give `Unhandled Exception ("RuntimeException")` in Diffusion: Unknown Object (Differential Revision).
Jul 12 2017, 3:38 AM · Subversion, Diffusion, Bug Report
chad awarded Bug Meister to recipient: keithzg.
Jul 12 2017, 3:00 AM
chad reopened T12915: SVN repositories give `Unhandled Exception ("RuntimeException")` in Diffusion as "Open".

Going to keep this Open and build out some proper blank states. This feel pretty janky.

Jul 12 2017, 3:00 AM · Subversion, Diffusion, Bug Report
chad added a comment to T12915: SVN repositories give `Unhandled Exception ("RuntimeException")` in Diffusion.

I pushed a hotfix up at HEAD of master, let us know if you're still having problems. Thanks for the report!

Jul 12 2017, 2:52 AM · Subversion, Diffusion, Bug Report
chad closed T12915: SVN repositories give `Unhandled Exception ("RuntimeException")` in Diffusion as Resolved by committing Restricted Diffusion Commit.
Jul 12 2017, 2:51 AM · Subversion, Diffusion, Bug Report
chad added a revision to T12915: SVN repositories give `Unhandled Exception ("RuntimeException")` in Diffusion: Unknown Object (Differential Revision).
Jul 12 2017, 2:50 AM · Subversion, Diffusion, Bug Report
keithzg created T12915: SVN repositories give `Unhandled Exception ("RuntimeException")` in Diffusion.
Jul 12 2017, 1:31 AM · Subversion, Diffusion, Bug Report

Jul 11 2017

chad merged task T11152: Some workflows of arc are broken on Windows into T8291: arc version fails on Windows.
Jul 11 2017, 11:31 PM · Arcanist, Windows, Bug Report
chad merged T11152: Some workflows of arc are broken on Windows into T8291: arc version fails on Windows.
Jul 11 2017, 11:31 PM · Windows, Arcanist
chad merged task T12914: Arcanist in Windows: ERROR 8: Undefined offset into T8821: arc commands (diff, version) fails on Windows - terrible fix inside.
Jul 11 2017, 11:30 PM · Bug Report
chad merged T12914: Arcanist in Windows: ERROR 8: Undefined offset into T8821: arc commands (diff, version) fails on Windows - terrible fix inside.
Jul 11 2017, 11:30 PM · Windows, libphutil, Arcanist
chad merged task T12913: `arc version` throws exception into T8291: arc version fails on Windows.
Jul 11 2017, 11:29 PM · Bug Report
chad merged T12913: `arc version` throws exception into T8291: arc version fails on Windows.
Jul 11 2017, 11:29 PM · Windows, Arcanist
chad added a comment to T12913: `arc version` throws exception.

Please provide reproduction instructions. There aren't enough details here.

Jul 11 2017, 11:16 PM · Bug Report
dwilches created T12914: Arcanist in Windows: ERROR 8: Undefined offset.
Jul 11 2017, 11:07 PM · Bug Report
dwilches created T12913: `arc version` throws exception.
Jul 11 2017, 11:06 PM · Bug Report
Luke081515.2 added a comment to T12839: Can't create one day event.

Hm, this screenshot made me dig a bit into the whole thing, so here is what I've found:
First, my date format is different because of my settings:

Jul 11 2017, 9:10 PM · Calendar, Bug Report
chad added a comment to T12839: Can't create one day event.

Why does your date format look different than ours? Is that a local modification or do we support this somewhere I'm not aware of.

Jul 11 2017, 8:40 PM · Calendar, Bug Report
chad changed the start date for E1571: one-day-event from Jul 11 2017 to Jul 12 2017.
Jul 11 2017, 8:38 PM
chad added a comment to T12912: Can't create one day event (again).

Merging over so there is one thread on this topic. We're happy to reopen if you have new reproduction steps we can follow or if you can verify on a Phacility test instance.

Jul 11 2017, 8:37 PM · Calendar, Bug Report
chad merged task T12912: Can't create one day event (again) into T12839: Can't create one day event.
Jul 11 2017, 8:36 PM · Calendar, Bug Report
chad merged T12912: Can't create one day event (again) into T12839: Can't create one day event.
Jul 11 2017, 8:36 PM · Calendar, Bug Report
chad added a comment to T12912: Can't create one day event (again).

I can't reproduce this (again)

Jul 11 2017, 8:30 PM · Calendar, Bug Report
chad created E1571: one-day-event.
Jul 11 2017, 8:29 PM
Luke081515.2 added a comment to T12912: Can't create one day event (again).

Here's the proof: F5042439

Jul 11 2017, 8:24 PM · Calendar, Bug Report
Luke081515.2 created T12912: Can't create one day event (again).
Jul 11 2017, 8:22 PM · Calendar, Bug Report
epriestley added a revision to T12681: Upcoming Support Product / End of Paid Prioritization: Unknown Object (Differential Revision).
Jul 11 2017, 6:56 PM · Guides, Phacility
jmeador created Image Macro "make-it-rain".
Jul 11 2017, 6:42 PM
joshuaspence added a revision to T12910: Allow SES to be used for inbound mail: Unknown Object (Differential Revision).
Jul 11 2017, 1:53 PM · Mail
nemobis awarded T10745: Emails are sent out about restricted changes a Heartbreak token.
Jul 11 2017, 12:33 PM · Restricted Project, Bug Report
• vijaykumarcmeseo added a comment to F5041649: evilsvgfile.svg.

<script>alert(xss)</script>

Jul 11 2017, 7:30 AM
remusvrm added a comment to Q633: Is it possible to change the width of the wiki page? (Answer 554).

Thanks @chad for the doc.

Jul 11 2017, 7:01 AM
chad added a comment to T10745: Emails are sent out about restricted changes.

FWIW, I don't think these are "pure" spam emails, as in, that they are 100% unwanted by everyone. Even if you cannot see the underlying diff, you've asked to be notified when work happens essentially on a task. I think it's difficult to have Phabricator assess whether this email is informative or not to each individual user who subscribed. To some who are waiting for the product, it's valuable, to you or others perhaps not at all. Client side filtering still seems like the correct workaround until T5791 (which is server side email filtering).

Jul 11 2017, 5:40 AM · Restricted Project, Bug Report
chad added a comment to T10745: Emails are sent out about restricted changes.

Hmm, what did you try for search, off hand, do you recall?

Jul 11 2017, 5:27 AM · Restricted Project, Bug Report
MZMcBride added a comment to T10745: Emails are sent out about restricted changes.
In T10745#229028, @chad wrote:

I'm not sure I follow what you're trying to tell us.

Jul 11 2017, 5:23 AM · Restricted Project, Bug Report
chad added a comment to T10745: Emails are sent out about restricted changes.

I'm not sure I follow what you're trying to tell us.

Jul 11 2017, 4:33 AM · Restricted Project, Bug Report
MZMcBride added a comment to T10745: Emails are sent out about restricted changes.

These types of e-mails are still being sent. Here's one from July 10, 2017:

Jul 11 2017, 4:01 AM · Restricted Project, Bug Report

Jul 10 2017

epriestley added a revision to T12681: Upcoming Support Product / End of Paid Prioritization: Unknown Object (Differential Revision).
Jul 10 2017, 11:53 PM · Guides, Phacility
epriestley added a comment to T12911: Better handling for users subscribed to objects they cannot see.

T4411 discusses some similar issues, although it predates Spaces and focuses on subscribers rather than assignees.

Jul 10 2017, 10:59 PM · Spaces, Feature Request
• sshannin created T12911: Better handling for users subscribed to objects they cannot see.
Jul 10 2017, 10:52 PM · Spaces, Feature Request
epriestley added a revision to T12681: Upcoming Support Product / End of Paid Prioritization: Unknown Object (Differential Revision).
Jul 10 2017, 10:10 PM · Guides, Phacility
cspeckmim added a comment to Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 555).

You configure this per-project in the .arcconfig to set

{
...
"history.immutable" : "false"
...
}
Jul 10 2017, 9:30 PM
cul8er updated the answer wiki for Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos.".
Jul 10 2017, 9:02 PM · Mercurial, Arcanist
cul8er added a comment to Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 557).

I read arcanist's source (src/workflow/ArcanistWorkflow.php)

Jul 10 2017, 9:01 PM
cul8er added a comment to Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 555).

I made it! great.

Jul 10 2017, 8:56 PM
cul8er added a comment to Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 557).

I also checked these places according to man hgrc:

Jul 10 2017, 8:31 PM
cul8er added a comment to Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 557).

I made sure that there is no aliases in ~/.bashrc and ~/.profile. And there is no ~/.bash_aliases

Jul 10 2017, 8:16 PM
cul8er added a comment to Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 557).

No... at least, as far as I can make sure.

Jul 10 2017, 8:10 PM
epriestley added a revision to T12681: Upcoming Support Product / End of Paid Prioritization: Unknown Object (Differential Revision).
Jul 10 2017, 7:42 PM · Guides, Phacility
cspeckmim added a comment to Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 557).

It doesn't even look like arcanist is using --merge argument -- the last command it tries before failing is hg checkout 'default'. Do you have an alias setup in .hgrc or something similar?

Jul 10 2017, 7:41 PM
epriestley updated the task description for T12904: Support Pact Errata.
Jul 10 2017, 7:32 PM · Phacility, Support
cul8er added a comment to Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 557).

mercurial is 4.2.
I updated the question, but paste the log here too.

Jul 10 2017, 7:21 PM
cul8er updated the question details for Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos.".
Jul 10 2017, 7:15 PM · Mercurial, Arcanist
chad added a comment to M1477: Diffusion - June 2017.

Gonna try the button and see if it sucks.

Jul 10 2017, 4:27 PM
chad added a comment to M1477: Diffusion - June 2017.

@epriestley I think "Clone" via a button -> dialog is like 20 minutes of work for me, and the dropdown 2-3 days of fighting javascript. Unsure which you prefer in the long run.

Jul 10 2017, 3:40 PM
joshuaspence created T12910: Allow SES to be used for inbound mail.
Jul 10 2017, 12:20 PM · Mail
ivo created T12909: Calendar ignores email preferences if a user is invited via a project.
Jul 10 2017, 9:45 AM · Calendar, Bug Report
joshuaspence updated the task description for T12908: Allow verbosity of Aphlict log to be reduced.
Jul 10 2017, 6:31 AM · Aphlict, Feature Request
joshuaspence created T12908: Allow verbosity of Aphlict log to be reduced.
Jul 10 2017, 6:31 AM · Aphlict, Feature Request
chad marked Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 556) as hidden.
Jul 10 2017, 4:08 AM
cspeckmim updated the answer details for Q635: arc land fails with error "Usage Exception: --merge is not currently supported for hg repos." (Answer 557).
Jul 10 2017, 4:03 AM