Page MenuHomePhabricator

benjumanji (Benjamin Edwards)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Tuesday

  • Clear sailing ahead.

User Details

User Since
Sep 29 2014, 10:11 PM (533 w, 5 d)
Availability
Available

Recent Activity

Jul 3 2017

benjumanji added a comment to Q630: I'd like to use the snazzy eye icon for a mattermost webhook, but it's served rather indirectly. (Answer 553).

That's what I suspected (I did find the rsrc link, but it seemed a bit suspect, and my default setup doesn't serve it anyway). Thanks!

Jul 3 2017, 8:23 AM
benjumanji closed Q630: I'd like to use the snazzy eye icon for a mattermost webhook, but it's served rather indirectly. as resolved.
Jul 3 2017, 8:22 AM

Jun 27 2017

benjumanji asked Q630: I'd like to use the snazzy eye icon for a mattermost webhook, but it's served rather indirectly..
Jun 27 2017, 12:53 PM
benjumanji awarded T12873: Ability to display custom message during externally linked login prompt a Love token.
Jun 27 2017, 9:28 AM · Feature Request

Nov 19 2016

benjumanji added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

Great thanks!

Nov 19 2016, 6:13 PM · Bug Report
benjumanji added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

Fixed and I'm back up and running, thanks!

Nov 19 2016, 4:19 PM · Bug Report
benjumanji added a comment to T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.

I can confirm that SHOW TABLES IN phabricator_xhpast works just fine as root mysql user, and dropping the credentials from the upgrade command yields exactly the same results as running under root. So that seems like it all ties up. I can work around this any number of ways and I will do just that.

Nov 19 2016, 3:55 PM · Bug Report
benjumanji changed the visibility for F1931085: trace.txt.
Nov 19 2016, 12:32 PM
benjumanji updated the task description for T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.
Nov 19 2016, 12:30 PM · Bug Report
benjumanji updated the task description for T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.
Nov 19 2016, 12:29 PM · Bug Report
benjumanji created T11893: storage upgrade claims it lacks permissions, but is running as root mysql user.
Nov 19 2016, 12:29 PM · Bug Report

Sep 1 2016

benjumanji added a comment to T10456: Conduit interprets all arguments from curl requests as strings.
params="                                                                                                                                                                                   
    {                                                                                                                                                                                      
        \"buildTargetPHID\": \"$PHID\",                                                                                                                                                    
        \"artifactKey\": \"jenkins.uri.$PHID\",                                                                                                                                            
        \"artifactType\": \"uri\",                                                                                                                                                         
        \"artifactData\": {                                                                                                                                                                
            \"uri\": \"http://build:8080/job/$JOB_NAME/$BUILD_NUMBER\",                                                                                                                    
            \"name\": \"Jenkins Build\",                                                                                                                                                   
            \"ui.external\": true                                                                                                                                                          
        },                                                                                                                                                                                 
        \"__conduit__\": { \"token\": \"$CONDUIT_API_TOKEN\" }                                                                                                                             
    }"
Sep 1 2016, 9:25 AM · Restricted Project, Conduit, Bug Report

Aug 31 2016

donaldguy awarded T10456: Conduit interprets all arguments from curl requests as strings a Dislike token.
Aug 31 2016, 11:24 PM · Restricted Project, Conduit, Bug Report

Jul 20 2016

benjumanji added a comment to Q447: Herald rules don't trigger on differential 'update'.

I won't comment on the specifics, but have you looked at the herald transcript? I normally find the answer hiding in there.

Jul 20 2016, 10:10 AM · Herald

Jul 13 2016

benjumanji updated Q442: Possible to save a custom workboard filter with a name? from to Possible to save a custom workboard filter with a name?.
Jul 13 2016, 10:19 AM

Jul 5 2016

benjumanji added a comment to Q437: How do I debug an ldap issue? (Answer 414).

That did the trick, thanks!

Jul 5 2016, 9:13 AM
benjumanji closed Q437: How do I debug an ldap issue? as resolved.
Jul 5 2016, 9:13 AM

Jul 4 2016

benjumanji updated Q437: How do I debug an ldap issue? from to How do I debug an ldap issue?.
Jul 4 2016, 1:55 PM

May 20 2016

