Page MenuHomePhabricator
Feed Advanced Search

Sep 13 2015

epriestley added a revision to T7785: Build a Phacility cluster service tier to sandbox Graphviz and Cowsay: D14099: Implement Cowsay in PHP.
Sep 13 2015, 4:23 PM · Phacility, Phacility Support
ivan-1986 closed Restricted Maniphest Task, a subtask of T9319: Accessing image file in Diffusion must not generate error (unhandled exception), as Spite.
Sep 13 2015, 10:00 AM · Phacility, Diffusion, Phacility Support

Sep 4 2015

oliver.fisher added a comment to T9342: Make it easier to discover application policy configuration.

Ah ha! Yes, I totally missed those little gear off to the right. Thanks very much!

Sep 4 2015, 1:36 PM · Policy, Applications
epriestley added a comment to T9342: Make it easier to discover application policy configuration.

You should be able to configure this in:

Sep 4 2015, 1:31 PM · Policy, Applications
oliver.fisher created T9342: Make it easier to discover application policy configuration.
Sep 4 2015, 1:14 PM · Policy, Applications

Sep 2 2015

chad moved T9322: Avoid spawning setup errors when changing parameters in application "Config" from Backlog to Onboarding/NUX on the Phacility board.
Sep 2 2015, 5:17 PM · Phacility Support, Phacility
chad added a project to T9322: Avoid spawning setup errors when changing parameters in application "Config": Phacility Support.
Sep 2 2015, 5:17 PM · Phacility Support, Phacility
epriestley added a parent task for T9319: Accessing image file in Diffusion must not generate error (unhandled exception): T9303: Improve Phacility Onboarding/NUX.
Sep 2 2015, 2:31 PM · Phacility, Diffusion, Phacility Support
epriestley moved T9319: Accessing image file in Diffusion must not generate error (unhandled exception) from Backlog to Onboarding/NUX on the Phacility board.
Sep 2 2015, 2:31 PM · Phacility, Diffusion, Phacility Support
epriestley added a project to T9319: Accessing image file in Diffusion must not generate error (unhandled exception): Phacility.
Sep 2 2015, 2:30 PM · Phacility, Diffusion, Phacility Support
epriestley triaged T9319: Accessing image file in Diffusion must not generate error (unhandled exception) as Normal priority.
Sep 2 2015, 2:30 PM · Phacility, Diffusion, Phacility Support

Sep 1 2015

epriestley removed a subtask for T9299: Send code reviews to non-Phabricator users: T1205: Allow grey users in some form or other.
Sep 1 2015, 3:20 PM · Phacility, People, Differential, Phacility Support
epriestley added a subtask for T9299: Send code reviews to non-Phabricator users: T9304: Workflow for inviting new users to a Phacility instance is unnecessarily confusing.
Sep 1 2015, 3:20 PM · Phacility, People, Differential, Phacility Support
epriestley renamed T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails from Allow arbitrary email addresses to be CC'd on billing emails to Allow arbitrary email addresses to be CC'd on Phacility billing emails.
Sep 1 2015, 3:14 PM · Phortune, Phacility
epriestley renamed T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails from Invoice/receipt emails for billing to Allow arbitrary email addresses to be CC'd on billing emails.
Sep 1 2015, 3:14 PM · Phortune, Phacility
epriestley moved T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails from Backlog to Onboarding/NUX on the Phacility board.
Sep 1 2015, 3:14 PM · Phortune, Phacility
epriestley added a project to T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails: Phacility.
Sep 1 2015, 3:13 PM · Phortune, Phacility
epriestley added a parent task for T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails: T9303: Improve Phacility Onboarding/NUX.
Sep 1 2015, 3:13 PM · Phortune, Phacility
epriestley added a parent task for T9299: Send code reviews to non-Phabricator users: T9303: Improve Phacility Onboarding/NUX.
Sep 1 2015, 3:13 PM · Phacility, People, Differential, Phacility Support
epriestley moved T9299: Send code reviews to non-Phabricator users from Backlog to Onboarding/NUX on the Phacility board.
Sep 1 2015, 3:12 PM · Phacility, People, Differential, Phacility Support
epriestley added a project to T9299: Send code reviews to non-Phabricator users: Phacility.
Sep 1 2015, 3:12 PM · Phacility, People, Differential, Phacility Support
epriestley added a comment to T9299: Send code reviews to non-Phabricator users.

Ignoring the actual invite workflow for now, the major technical issue with supporting this is handling the eventual merging of accounts.

