Page MenuHomePhabricator
Feed Advanced Search

May 15 2015

epriestley updated the task description for T8210: Phacility Cluster: Bastion host stopped responding.
May 15 2015, 2:51 PM · Ops, Phacility
epriestley updated subscribers of T7338: Build more status tools for monitoring Phacility cluster health.
May 15 2015, 1:19 PM · Almanac, Phacility
epriestley renamed T7338: Build more status tools for monitoring Phacility cluster health from BES EOL determine strategy to sunset in place to Build more status tools for monitoring Phacility cluster health.
May 15 2015, 1:18 PM · Almanac, Phacility
epriestley added a parent task for T8205: Add timeouts to commands executed through the bastion: T8210: Phacility Cluster: Bastion host stopped responding.
May 15 2015, 1:10 PM · Phacility
epriestley added a parent task for T8206: Make authorize-user part of bastion deployment: T8210: Phacility Cluster: Bastion host stopped responding.
May 15 2015, 1:10 PM · Phacility
epriestley added a parent task for T8203: Make Phacility account import default to "ignore" and confirm imports: T8210: Phacility Cluster: Bastion host stopped responding.
May 15 2015, 1:10 PM · Phacility
epriestley added a parent task for T8204: Stop using EC2 micro instances: T8210: Phacility Cluster: Bastion host stopped responding.
May 15 2015, 1:10 PM · Phacility
epriestley added subtasks for T8210: Phacility Cluster: Bastion host stopped responding: T8206: Make authorize-user part of bastion deployment, T8205: Add timeouts to commands executed through the bastion, T8204: Stop using EC2 micro instances, T8203: Make Phacility account import default to "ignore" and confirm imports.
May 15 2015, 1:10 PM · Ops, Phacility
epriestley created T8210: Phacility Cluster: Bastion host stopped responding.
May 15 2015, 1:09 PM · Ops, Phacility
epriestley added a comment to T8208: Create a plan for consistent use of icons, colors and text style when rendering Calendar events.

Depending on exactly how T8026 sorts out, we might also have a stronger "public event" vs "private event" state in the future.

May 15 2015, 1:00 PM · Calendar
epriestley renamed T8208: Create a plan for consistent use of icons, colors and text style when rendering Calendar events from Placeholder for the fate of the dots in user handles (previously used for labeling "away" or "sporadic" status of a user, based on Calendar availability) to Create a plan for consistent use of icons, colors and text style when rendering Calendar events.
May 15 2015, 12:56 PM · Calendar
epriestley added a comment to T4571: Allow Phabricator to run in Read-Only Mode.

In T8209, we hit an issue where a migration performed a cache fill as a side effect of loading handles. This created a problem because the cache fill happened from a migration that runs before the cache was introduced.

May 15 2015, 12:54 PM · Phacility
epriestley added a comment to T8209: Migrations which use handles may fail after introduction of cache columns.

In the long term, I think we should probably add an interaction with T4571 to reduce the likelihood of these issues. I'll make some notes there.

May 15 2015, 12:47 PM
epriestley added a revision to T8209: Migrations which use handles may fail after introduction of cache columns: Unknown Object (Differential Revision).
May 15 2015, 12:46 PM
epriestley added a comment to T8183: Remove "Away" and "Sporadic" statuses..

D12857 should fix the issue with list views (including /upcoming/).

May 15 2015, 12:39 PM · Calendar
epriestley added a revision to T8183: Remove "Away" and "Sporadic" statuses.: Unknown Object (Differential Revision).
May 15 2015, 12:39 PM · Calendar
epriestley added a comment to T8183: Remove "Away" and "Sporadic" statuses..

Ah, thanks! I'll clean that up.

May 15 2015, 12:32 PM · Calendar
epriestley created T8209: Migrations which use handles may fail after introduction of cache columns.
May 15 2015, 12:31 PM
epriestley added a comment to T8208: Create a plan for consistent use of icons, colors and text style when rendering Calendar events.

One other signal we could use is adding or removing backgrounds from icons, if we add them:

