Page MenuHomePhabricator
Feed Advanced Search

Sep 20 2016

epriestley merged T11666: Checkout is failing from Phacility into T11665: repo002.phacility.net is heavily loaded.
Sep 20 2016, 1:51 AM · Ops, Phacility
swisspol added a comment to T11666: Checkout is failing from Phacility.

Pushing appears to work (as in the commits appear in Diffusion) but errors are printed:

$ git push
# Push received by "web.phacility.net", forwarding to cluster host.
# Waiting up to 120 second(s) for a cluster write lock...
# Acquired write lock immediately.
# Waiting up to 120 second(s) for a cluster read lock on "repo002.phacility.net"...
# Acquired read lock immediately.
# Device "repo002.phacility.net" is already a cluster leader and does not need to be synchronized.
# Ready to receive on cluster host "repo002.phacility.net".
Counting objects: 11, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (11/11), 1.90 KiB | 0 bytes/s, done.
Total 11 (delta 10), reused 0 (delta 0)
# Released cluster write lock.
PHP Fatal error:  Uncaught exception 'FilesystemException' with message 'Requested path '/core/log/hyper' is not writable.' in /core/lib/libphutil/src/filesystem/Filesystem.php:1107
Stack trace:
#0 /core/lib/libphutil/src/filesystem/Filesystem.php(650): Filesystem::assertWritable('/core/log/hyper')
#1 /core/lib/libphutil/src/filesystem/PhutilDeferredLog.php(183): Filesystem::createDirectory('/core//log/hype...', 493, true)
#2 /core/lib/libphutil/src/filesystem/PhutilDeferredLog.php(155): PhutilDeferredLog->write()
#3 [internal function]: PhutilDeferredLog->__destruct()
#4 {main}
  thrown in /core/lib/libphutil/src/filesystem/Filesystem.php on line 1107
To ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git
   d77b984..6eb6700  master -> master
error: failed to push some refs to 'ssh://hyper@vault.phacility.com/diffusion/HYPER/hyper.git'
Sep 20 2016, 1:50 AM · Bug Report
swisspol created T11666: Checkout is failing from Phacility.
Sep 20 2016, 1:48 AM · Bug Report
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

One remaining issue with backoff/cooldown is that mirrors aren't included: a regularly-updated repository with broken mirrors still tries mirroring. This is somewhat more involved to fix, though (see T6131).

Sep 20 2016, 1:07 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

Those changes seem stable. Unclear how much they're really helping, but load is looking at least somewhat better. I'm going to dig into things a bit more and see if I can get a better sense of whether we're on the path to recovery or not.

Sep 20 2016, 1:06 AM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

