Page MenuHomePhabricator
Feed Advanced Search

Feb 26 2016

epriestley removed 1 mock(s) for T10433: Repo in Diffusion has no padding after Branches box if footer is not configured: Restricted Pholio Mock.
Feb 26 2016, 1:55 PM · Diffusion, Bug Report
epriestley updated the task description for T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails.
Feb 26 2016, 1:54 PM · Phortune, Phacility
epriestley closed T10455: technology"... as Spite.
Feb 26 2016, 1:53 PM
• 9999999999 created T10455: technology"....
Feb 26 2016, 1:14 PM
• 9999999999 created T10454: represents-Thenth,.
Feb 26 2016, 1:10 PM
• 9999999999 added 1 mock(s) for T10450: Owner locked out of project: Restricted Pholio Mock.
Feb 26 2016, 1:07 PM · Projects, Bug Report
• 9999999999 added 1 mock(s) for T10433: Repo in Diffusion has no padding after Branches box if footer is not configured: Restricted Pholio Mock.
Feb 26 2016, 1:07 PM · Diffusion, Bug Report
• 9999999999 updated the task description for T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails.
Feb 26 2016, 11:56 AM · Phortune, Phacility
michaeloa added a comment to T10450: Owner locked out of project.

Awesome. That worked, thanks.

Feb 26 2016, 9:47 AM · Projects, Bug Report
chad closed T10453: How to import remote repository? as Invalid.

Please see Support Resources for how to get support using Phabricator. This doesn't appear to be a bug report.

Feb 26 2016, 7:17 AM · Bug Report
chorulex created T10453: How to import remote repository?.
Feb 26 2016, 7:05 AM · Bug Report

Feb 25 2016

epriestley added a comment to T10450: Owner locked out of project.

Subscribers were removed from projects, so this policy no longer functions.

Feb 25 2016, 4:31 PM · Projects, Bug Report
michaeloa created T10450: Owner locked out of project.
Feb 25 2016, 4:18 PM · Projects, Bug Report
clemvangelis added a project to T5229: Fixing up invalid encoding confuses archanists utf8-check: Bug Report.

I can confirm this behavior. Is there any way to fix that ?

Feb 25 2016, 11:34 AM · Bug Report, Arcanist

Feb 24 2016

Firehed closed T10420: Code coverage reports from `arc unit` are sometimes inaccurate as Resolved by committing Restricted Diffusion Commit.
Feb 24 2016, 8:58 PM · Arcanist, Bug Report
Firehed added a revision to T10420: Code coverage reports from `arc unit` are sometimes inaccurate: Restricted Differential Revision.
Feb 24 2016, 8:55 PM · Arcanist, Bug Report
epriestley added a comment to T10420: Code coverage reports from `arc unit` are sometimes inaccurate.

Sure, happy to bring those both upstream.

Feb 24 2016, 8:42 PM · Arcanist, Bug Report
Firehed added a comment to T10420: Code coverage reports from `arc unit` are sometimes inaccurate.

Implementation of what's described above - also seems to provide the expected results in my case, and limits impact to PHPUnit results. If a file is executed at all, it should show up normally; when completely untouched (which tends to show up with phpunit whitelisting, now necessary in the newest versions).

Feb 24 2016, 8:40 PM · Arcanist, Bug Report
Firehed added a comment to T10420: Code coverage reports from `arc unit` are sometimes inaccurate.

That seems totally reasonable. It seems like a bit of a stretch for PHP code specifically, but sane as a general-purpose answer.

Feb 24 2016, 5:36 PM · Arcanist, Bug Report
epriestley added a comment to T10446: Unable to simply mark a Diffusion comment as Done.

Ah, I missed "In Diffusion". Let me see if I can reproduce there.

Feb 24 2016, 4:58 PM · Audit, Bug Report
joshkel added a comment to T10446: Unable to simply mark a Diffusion comment as Done.

Your steps match mine. However, I'm not getting an "Action With No Effect" prompt; instead, the server acts like it's processing, and redisplays the page, but doesn't actually make any changes.

Feb 24 2016, 4:57 PM · Audit, Bug Report
epriestley added a comment to T10446: Unable to simply mark a Diffusion comment as Done.