benjumanji added a comment to T5000: Using Differential with plain Git, without requiring Arc.

If this is the outcome of you using arc land then you are doing it wrong. Just flick the history to immutable if you don't want it squashed.

May 20 2016, 4:25 PM · KDE, FreeBSD, Restricted Project, Haskell.org, Arcanist, Differential
benjumanji added a comment to T4214: Allow installs to change or supplement the "Phabricator" icon.

Would definitely like the ability to swap out the eye via config à la wikimedia install.

May 20 2016, 12:07 PM · Design
benjumanji awarded T4214: Allow installs to change or supplement the "Phabricator" icon a Love token.
May 20 2016, 12:05 PM · Design

May 6 2016

benjumanji closed Q384: Where does one configure arcanist commit rules as resolved.
May 6 2016, 5:00 PM
benjumanji added a comment to Q384: Where does one configure arcanist commit rules (Answer 375).

Ahh! It's right there on the bloody page. Sorry.

May 6 2016, 5:00 PM
benjumanji updated the question details for Q384: Where does one configure arcanist commit rules.
May 6 2016, 4:11 PM
benjumanji updated Q384: Where does one configure arcanist commit rules from to Where does one configure arcanist commit rules.
May 6 2016, 4:11 PM

May 4 2016

benjumanji awarded T10917: Build Glorious High-Security Notification/Audit Infrastructure a Piece of Eight token.
May 4 2016, 10:06 AM · Auth, Prioritized, Security, Feature Request

May 3 2016

benjumanji added a comment to T10914: Allow arcanist to use immutable or mutable history on a branch by branch basis.

Yeah, I'd like to encourage people to work like that (review each change to the shared branch and then land it to the shared branch) but just getting people to use arc "normally" is proving more than some people can bear (I recently pushed a us onto git from svn, the transition has not been without pain for some). I hadn't considering commiting then reverting the change to .arcconfig.

May 3 2016, 4:41 PM · Arcanist, Feature Request
benjumanji added a comment to T10914: Allow arcanist to use immutable or mutable history on a branch by branch basis.

That would work for us, yes.

May 3 2016, 4:17 PM · Arcanist, Feature Request
benjumanji updated the task description for T10914: Allow arcanist to use immutable or mutable history on a branch by branch basis.
May 3 2016, 4:13 PM · Arcanist, Feature Request
benjumanji created T10914: Allow arcanist to use immutable or mutable history on a branch by branch basis.
May 3 2016, 4:12 PM · Arcanist, Feature Request

Apr 29 2016

benjumanji added a comment to T10898: testing.

Right at the top of the home page...

Apr 29 2016, 8:59 AM · Feature Request

Apr 28 2016

benjumanji added a comment to T7789: Support Git Large File Storage.

Can confirm that on latest stable my lfs-test repository now succeeds with both jars and jpegs. Huzzah!

Apr 28 2016, 3:35 PM · Customer Impact, Restricted Project, Files, Diffusion

Apr 26 2016

benjumanji added a comment to T10879: Daily TODO list.
  1. make a private project
  2. add tasks you like to it
  3. ???
  4. profit
Apr 26 2016, 3:19 PM · Feature Request

Apr 19 2016

benjumanji added a comment to T10837: Arc patch update.

@anthony-verge just edit the repo, scroll down, branch stuff is somewhere around there.

Apr 19 2016, 2:51 PM · Feature Request
benjumanji added a comment to T10837: Arc patch update.

I'm not sure if this solves your problem, but just switch off autoclose for some prefix? We just have autoclose on master, as we don't consider anything on any branch that isn't master as done.

Apr 19 2016, 2:25 PM · Feature Request

Apr 14 2016

benjumanji added a comment to T10807: CI setup blocked by commit import bug?.

You don't need any plugins with staging. Just set up a parameterised build, use a parameter for the commit and call directly from harbormaster. That's reason for staging repos, regular git tooling can work with them.

Apr 14 2016, 9:47 PM · Bug Report
benjumanji added a comment to T10807: CI setup blocked by commit import bug?.

I am not sure exactly what you are trying to achieve, but if you want to do CI with differentials, just set up a herald rule to trigger a build plan when a diff is created / updated. This will trigger whether the staging commits are imported by phabricator or not.

