Page MenuHomePhabricator
Feed Advanced Search

May 4 2016

epriestley created T10923: Diffusion API/URI Errata.
May 4 2016, 11:45 PM · Diffusion (v3)
epriestley closed T10366: General support for multiple URIs for a repository, a subtask of T4292: Implement repository replication, as Resolved.
May 4 2016, 11:32 PM · Restricted Project, Diffusion
epriestley closed T4039: Prevent a repository from being set to the same "local path" as some other repository as Resolved.

D15837 appears to have deployed cleanly here.

May 4 2016, 11:15 PM · Restricted Project, Diffusion
epriestley created T10922: Upgrading: New Repository UI, APIs, and URI management.
May 4 2016, 10:18 PM · Diffusion (v3), Installing & Upgrading

May 3 2016

epriestley added a revision to T4039: Prevent a repository from being set to the same "local path" as some other repository: Restricted Differential Revision.
May 3 2016, 7:40 PM · Restricted Project, Diffusion

May 2 2016

chad added projects to T10866: Missing Information in Diffusion User Guide: Repo Hosting: Documentation, Diffusion.
May 2 2016, 6:55 PM · Diffusion, Documentation, Bug Report
chad added a project to T10893: repo update fails for repos with 1000s of closing branches: Diffusion.
May 2 2016, 6:54 PM · Diffusion, Bug Report
epriestley merged task T4805: Make sure Conduit repositories get the "x created this repository" transaction into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
May 2 2016, 12:43 PM · Diffusion
epriestley added a comment to T4805: Make sure Conduit repositories get the "x created this repository" transaction.

I'm going to merge this into T10748: the modern diffusion.repository.edit endpoint it provides is the pathway forward here.

May 2 2016, 12:43 PM · Diffusion
fedegiova closed Q351: How to strip some mercurial commits as resolved.
May 2 2016, 7:31 AM · Diffusion, Mercurial

May 1 2016

epriestley merged task T10241: Modifying an externally-hosted repository's URL/path can unintentionally change the credentials into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
May 1 2016, 4:42 PM · Passphrase, Diffusion, Bug Report
epriestley added a comment to T10241: Modifying an externally-hosted repository's URL/path can unintentionally change the credentials.

I'm going to merge this into T10748. It should technically be fixed by D15829 on its own, but will be fixed even harder once we cut over to the changes in T10748.

May 1 2016, 4:42 PM · Passphrase, Diffusion, Bug Report
epriestley merged task T7221: External git repository SSH credentials are not optional into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
May 1 2016, 4:40 PM · Diffusion
epriestley added a comment to T7221: External git repository SSH credentials are not optional.

I'm going to merge this into T10748: this should be fixed by D15829, but won't go live until after we cut over to the new UI as part of the changes in T10748.

May 1 2016, 4:40 PM · Diffusion
epriestley added a revision to T7221: External git repository SSH credentials are not optional: Restricted Differential Revision.
May 1 2016, 4:39 PM · Diffusion
epriestley added a revision to T10241: Modifying an externally-hosted repository's URL/path can unintentionally change the credentials: Restricted Differential Revision.
May 1 2016, 4:39 PM · Passphrase, Diffusion, Bug Report

Apr 30 2016

avivey added a project to T10900: Repository content search with `hg grep` returns matches found in old versions of files: Mercurial.
Apr 30 2016, 6:15 PM · Mercurial, Diffusion

Apr 29 2016

rftfaria added a comment to T9213: Unable to set up a GIT hosted repository.

@storrgie, since it was a test environment, I disabled it

Apr 29 2016, 6:22 PM · Diffusion
frederic.marbach created T10900: Repository content search with `hg grep` returns matches found in old versions of files.
Apr 29 2016, 5:08 PM · Mercurial, Diffusion
epriestley added a comment to T7789: Support Git Large File Storage.

Great, I'm glad to hear things are working for you. Thanks for your help testing this out!

Apr 29 2016, 1:14 AM · Customer Impact, Restricted Project, Files, Diffusion

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
fooishbar updated the task description for T10742: PhabricatorRepositoryCommitParserWorker fails on deleted commits.
Apr 28 2016, 10:02 AM · Diffusion, Bug Report