May 15 2015, 4:00 AM · Calendar
epriestley added a comment to T8208: Create a plan for consistent use of icons, colors and text style when rendering Calendar events.

Few different parts:

May 15 2015, 3:50 AM · Calendar
epriestley created T8206: Make authorize-user part of bastion deployment.
May 15 2015, 1:47 AM · Phacility
epriestley created T8205: Add timeouts to commands executed through the bastion.
May 15 2015, 1:30 AM · Phacility
epriestley created T8204: Stop using EC2 micro instances.
May 15 2015, 1:29 AM · Phacility
epriestley added a comment to T550: Build an SSH conduit client.

... on the host

May 15 2015, 12:39 AM · Prioritized, Arcanist, Restricted Project, Restricted Project, Conduit
epriestley added a comment to T550: Build an SSH conduit client.

I'd exec ssh directly. If there's some major advantage to using libssh2 (but what?), I'd possibly use that optionally if it was available on the host.

May 15 2015, 12:37 AM · Prioritized, Arcanist, Restricted Project, Restricted Project, Conduit
epriestley closed T8201: Breaking UserConduitAPIMethod.php error as Resolved.

Should be fixed in HEAD. Thanks for the report!

May 15 2015, 12:24 AM · People, Conduit
epriestley added a revision to T8201: Breaking UserConduitAPIMethod.php error : Unknown Object (Differential Revision).
May 15 2015, 12:24 AM · People, Conduit
epriestley renamed T8203: Make Phacility account import default to "ignore" and confirm imports from Make phacility account import default to "ignore" and confirm imports to Make Phacility account import default to "ignore" and confirm imports.
May 15 2015, 12:21 AM · Phacility
epriestley created T8203: Make Phacility account import default to "ignore" and confirm imports.
May 15 2015, 12:21 AM · Phacility

May 14 2015

epriestley closed T7195: Page up / page down don't work in chrome after failed text search as Resolved.

Presumably obsoleted by T8151, reopen if this is still an issue at HEAD.

May 14 2015, 11:10 PM · PHUI
epriestley closed T7209: Scollbar with Trackpad inconsistent with Desktop/Mouse as Resolved.
May 14 2015, 11:10 PM · PHUI
epriestley closed T7466: Chome 41 has weird scroll jitter with Paste embeds as Resolved.
May 14 2015, 11:10 PM · PHUI
epriestley added a comment to T7466: Chome 41 has weird scroll jitter with Paste embeds.

Presumably obsoleted by T8151, reopen if this is still an issue at HEAD. This is probably a Chrome bug if it's not fixed at HEAD.

May 14 2015, 11:10 PM · PHUI
epriestley added a comment to T7209: Scollbar with Trackpad inconsistent with Desktop/Mouse.

Presumably obsoleted by T8151, reopen if this is still an issue at HEAD.

May 14 2015, 11:09 PM · PHUI
epriestley closed T7128: No scrollbar when using trackpad on Dashboards as Resolved.

Presumably obsoleted by T8151, reopen if this is still an issue at HEAD.

May 14 2015, 11:09 PM · PHUI
epriestley closed T7494: Scrolling is... super bad in Chrome? as Resolved.

Presumably obsoleted in HEAD.

May 14 2015, 11:08 PM · PHUI
epriestley closed T7323: disabling fancy scrollbars workaround still breaks keyboard scrolling as Resolved.

Obsoleted in HEAD.

May 14 2015, 11:08 PM · PHUI
epriestley closed T7109: Mousewheel scrolling is sluggish in Firefox/Win as Resolved.
May 14 2015, 11:07 PM · Firefox, Windows, PHUI
epriestley closed T7740: Chrome exhibits weird scrolling jumping with anchor in the URL as Resolved.

This should be resolved in HEAD because we no longer frame the page content. Chrome still has the underlying bug.

May 14 2015, 11:07 PM · Google Chrome
epriestley closed T8151: Decide the fate of JX.Scrollbar as Resolved.

I'm going to tentatively call this resolved since things have been quiet for a few days now, but I'll keep an ear open for more feedback.

