Page MenuHomePhabricator
Feed Advanced Search

Jun 6 2015

cburroughs created T8457: harbormaster.sendmessage return with invalid type is a bit too terse.
Jun 6 2015, 9:46 PM · Conduit, Harbormaster

Jun 5 2015

cburroughs added a member for Space Test Users: cburroughs.
Jun 5 2015, 6:36 PM
cburroughs added a comment to D13156: Provide core policy support for Spaces.

Most people I've talked to just want to keep OPS separate from IT separate from ENG. stuff usually doesn't move between.

Jun 5 2015, 12:26 PM

Jun 2 2015

cburroughs added a comment to T8387: Convert Mailing Lists into special users, similar to bot users.

D13124 shows up as Unknown Object (Differential Revision). for me. Something meta going on here?

Jun 2 2015, 6:53 PM · FreeBSD, Mailing Lists, Mail
cburroughs abandoned D10565: Make "Active" the default query for People.
Jun 2 2015, 6:30 PM
cburroughs awarded T8388: Search Closed Tasks by Award Tokens a Baby Tequila token.
Jun 2 2015, 4:33 PM · Tokens

May 30 2015

cburroughs added a comment to T8363: Include reply action ("!action") hints in outbound email.

I like the help live saver thing in the top right. that's where I always find them.

May 30 2015, 6:10 PM · llvm, Restricted Project, Mail

May 29 2015

cburroughs added a comment to T7780: Add linter for Foodcritic.

D10105 has an example of a linter that's more 'I need to look at the whole repo' instead of individual files.

May 29 2015, 12:27 PM · Restricted Project, Lint

May 28 2015

cburroughs added a comment to T8238: Formally support side-band change handoff in external repositories.

"You need to cron some stuff syncing master or it will sometimes be kind of slow" is acceptable from my point of view. Things I can solve with code is preferable to 'please debug my snowflake workstation where I did strange where I did strange things with arc and git'.

May 28 2015, 1:19 AM · Restricted Project, Restricted Project, Diffusion, Harbormaster

May 26 2015

cburroughs added a comment to T8238: Formally support side-band change handoff in external repositories.

If I'm reading this right, with the tag approach & staging != main-repo, unless someone has specified things manually in .git/config they (well arc) can do git push to arbitrary places but since the staging repo isn't a configured remote none of {fetch,pull,clone} will know anything about it. I think that is a satisfactory level of cleanliness for both the main & local repos.

May 26 2015, 6:27 PM · Restricted Project, Restricted Project, Diffusion, Harbormaster
cburroughs closed T7484: Lock is not locked by this process as Resolved.

I have not seen this since moving our install a month ago. I don't have the logs from the old server to verify if that was before or after D12574. Maybe the old server was haunted? ¯\_(ツ)_/¯

May 26 2015, 1:45 PM · Restricted Project, Daemons

May 20 2015

cburroughs created T8280: email notifications appear to bypass some project visibility rules.
May 20 2015, 6:22 PM
cburroughs added a comment to T5258: Memes use older image transform code without modern error handling.

On a possibly related note, when messing with increasing the timeout to get a meme to work (memes are very important for delivering business value) I once got:

May 20 2015, 3:22 PM · Restricted Project, Macros, Files
cburroughs retitled D12950: fix trvial space typo in user guide from to fix trvial space typo in user guide.
May 20 2015, 2:18 PM

May 19 2015

cburroughs created T8258: Allow comments in JSON.
May 19 2015, 3:56 PM · Restricted Project, libphutil, Arcanist

May 18 2015

cburroughs awarded T8236: `arc weld` should do something a Manufacturing Defect? token.
May 18 2015, 11:02 PM · Contributor Onboarding, Arcanist
cburroughs awarded rP00136f05dd6d: Allow adding Auditors with the singular noun. a Piece of Eight token.
May 18 2015, 12:56 PM

May 14 2015

cburroughs added a comment to rPb1b97dddc771: Merge branch 'master' into redesign-2015.

Tangential question: For the rare public-long-lived-topic-branch why merge over rebase?

May 14 2015, 7:11 PM
cburroughs added a comment to T8135: drag & drop on a workboard sorted by priority behavior is surprising.

Broadly the way forward make sense to me.

May 14 2015, 1:55 PM · Restricted Project, Workboards (v3), Restricted Project
cburroughs created T8196: cburrough's total normal ticket.
May 14 2015, 1:41 PM · Restricted Project
cburroughs added a comment to T8135: drag & drop on a workboard sorted by priority behavior is surprising.