D16574 retains update cooldowns (so we don't immediately update everything after a restart) and sets up for the following change.

Sep 20 2016, 12:32 AM · Ops, Phacility
epriestley added a revision to T11665: repo002.phacility.net is heavily loaded: Unknown Object (Differential Revision).
Sep 20 2016, 12:28 AM · Ops, Phacility

Sep 19 2016

epriestley closed T7865: After restarting, repository daemons all do a lot of work as Resolved by committing Restricted Diffusion Commit.
Sep 19 2016, 11:49 PM · Phacility
epriestley added a revision to T7865: After restarting, repository daemons all do a lot of work: Unknown Object (Differential Revision).
Sep 19 2016, 11:48 PM · Phacility
epriestley added a revision to T11665: repo002.phacility.net is heavily loaded: Restricted Differential Revision.
Sep 19 2016, 11:48 PM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

Repository retries (failed mirrors, failed fetches) have no real backoff

Sep 19 2016, 11:03 PM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.

I could alternatively throttle down the highly active instances (or all instances on this shard) but that's not too great.

Sep 19 2016, 10:51 PM · Ops, Phacility
epriestley added a comment to T11665: repo002.phacility.net is heavily loaded.
  • Maybe a good time to launch new shards? Last tier size increase was 10 months ago (late November, 2015) when I fixed the allocation algorithm to be not a huge wasteful pile of garbage that required us to constantly throw hardware at it.
    • Maybe the allocation algorithm should consider active users, not just active instances? I think the 002 shard has the most active users by a hefty margin, just by luck of the draw. But this will tend to be moot at scale.
    • Maybe start moving instances between shards? This seems hard to do without downtime today, and doing all the legwork to reduce downtime through clustering is foreboding.
Sep 19 2016, 10:38 PM · Ops, Phacility
epriestley created T11665: repo002.phacility.net is heavily loaded.
Sep 19 2016, 10:29 PM · Ops, Phacility
fushi created T11664: Add option to allow owner packages to be added as reviewers to diffs, when the diff author is in the owner package.
Sep 19 2016, 10:21 PM · Restricted Project, Feature Request
epriestley renamed T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained from Remarkup navigation component is not able to have text starting with equals sign to Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.
Sep 19 2016, 9:44 PM · Documentation, Remarkup
cspeckmim added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

Ohhh nice, thank you I missed that
=C124

Sep 19 2016, 9:32 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

And you can technically:

Sep 19 2016, 9:32 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

Oh, right. That works, but I think isn't documented:

Sep 19 2016, 9:31 PM · Documentation, Remarkup
cspeckmim added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

It's an off-case for sure. We have a ton of data identifiers which all end with an equals-sign and a couple letters/numbers, which we abbreviate when referring to the datasets. I was actually trying to render the equals sign as part of the nav sequence and not just C124.

Sep 19 2016, 9:30 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.

We could conceivably special-case this since setting an empty attribute to a value isn't meaningful, I suppose.

Sep 19 2016, 9:25 PM · Documentation, Remarkup
epriestley added a comment to T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.
{nav name=C124}
Sep 19 2016, 9:24 PM · Documentation, Remarkup
cspeckmim created T11663: Escaping for properties in remarkup rules like "{nav ...}" is not clearly explained.
Sep 19 2016, 9:04 PM · Documentation, Remarkup
epriestley created T11662: Draft comments which reply to ghost inlines from lines which no longer exist can not be edited or deleted until the page is reloaded.
Sep 19 2016, 8:43 PM · Inline Comments, Customer Impact, Differential
jcox created T11661: Change PhabricatorPhurlURLViewController to use EditEngine for commenting.
Sep 19 2016, 8:15 PM · Contributor Onboarding, Modernization
jonah214 added a comment to T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.

In case someone else ends up with my original problem, I'll mention that https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1518457/comments/69 fixed it.

Sep 19 2016, 7:57 PM · Documentation, Setup
epriestley added a comment to T10747: Import and export ICS from Calendar.

Some somewhat-good news is that Google Calendar just gives you an .ics blob of garbage to describe recurrence rules:

Sep 19 2016, 7:11 PM · Restricted Project, Prioritized, Calendar
epriestley added a comment to T10747: Import and export ICS from Calendar.

Things don't look promising.

Sep 19 2016, 7:02 PM · Restricted Project, Prioritized, Calendar
jcox added a revision to T10673: Convert Phurl to use EditEngine: Unknown Object (Differential Revision).
Sep 19 2016, 6:11 PM · Contributor Onboarding, Restricted Project, Phurl
avivey updated the task description for T11660: Diff creation fails when submitting too long file paths.
Sep 19 2016, 5:53 PM · Differential, Bug Report
avivey added a comment to Q481: base URI & Allowed URI change issues (Answer 451).

When you load the configuration web page, the effective configuration is highlighted yellow. If the database value is not null, it might be overriding it, but ./bin/config get should show all the configured values.

Sep 19 2016, 5:37 PM
epriestley added a comment to T10747: Import and export ICS from Calendar.

I think we should be able to accommodate that fairly easily.

Sep 19 2016, 5:30 PM · Restricted Project, Prioritized, Calendar
ksmith added a comment to T4171: Building reporting and data systems.

@lyahdav : That looks pretty cool. It's not quite clear to me where it runs. Is this a script that one would install on the same box that phab itself is running on?

Sep 19 2016, 5:26 PM · Restricted Project, FreeBSD, Wikimedia, Facts
jcox closed T11642: Add "name" field to `harbormaster.build.search` results as Resolved by committing Restricted Diffusion Commit.
Sep 19 2016, 5:17 PM · Restricted Project, Conduit, Harbormaster
benjamin.cohen-solal added a project to T11660: Diff creation fails when submitting too long file paths: Differential.
Sep 19 2016, 2:52 PM · Differential, Bug Report
benjamin.cohen-solal created T11660: Diff creation fails when submitting too long file paths.
Sep 19 2016, 2:52 PM · Differential, Bug Report
cspeckmim created T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.
Sep 19 2016, 1:41 PM · Differential, Bug Report
jcox added a revision to T8628: Modernize AphrontControllers to implement handleRequest: Unknown Object (Differential Revision).
Sep 19 2016, 1:19 PM · Contributor Onboarding, Modernization, Badge Awarded, Aphront
chad merged T11658: Ability to Receive Notification when I'm mentioned in a task description or comment into T4654: Add 'always email me' type option for direct mentions, ccs, assignments, etc.
Sep 19 2016, 12:52 PM · Customer Impact, Restricted Project, Mail
chad merged task T11658: Ability to Receive Notification when I'm mentioned in a task description or comment into T4654: Add 'always email me' type option for direct mentions, ccs, assignments, etc.
Sep 19 2016, 12:52 PM · Feature Request
jcox added a revision to T8628: Modernize AphrontControllers to implement handleRequest: Unknown Object (Differential Revision).
Sep 19 2016, 12:42 PM · Contributor Onboarding, Modernization, Badge Awarded, Aphront
jcox added a revision to T8628: Modernize AphrontControllers to implement handleRequest: Unknown Object (Differential Revision).
Sep 19 2016, 12:38 PM · Contributor Onboarding, Modernization, Badge Awarded, Aphront
jcox added a revision to T11642: Add "name" field to `harbormaster.build.search` results: Unknown Object (Differential Revision).
Sep 19 2016, 12:04 PM · Restricted Project, Conduit, Harbormaster
lostinspace updated Q492: new base URI not working from to new base URI not working.
Sep 19 2016, 10:15 AM
lostinspace added a comment to Q481: base URI & Allowed URI change issues (Answer 451).

I found this link: https://secure.phabricator.com/book/phabricator/article/advanced_configuration/

Sep 19 2016, 9:39 AM
barron.ernst created T11658: Ability to Receive Notification when I'm mentioned in a task description or comment.
Sep 19 2016, 8:20 AM · Feature Request
20after4 added a comment to T7944: Add Conduit support to Calendar.

Here are 2 use-cases which are both essentially the same in all the important ways:

Sep 19 2016, 5:31 AM · Conduit, Calendar

Sep 18 2016

hach-que added a comment to T10747: Import and export ICS from Calendar.

(comment posted twice for some reason)

Sep 18 2016, 10:46 PM · Restricted Project, Prioritized, Calendar
hach-que added a comment to T10747: Import and export ICS from Calendar.

I thought I'd throw our use case in here in case it helps:

Sep 18 2016, 10:46 PM · Restricted Project, Prioritized, Calendar
chad closed T11654: Keyboard shortcuts (?) show as white on gray (unreadable) as Resolved by committing Restricted Diffusion Commit.
Sep 18 2016, 10:35 PM · Bug Report
chad added a revision to T11654: Keyboard shortcuts (?) show as white on gray (unreadable): Unknown Object (Differential Revision).
Sep 18 2016, 10:18 PM · Bug Report
chad closed T11657: Phabricator Account Created as Spite.
Sep 18 2016, 9:49 PM · Phabricator
chad closed T11656: Phabricator as Spite.
Sep 18 2016, 9:49 PM · Bug Report
chad closed T11655: Phabricator as Spite.
Sep 18 2016, 9:48 PM · Bug Report
• lpetov.ad36 awarded T11657: Phabricator Account Created a Dislike token.
Sep 18 2016, 9:18 PM · Phabricator
• lpetov.ad36 edited projects for T11657: Phabricator Account Created, added: Phabricator; removed Phabot.
Sep 18 2016, 9:16 PM · Phabricator
• lpetov.ad36 edited projects for T11657: Phabricator Account Created, added: Phabot; removed Feature Request.
Sep 18 2016, 9:14 PM · Phabricator
• lpetov.ad36 created T11657: Phabricator Account Created.
Sep 18 2016, 9:08 PM · Phabricator
• lpetov.ad36 created T11656: Phabricator.
Sep 18 2016, 9:04 PM · Bug Report
• lpetov.ad36 created T11655: Phabricator.
Sep 18 2016, 9:03 PM · Bug Report
carlokok created T11654: Keyboard shortcuts (?) show as white on gray (unreadable).
Sep 18 2016, 6:09 PM · Bug Report
epriestley added a comment to T10747: Import and export ICS from Calendar.

More generally, here's how I'm planning to model imports, which seems largely consistent with other systems:

Sep 18 2016, 2:02 PM · Restricted Project, Prioritized, Calendar
epriestley added a comment to T10747: Import and export ICS from Calendar.

Yeah, I fiddled with that but didn't immediately have any luck. It's possible that there's some complex condition like non-self organizer + non-self sender or something. If it only requires non-organizer, it should already work today (e.g., on this install).

Sep 18 2016, 1:18 PM · Restricted Project, Prioritized, Calendar
epriestley added a comment to T7944: Add Conduit support to Calendar.

Sort of. I'd like to have use cases (what are you trying to do?) first, and I'm not certain what the best representation of dates and times is for external consumption.

Sep 18 2016, 1:15 PM · Conduit, Calendar
20after4 added a comment to T7944: Add Conduit support to Calendar.

@epriestley: Are you open to patches that add more information to the calendar.event.search response? It currently does not even include the event time which is a pretty crucial detail to make the results useful.

Sep 18 2016, 4:00 AM · Conduit, Calendar

Sep 17 2016

lyahdav added a comment to T4171: Building reporting and data systems.

For anyone trying to get Kanban statistics / reports in the meantime, you can try this project I recently started working on:
https://github.com/lyahdav/analytics-limn-analytics-data/tree/kanban_stats. It generates a CFD (Cumulative Flow Diagram) and computes cycle time and lead time.

Sep 17 2016, 11:58 PM · Restricted Project, FreeBSD, Wikimedia, Facts
epriestley edited the content of Changelog.
Sep 17 2016, 9:25 PM
epriestley edited the content of 2016 Week 38 (Mid September).
Sep 17 2016, 9:25 PM
epriestley created 2016 Week 38 (Mid September).
Sep 17 2016, 9:25 PM
d.maznekov merged task T11653: Take over (forcibly) Permisions of Project into T10255: Edit permissions for Project (before and after some update of Phabricator).
Sep 17 2016, 10:04 AM · Bug Report
d.maznekov merged T11653: Take over (forcibly) Permisions of Project into T10255: Edit permissions for Project (before and after some update of Phabricator).
Sep 17 2016, 10:04 AM · Bug Report
d.maznekov added a comment to T11653: Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:59 AM · Bug Report
d.maznekov merged task T10255: Edit permissions for Project (before and after some update of Phabricator) into T11653: Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:59 AM · Bug Report
d.maznekov merged T10255: Edit permissions for Project (before and after some update of Phabricator) into T11653: Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:58 AM · Bug Report
d.maznekov renamed T11653: Take over (forcibly) Permisions of Project from Permisions of Project to Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:53 AM · Bug Report
d.maznekov created T11653: Take over (forcibly) Permisions of Project.
Sep 17 2016, 9:52 AM · Bug Report
seungrye awarded T3027: make conpherence transactions automagically scroll a Like token.
Sep 17 2016, 7:10 AM · Conpherence
lyahdav added a comment to T5214: Add Conduit API methods for workboards.

@raincoastchris the context of this issue caught my eye because I've really wanted Phab to generate a cumulative flow chart and Kanban statistics for some time now. I took a stab at it today and modified an existing python script to generate a cumulative flow chart and Kanban statistics (https://github.com/lyahdav/analytics-limn-analytics-data/tree/kanban_stats) but configuring it is a bit painful because there is no easy way to get the names of all the columns in a workboard right now.

Sep 17 2016, 6:13 AM · Restricted Project, Conduit, Maniphest, Workboards
chad closed T11645: Generate feed story for new (all user) rooms? as Resolved.
Sep 17 2016, 4:39 AM · Conpherence (v4)
chad changed the visibility for T3027: make conpherence transactions automagically scroll.
Sep 17 2016, 3:32 AM · Conpherence
chad closed T11628: It's not simple to join a Conpherence as Resolved.

I think this is nearly all resolved with NUX.

Sep 17 2016, 2:27 AM · Conpherence
epriestley added a comment to T11644: Allow editing of Conpherence messages.

This probably already works, on the backend, but needs:

Sep 17 2016, 1:14 AM · Conpherence
chad added a comment to T11644: Allow editing of Conpherence messages.

Let me know if there is a conversion diff I can follow.

Sep 17 2016, 12:22 AM · Conpherence

Sep 16 2016

bizrad6 added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

Awesome! Thanks for the quick fix @epriestley!

Sep 16 2016, 9:37 PM · Restricted Project, Bug Report
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

I think the first part of this should be fixed in HEAD now, thanks for the report!

Sep 16 2016, 9:07 PM · Restricted Project, Bug Report
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

The "multiple packages" issue may have been a symptom of the dominion/archive problem, here's a case I hit while testing that made it look like there was a second issue:

Sep 16 2016, 9:00 PM · Restricted Project, Bug Report
epriestley created T11652: `arc which` lists repositories by monogram instead of display name.
Sep 16 2016, 8:40 PM · Arcanist
epriestley created T11651: Owners empty states are lookin' weird.
Sep 16 2016, 8:37 PM · Owners, Design
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

D16564 might fix this but I need to actually test it.

Sep 16 2016, 7:10 PM · Restricted Project, Bug Report
epriestley added a revision to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived: Unknown Object (Differential Revision).
Sep 16 2016, 7:09 PM · Restricted Project, Bug Report
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

Also, what is the expected behavior if two Owners packages point to the same path?

Sep 16 2016, 7:02 PM · Restricted Project, Bug Report
bizrad6 added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

Is it possible that this is slightly narrower: "package dominion rules incorrectly consider archived packages"? That seems to fit with the observations, but maybe I'm overlooking something.

Sep 16 2016, 6:55 PM · Restricted Project, Bug Report
epriestley added a comment to T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.

Is it possible that this is slightly narrower: "package dominion rules incorrectly consider archived packages"? That seems to fit with the observations, but maybe I'm overlooking something.

Sep 16 2016, 6:52 PM · Restricted Project, Bug Report
eadler moved T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Sep 16 2016, 6:28 PM · Restricted Project, Bug Report
chad added a comment to T11636: Herald condition: count of project tags.

Nuance is our plan for routing tasks, we don't anticipate needing this functionality in Herald.

Sep 16 2016, 5:32 PM · Herald, Feature Request
bizrad6 created T11650: Owners does not assign as reviewer if an Owners package with the same path has been archived.
Sep 16 2016, 5:22 PM · Restricted Project, Bug Report
maxie added a comment to T11636: Herald condition: count of project tags.

Ok, read that. Here is a breakdown of our wishes and what we did to implement them:

Sep 16 2016, 2:34 PM · Herald, Feature Request
epriestley merged T11649: User changing the task access to a project which another user is not member of, leads to a not visible active notification. into T8953: Phantom unread notifications when the user is subscribed to something he's not allowed to view.
Sep 16 2016, 1:24 PM · Wikimedia, Notifications
epriestley merged task T11649: User changing the task access to a project which another user is not member of, leads to a not visible active notification. into T8953: Phantom unread notifications when the user is subscribed to something he's not allowed to view.
Sep 16 2016, 1:24 PM · Bug Report
simevo edited the content of Community Resources.
Sep 16 2016, 12:58 PM