Apr 27 2016

epriestley merged task T9679: Maybe rename "Import an Existing External Repository" in New Repo Wizard into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 10:46 PM · Diffusion
epriestley added a comment to T9679: Maybe rename "Import an Existing External Repository" in New Repo Wizard.

I'm going to merge this into T10748, which is now more clearly the way forward.

Apr 27 2016, 10:46 PM · Diffusion
epriestley merged task T9604: Creating an imported repository via Conduit results in filesystem error into T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.
Apr 27 2016, 1:58 PM · Restricted Project, Phacility Support, Diffusion, Conduit
epriestley added a comment to T9604: Creating an imported repository via Conduit results in filesystem error.

I'm going to merge this into T10748, since that's now clearly the way forward.

Apr 27 2016, 1:57 PM · Restricted Project, Phacility Support, Diffusion, Conduit
epriestley added a comment to T10883: Allow repository cluster nodes to be read-only.

See also T10884 for related work: ranking similar nodes more intelligently.

Apr 27 2016, 11:17 AM · Restricted Project, Diffusion, Clusters
epriestley created T10883: Allow repository cluster nodes to be read-only.
Apr 27 2016, 11:13 AM · Restricted Project, Diffusion, Clusters

Apr 26 2016

avivey added a subtask for T9287: Working with many repositories: T10874: Add repository or project to email subject.
Apr 26 2016, 5:58 PM · Restricted Project, Restricted Project, Diffusion
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 26 2016, 2:47 AM · Restricted Project, Diffusion

Apr 25 2016

eadler moved T5029: Fill in missing context for low-context diffs against tracked repositories from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 25 2016, 6:02 PM · KDE, Restricted Project, Diffusion, Differential
epriestley closed T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too as Resolved by committing Restricted Diffusion Commit.
Apr 25 2016, 3:16 PM · Diffusion, Documentation
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 25 2016, 1:38 PM · Restricted Project, Diffusion
epriestley added a comment to T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too.

This also reproduced locally for me (OS X Yosemite, openssh-6.7p1), and chmod on the parent directory alone was reliably sufficient to prevent SSH from working. The instructions already suggest a root-owned location, but D15794 notes this requirement explicitly.

Apr 25 2016, 12:42 PM · Diffusion, Documentation
epriestley added a revision to T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too: Restricted Differential Revision.
Apr 25 2016, 12:40 PM · Diffusion, Documentation
epriestley claimed T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too.
Apr 25 2016, 12:33 PM · Diffusion, Documentation
epriestley added a comment to T5463: Could not stat AuthorizedKeysCommand .

I can confirm T9560 locally, at least, so that might be the problem or part of it -- maybe start there? If you can provide details we're happy to fix this properly.

Apr 25 2016, 12:32 PM · Diffusion
florianb added a comment to T5463: Could not stat AuthorizedKeysCommand .

Thanks @epriestley, i will try to investigate further. It seems that i have still something misconfigured. I'll get back as soon as i have to add something useful. :-)

Apr 25 2016, 12:31 PM · Diffusion
epriestley added a comment to T5463: Could not stat AuthorizedKeysCommand .

Yes, you should NOT set AuthorizedKeysCommandUser root, this is needlessly dangerous and gives the script substantially more permission than it needs.

Apr 25 2016, 12:28 PM · Diffusion
florianb added a comment to T5463: Could not stat AuthorizedKeysCommand .

I ran into the same issue, it turned out, that the permissions for /phabricator, /libphutil and /phabricator as well as for the preceding parent-directory had to be 755.

Apr 25 2016, 12:21 PM · Diffusion

Apr 24 2016

epriestley lowered the priority of T10860: After an inconsistent cluster repository write, consider just ignoring the lock from Normal to Low.

After we receive a push, try very very very very hard to release the lock if we're the thread holding it, even if we lost the global lock