Apr 14 2016, 2:23 PM · Bug Report
benjumanji updated Q364: Is it possible to create a herald to reject non-fastforward pushes? from Is it possible to teach herald to reject non-fastforward pushes? to Is it possible to create a herald to reject non-fastforward pushes?.
Apr 14 2016, 2:16 PM · Herald
benjumanji updated Q364: Is it possible to create a herald to reject non-fastforward pushes? from to Is it possible to teach herald to reject non-fastforward pushes?.
Apr 14 2016, 2:16 PM · Herald
benjumanji added Q359: HTTP repository serving (Answer 359).
Apr 14 2016, 10:47 AM

Apr 13 2016

benjumanji closed Q362: LDAP credential caching as resolved.
Apr 13 2016, 2:43 PM · Auth
benjumanji added a comment to Q362: LDAP credential caching (Answer 358).

I'm not worried about immediate logout. I agree that the effort / reward tradeoff is all wrong there. I was more thinking about the case where you are setup for anonymous search. Every X a check is performed to see if users still exist.

Apr 13 2016, 2:43 PM
benjumanji updated Q362: LDAP credential caching from to LDAP credential caching.
Apr 13 2016, 1:24 PM · Auth

Apr 8 2016

benjumanji added a comment to T10749: Add the Herald rule condition "when the revision has been closed".

Interesting. Time to update my bot. Thanks!

Apr 8 2016, 2:23 PM · Restricted Project, Herald, Harbormaster, Feature Request
benjumanji added a comment to T10749: Add the Herald rule condition "when the revision has been closed".

I have build kind of thing out myself, using polling to check what revisions are open, then diff my list of "envs / build jobs" that I have running and then make the two line up. It would nice if it could be event based and triggered by herald.

Apr 8 2016, 11:10 AM · Restricted Project, Herald, Harbormaster, Feature Request
benjumanji awarded T10749: Add the Herald rule condition "when the revision has been closed" a Like token.
Apr 8 2016, 11:07 AM · Restricted Project, Herald, Harbormaster, Feature Request

Apr 7 2016

benjumanji awarded T10742: PhabricatorRepositoryCommitParserWorker fails on deleted commits a The World Burns token.
Apr 7 2016, 12:24 PM · Diffusion, Bug Report
benjumanji awarded T10742: PhabricatorRepositoryCommitParserWorker fails on deleted commits a Like token.
Apr 7 2016, 12:24 PM · Diffusion, Bug Report
benjumanji added a comment to T10742: PhabricatorRepositoryCommitParserWorker fails on deleted commits.

I've triggered similar problems running reparse on a repository where branches had been deleted that were tracked.

Apr 7 2016, 12:24 PM · Diffusion, Bug Report
benjumanji awarded T5187: Make "Upload File" dialog have a vanilla file upload control a Like token.
Apr 7 2016, 11:53 AM · Prioritized, FreeBSD, User Delight, Remarkup, Files, Wikimedia
benjumanji changed the visibility for F1209330: lfs_trace.log.
Apr 7 2016, 11:26 AM
benjumanji added a comment to T7789: Support Git Large File Storage.

Unfortunately, no dice. I removed the jar file, and added a jpeg. I have attached a trace of running GIT_TRACE=1 git lfs fetch origin.

Apr 7 2016, 11:26 AM · Customer Impact, Restricted Project, Files, Diffusion
benjumanji added a comment to T7789: Support Git Large File Storage.

Can confirm that navigating to the file in diffusion will let me download it, and it really does give me back the right file.

Apr 7 2016, 10:59 AM · Customer Impact, Restricted Project, Files, Diffusion
benjumanji added a comment to T7789: Support Git Large File Storage.

As an addendum, I can definitely find the zip file on the box...

Apr 7 2016, 10:34 AM · Customer Impact, Restricted Project, Files, Diffusion
benjumanji added a comment to T7789: Support Git Large File Storage.

I'm on latest stable (fea2389), and I just tried the following