Sep 1 2015, 2:57 PM · Phacility, People, Differential, Phacility Support
oliver.fisher added a comment to T9299: Send code reviews to non-Phabricator users.

Yep, @epriestley has nailed the scenario exactly. I'd like use of Phabricator to spread organically (hopefully virally) through the org. That makes user commitment to it much stronger - and it gives me a measure of how enthusiastic people are about the tools and whether we should commit to it.

Sep 1 2015, 1:05 PM · Phacility, People, Differential, Phacility Support
epriestley added a comment to T9299: Send code reviews to non-Phabricator users.

You can just type 500 email addresses into this box (and, at least locally, it works fine, although admittedly I haven't done it in production personally):

Sep 1 2015, 3:10 AM · Phacility, People, Differential, Phacility Support
chad renamed T9299: Send code reviews to non-Phabricator users from Send code reviews to non-Phabricator users. to Send code reviews to non-Phabricator users.
Sep 1 2015, 3:01 AM · Phacility, People, Differential, Phacility Support
avivey added a comment to T9299: Send code reviews to non-Phabricator users.

I think that @oliver.fisher's use-case - this ticket and T9297 has more to do with initial on-boarding to Phacility/Phabricator, and bulk-creation of accounts.

Sep 1 2015, 2:56 AM · Phacility, People, Differential, Phacility Support
cspeckmim added a comment to T9299: Send code reviews to non-Phabricator users.

I'm not familiar with @oliver.fisher's case or the numerous situations that Phabricator would need to account for, but this problem sounded relate-able to ones we've come across in the past.

Sep 1 2015, 12:39 AM · Phacility, People, Differential, Phacility Support

Aug 31 2015

joshuaspence added projects to T9299: Send code reviews to non-Phabricator users: Differential, People.
Aug 31 2015, 9:28 PM · Phacility, People, Differential, Phacility Support
epriestley added a subtask for T9299: Send code reviews to non-Phabricator users: T1205: Allow grey users in some form or other.
Aug 31 2015, 8:28 PM · Phacility, People, Differential, Phacility Support
epriestley added a comment to T9299: Send code reviews to non-Phabricator users.

This is desirable, but a bit involved, particularly in the cluster.

Aug 31 2015, 8:28 PM · Phacility, People, Differential, Phacility Support
epriestley closed T9297: How do we get 500 devs invited to an instance (with or without invites)? as Resolved.

Awesome! I'll see if we can clean up that login-error-state stuff soonish. Let us know if you run into anything else.

Aug 31 2015, 7:58 PM · Phacility Support
oliver.fisher created T9299: Send code reviews to non-Phabricator users.
Aug 31 2015, 7:56 PM · Phacility, People, Differential, Phacility Support
oliver.fisher added a comment to T9297: How do we get 500 devs invited to an instance (with or without invites)?.

Thanks for the quick turnaround! Just went through it with a couple of people and seems to be working great.

Aug 31 2015, 7:53 PM · Phacility Support
epriestley added a comment to T9297: How do we get 500 devs invited to an instance (with or without invites)?.

This is now deployed to the cluster and I've updated the configuration for your instance. You should be able to see the domain on your instance page, and the configuration change in the transaction log in case I spelled it wrong or something silly like that.

Aug 31 2015, 7:18 PM · Phacility Support
epriestley added a revision to T9297: How do we get 500 devs invited to an instance (with or without invites)?: Restricted Differential Revision.
Aug 31 2015, 6:57 PM · Phacility Support
oliver.fisher added a comment to T9297: How do we get 500 devs invited to an instance (with or without invites)?.

That would be awesome! I've shown Phabricator (Differential specifically) to a few people and used it for some changes. They've basically gone crazy about how awesome it is (they've only known github PRs). I'd like to have them start spreading the word virally in the org without me. Something like a multi-use invite link would work well for that, although are certainly other approaches. Thanks!

Aug 31 2015, 6:02 PM · Phacility Support
epriestley added a comment to T9297: How do we get 500 devs invited to an instance (with or without invites)?.

There isn't really much of a workaround available at the moment, but I think we can get the feature deployed in some reasonable form by later today.

Aug 31 2015, 5:58 PM · Phacility Support
oliver.fisher added a comment to T9297: How do we get 500 devs invited to an instance (with or without invites)?.

Yep, that sounds pretty much perfect.

Aug 31 2015, 5:54 PM · Phacility Support
epriestley added a comment to T9297: How do we get 500 devs invited to an instance (with or without invites)?.

This isn't currently supported, but I think it's reasonable to implement. Behavior would roughly be something like this:

Aug 31 2015, 5:40 PM · Phacility Support
oliver.fisher created T9297: How do we get 500 devs invited to an instance (with or without invites)?.
Aug 31 2015, 5:34 PM · Phacility Support

Aug 24 2015

joshuaspence added projects to T8977: Unable to push to repo on Phacility due to pre-receive hook failing because of Herald exception: Phacility Support, Herald, Diffusion.
Aug 24 2015, 9:19 PM · Diffusion, Herald, Phacility Support
joshuaspence added a comment to T9003: Herald rule not working.

This task made for a good read.

Aug 24 2015, 9:05 PM · Herald, Phacility Support
joshuaspence added a project to T9003: Herald rule not working: Herald.
Aug 24 2015, 9:05 PM · Herald, Phacility Support

Aug 9 2015

epriestley added a comment to T8973: Encoding JSON for presentation could use more human-friendly escaping rules.

The specific UI (PhacilityInstances(Choose an Instance)Adjust Configuration) in question doesn't actually use PhutilJSON to format the output right now, so it's still rendering the less-preferred form:

Aug 9 2015, 4:25 AM · Customer Impact
chad added a comment to T8973: Encoding JSON for presentation could use more human-friendly escaping rules.

Is this complete?

Aug 9 2015, 4:11 AM · Customer Impact

Aug 5 2015

epriestley added a comment to T9080: Service-based repositories in Phacility cluster fail too abruptly if cloning fails.

Awesome! I'll keep this around as context for getting the root issue fixed. Thanks for the report, and let us know if you run into anything else.

Aug 5 2015, 4:15 PM · Diffusion, Phacility
epriestley added a comment to T9080: Service-based repositories in Phacility cluster fail too abruptly if cloning fails.

Another possible workaround would be to go find the credential in Passphrase and just update that, assuming the rest of the configuration is correct.

Aug 5 2015, 4:14 PM · Diffusion, Phacility
samplesizeofone added a comment to T9080: Service-based repositories in Phacility cluster fail too abruptly if cloning fails.

The edit link you provided let me fix my private key. I'm pretty sure I'm good to go now.

Aug 5 2015, 4:14 PM · Diffusion, Phacility
epriestley added a comment to T9080: Service-based repositories in Phacility cluster fail too abruptly if cloning fails.

Er, sorry, had the wrong URL in the previous comment. Should be correct now, I think.

Aug 5 2015, 4:12 PM · Diffusion, Phacility
epriestley added a comment to T9080: Service-based repositories in Phacility cluster fail too abruptly if cloning fails.

If you manually go to /diffusion/COLP/edit/ does that work?

Aug 5 2015, 4:12 PM · Diffusion, Phacility
samplesizeofone added a comment to T9080: Service-based repositories in Phacility cluster fail too abruptly if cloning fails.

I'm almost certain I put in the wrong SSH key.

Aug 5 2015, 4:10 PM · Diffusion, Phacility
epriestley added a comment to T9080: Service-based repositories in Phacility cluster fail too abruptly if cloning fails.

We should improve the error handling behavior here. Because this is a clustered repository, we're failing more abruptly than we would with a non-service repository, but we should make the behavior more similar.

Aug 5 2015, 4:04 PM · Diffusion, Phacility
samplesizeofone created T9080: Service-based repositories in Phacility cluster fail too abruptly if cloning fails.
Aug 5 2015, 4:01 PM · Diffusion, Phacility

Aug 4 2015

epriestley added a revision to T8973: Encoding JSON for presentation could use more human-friendly escaping rules: D13787: Add encodeAsList to PhutilJSON and make formatting prettier.
Aug 4 2015, 7:28 PM · Customer Impact

Aug 3 2015

epriestley added a comment to T9003: Herald rule not working.

Just to follow up on this, the transcript will show this at HEAD once T8726 lands:

Aug 3 2015, 8:42 PM · Herald, Phacility Support

Aug 1 2015

chad added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

Have a free badge.

Aug 1 2015, 1:58 AM · Phacility Support
bagashe added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

happyzebra

Aug 1 2015, 1:56 AM · Phacility Support
epriestley closed T9035: Import database from custom installation of phabricator into phacility's hosted service. as Resolved.

Looks like this is in good shape now.

Aug 1 2015, 1:40 AM · Phacility Support
epriestley added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

Ah, perfect. Yeah, either contact@ or support@ will work fine. Thanks!

Aug 1 2015, 12:16 AM · Phacility Support
epriestley added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

Cool, I'll get this importing. Do you want to shoot us an email at support@phacility.com to coordinate the rest of the import? There might be some private questions later on in the process about data specifics in case we run into any issues.

Aug 1 2015, 12:16 AM · Phacility Support
bagashe added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

I'll email contact@phacility.com with this info. Thanks again! :)

Aug 1 2015, 12:15 AM · Phacility Support
bagashe added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..
Aug 1 2015, 12:15 AM · Phacility Support
bagashe added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

Ah! Drag-n-drop worked like a charm.

Aug 1 2015, 12:14 AM · Phacility Support
chad added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

Are you dragging and dropping or using the upload button? The drag and drop should chunk the file.

Aug 1 2015, 12:12 AM · Phacility Support
bagashe added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

Looks like my phabricator_data.sql.gz file is bigger than the post_max_size setting.

Aug 1 2015, 12:08 AM · Phacility Support

Jul 31 2015

chad added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

Also https://admin.phacility.com/book/phacility/article/importing/ has a rough overview of the process and https://admin.phacility.com/book/phacility/article/limitations/ notes any limitations, I should have mentioned those first.

Jul 31 2015, 10:17 PM · Phacility Support
chad added a comment to T9035: Import database from custom installation of phabricator into phacility's hosted service..

Not a problem, go ahead and make an account on Phacility and whenever you're ready:

Jul 31 2015, 10:15 PM · Phacility Support
bagashe created T9035: Import database from custom installation of phabricator into phacility's hosted service..
Jul 31 2015, 9:39 PM · Phacility Support

Jul 29 2015

epriestley closed T9003: Herald rule not working as Resolved.

In any case, glad to hear this is working now! I think the UI should improve pretty soon (next week or two-ish) for this specific class of problem, at least. Let us know if you run into anything else.

Jul 29 2015, 11:41 PM · Herald, Phacility Support
epriestley added a comment to T9003: Herald rule not working.

Upcoming changes related to T5791 and T8726 should increase debug-ability, but giving web users access to mail is tricky while still respecting policies (e.g., administrators shouldn't be able to violate policies by just reading other users mail) and limiting the ability for an attacker to escalate access (e.g., an attacker who gets an admin account shouldn't have a blank check to access every other account by reading other users' "Welcome" or "Password Reset" mail).

Jul 29 2015, 11:39 PM · Herald, Phacility Support
chad added a comment to T9003: Herald rule not working.

I'm learning stuff here today too, not a waste at all.

Jul 29 2015, 11:38 PM · Herald, Phacility Support
oliverdain added a comment to T9003: Herald rule not working.

Never mind! Not only did my latest test task come through, all the previous ones did too. There was a few minutes delay but it seems to be working now. Thanks a ton for you help.

Jul 29 2015, 11:34 PM · Herald, Phacility Support
oliverdain added a comment to T9003: Herald rule not working.

Test above wasn't quite right. I have now added the user to the "Signifyd Employees" project as well:

Jul 29 2015, 11:30 PM · Herald, Phacility Support
oliverdain added a comment to T9003: Herald rule not working.

Suggestion makes sense, but does not seem to have worked. I added the user in question to the project:

Jul 29 2015, 11:26 PM · Herald, Phacility Support
epriestley added a comment to T9003: Herald rule not working.

Specifically, you could get the behavior you want by changing the visibility policy of the task (so the user can see it) or adding the user to the project (so they satisfy the current visibility policy).

Jul 29 2015, 11:03 PM · Herald, Phacility Support
epriestley added a comment to T9003: Herald rule not working.

It looks like T1603 on your install has a project visibility policy (i.e., the task is visible to members of a specific project). However, I think the user customer_support_pagerduty is not a member of the project, so they can not see the task. (I'm just looking at the database here, so it's possible I got the WHERE clauses wrong.)

Jul 29 2015, 11:02 PM · Herald, Phacility Support
oliverdain added a comment to T9003: Herald rule not working.

Thanks for the quick debugging. However, I don't think that fixes my real problem (not your fault since I didn't tell you what that is). The rule was originally to trigger an email to a mailing list user (a PagerDuty integration email specifically). That "user" does not have the don't email for self actions action set (and even if they did, I'm the one creating the bug so it shouldn't apply). Those emails weren't going through. So I changed the rule to email me as a debugging step and found that I'm not getting the email so figured that was the root cause. As you suggest, if I change my preference so I get emailed on self-actions I then do get emails. However, if I change the email back to the mailing list it is not getting emails. Perhaps there's something else blocking these??

Jul 29 2015, 9:14 PM · Herald, Phacility Support
chad added a comment to T9003: Herald rule not working.

Oh, I thought all Herald rules were stronger than preferences.

Jul 29 2015, 7:50 PM · Herald, Phacility Support
epriestley added a comment to T9003: Herald rule not working.

Here's why you didn't receive the mail:

Jul 29 2015, 7:44 PM · Herald, Phacility Support
epriestley added a comment to T9003: Herald rule not working.

I can't reproduce this. I made this rule:

Jul 29 2015, 7:40 PM · Herald, Phacility Support
chad added a comment to T9003: Herald rule not working.

I can confirm, Rule says it passed and it emailed me, but I received nothing.

Jul 29 2015, 6:45 PM · Herald, Phacility Support
oliverdain added a comment to T9003: Herald rule not working.

Attaching screen shots of the rule and the task that I think should have triggered it:

Jul 29 2015, 6:03 PM · Herald, Phacility Support
oliverdain created T9003: Herald rule not working.
Jul 29 2015, 6:02 PM · Herald, Phacility Support

Jul 28 2015

chad added a comment to T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails.

T7607 might help here as well.

Jul 28 2015, 6:24 PM · Phortune, Phacility
kislyuk added a comment to T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails.

Yes - it would be great to have a "billing contact" email for these kinds of things.

Jul 28 2015, 6:02 PM · Phortune, Phacility

Jul 27 2015

epriestley renamed T8973: Encoding JSON for presentation could use more human-friendly escaping rules from Instance Configuration Improperly Escapes Characters to Encoding JSON for presentation could use more human-friendly escaping rules.
Jul 27 2015, 9:28 PM · Customer Impact
epriestley added a comment to T8973: Encoding JSON for presentation could use more human-friendly escaping rules.

This is a valid, compliant JSON-encoding of the string, which allows / to be escaped -- here's the grammar diagram from json.org:

Jul 27 2015, 9:27 PM · Customer Impact
cmelbye created T8973: Encoding JSON for presentation could use more human-friendly escaping rules.
Jul 27 2015, 6:09 PM · Customer Impact

Jul 7 2015

epriestley closed T8781: Not receiving emails as Resolved.

This looks like it was primarily caused by operator error. I stopped some services on the paired repo host in order to recover db001 in connection with T8764 yesterday. This worked, and let me revive the host immediately rather than needing to restart it, but I then failed to completely restart services later.

Jul 7 2015, 6:47 PM · Phacility Support
scottybollinger added a comment to T8781: Not receiving emails.

Thanks! We are receiving emails now

Jul 7 2015, 5:06 PM · Phacility Support
epriestley added a comment to T8781: Not receiving emails.

I think the immediate issue is now resolved, I'll look at this in more detail and post a followup a little later.

Jul 7 2015, 5:05 PM · Phacility Support
epriestley added a comment to T8781: Not receiving emails.

Looking into this now.

Jul 7 2015, 4:50 PM · Phacility Support
scottybollinger created T8781: Not receiving emails.
Jul 7 2015, 4:15 PM · Phacility Support

Jun 17 2015

dkillebrew added a comment to T8574: Users receiving many duplicate emails.

Thank you for the fast response, as well as the credit. We appreciate it. Cheers.

Jun 17 2015, 4:30 AM · Phacility Support
epriestley closed T8574: Users receiving many duplicate emails as Resolved.

I believe this is now resolved. Thanks for the report, and sorry for the duplicate mail. I've issued your instance a service credit for the last 48 hours, which will be reflected on your next bill.

Jun 17 2015, 12:03 AM · Phacility Support

Jun 16 2015

epriestley added a revision to T8574: Users receiving many duplicate emails: D13320: Dramatically limit the range of failures which can cause duplicate mail.
Jun 16 2015, 11:22 PM · Phacility Support
epriestley added a comment to T8574: Users receiving many duplicate emails.

Repository tier is restarting now with D13319, so this should be fixed in about a minute.

Jun 16 2015, 11:09 PM · Phacility Support
epriestley added a comment to T8574: Users receiving many duplicate emails.

D13319 should fix this specific issue.

Jun 16 2015, 11:02 PM · Phacility Support
epriestley added a revision to T8574: Users receiving many duplicate emails: D13319: Fail gracefully on bad API call when building commit mail.
Jun 16 2015, 10:56 PM · Phacility Support
epriestley added a comment to T8574: Users receiving many duplicate emails.

The issue is roughly:

Jun 16 2015, 10:49 PM · Phacility Support