Apr 24 2016, 6:39 PM · Clusters, Diffusion
epriestley added a revision to T10860: After an inconsistent cluster repository write, consider just ignoring the lock: Restricted Differential Revision.
Apr 24 2016, 5:59 PM · Clusters, Diffusion
epriestley added a revision to T10860: After an inconsistent cluster repository write, consider just ignoring the lock: Restricted Differential Revision.
Apr 24 2016, 4:58 PM · Clusters, Diffusion
epriestley added a revision to T10860: After an inconsistent cluster repository write, consider just ignoring the lock: Restricted Differential Revision.
Apr 24 2016, 4:07 PM · Clusters, Diffusion
epriestley added a revision to T10860: After an inconsistent cluster repository write, consider just ignoring the lock: Restricted Differential Revision.
Apr 24 2016, 3:33 PM · Clusters, Diffusion

Apr 23 2016

epriestley created T10861: Provide a tool to rewind the push log for a repository.
Apr 23 2016, 1:42 PM · Diffusion
epriestley created T10860: After an inconsistent cluster repository write, consider just ignoring the lock.
Apr 23 2016, 1:06 PM · Clusters, Diffusion

Apr 22 2016

epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 22 2016, 12:41 PM · Restricted Project, Diffusion
salvian added a comment to T7789: Support Git Large File Storage.

Thanks!
Yes, we will consider setting up alternate file domain

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

I think D15784 should fix the issue. It should land in master later today and be available in stable in about 24 hours. Let me know if you're still seeing issues once you update past that.

Apr 22 2016, 11:47 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a revision to T7789: Support Git Large File Storage: Restricted Differential Revision.
Apr 22 2016, 11:45 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a comment to T7789: Support Git Large File Storage.

I think I know what's going on, although I'm unsure why the <2MB files would work, unless you happened to push those to a public ("Visible To: Public (No Login Required)") repository and pushed the large files to a private repository?

Apr 22 2016, 11:17 AM · Customer Impact, Restricted Project, Files, Diffusion
salvian added a comment to T7789: Support Git Large File Storage.
  1. It's not configured
  2. Yes
  3. I'm not sure, these are my install versions (stable branch) :
    • phabricator f75b1cf562c0c3646324864851d693ef1069a068 (Mon, Apr 18)
    • arcanist 01b6fe8bb0239a4bce03f58d6288a3a52ad83a91 (Sat, Apr 16)
    • phutil b8c65df2a910a7028d39bec602d181428b6ce01f (Sat, Apr 16)

(edit, it has rP37b93f4)