$ git clone ssh://git@phabricator.example.com:222/diffusion/18/lfs-test.git                                                                     [10:54:36]
Cloning into 'lfs-test'...
warning: You appear to have cloned an empty repository.
Checking connectivity... done.                                                                                                                       
$ cd lfs-test/                                                                                                                                  [10:56:39]
$ git lfs install                                                                                                                               [10:56:42]
Updated pre-push hook.
Git LFS initialized.                                                                                                         
$ git lfs track "*.jar"                                                                                                                         [10:57:05]
Tracking *.jar                                                                                                  
$ cp ../some/path/antlr-3.2.jar .                                                                                                               [10:57:19]
$ ls                                                                                                                                            [10:57:52]
antlr-3.2.jar                                                                                                       
$ git add antlr-3.2.jar                                                                                                                         [10:57:54]
$ git commit -m "testing"                                                                                                                       [10:57:57]
[master (root-commit) 2690d02] testing
 1 file changed, 3 insertions(+)
 create mode 100644 antlr-3.2.jar
$ git push -u origin master                                                                                                                     [10:58:04]
Git LFS: (1 of 1 files) 1.84 MB / 1.84 MB                                                                                                                
Counting objects: 3, done.
Delta compression using up to 6 threads.
Compressing objects: 100% (2/2), done.
Writing objects: 100% (3/3), 329 bytes | 0 bytes/s, done.
Total 3 (delta 0), reused 0 (delta 0)
To ssh://git@phabricator.example.com:222/diffusion/18/lfs-test.git
 * [new branch]      master -> master
Branch master set up to track remote branch master from origin.

so far so good. however...

$ cd ..                                                                                                                                         [10:58:18]
$ rm -rf lfs-test/                                                                                                                              [10:59:45]
$ git clone ssh://git@phabricator.example.com:222/diffusion/18/lfs-test.git                                                                     [10:59:50]
Cloning into 'lfs-test'...
remote: Counting objects: 3, done.
remote: Compressing objects: 100% (2/2), done.
remote: Total 3 (delta 0), reused 0 (delta 0)
Receiving objects: 100% (3/3), done.
Checking connectivity... done.                                                                                                                     
$ cd lfs-test/                                                                                                                                  [11:00:15]
$ ls                                                                                                                                            [11:00:19]
antlr-3.2.jar
$ cat antlr-3.2.jar                                                                                                                             [11:00:45]
version https://git-lfs.github.com/spec/v1
oid sha256:4c8737014e7ca0d2c85171edf37f5a26b2d8d8237c283357b81a3269b6848d38
size 1928009                                                                                         
$ git lfs fetch origin                                                                                                                          [11:05:19]
Fetching master
Git LFS: (2 of 1 files) 0 B / 1.84 MB                                                                                                                    
Repository or object not found: http://phabricator.example.com/file/data/w5mihitt7ijile3b7n44/PHID-FILE-kbhl75cnfm6nwks7ojmf/dosd5iqevvn3rune/lfs-4c8737014e7ca0d2c85171edf37f5a26b2d8d8237c283357b81a3269b6848d38
Check that it exists and that you have proper access to it
Warning: errors occurred                 
Apr 7 2016, 10:24 AM · Customer Impact, Restricted Project, Files, Diffusion
benjumanji awarded T1816: Support downloading a compressed working copy (tar/zip) of a repository at a specific commit or tag a Like token.
Apr 7 2016, 9:44 AM · Drydock, Diffusion

Mar 7 2016

benjumanji awarded T6848: It should be possible to view a repository's policies without necessarily being able to edit it a Like token.
Mar 7 2016, 5:30 PM · Diffusion (v3)
benjumanji created T10534: No ability to view push permissions without edit rights..
Mar 7 2016, 5:00 PM · Diffusion, Feature Request

Mar 4 2016

benjumanji added a comment to Q331: Is it possible to reindex commit metadata? (Answer 339).

Perfect!

Mar 4 2016, 5:55 PM
benjumanji closed Q331: Is it possible to reindex commit metadata? as resolved.
Mar 4 2016, 5:55 PM · Diffusion
benjumanji updated Q331: Is it possible to reindex commit metadata? from to Is it possible to reindex commit metadata?.
Mar 4 2016, 11:50 AM · Diffusion
benjumanji closed Q326: Impossible to set boolean param with curl? as resolved.
Mar 4 2016, 11:05 AM · Conduit
benjumanji awarded T3794: Rebuild Diffusion/Audit on top of CustomField infrastructure a Like token.
Mar 4 2016, 9:23 AM · Restricted Project, Diffusion, Custom Fields

Feb 29 2016