May 14 2015, 11:06 PM · Conpherence, PHUI
epriestley closed T8202: Comment Preview JS error as Wontfix.

If this doesn't do anything bad and you can't reproduce it reliably, I'm just going to close this until someone is materially affected by it.

May 14 2015, 11:04 PM · Quicksand
epriestley claimed T8201: Breaking UserConduitAPIMethod.php error .
May 14 2015, 10:42 PM · People, Conduit
epriestley added a revision to T8199: Inbound email kind-of working (but kind of only).: Unknown Object (Differential Revision).
May 14 2015, 7:08 PM · Mail
epriestley added a comment to T8199: Inbound email kind-of working (but kind of only)..

Is "2015-05-14 19:34 GMT+02:00 Me <its.me@mario>" a standard header for Gmail quoted text in French? Is it a standard signature? I'm confused because neither of your examples actually contain any quoted text.

May 14 2015, 7:02 PM · Mail
epriestley added a comment to T8199: Inbound email kind-of working (but kind of only)..

What mail client did you use to send that mail?

May 14 2015, 5:57 PM · Mail
epriestley closed T8198: Unguarded write error on every page as Resolved by committing Restricted Diffusion Commit.
May 14 2015, 5:39 PM · Aphront
epriestley added a revision to T8198: Unguarded write error on every page: Unknown Object (Differential Revision).
May 14 2015, 5:21 PM · Aphront
epriestley added a revision to T8198: Unguarded write error on every page: Unknown Object (Differential Revision).
May 14 2015, 5:14 PM · Aphront
epriestley claimed T8198: Unguarded write error on every page.

Ah, perfect.

May 14 2015, 4:57 PM · Aphront
epriestley added a comment to T8198: Unguarded write error on every page.