I tried carefully dragging a Low priority task from column A to the bottom of column B where the lowest priority so far was Normal. This still changed the priority from Low to Normal.

May 14 2015, 12:31 PM · Restricted Project, Workboards (v3), Restricted Project

May 13 2015

cburroughs updated the diff for D10610: A batch editable page for viewing a single board column.
  • adapt for changes in D12526
  • handle some quicksand stuff I do not understand from D12599
May 13 2015, 7:25 PM
flash awarded T8181: undefined method 'year' when trying to spy on epriestley a Like token.
May 13 2015, 6:56 PM · Calendar
cburroughs planned changes to D10610: A batch editable page for viewing a single board column.

Whoops, need to account for D12526.

May 13 2015, 5:20 PM
cburroughs created T8181: undefined method 'year' when trying to spy on epriestley.
May 13 2015, 5:13 PM · Calendar

May 12 2015

cburroughs updated the diff for D10610: A batch editable page for viewing a single board column.
  • rebase
May 12 2015, 9:05 PM
cburroughs updated the diff for D10289: support phids in repository.query.
  • rebase
May 12 2015, 8:58 PM
cburroughs updated the diff for D10565: Make "Active" the default query for People.
  • rebase
May 12 2015, 7:58 PM
cburroughs added a comment to T5132: Document the special syntax you can use in commit messages to cause effects.

This might be nice to move to the new-ish help dropdown.

May 12 2015, 3:34 PM · Guides, Diffusion

May 11 2015

cburroughs added a comment to T8135: drag & drop on a workboard sorted by priority behavior is surprising.

Huh weird, our desires for what this feature would do are just complete opposites.

May 11 2015, 9:51 PM · Unknown Object (Project), Workboards (v3), Restricted Project
cburroughs added a comment to T8151: Decide the fate of JX.Scrollbar.

With the durable column open, you'll get a double scrollbar. Hopefully, this isn't too visually confusing. If the double scrollbar is worse than the old stuff was, let me know.

May 11 2015, 7:36 PM · Conpherence, PHUI
cburroughs awarded T8151: Decide the fate of JX.Scrollbar a Love token.
May 11 2015, 5:45 PM · Conpherence, PHUI
cburroughs added a comment to T8151: Decide the fate of JX.Scrollbar.

(5) Stop framing page content. Use JX.Scrollbar only in Conpherence panels. Accept double scrollbar.

May 11 2015, 5:18 PM · Conpherence, PHUI
cburroughs added a comment to T8145: Support global/broadcast notifications.

See also: T5231

May 11 2015, 12:23 PM · Notifications
cburroughs added a comment to T8135: drag & drop on a workboard sorted by priority behavior is surprising.

See T5474. I'd expect this to get resolved as having this automagical prioritization trigger be made more explicit alongside the rest of T5474. I should go on the record that I love this behavior and wouldn't want to simply remove it. :D

May 11 2015, 12:22 PM · Unknown Object (Project), Workboards (v3), Restricted Project

May 8 2015

cburroughs created T8135: drag & drop on a workboard sorted by priority behavior is surprising.
May 8 2015, 10:12 PM · Unknown Object (Project), Workboards (v3), Restricted Project

May 7 2015