I'm having trouble reproducing this -- I attempted to follow your steps, but didn't experience the issue. Here's what I did:

Feb 24 2016, 4:22 PM · Audit, Bug Report
joshkel created T10446: Unable to simply mark a Diffusion comment as Done.
Feb 24 2016, 4:11 PM · Audit, Bug Report
mavit added a comment to T10300: On branch "xyz", "arc land --onto xyz abc" does not result in update to "xyz" even when "xyz" is not ahead of the remote.

This has tripped me up a few times (maybe since arc was made great again?) and I think it happens when you do arc land foo instead of arc land in foo.

Feb 24 2016, 2:41 PM · Arcanist
• Persona created T10442: What should code review look like to the reviewer?.
Feb 24 2016, 11:30 AM
• Persona created T10440: Sample Process.
Feb 24 2016, 11:21 AM
chad closed T10435: hello as Invalid.
Feb 24 2016, 7:15 AM · Bug Report
• faridtechm created T10435: hello.
Feb 24 2016, 7:14 AM · Bug Report
epriestley added a comment to T10433: Repo in Diffusion has no padding after Branches box if footer is not configured.

I guess "off" is default, so "disabled" isn't quite accurate.

Feb 24 2016, 1:30 AM · Diffusion, Bug Report
epriestley renamed T10433: Repo in Diffusion has no padding after Branches box if footer is not configured from Repo in Diffusion has no padding after Branches box if footer is disabled to Repo in Diffusion has no padding after Branches box if footer is not configured.
Feb 24 2016, 1:30 AM · Diffusion, Bug Report
epriestley renamed T10433: Repo in Diffusion has no padding after Branches box if footer is not configured from Repo in Diffusion has no padding after Branches box to Repo in Diffusion has no padding after Branches box if footer is disabled.
Feb 24 2016, 1:02 AM · Diffusion, Bug Report
Mnkras created T10433: Repo in Diffusion has no padding after Branches box if footer is not configured.
Feb 24 2016, 12:26 AM · Diffusion, Bug Report

Feb 23 2016

epriestley added a comment to T10432: Attempting to access attached data on AlmanacBinding.

I think I got 'em all, but let me know if you catch any more.

Feb 23 2016, 9:58 PM · Almanac, Bug Report
joshuaspence added a comment to T10432: Attempting to access attached data on AlmanacBinding.

Thanks :)

Feb 23 2016, 9:51 PM · Almanac, Bug Report
epriestley closed T10432: Attempting to access attached data on AlmanacBinding as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 9:50 PM · Almanac, Bug Report
epriestley added a revision to T10432: Attempting to access attached data on AlmanacBinding: Restricted Differential Revision.
Feb 23 2016, 9:49 PM · Almanac, Bug Report
joshuaspence added a comment to T10432: Attempting to access attached data on AlmanacBinding.

This seems to be happening consistently for all new commits.

Feb 23 2016, 9:19 PM · Almanac, Bug Report
joshuaspence created T10432: Attempting to access attached data on AlmanacBinding.
Feb 23 2016, 9:18 PM · Almanac, Bug Report
epriestley closed T10413: Archived project tags are hidden everywhere, not just on Workboard cards as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 6:47 PM · Projects (v3), Bug Report
epriestley added a revision to T10413: Archived project tags are hidden everywhere, not just on Workboard cards: Restricted Differential Revision.
Feb 23 2016, 6:45 PM · Projects (v3), Bug Report
epriestley closed T10414: No natural sort on space dropdown as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 6:41 PM · Bug Report
epriestley added a revision to T10414: No natural sort on space dropdown: Restricted Differential Revision.
Feb 23 2016, 6:31 PM · Bug Report
epriestley renamed T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded from Can not Download large diffs in diffusion "No configured storage engine can store this file" to Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded.
Feb 23 2016, 6:26 PM · Phacility, Files, Diffusion, Bug Report
rbalik added a comment to T10427: Errors when adding person to subproject.

I can't make a workboard either for the subproject. When I click "Create Workboard" it doesn't do anything

Feb 23 2016, 6:19 PM · Projects
epriestley added a comment to T10428: Behavior definitions not registered in some project dialogs.

I'm having difficulty reproducing this. Here's what I did:

Feb 23 2016, 6:18 PM · Bug Report (Needs Information)
chad added a comment to T10428: Behavior definitions not registered in some project dialogs.

Generally, multiple JS issues point towards an issue outside of Phabricator (proxy, cache, update procedures).

Feb 23 2016, 6:15 PM · Bug Report (Needs Information)
chad added a comment to T10428: Behavior definitions not registered in some project dialogs.

Nothing here reproduces for me. Are you able to reproduce this issue on a known good installation of Phabricator? Either here or on http://phacility.com?

Feb 23 2016, 6:14 PM · Bug Report (Needs Information)
urzds added a comment to T10427: Errors when adding person to subproject.

Also interesting:

pasted_file (728×1 px, 119 KB)

Feb 23 2016, 6:13 PM · Projects
rbalik added a comment to T10427: Errors when adding person to subproject.

pasted_file (201×446 px, 13 KB)

Feb 23 2016, 6:11 PM · Projects
urzds added a comment to T10427: Errors when adding person to subproject.

From the DB: P has viewPolicy: P, editPolicy: admin, joinPolicy: admin.

Feb 23 2016, 6:10 PM · Projects
epriestley closed T10426: assignee is added to subscribers after task edit as Resolved by committing Restricted Diffusion Commit.
Feb 23 2016, 6:10 PM · Maniphest, Bug Report
epriestley added a comment to T10427: Errors when adding person to subproject.

What are the view, edit and join policies of the parent project set to?

Feb 23 2016, 6:09 PM · Projects
WTreutterer created T10428: Behavior definitions not registered in some project dialogs.
Feb 23 2016, 6:08 PM · Bug Report (Needs Information)
urzds added a comment to T10427: Errors when adding person to subproject.

I just encountered the same error message.

Feb 23 2016, 6:06 PM · Projects
rbalik added a comment to T10427: Errors when adding person to subproject.

Oh ok, so I guess that doesn't matter then

Feb 23 2016, 6:04 PM · Projects
epriestley added a comment to T10427: Errors when adding person to subproject.

Note that being an administrator doesn't matter: administrators have no special privileges, and can not bypass policies. See:

Feb 23 2016, 6:03 PM · Projects
rbalik added a comment to T10427: Errors when adding person to subproject.

Also interesting:

pasted_file (728×1 px, 119 KB)

Feb 23 2016, 6:02 PM · Projects
epriestley added a revision to T10426: assignee is added to subscribers after task edit : Restricted Differential Revision.
Feb 23 2016, 6:01 PM · Maniphest, Bug Report
rbalik added a comment to T10427: Errors when adding person to subproject.

Hmm, that looks like the same process.

Feb 23 2016, 5:59 PM · Projects
epriestley added a comment to T10426: assignee is added to subscribers after task edit .

Thanks, your reproduction instructions are clear and complete. I was able to reproduce this by following your instructions and narrow it down to a root cause. This should be fixed shortly.

Feb 23 2016, 5:57 PM · Maniphest, Bug Report
urzds added a comment to T10425: Phame - Unhandled Exception ("RuntimeException") - Undefined variable: description.

Thanks! This will be in next weeks stable release?

Feb 23 2016, 5:56 PM · Bug Report
epriestley added a comment to T10427: Errors when adding person to subproject.

I'm having trouble reproducing this. I tried to follow the instructions you provided to recreate the issue, but didn't seem to experience it. Here's what I did.

Feb 23 2016, 5:53 PM · Projects
rbalik added a comment to T10427: Errors when adding person to subproject.

Yeah, running a version from Friday: 5eecff91cdd3

Feb 23 2016, 5:50 PM · Projects
epriestley added a comment to T10427: Errors when adding person to subproject.

Hi! Can you please let us know what version of Phabricator you're running?

Feb 23 2016, 5:48 PM · Projects
rbalik created T10427: Errors when adding person to subproject.
Feb 23 2016, 5:39 PM · Projects
danielberger created T10426: assignee is added to subscribers after task edit .
Feb 23 2016, 5:04 PM · Maniphest, Bug Report
ppggff added a comment to T10264: git clone using http hangs after update.

@epriestley, test files in last comment indicate 31 can decompress gzip properly.
please check it.