Just doing phlog($ex) might be useful (or phlog(new Exception()) if you aren't actually catching one).

May 14 2015, 4:54 PM · Aphront
epriestley added a comment to T8198: Unguarded write error on every page.

I can't immediately reproduce this -- do you have a trace in the Apache/php-fpm error log?

May 14 2015, 4:48 PM · Aphront
epriestley added a revision to T8183: Remove "Away" and "Sporadic" statuses.: Unknown Object (Differential Revision).
May 14 2015, 4:31 PM · Calendar
epriestley added a revision to T8183: Remove "Away" and "Sporadic" statuses.: Unknown Object (Differential Revision).
May 14 2015, 4:30 PM · Calendar
epriestley added a revision to T7707: Implement a read-through cache layer: Unknown Object (Differential Revision).
May 14 2015, 3:55 PM · Performance
epriestley added a comment to T8135: drag & drop on a workboard sorted by priority behavior is surprising.

Yeah, I think we can probably slim them down a hair and make them work. Installs that define 200 priorities will get questionable results, but I don't feel too bad about that.

May 14 2015, 2:40 PM · Restricted Project, Workboards (v3), Restricted Project
epriestley created T8197: Dropping workboard cards on the bottom of a priority column is buggy.
May 14 2015, 2:32 PM · Workboards
epriestley added a comment to T8135: drag & drop on a workboard sorted by priority behavior is surprising.
In T8135#113908, @chad wrote:

I had originally designed dividers into the columns for clarity. We removed them since typically they're clutter, but I wonder if we could instead just activate them when hovering a card over a column.

May 14 2015, 2:31 PM · Restricted Project, Workboards (v3), Restricted Project
epriestley added a comment to T8135: drag & drop on a workboard sorted by priority behavior is surprising.

I'm not sure what you mean by 'lightly used'.

May 14 2015, 2:28 PM · Restricted Project, Workboards (v3), Restricted Project
epriestley added a comment to T7076: Show status of linked revisions for tasks.

Sorry, this is a lot harder to build than it seems like it might be, and probably not appropriate for a contributor. In particular:

May 14 2015, 1:19 PM · Restricted Project, Differential, Maniphest
epriestley added a comment to T8135: drag & drop on a workboard sorted by priority behavior is surprising.

I think the likely way forward is:

May 14 2015, 1:05 PM · Restricted Project, Workboards (v3), Restricted Project
epriestley closed T8195: Error running arc diff command: sprintf(): Too few arguments at ArcanistWorkflow.php:1024 as Resolved by committing Restricted Diffusion Commit.
May 14 2015, 12:59 PM · Arcanist
epriestley added a comment to T8194: Replying to a conpherence hides all previous messages from the UI.

To clarify, this is a visual glitch only.

May 14 2015, 1:31 AM · Conpherence
epriestley renamed T8194: Replying to a conpherence hides all previous messages from the UI from Replying to a conpherence erases all previous messages to Replying to a conpherence hides all previous messages from the UI.
May 14 2015, 1:31 AM · Conpherence
epriestley created T8194: Replying to a conpherence hides all previous messages from the UI.
May 14 2015, 1:31 AM · Conpherence
greggrossmeier awarded T2714: Move Releeph to Beta a Love token.
May 14 2015, 1:24 AM · Releeph
epriestley added a revision to T7707: Implement a read-through cache layer: Unknown Object (Differential Revision).
May 14 2015, 12:00 AM · Performance
epriestley added a revision to T8183: Remove "Away" and "Sporadic" statuses.: Unknown Object (Differential Revision).
May 14 2015, 12:00 AM · Calendar

May 13 2015

epriestley added a revision to T7707: Implement a read-through cache layer: Unknown Object (Differential Revision).
May 13 2015, 10:57 PM · Performance
epriestley added a comment to T8183: Remove "Away" and "Sporadic" statuses..

In a future where we have automatic escalation rules, "escalate automatically" vs "do not escalate automatically" makes sense.

May 13 2015, 10:00 PM · Calendar
epriestley added a comment to T8062: Conpherence custom image refinements.

More accurately, this stuff would be awesome, just building / supporting it versus the user alternative of busting out photoshop or paint or what have you isn't worth it.

May 13 2015, 9:49 PM · Conpherence (v4)
epriestley added a comment to T5258: Memes use older image transform code without modern error handling.

This is now resolved for non-meme cases.

May 13 2015, 6:45 PM · Restricted Project, Files, Macros
epriestley renamed T5258: Memes use older image transform code without modern error handling from `CommandException` when trying to create a thumbnail to Memes use older image transform code without modern error handling.
May 13 2015, 6:45 PM · Restricted Project, Files, Macros
epriestley closed Restricted Maniphest Task, a subtask of T7707: Implement a read-through cache layer, as Resolved.
May 13 2015, 6:44 PM · Performance
epriestley closed T7879: Profile pictures should have policy settings forced to public/most-open as Resolved by committing Restricted Diffusion Commit.
May 13 2015, 6:40 PM · Files
epriestley closed T7879: Profile pictures should have policy settings forced to public/most-open, a subtask of T7707: Implement a read-through cache layer, as Resolved.
May 13 2015, 6:40 PM · Performance
epriestley closed T4406: Allow for 100x100 images in profile and projects as Resolved by committing Restricted Diffusion Commit.
May 13 2015, 6:40 PM · Projects, Design, People
epriestley closed T4406: Allow for 100x100 images in profile and projects, a subtask of T4400: Allow profile pictures to go on object cards, as Resolved.
May 13 2015, 6:40 PM · Projects, Design, People
epriestley closed T4406: Allow for 100x100 images in profile and projects, a subtask of T7707: Implement a read-through cache layer, as Resolved.
May 13 2015, 6:40 PM · Performance
epriestley closed T4724: Some images appear with the wrong aspect ratio as Resolved by committing Restricted Diffusion Commit.
May 13 2015, 6:39 PM · Remarkup
epriestley closed T4724: Some images appear with the wrong aspect ratio, a subtask of T7707: Implement a read-through cache layer, as Resolved.
May 13 2015, 6:39 PM · Performance
epriestley added a comment to T8181: undefined method 'year' when trying to spy on epriestley.

This is how I sneak around >_> <_<

May 13 2015, 6:35 PM · Calendar
epriestley updated subscribers of T8183: Remove "Away" and "Sporadic" statuses..

@btrahan, @chad, curious if you guys have any use cases for "Sporadic" vs "Away" -- I couldn't really come up with any, and if we don't get rid of this we probably need to make it per-user instead of per-event so it would be simpler to toss it if we don't have a good use case.

May 13 2015, 6:28 PM · Calendar
epriestley created T8182: Calendar preview has timeline lines going through it.
May 13 2015, 6:15 PM · Calendar
epriestley assigned T8181: undefined method 'year' when trying to spy on epriestley to lpriestley.
May 13 2015, 6:13 PM · Calendar
epriestley added a revision to T7707: Implement a read-through cache layer: Unknown Object (Differential Revision).
May 13 2015, 4:51 PM · Performance
epriestley added a comment to T4830: Applications with Public permissions requiring users to login.

(@btrahan, you're the only repro I have, so don't change your profile picture!)

May 13 2015, 3:57 PM · Facts, Policy, People
epriestley updated subscribers of T7879: Profile pictures should have policy settings forced to public/most-open.
May 13 2015, 3:56 PM · Files
epriestley merged task T6354: Register a new user redirect to a png file instead of email confirmation page into T7879: Profile pictures should have policy settings forced to public/most-open.
May 13 2015, 3:56 PM · Support Impact, Wikimedia, Auth
epriestley added a comment to T6354: Register a new user redirect to a png file instead of email confirmation page.

I believe T7879 is the root issue here, which will be resolved shortly in connection with T7707.

May 13 2015, 3:56 PM · Support Impact, Wikimedia, Auth
epriestley added a comment to T7707: Implement a read-through cache layer.

Or, more specifically:

May 13 2015, 3:25 PM · Performance
epriestley added a revision to T7707: Implement a read-through cache layer: Unknown Object (Differential Revision).
May 13 2015, 3:24 PM · Performance
epriestley added a comment to T7707: Implement a read-through cache layer.

Oh, it doesn't actually create any peril until we try to use images larger than 50x50px (I was confusing it with some other unrelated issues), but I think it's still better to fix it now. It looks like using 100% works fine.

May 13 2015, 3:21 PM · Performance
epriestley requested changes to D5873: Add initial jabberbot support.

At this point, I strongly suspect we'll never support XMPP in the upstream: it's just too large and complex a dependency for too little value.

May 13 2015, 1:50 PM
epriestley added a comment to T7707: Implement a read-through cache layer.

D12821 creates a bit of a problem: the CSS will expect 100x100 images, but all old profile images will be 50x50.

May 13 2015, 1:46 PM · Performance
epriestley added a comment to T8062: Conpherence custom image refinements.

For T7707, I'd tentatively like to do this:

May 13 2015, 1:39 PM · Conpherence (v4)
epriestley added a comment to T8173: Red "X" for requesting changes on code reviews is rather harsh.

Tentatively, the bucketing rules might change to:

May 13 2015, 1:22 PM
epriestley added a comment to T8173: Red "X" for requesting changes on code reviews is rather harsh.

I was planning to try make "comment" work somewhat like a "soft reject" at some point, but this was before T1279. After T1279, I haven't seen much feedback to the effect that this is important (I think essentially all feedback since T1279 -- other than this case -- is related to cases like the one discussed in T8080 -- roughly, revisions appear in a bucket based on the overall revision state right now, but viewers expect them to appear in a bucket based on their personal reviewer state).

May 13 2015, 12:59 PM
epriestley added a revision to T7879: Profile pictures should have policy settings forced to public/most-open: Unknown Object (Differential Revision).
May 13 2015, 12:37 AM · Files
epriestley added a revision to T7707: Implement a read-through cache layer: Unknown Object (Differential Revision).
May 13 2015, 12:37 AM · Performance
epriestley added a revision to T4406: Allow for 100x100 images in profile and projects: Unknown Object (Differential Revision).
May 13 2015, 12:37 AM · Projects, Design, People