Apr 22 2016, 11:13 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a comment to T7789: Support Git Large File Storage.
  • Is security.alternate-file-domain configured on your install?
  • When you say "redirect to the correct file", do you mean you get taken to the file detail page /F1234?
  • Also, do you have rP37b93f4 locally (April 7)? (You can grab the hashes in Config → Versions if you aren't immediately sure and I can figure it out from there.)
Apr 22 2016, 11:08 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 22 2016, 11:03 AM · Restricted Project, Diffusion
salvian added a comment to T7789: Support Git Large File Storage.

Ah, is the auth token (first [shortened]) same for every user? It is in my case

  1. If I visit the URI while being logged in to the app, the browser redirects to the correct file, the download link works
  2. The download URI is the same as the logged URI
  3. Visiting the URI with private window returns normal 404 page
Apr 22 2016, 11:03 AM · Customer Impact, Restricted Project, Files, Diffusion
epriestley added a comment to T7789: Support Git Large File Storage.

You shouldn't need to configure diffusion.allow-http-auth -- the auth token is basically the first [shortened] in the URI, so that appears to be working properly.

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

I'm using S3 as Files storage backend, phabricator runs behind nginx, and the diffusion is hosted over SSH (diffusion.allow-http-auth=false, HTTP is not configured anyway). I'm having an issue:

  • pushing and pulling small files (<2MB) tracked as LFS -> works flawlessly
  • pushing 11x 1 GB files works, but can't do git clone / git lfs fetch (got the same issue as @benjumanji, the lfs_trace log is similar, 404 errors when getting the lfs-* files)
  • pushing a 11GB file, doesn't work. forgot the error message, will add it later if needed
Apr 22 2016, 9:03 AM · Customer Impact, Restricted Project, Files, Diffusion
Jacksgong added a comment to T5132: Document the special syntax you can use in commit messages to cause effects.

Very Useful!

Apr 22 2016, 4:14 AM · Guides, Diffusion
eadler updated the task description for T5132: Document the special syntax you can use in commit messages to cause effects.
Apr 22 2016, 4:12 AM · Guides, Diffusion

Apr 21 2016

chad closed T6803: Diffusion: sudo rules documentation refinement as Wontfix.
Apr 21 2016, 7:13 PM · Diffusion, Documentation
vinzent added a comment to T6803: Diffusion: sudo rules documentation refinement.

so would you mind closing the task? I can't do it myself.

Apr 21 2016, 7:12 PM · Diffusion, Documentation
chad added a comment to T6803: Diffusion: sudo rules documentation refinement.

I have no idea if we plan to add it or not. See Planning for how we prioritize time.

Apr 21 2016, 7:11 PM · Diffusion, Documentation
eadler added a project to T5029: Fill in missing context for low-context diffs against tracked repositories: Restricted Project.
Apr 21 2016, 7:07 PM · KDE, Restricted Project, Diffusion, Differential
vinzent added a comment to T6803: Diffusion: sudo rules documentation refinement.

If you don't plan to add it: just reject the task - no problem. I just wanted to let you guys no that there is a slightly more secure way of doing what you provide to all the novice users out there.

Apr 21 2016, 7:01 PM · Diffusion, Documentation
epriestley added a comment to T4292: Implement repository replication.

The diffusion.querycommits method needs to sync-before-read (at least, if bypassCache is provided?) but currently does not. This can lead to tasks failing on the daemon on an un-synchronized node. Things self-heal, but it would be nice to prevent this.

Apr 21 2016, 6:59 PM · Restricted Project, Diffusion
avivey added a subtask for T9287: Working with many repositories: T10850: Search revisions by "Repository Projects".
Apr 21 2016, 6:44 PM · Restricted Project, Restricted Project, Diffusion
chad added a comment to T6803: Diffusion: sudo rules documentation refinement.

Accepting contributions isn't free to us. We bear the cost of testing, accepting, documenting, and providing lifetime support. See https://secure.phabricator.com/book/phabcontrib/article/contributing_code/ for more details.

Apr 21 2016, 3:44 PM · Diffusion, Documentation
vinzent added a comment to T6803: Diffusion: sudo rules documentation refinement.

this still applies 2 years after sending a diff. is there missing something?

Apr 21 2016, 2:31 PM · Diffusion, Documentation

Apr 20 2016

epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 20 2016, 7:32 PM · Restricted Project, Diffusion
cburroughs moved T8238: Formally support side-band change handoff in external repositories from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 20 2016, 4:02 PM · Restricted Project, Restricted Project, Diffusion, Harbormaster
cburroughs added a project to T8238: Formally support side-band change handoff in external repositories: Restricted Project.
Apr 20 2016, 4:02 PM · Restricted Project, Restricted Project, Diffusion, Harbormaster
cburroughs moved T9210: break apart Publish/Notify repo setting from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 20 2016, 3:57 PM · Restricted Project, Restricted Project, Diffusion
cburroughs added a project to T9210: break apart Publish/Notify repo setting: Restricted Project.
Apr 20 2016, 3:12 PM · Restricted Project, Restricted Project, Diffusion
epriestley added a comment to T8936: Unable to identify the reftype of 'refs/remotes/origin/master'. Rejecting push..

This is probably ultimately the same as T6878 -- T6878#167273 has the current plans here.

Apr 20 2016, 1:52 PM · Diffusion
epriestley updated subscribers of T8936: Unable to identify the reftype of 'refs/remotes/origin/master'. Rejecting push..
Apr 20 2016, 1:51 PM · Diffusion

Apr 19 2016

eadler added a subtask for T4292: Implement repository replication: T10366: General support for multiple URIs for a repository.
Apr 19 2016, 9:17 PM · Restricted Project, Diffusion
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 19 2016, 7:37 PM · Restricted Project, Diffusion
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 19 2016, 5:13 PM · Restricted Project, Diffusion
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 19 2016, 5:02 PM · Restricted Project, Diffusion
epriestley updated subscribers of T10691: Support GitHub-like forking of repositories.
Apr 19 2016, 2:56 PM · Restricted Project, Diffusion, Feature Request
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 19 2016, 2:49 PM · Restricted Project, Diffusion
epriestley renamed T5701: Allow custom pygments styles from Allow custom pygyments styles to Allow custom pygments styles.
Apr 19 2016, 2:05 PM · libphutil, Diffusion
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 19 2016, 1:28 PM · Restricted Project, Diffusion
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 19 2016, 1:03 PM · Restricted Project, Diffusion
eadler added a revision to T10326: Property Lists use "Projects", not "Tags": Restricted Differential Revision.
Apr 19 2016, 6:26 AM · Projects, Diffusion, Bug Report
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 19 2016, 12:16 AM · Restricted Project, Diffusion

Apr 18 2016

epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 18 2016, 3:30 PM · Restricted Project, Diffusion
epriestley added a revision to T4292: Implement repository replication: Restricted Differential Revision.
Apr 18 2016, 3:20 PM · Restricted Project, Diffusion

Apr 17 2016

eadler moved T10767: fold empty directories in Browse view from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 17 2016, 6:26 PM · Restricted Project, Diffusion, Feature Request
eadler moved T491: Inline Comments for Paste / Diffusion Files from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 17 2016, 6:25 PM · Restricted Project, Paste, Diffusion

Apr 16 2016

avivey renamed T10826: Config becomes inaccesible after changing visibility when all spaces are archived from Config becomes inaccesible after changing visibility to Config becomes inaccesible after changing visibility when all spaces are archived.
Apr 16 2016, 8:00 PM · Spaces, Bug Report
avivey added a project to T7022: diffusion - doesn't understand svn Replace operation: Bug Report.
Apr 16 2016, 7:42 PM · Bug Report, Diffusion, Subversion
• sshannin added a comment to T7022: diffusion - doesn't understand svn Replace operation.

I suspect it's due to the replace status code. Specifically, if a specific path is removed + added, it shows as different than a modify. When this is then copied, it looks like SVN is showing this code for the new path of the file too. See /rSVNTEST8.

Apr 16 2016, 7:41 PM · Bug Report, Diffusion, Subversion
JustHev added a comment to T10826: Config becomes inaccesible after changing visibility when all spaces are archived.

Turns out that because all spaces were archived, repositories did not get an space assigned to them, explaining the error. After I re-enabled S1, I saw a box for selecting the space appear.

Apr 16 2016, 6:48 PM · Spaces, Bug Report
chad added a comment to T10826: Config becomes inaccesible after changing visibility when all spaces are archived.

Yeah we should fix that, for sure. Thanks for digging into this.

Apr 16 2016, 6:46 PM · Spaces, Bug Report
JustHev added a comment to T10826: Config becomes inaccesible after changing visibility when all spaces are archived.

Ok, Whoops. I toyed with it a bit earlier and created a space, then had the balls to archive it. I unarchived it and everything starts working again: Repositories get shown again in the Diffusion menu, old broken repos stay broken, but new repos can be edited just fine.

Apr 16 2016, 6:42 PM · Spaces, Bug Report
chad added a comment to T10826: Config becomes inaccesible after changing visibility when all spaces are archived.

Have you set up Spaces?

Apr 16 2016, 6:16 PM · Spaces, Bug Report
JustHev updated the task description for T10826: Config becomes inaccesible after changing visibility when all spaces are archived.
Apr 16 2016, 6:04 PM · Spaces, Bug Report

Apr 15 2016

jdickson added a comment to T9548: Support Mercurial's bundle2 wire protocol.

I've been looking at this the last couple of days as it's a pain point for me -- I'd like to host my repositories on Diffusion, and I would like to stay with Mercurial rather than git..

Apr 15 2016, 3:05 PM · Diffusion, Mercurial