benjumanji awarded Blog Post: Advanced Internet Usage (Experts ONLY!) a Mountain of Wealth token.
Feb 29 2016, 5:00 PM
benjumanji updated the question details for Q326: Impossible to set boolean param with curl? .
Feb 29 2016, 2:18 PM · Conduit
benjumanji added a comment to Q326: Impossible to set boolean param with curl? .
Feb 29 2016, 2:17 PM · Conduit

Feb 27 2016

totova awarded T10456: Conduit interprets all arguments from curl requests as strings a Heartbreak token.
Feb 27 2016, 10:29 AM · Restricted Project, Conduit, Bug Report

Feb 26 2016

benjumanji added a project to T10456: Conduit interprets all arguments from curl requests as strings: Conduit.
Feb 26 2016, 2:58 PM · Restricted Project, Conduit, Bug Report
benjumanji created T10456: Conduit interprets all arguments from curl requests as strings.
Feb 26 2016, 2:57 PM · Restricted Project, Conduit, Bug Report
benjumanji added a comment to Q326: Impossible to set boolean param with curl? (Answer 335).

Thanks!

Feb 26 2016, 2:53 PM
benjumanji added a comment to Q326: Impossible to set boolean param with curl? (Answer 335).

I tried that and it gives the same complaint.

Feb 26 2016, 2:50 PM
benjumanji updated the question details for Q326: Impossible to set boolean param with curl? .
Feb 26 2016, 2:50 PM · Conduit
benjumanji updated the question details for Q326: Impossible to set boolean param with curl? .
Feb 26 2016, 2:46 PM · Conduit
benjumanji updated Q326: Impossible to set boolean param with curl? from to Impossible to set boolean param with curl? .
Feb 26 2016, 2:46 PM · Conduit

Feb 17 2016

benjumanji closed Q313: Diffusion browsing broken on recently upgraded / migrated install as resolved.
Feb 17 2016, 4:52 PM
benjumanji added Q313: Diffusion browsing broken on recently upgraded / migrated install (Answer 331).
Feb 17 2016, 4:39 PM

Feb 16 2016

benjumanji added a comment to Q313: Diffusion browsing broken on recently upgraded / migrated install.

The other thing that's probably worth mentioning is that when I tried to create a new repo on the updated install, it has the same problems. Running the failing command in the repository root also succeeds, so it seems to be a paths thing... although the '.git' at the end of the command freaks me out, because the repos are bare, so there is no .git folder.

Feb 16 2016, 5:29 PM
benjumanji added a comment to Q313: Diffusion browsing broken on recently upgraded / migrated install.

With the results being the same either way. The previous install was @ 6a035d977f22aaf1 (I will admit this was a bit lazy.. that's from November of last year).

Feb 16 2016, 5:26 PM
benjumanji updated Q313: Diffusion browsing broken on recently upgraded / migrated install from to Diffusion browsing broken on recently upgraded / migrated install.
Feb 16 2016, 5:04 PM

Jul 1 2015

benjumanji awarded T8549: Feedback on 2015 Redesign a Love token.
Jul 1 2015, 4:37 PM · Design

Apr 2 2015

benjumanji added a comment to T4139: Add support for desktop notifications.

Thanks, I'll get on it.

Apr 2 2015, 8:04 PM · Aphlict
benjumanji added a comment to T4139: Add support for desktop notifications.

Very interested in this. Willing to contribute code if someone will have time to look at the patch (it doesn't have to be accepted, I just want to know that I am not stepping into something that is unlikely to get merged any time soon).

Apr 2 2015, 5:08 PM · Aphlict
benjumanji awarded T4139: Add support for desktop notifications a Like token.
Apr 2 2015, 5:01 PM · Aphlict

Mar 29 2015

benjumanji added a comment to D10297: Support Azure AD / Office 365 as an OAuth provider.

Thanks for the insight. I guess my interest is registered, and I'll probably try to keep a patch like this maintained locally, and keep watching to see if azure does anything to lower the testing barrier.

Mar 29 2015, 1:58 PM
benjumanji added a comment to D10297: Support Azure AD / Office 365 as an OAuth provider.

What assistance could be offered to help this along? Setting up a test azure as app with access for the phabricator team?

Mar 29 2015, 9:42 AM