Feb 23 2016, 4:59 PM · Bug Report, Diffusion
chad added a comment to T10425: Phame - Unhandled Exception ("RuntimeException") - Undefined variable: description.

Should be resolved at HEAD. Thanks!

Feb 23 2016, 4:57 PM · Bug Report
chad merged task T10425: Phame - Unhandled Exception ("RuntimeException") - Undefined variable: description into T9897: Unbeta Feedback on Phame.
Feb 23 2016, 4:49 PM · Bug Report
urzds created T10425: Phame - Unhandled Exception ("RuntimeException") - Undefined variable: description.
Feb 23 2016, 4:39 PM · Bug Report
urzds added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

Reading the code around /usr/lib/python2.7/dist-packages/mercurial/sshserver.py:32:

def getargs(self, args):
    data = {}
    keys = args.split()
    for n in xrange(len(keys)):
        argline = self.fin.readline()[:-1]
        arg, l = argline.split()
        if arg not in keys:
            raise util.Abort("unexpected parameter %r" % arg)
        if arg == '*':
            star = {}
            for k in xrange(int(l)):
                argline = self.fin.readline()[:-1]
                arg, l = argline.split()
                val = self.fin.read(int(l))
                star[arg] = val
            data['*'] = star
        else:
            val = self.fin.read(int(l))
            data[arg] = val
    return [data[k] for k in keys]
Feb 23 2016, 4:37 PM · Mercurial, Bug Report
pouyana added a comment to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded.

Thanks for the heads up. I will contact my user with this information. What you said make totally sense to me :).

Feb 23 2016, 12:45 PM · Phacility, Files, Diffusion, Bug Report
epriestley added a project to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded: Diffusion.
Feb 23 2016, 12:37 PM · Phacility, Files, Diffusion, Bug Report
epriestley triaged T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded as Wishlist priority.

Thanks, that's helpful.

Feb 23 2016, 12:36 PM · Phacility, Files, Diffusion, Bug Report
pouyana added a comment to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded.

Sorry I totally forget the version:
phabricator

023cfbb23af36b7f15be86a1f5950b62d68b246d (Mon, Feb 22)

arcanist

fcc11b3a278199fa15b90692aba65f54fef5d7f7 (Wed, Feb 10)

phutil

f43291e99d36045bc459e5133454c0d8fd8768ea (Jan 21 2016)
Feb 23 2016, 12:21 PM · Phacility, Files, Diffusion, Bug Report
epriestley added a comment to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded.

Hi! Can you tell me what version of Phabricator you're running?

Feb 23 2016, 12:16 PM · Phacility, Files, Diffusion, Bug Report
epriestley added a comment to T10420: Code coverage reports from `arc unit` are sometimes inaccurate.

In the very, very general case, I think it's undesirable to favor "N" over "U" because the reports could be coming from two different engines, only one of which knows how to execute the lines.

Feb 23 2016, 12:08 PM · Arcanist, Bug Report
pouyana created T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded.
Feb 23 2016, 12:07 PM · Phacility, Files, Diffusion, Bug Report
• williamlohr added a comment to T10422: How to Use Direct Marketing Effectively.
Feb 23 2016, 6:31 AM
• williamlohr created T10422: How to Use Direct Marketing Effectively.
Feb 23 2016, 6:29 AM
sophie added a comment to T10418: Running arc land in a repository running git with unstaged changes that conflict with master wipes out those changes..

thanks! :)

Feb 23 2016, 2:01 AM · Arcanist, Bug Report
sophie closed T10418: Running arc land in a repository running git with unstaged changes that conflict with master wipes out those changes. as Invalid.
Feb 23 2016, 2:01 AM · Arcanist, Bug Report
epriestley added a comment to T10418: Running arc land in a repository running git with unstaged changes that conflict with master wipes out those changes..

I added you to Community if you want to do the honors. :)

Feb 23 2016, 1:59 AM · Arcanist, Bug Report
sophie added a comment to T10418: Running arc land in a repository running git with unstaged changes that conflict with master wipes out those changes..

Ah, ok. sorry! This was a local issue. But now I can't close this task...