cburroughs added a comment to T8117: Choose the most straightforward possible name for a package management application.
  • ppm is already taken :-(
  • phump: I tried to figure out how many consecutive ps I could string together and started making a sound like this
  • pkg Somehow not taken yet, hurry!
May 7 2015, 7:56 PM
cburroughs added a comment to T8115: arcanist plugin system.

The use case I'm thinking of is mostly "Make sure my company's custom lint is installed/configured, when I have lots of repos and I don't control the user's setup".

May 7 2015, 7:52 PM · Arcanist
cburroughs awarded D12749: Allow Phabricator to parse bare revision IDs from "Differential Revision:" fields a Orange Medal token.
May 7 2015, 4:24 PM
cburroughs added inline comments to D12748: Allow Arcanist to parse bare revision IDs from "Differential Revisions:" field.
May 7 2015, 4:23 PM
cburroughs awarded T3820: Implement top-level "Spaces" that provide policy isolation to groups of objects a Pterodactyl token.
May 7 2015, 3:41 PM · Spaces, Policy, Wikimedia
cburroughs added a comment to T8110: project ACL dropdowns cause native & kappa twitch scrollbar to both appear.

Linux. I tried in FF 31 and Chrome 37

May 7 2015, 3:19 PM · PHUI, Policy
cburroughs created T8110: project ACL dropdowns cause native & kappa twitch scrollbar to both appear.
May 7 2015, 3:16 PM · PHUI, Policy

Apr 30 2015

cburroughs added a comment to T8003: Total vote count is one less than sum of all options - V27.

192162_5_.jpg (425×300 px, 28 KB)

Apr 30 2015, 5:57 PM · Slowvote
cburroughs awarded T7992: Design "Badges" or "Medals" a Pterodactyl token.
Apr 30 2015, 3:55 PM · Badges, Design, People

Apr 28 2015

cburroughs added a comment to T4131: Store LDAP domain as credential source for LDAP external accounts.

This escalation requires control of DNS/networking or something to point phabricator at the new LDAP server, correct?

Apr 28 2015, 2:01 PM · Auth, Security

Apr 24 2015

cburroughs added a comment to T3868: Implement a "Link" StandardCustomField.

FWIW It would be cool to have a list of links. We use a free text field for support cases (many external tickets:one internal ticket) which mostly working but sometimes people type random junk in the next block to celebrate their freedom.

Apr 24 2015, 4:38 PM · Maniphest, Custom Fields

Apr 23 2015

cburroughs claimed T7893: placing two memes side by side results in blank comment.

Not reproducing on this this install...

Apr 23 2015, 2:06 AM · Remarkup
cburroughs added a comment to T7893: placing two memes side by side results in blank comment.

Macro sfken:  Macro sfken:

Apr 23 2015, 2:04 AM · Remarkup
cburroughs created T7893: placing two memes side by side results in blank comment.
Apr 23 2015, 2:04 AM · Remarkup

Apr 20 2015

cburroughs added a comment to T5231: Allow Broadcasting on all channels.

I ran into this need when trying to cover all of my bases for an extended maintenance notification. We don't have an email list that definitely covers 'all users of phabricator' that isn't absurdly over-broad (like 'all employees'). I suspect most corporate users will run into a similar convoluted problem at some point.

Apr 20 2015, 10:51 PM · Notifications
cburroughs added a comment to D12475: Add a "Batch Edit Tasks..." action to workboard columns.

I think if this kept the filter & export buttons it would cover most of T5024 and obsolete my sketchy D10610 as well.

Apr 20 2015, 12:29 PM · Prioritized

Apr 17 2015

cburroughs added a comment to T7847: send message button on user profile appears even when Conpherence is disabled.
In T7847#107532, @chad wrote:

y u no like Conpherence.

Apr 17 2015, 8:03 PM · Conpherence
cburroughs created T7847: send message button on user profile appears even when Conpherence is disabled.
Apr 17 2015, 7:44 PM · Conpherence
cburroughs added a comment to T7484: Lock is not locked by this process.

Occurring on yet another repository after the latest rebase:

Apr 17 2015, 5:26 PM · Restricted Project, Daemons
cburroughs closed T7804: Implement an "Acting User" Herald condition as Invalid.

Okay figured it out from the mail headers and transcript. The user did indeed have a personal rule.

Apr 17 2015, 2:35 PM · Restricted Project, Herald, Mail
cburroughs added a comment to T7804: Implement an "Acting User" Herald condition.

So I think Herald rules are the culprit on our end, but I can't figure out how to fix it:

Apr 17 2015, 2:16 PM · Restricted Project, Herald, Mail

Apr 14 2015

cburroughs added a comment to T7804: Implement an "Acting User" Herald condition.

I've only confirmed one so far.

Apr 14 2015, 1:53 PM · Restricted Project, Herald, Mail

Apr 10 2015

cburroughs added a comment to T7804: Implement an "Acting User" Herald condition.

Not sure what would be needed to debug in this case.

Apr 10 2015, 10:57 PM · Restricted Project, Herald, Mail
cburroughs created T7804: Implement an "Acting User" Herald condition.
Apr 10 2015, 10:56 PM · Restricted Project, Herald, Mail
cburroughs added a comment to D12346: Remove Herald rule edit log.

FWIW The log has solved at least one 'what changed when' mystery for us.

Apr 10 2015, 6:09 PM

Apr 9 2015

cburroughs added a comment to T7780: Add linter for Foodcritic.

It's included in the 'chef development kit' now so almost everyone using chef has it. So at least as popular as the puppet linter ;-)

Apr 9 2015, 8:13 PM · Restricted Project, Lint
cburroughs added a comment to T7484: Lock is not locked by this process.

Spooky problem has not occurred in the few minutes since I rebased off of e0473aa70265a222d440c7e2e2680d2d1959ab20. Will continue to watch.

Apr 9 2015, 12:43 PM · Restricted Project, Daemons

Apr 8 2015

cburroughs added a comment to T7781: Make feed.http-hooks staff-editable or user-editable.

Unknown Object (Revision) / Unknown Object (Commit)

Apr 8 2015, 6:42 PM · Setup, Phacility
cburroughs added a comment to T7323: disabling fancy scrollbars workaround still breaks keyboard scrolling.

we'll need that when we eventually have an optional disable anyway.

Apr 8 2015, 6:36 PM · PHUI
cburroughs reopened T7484: Lock is not locked by this process as "Open".

We are seeing this again with both D12263 and D12253 applied.

Apr 8 2015, 6:20 PM · Restricted Project, Daemons

Apr 6 2015

cburroughs added a comment to T7484: Lock is not locked by this process.
Apr 6 2015, 4:16 PM · Restricted Project, Daemons
cburroughs updated the diff for D10738: Added RuboCop linter.
  • avoid calling setSeverity directly
  • handle all rubocop severity values
Apr 6 2015, 4:09 PM
cburroughs added inline comments to D11657: Add a ShellCheck linter.
Apr 6 2015, 4:01 PM
cburroughs planned changes to D10738: Added RuboCop linter.

Minor severity-mapping updates on the way.

Apr 6 2015, 12:17 PM

Apr 2 2015

cburroughs added a comment to D12256: Remove all checks for untracked files from Arcanist.

nevergiveup.png (590×584 px, 483 KB)

Apr 2 2015, 12:46 PM
cburroughs added a comment to T7484: Lock is not locked by this process.

No errors with that patch either...

Apr 2 2015, 12:31 PM · Restricted Project, Daemons
cburroughs added a comment to T7484: Lock is not locked by this process.
  • Shorten patch: Works
  • Spooky Patch: Works
  • Roll back to no patch: Confirmed still broken.
Apr 2 2015, 1:53 AM · Restricted Project, Daemons
cburroughs added a comment to T7484: Lock is not locked by this process.

With that patch the daemon log is clean and the repositories show recent updates.

Apr 2 2015, 1:04 AM · Restricted Project, Daemons
cburroughs added a comment to T7484: Lock is not locked by this process.
mysql> SELECT GET_LOCK('global:phabricator:PhabricatorRepositoryManagementUpdateWorkflow:HYDRA', 0);
+---------------------------------------------------------------------------------------+
| GET_LOCK('global:phabricator:PhabricatorRepositoryManagementUpdateWorkflow:HYDRA', 0) |
+---------------------------------------------------------------------------------------+
|                                                                                     1 |
+---------------------------------------------------------------------------------------+
Apr 2 2015, 12:45 AM · Restricted Project, Daemons
cburroughs added a comment to T7484: Lock is not locked by this process.

I'm not sure I find the new log message that much more different or illuminating.

Apr 2 2015, 12:40 AM · Restricted Project, Daemons
cburroughs added a comment to T7484: Lock is not locked by this process.

Updated log message with D12253 applied and daemons restarted

Apr 2 2015, 12:08 AM · Restricted Project, Daemons

Apr 1 2015

cburroughs added a comment to T7484: Lock is not locked by this process.
mysql> SELECT IS_USED_LOCK('global:phabricator:PhabricatorRepositoryManagementUpdateWorkflow:TEMPORALANOMALY');
+--------------------------------------------------------------------------------------------------+
| IS_USED_LOCK('global:phabricator:PhabricatorRepositoryManagementUpdateWorkflow:TEMPORALANOMALY') |
+--------------------------------------------------------------------------------------------------+
|                                                                                             NULL |
+--------------------------------------------------------------------------------------------------+
1 row in set (0.00 sec)
Apr 1 2015, 11:44 PM · Restricted Project, Daemons
cburroughs added a comment to T7484: Lock is not locked by this process.

The root cause is very likely to be a stray copy of the daemon running somewhere.

Apr 1 2015, 11:32 PM · Restricted Project, Daemons
cburroughs added a comment to T7484: Lock is not locked by this process.

I'm now seeing this on a second repository. After some digging it appears to be a magical mysql GET_LOCK and not something simple on the file system I can just clear.

Apr 1 2015, 6:53 PM · Restricted Project, Daemons

Mar 30 2015

cburroughs updated subscribers of D10738: Added RuboCop linter.
Mar 30 2015, 6:04 PM
cburroughs added a comment to T7484: Lock is not locked by this process.

I'm seeing this on a build from March 12. A repository stopped updating on Friday and the daemon logs have been printing out this message ever few minutes and it is still occuring on Monday. No change after phd restart. diffusion/edit says 'Initializing Working Copy Daemons are initializing the working copy.'

Mar 30 2015, 5:36 PM · Restricted Project, Daemons

Mar 17 2015

cburroughs added a comment to D10738: Added RuboCop linter.

rubocop.conf can be used to point at whichever configuration file you like.

Mar 17 2015, 3:25 PM
easyrasta awarded D10610: A batch editable page for viewing a single board column a Like token.
Mar 17 2015, 9:38 AM

Mar 13 2015

cburroughs added a comment to D11863: Add a utility class for getting system memory information.

/proc/meminfo is not cgroup aware so from LXC/Docker/whatever you will see the free memory of the physical host, not your little container.

Mar 13 2015, 12:42 AM

Mar 2 2015

cburroughs added a comment to T7408: Consider dropping support for older versions of PHP.

FWIW RHEL6 is on php 5.3.3.

Mar 2 2015, 4:00 PM · Restricted Project, Infrastructure

Feb 27 2015

cburroughs added a comment to D10072: trivial typo.

"You signed this document on Aug 19 2014, 2:08 PM."

Feb 27 2015, 11:55 PM

Feb 25 2015

cburroughs added a comment to T7351: Improve storage scalability in the cluster.

I have no idea if we are a typical corporate user or not, not if on-premise users will be like SaaS users. Anyway if it happens to be a useful example we have:

  • 2 GiB of files
  • 3 GiB of repos
  • mysql dump gzipped is 1.6 GiB
Feb 25 2015, 3:31 PM · Phacility

Feb 19 2015

cburroughs created T7323: disabling fancy scrollbars workaround still breaks keyboard scrolling.
Feb 19 2015, 1:43 PM · PHUI

Feb 18 2015

cburroughs added a comment to D10738: Added RuboCop linter.

@joshuaspence I think you already have the accepted flag set, but does this look good to you for realz now?

Feb 18 2015, 4:36 PM

Feb 11 2015

cburroughs updated the diff for D10738: Added RuboCop linter.
  • fancier pht games
  • color setting redundant with json format
  • idiomatic message chaining
  • goodbye confusing flag
Feb 11 2015, 7:58 PM
cburroughs added inline comments to D10738: Added RuboCop linter.
Feb 11 2015, 7:58 PM
cburroughs added a comment to T7166: Putting project to CCs as a condition of Herald Rule doesn't get triggered.

Is this the same as the herald based replacement for project mentions + email (T6819) ?

Feb 11 2015, 2:10 PM · Herald

Feb 10 2015

cburroughs retitled D11728: losen pep8 version check from to losen pep8 version check.
Feb 10 2015, 3:23 AM
cburroughs updated the diff for D10738: Added RuboCop linter.
  • updats for review feedback and changing test names
  • satisfy the linting gods
  • I updated the library map...
Feb 10 2015, 2:54 AM
cburroughs commandeered D10738: Added RuboCop linter.
Feb 10 2015, 2:48 AM
cburroughs created T7224: pep8 version check fails on two digit versions.
Feb 10 2015, 2:32 AM · Arcanist

Feb 9 2015

cburroughs added a comment to D10738: Added RuboCop linter.

@remon sorry to poke you again, are you still working on joshuaspence's inline comments? If not I can take a stab at commandeering.

Feb 9 2015, 9:02 PM

Feb 6 2015

cburroughs awarded T6819: Stop project mentions from auto-associating projects a Love token.
Feb 6 2015, 9:07 PM · Transactions, Projects

Feb 2 2015

cburroughs added a comment to T7106: Importing a repo with invalid credentials will cause a lot of retries in too short a period of time.

(Oh and we last rebased on upstream on 2015-01-07)

Feb 2 2015, 4:45 PM · Diffusion