Feb 23 2016, 1:58 AM · Arcanist, Bug Report
Firehed created T10420: Code coverage reports from `arc unit` are sometimes inaccurate.
Feb 23 2016, 1:48 AM · Arcanist, Bug Report
epriestley added a comment to T10418: Running arc land in a repository running git with unstaged changes that conflict with master wipes out those changes..

I can't immediately reproduce this, here's what I get:

Feb 23 2016, 1:09 AM · Arcanist, Bug Report
sophie created T10418: Running arc land in a repository running git with unstaged changes that conflict with master wipes out those changes..
Feb 23 2016, 12:53 AM · Arcanist, Bug Report

Feb 22 2016

eadler added a comment to T10417: Active Differentials are wrongly indicated as blocking when "request changes" is used.

I promise I searched first! Thanks for the info

Feb 22 2016, 11:44 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T10416: 7b8d38cd causes hangs with SSH master connection.

Since only the first connection has an issue, you could also conceivably add a slightly-more-than-one-line patch to make arc land run some no-op remote operation (git remote show origin?) and then immediately kill the subprocess, creating and throwing away the bad connection so the followup doesn't hang.

Feb 22 2016, 8:56 PM · Arcanist, Bug Report
epriestley closed T10416: 7b8d38cd causes hangs with SSH master connection as Wontfix.

If you're open to patching one site, you can just comment out the feof($stderr) to retain the bulk of the change (i.e., still wait for stdout).

Feb 22 2016, 8:52 PM · Arcanist, Bug Report
epriestley added a comment to T10414: No natural sort on space dropdown.

I can sort of see arguments either way, although I think I'm inclined to names too.

Feb 22 2016, 8:49 PM · Bug Report
alexmv added a comment to T10416: 7b8d38cd causes hangs with SSH master connection.

Using SSH master connections has two advantages:

  1. Connections are indeed faster. In a minimal case (SSH'ing to an adjacent machine on the same network), it reduces the time for a no-op SSH connection from 0.14s to 0.01s. The utility of this grows as one needs to tunnel SSH connections in order to access a machine; for instance, SSH'ing to our Phabricator instance requires bouncing through a bastion host; using SSH master connections reduces the time to connect from ~0.7s to 0.01s. This is notable since arc may produce mulitiple git commands which attempt to fetch from upstream.
  2. Authentication information is preserved. If the connection does two-factor authentication, for instance, the user may be supplied with an authentication prompt when the master is set up. Re-using the master preserves this authenticated session, and reduces the amount of re-authorization that the user must do.
Feb 22 2016, 8:46 PM · Arcanist, Bug Report
chad added a comment to T10414: No natural sort on space dropdown.

I would go for Space Name personally.

Feb 22 2016, 8:45 PM · Bug Report
epriestley added a comment to T10414: No natural sort on space dropdown.

Is sorting by Space ID (instead of Space name) definitely your preferred sort?

Feb 22 2016, 8:42 PM · Bug Report
epriestley renamed T10413: Archived project tags are hidden everywhere, not just on Workboard cards from Relation between task and project lost after milestone is archived to Archived project tags are hidden everywhere, not just on Workboard cards.
Feb 22 2016, 8:41 PM · Projects (v3), Bug Report
epriestley claimed T10413: Archived project tags are hidden everywhere, not just on Workboard cards.
Feb 22 2016, 8:40 PM · Projects (v3), Bug Report
chad added a project to T10416: 7b8d38cd causes hangs with SSH master connection: Arcanist.
Feb 22 2016, 8:26 PM · Arcanist, Bug Report
chad added a comment to T10417: Active Differentials are wrongly indicated as blocking when "request changes" is used.

See also T1279#42118, T9372, T10031, T8080, T6596

Feb 22 2016, 8:15 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T10416: 7b8d38cd causes hangs with SSH master connection.

The benefit of enabling ControlPersist is only very slightly faster SSH connections, right?

Feb 22 2016, 8:06 PM · Arcanist, Bug Report
chad merged task T10417: Active Differentials are wrongly indicated as blocking when "request changes" is used into T4144: Users disagree on what "Blocking Others", "Action Required" and "Waiting on Others" mean, and/or which groups should exist.
Feb 22 2016, 8:06 PM · Restricted Project, Differential, Bug Report