Page MenuHomePhabricator
Feed Advanced Search

May 26 2016

epriestley added a comment to T11041: Not possible to move a URI without destroying the old one.

I can't reproduce this. Here's the first repository (I/O off, disabled);

May 26 2016, 5:21 PM · Diffusion, Bug Report
Luke081515.2 created T11041: Not possible to move a URI without destroying the old one.
May 26 2016, 5:19 PM · Diffusion, Bug Report
epriestley added a comment to T11038: Herald rules linked to projects don't work as expected.

I don't think the behavior of the fields is unclear and we haven't seen other users run into this particular difficulty.

May 26 2016, 1:06 PM · Owners, Herald, Bug Report
epriestley closed T11039: Trello to Phabricator Migration issue with trello_create.py and trello_makePHIDs.py as Invalid.

Unfortunately, we can't help you with this problem. We didn't write this script, do not maintain it, and have never seen it before. You should try getting in touch with the script maintainers instead. We don't have the resources to provide support for third-party code.

May 26 2016, 12:39 PM · Bug Report
sharmakhimlal created T11039: Trello to Phabricator Migration issue with trello_create.py and trello_makePHIDs.py.
May 26 2016, 6:41 AM · Bug Report
aristedes added a comment to T11038: Herald rules linked to projects don't work as expected.

OK, thanks. Because there is no documentation about what any of these choices mean would you like to make this task a documentation fixing issue?

May 26 2016, 6:25 AM · Owners, Herald, Bug Report
epriestley closed T11038: Herald rules linked to projects don't work as expected as Invalid.

Code being owned by a package does not tag it with a project.

May 26 2016, 6:19 AM · Owners, Herald, Bug Report
aristedes created T11038: Herald rules linked to projects don't work as expected.
May 26 2016, 3:25 AM · Owners, Herald, Bug Report
epriestley added a comment to T10811: `phd-daemon` crash after automatic restart following config change.

D15977 improves some behaviors slightly and adds more extensive logging for signal handling. I don't think I caught anything with real meat to it.

May 26 2016, 3:23 AM · Daemons, Bug Report
epriestley added a revision to T10811: `phd-daemon` crash after automatic restart following config change: Restricted Differential Revision.
May 26 2016, 3:19 AM · Daemons, Bug Report
epriestley added a comment to T10811: `phd-daemon` crash after automatic restart following config change.

D15975 should fix the (Exception) No such daemon "39084:6ra7uz"! error. I'm not sure if it will do anything for stability or not, but it should reduce noise. I didn't uncover anything interesting in resolving it (it wasn't really even a race, just a fairly straightforward set of state management bugs).

May 26 2016, 1:17 AM · Daemons, Bug Report
epriestley added a revision to T10811: `phd-daemon` crash after automatic restart following config change: Restricted Differential Revision.
May 26 2016, 1:13 AM · Daemons, Bug Report

May 25 2016

epriestley added a comment to T11030: Cannot clone public repository over HTTP anonymously..

This should be fixed in HEAD of master. Thanks for the report! Let us know if you run into anything else.

May 25 2016, 4:11 PM · Diffusion, Bug Report
epriestley added a comment to T11032: Repository URL missing port if phabricator is installed on non-80 port.

This should be fixed in HEAD of master. Thanks for the report! Let us know if you run into anything else.

May 25 2016, 4:09 PM · Diffusion, Bug Report
epriestley closed T11030: Cannot clone public repository over HTTP anonymously. as Resolved by committing Restricted Diffusion Commit.
May 25 2016, 4:07 PM · Diffusion, Bug Report
epriestley closed T11032: Repository URL missing port if phabricator is installed on non-80 port as Resolved by committing Restricted Diffusion Commit.
May 25 2016, 4:07 PM · Diffusion, Bug Report
epriestley added a project to T11030: Cannot clone public repository over HTTP anonymously.: Diffusion.
May 25 2016, 3:13 PM · Diffusion, Bug Report
epriestley claimed T11030: Cannot clone public repository over HTTP anonymously..
May 25 2016, 3:13 PM · Diffusion, Bug Report
epriestley claimed T11032: Repository URL missing port if phabricator is installed on non-80 port.
May 25 2016, 3:12 PM · Diffusion, Bug Report
epriestley added a revision to T11030: Cannot clone public repository over HTTP anonymously.: Restricted Differential Revision.
May 25 2016, 3:10 PM · Diffusion, Bug Report
epriestley added a revision to T11032: Repository URL missing port if phabricator is installed on non-80 port: Restricted Differential Revision.
May 25 2016, 3:10 PM · Diffusion, Bug Report
epriestley closed T11031: It is possible to create repositories with invalid ssh urls as Invalid.

Please read Contributing Bug Reports carefully and make sure you provide all the required information, particularly reproduction steps (see Providing Reproduction Steps). Thanks!

May 25 2016, 1:29 PM · Bug Report
RuralHunter created T11032: Repository URL missing port if phabricator is installed on non-80 port.
May 25 2016, 9:00 AM · Diffusion, Bug Report
rabahmeradi created T11031: It is possible to create repositories with invalid ssh urls.
May 25 2016, 6:36 AM · Bug Report
tail updated the task description for T11030: Cannot clone public repository over HTTP anonymously..
May 25 2016, 5:03 AM · Diffusion, Bug Report
tail created T11030: Cannot clone public repository over HTTP anonymously..
May 25 2016, 5:02 AM · Diffusion, Bug Report

May 24 2016

CodeMouse92 added a comment to T10779: Cannot manage base project milestones if base project has subprojects.

Confirmed workaround: set base project edit permissions to something other than Project Members.

May 24 2016, 3:16 PM · Restricted Project, Projects, Bug Report
CodeMouse92 added a comment to T10661: Can't create milestone.

@sascha-egerer that would seem to be the main factor, and the reason this couldn't be duplicated initially. It isn't all milestones - it is specifically milestones for projects that have subprojects.

May 24 2016, 3:11 PM · Projects (v3), Bug Report
epriestley added a comment to T11023: Branch contains condition in Herald rule not behaving as expected .

Commit objects are not GC'ed in Phabricator?

May 24 2016, 2:38 PM · Bug Report
swisspol added a comment to T11023: Branch contains condition in Herald rule not behaving as expected .

Deleting the branch first won't have any effect, since the commit still exists.

May 24 2016, 2:37 PM · Bug Report
epriestley merged task T11023: Branch contains condition in Herald rule not behaving as expected into T6522: "Commit's branches contains" condition is not re-evaluated for commits in multiple branches.
May 24 2016, 2:36 PM · Bug Report
epriestley added a comment to T11023: Branch contains condition in Herald rule not behaving as expected .

Deleting the branch first won't have any effect, since the commit still exists.

May 24 2016, 2:36 PM · Bug Report
swisspol added a comment to T11023: Branch contains condition in Herald rule not behaving as expected .

Sounds like a dup indeed. What's the work around?

  • Doing a rebase instead of FF-merge?
  • Deleting the topic branch in Diffusion first, then FF-merge and push?
May 24 2016, 2:29 PM · Bug Report
epriestley closed T11016: When owner of Owners package is a project, audit triggers even when member of project signs off as Resolved.
May 24 2016, 1:46 PM · Owners, Bug Report
epriestley added a comment to T11015: Owners has perplexing behavior when directory paths are not terminated with a slash.

We have a somewhat-legitimate technical reason for this behavior: when we compute which-packages-own(list-of-paths), we do so by splitting the paths into components and then querying for the components. a/b/ is the best database representation for this.

May 24 2016, 1:29 PM · Owners, Bug Report
epriestley claimed T11016: When owner of Owners package is a project, audit triggers even when member of project signs off.
May 24 2016, 1:09 PM · Owners, Bug Report
epriestley added a comment to T11016: When owner of Owners package is a project, audit triggers even when member of project signs off.

This was a legitimate upstream bug and I was able to reproduce it by following your steps, thanks for the report. D15971 should resolve it.

May 24 2016, 1:09 PM · Owners, Bug Report
epriestley added a revision to T11016: When owner of Owners package is a project, audit triggers even when member of project signs off: Restricted Differential Revision.
May 24 2016, 1:08 PM · Owners, Bug Report
epriestley added a comment to T11023: Branch contains condition in Herald rule not behaving as expected .

Is this the same as T6522? I'm not sure where the last screenshot came from, I wouldn't expect an FF-merge + push to trigger another evaluation of Herald rules.

May 24 2016, 12:44 PM · Bug Report
epriestley closed T11028: Can't push/pull into a hosted repository since 2016 Week 20 (Mid May) update as Resolved.

Great, glad things are working. Let us know if you run into anything else.

May 24 2016, 12:41 PM · Diffusion, Bug Report
• antx42 added a comment to T11028: Can't push/pull into a hosted repository since 2016 Week 20 (Mid May) update.

Looks like that value was removed when the update happened as it was there before (checked with a backup).

May 24 2016, 12:40 PM · Diffusion, Bug Report
• antx42 added a comment to T11028: Can't push/pull into a hosted repository since 2016 Week 20 (Mid May) update.

Should be as I've only used SSH for the hosted repos - I will double check now.

May 24 2016, 12:16 PM · Diffusion, Bug Report
epriestley added a comment to T11028: Can't push/pull into a hosted repository since 2016 Week 20 (Mid May) update.

Does your configuration satisfy the conditions described in the documentation?

May 24 2016, 12:13 PM · Diffusion, Bug Report
• antx42 created T11028: Can't push/pull into a hosted repository since 2016 Week 20 (Mid May) update.
May 24 2016, 12:10 PM · Diffusion, Bug Report
epriestley closed T11024: Can't create account of Apple Computer Wiki as Invalid.

This service has no relationship to Apple Computer Wiki, and can not help resolve your problem.

May 24 2016, 11:39 AM · Bug Report
• mtherjs created T11024: Can't create account of Apple Computer Wiki.
May 24 2016, 7:45 AM · Bug Report
swisspol created T11023: Branch contains condition in Herald rule not behaving as expected .
May 24 2016, 6:14 AM · Bug Report
sascha-egerer added a comment to T10661: Can't create milestone.

@CodeMouse92 yes the project has a subproject. But changing the policies as described above works around that issue.

May 24 2016, 4:38 AM · Projects (v3), Bug Report
RuralHunter added a comment to T11012: Audit not Triggered Sometimes in Owners/Packages Application.

I just looked the description in 74e117ae41dd.

May 24 2016, 3:38 AM · Restricted Project, Owners, Bug Report
RuralHunter added a comment to T11012: Audit not Triggered Sometimes in Owners/Packages Application.
In T11012#176923, @chad wrote:

We require reproduction steps on all bug reports, this way we know we're fixing your actual bug, and have a means to test and verify any fix. As Owners was recently overhauled with many new features, it's hard to identify what case we may have missed in testing.

I understand that. Will try to find reproduce steps.

May 24 2016, 2:32 AM · Restricted Project, Owners, Bug Report
chad added a comment to T11012: Audit not Triggered Sometimes in Owners/Packages Application.

I have not found a concrete way to reproduce it. Sometimes the status is correct and sometimes it's not and I can not find any difference on those commits yet.

May 24 2016, 2:25 AM · Restricted Project, Owners, Bug Report
RuralHunter added a comment to T11012: Audit not Triggered Sometimes in Owners/Packages Application.
In T11012#176731, @chad wrote:

This may also be T11015.

May 24 2016, 2:07 AM · Restricted Project, Owners, Bug Report
RuralHunter added a comment to T11012: Audit not Triggered Sometimes in Owners/Packages Application.
In T11012#176668, @chad wrote:

Please follow all the steps in Contributing Bug Reports. Specifically we need to know how to reproduce the issue locally.

May 24 2016, 2:00 AM · Restricted Project, Owners, Bug Report
epriestley closed T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user as Resolved by committing Restricted Diffusion Commit.
May 24 2016, 12:55 AM · Owners, Bug Report
epriestley claimed T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user.
May 24 2016, 12:34 AM · Owners, Bug Report
epriestley added a revision to T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user: Restricted Differential Revision.
May 24 2016, 12:33 AM · Owners, Bug Report
epriestley merged task T11018: Restricted credentials show as unknown in Drydock into T9771: Objects which are restricted because of Spaces render as "Unknown" instead of "Restricted".
May 24 2016, 12:20 AM · Bug Report, Drydock
epriestley added a comment to T11018: Restricted credentials show as unknown in Drydock .

This is because that credential is in a restricted Space so this is ultimately the same as T9771, I'm just going to merge it there. The actual Query looks to be implemented fine so I think this is purely a Space issue.

May 24 2016, 12:20 AM · Bug Report, Drydock
CodeMouse92 added a comment to T10661: Can't create milestone.

I know this was linked against my question, but I wanted to check and see if it is the same thing...

May 24 2016, 12:19 AM · Projects (v3), Bug Report
epriestley closed T11021: Argument 1 passed to DifferentialReviewersField::inflateReviewers() must be of the type array, null given as Resolved by committing Restricted Diffusion Commit.
May 24 2016, 12:16 AM · Differential, Bug Report
epriestley added a comment to T11021: Argument 1 passed to DifferentialReviewersField::inflateReviewers() must be of the type array, null given.

Thanks!

May 24 2016, 12:06 AM · Differential, Bug Report
epriestley added a revision to T11021: Argument 1 passed to DifferentialReviewersField::inflateReviewers() must be of the type array, null given: Restricted Differential Revision.
May 24 2016, 12:06 AM · Differential, Bug Report
epriestley claimed T11021: Argument 1 passed to DifferentialReviewersField::inflateReviewers() must be of the type array, null given.
May 24 2016, 12:05 AM · Differential, Bug Report

May 23 2016

joshuaspence added a comment to T11021: Argument 1 passed to DifferentialReviewersField::inflateReviewers() must be of the type array, null given.

This seemed to fix the issue:

May 23 2016, 10:57 PM · Differential, Bug Report
joshuaspence added a comment to T11021: Argument 1 passed to DifferentialReviewersField::inflateReviewers() must be of the type array, null given.

From the server logs:

May 23 2016, 10:55 PM · Differential, Bug Report
joshuaspence added a comment to T11021: Argument 1 passed to DifferentialReviewersField::inflateReviewers() must be of the type array, null given.

Phabricator version is rPe1ad312fdd2c570eeaacd3e02eda2142cc5d9849.

May 23 2016, 10:54 PM · Differential, Bug Report
joshuaspence created T11021: Argument 1 passed to DifferentialReviewersField::inflateReviewers() must be of the type array, null given.
May 23 2016, 10:54 PM · Differential, Bug Report
chad renamed T11016: When owner of Owners package is a project, audit triggers even when member of project signs off from when owner of Owners package is a project, audit triggers even when member of project signs off to When owner of Owners package is a project, audit triggers even when member of project signs off.
May 23 2016, 10:43 PM · Owners, Bug Report
chad added a project to T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user: Owners.
May 23 2016, 10:42 PM · Owners, Bug Report
eadler added a comment to T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user.

ref T9412

May 23 2016, 10:04 PM · Owners, Bug Report
joshuaspence created T11018: Restricted credentials show as unknown in Drydock .
May 23 2016, 9:42 PM · Bug Report, Drydock
edibiase added a comment to T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user.

That sounds like it might be what's going on. I apologize for not having better steps to reproduce; I'm working backward from example emails that folks are receiving, so it's been a bit of a guessing game as to how to make this happen.

May 23 2016, 9:00 PM · Owners, Bug Report
epriestley added a comment to T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user.

I think I have a pretty good guess about this one.

May 23 2016, 8:56 PM · Owners, Bug Report
chad added a comment to T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user.

Can you be more specific about the steps to reproduce?

May 23 2016, 8:39 PM · Owners, Bug Report
edibiase created T11017: Phabricator sending unexpected email for commits that aren't obviously related to the user.
May 23 2016, 8:32 PM · Owners, Bug Report
allixsenos added a comment to T11016: When owner of Owners package is a project, audit triggers even when member of project signs off.
  1. manually restarted php5-fpm and phd services
  2. colleague, member of ops submitted review touching src/ansible
  3. that review doesn't explicitly list "ops" as blocking reviewers because he's also an owner of ansible, but it does highlight my review in yellow
    Screen Shot 2016-05-23 at 22.03.22.png (43×179 px, 5 KB)
  4. he lands it after I accept it
  5. his commit is flagged for review despite both of us being members of ops
May 23 2016, 8:04 PM · Owners, Bug Report
epriestley closed T11010: Http502 when submitting or updating a diff as Resolved by committing Restricted Diffusion Commit.
May 23 2016, 7:48 PM · Bug Report
allixsenos added a comment to T11016: When owner of Owners package is a project, audit triggers even when member of project signs off.

I will manually restart everything and reproduce again

May 23 2016, 7:45 PM · Owners, Bug Report
epriestley added a comment to T11016: When owner of Owners package is a project, audit triggers even when member of project signs off.

Before I dig into this, is there any possibility this was a daemon running old code? I don't remember if I tested this explicitly and it could quite reasonably be an upstream bug, but the code at least attempts to check for this, and D15939 theoretically fixed issues with this.

May 23 2016, 7:45 PM · Owners, Bug Report
allixsenos updated the task description for T11016: When owner of Owners package is a project, audit triggers even when member of project signs off.
May 23 2016, 7:43 PM · Owners, Bug Report
allixsenos created T11016: When owner of Owners package is a project, audit triggers even when member of project signs off.
May 23 2016, 7:16 PM · Owners, Bug Report
epriestley added a revision to T11010: Http502 when submitting or updating a diff: Restricted Differential Revision.
May 23 2016, 5:43 PM · Bug Report
chad added a comment to T11012: Audit not Triggered Sometimes in Owners/Packages Application.

This may also be T11015.

May 23 2016, 5:35 PM · Restricted Project, Owners, Bug Report
epriestley renamed T11015: Owners has perplexing behavior when directory paths are not terminated with a slash from Owners not picking up code reviews, unclear if properly configured to Owners has perplexing behavior when directory paths are not terminated with a slash.
May 23 2016, 4:34 PM · Owners, Bug Report
epriestley added a comment to T11015: Owners has perplexing behavior when directory paths are not terminated with a slash.

Yeah, current behavior is garbage. Not sure if I changed it by accident while doing other changes or we just didn't run into it before, but there's no technical or product reason for it to work like it currently does.

May 23 2016, 4:31 PM · Owners, Bug Report
eadler added a comment to T11015: Owners has perplexing behavior when directory paths are not terminated with a slash.

I ran into an identical issue (and debugged on chat). Agreed that there could be better messaging around this.

May 23 2016, 4:24 PM · Owners, Bug Report
allixsenos added a comment to T11015: Owners has perplexing behavior when directory paths are not terminated with a slash.

that worked. there's no indication in the UI that the trailing slash is required, the link rendered works fine

May 23 2016, 3:48 PM · Owners, Bug Report
epriestley added a comment to T11015: Owners has perplexing behavior when directory paths are not terminated with a slash.

Try /wsl_root/src/ansible/ (with a trailing slash)?

May 23 2016, 3:09 PM · Owners, Bug Report
allixsenos created T11015: Owners has perplexing behavior when directory paths are not terminated with a slash.
May 23 2016, 2:47 PM · Owners, Bug Report
chad added a comment to T11012: Audit not Triggered Sometimes in Owners/Packages Application.

Please follow all the steps in Contributing Bug Reports. Specifically we need to know how to reproduce the issue locally.

May 23 2016, 1:46 PM · Restricted Project, Owners, Bug Report
joshuaspence created T11013: "Unknown Object" when attempting to create a new Almanac network.
May 23 2016, 1:33 PM · Almanac, Bug Report
RuralHunter created T11012: Audit not Triggered Sometimes in Owners/Packages Application.
May 23 2016, 9:52 AM · Restricted Project, Owners, Bug Report
allixsenos closed T11009: owners packages details screen throws exception as Invalid.

I apologize, my ansible deploy script *should* have restarted everything that needs restarting, but it appears it didn't.

May 23 2016, 9:13 AM · Bug Report
joshuaspence added a comment to T11011: Can't connect to MySQL server.

The following queries are full table scans:

May 23 2016, 5:31 AM · Performance, Bug Report
joshuaspence created T11011: Can't connect to MySQL server.
May 23 2016, 5:27 AM · Performance, Bug Report
• donotreply created T11010: Http502 when submitting or updating a diff.
May 23 2016, 4:05 AM · Bug Report
avivey added a comment to T11009: owners packages details screen throws exception.

Did you restart all webservers/php-fpm/etc. after upgrading? Did you upgrade the storage?
I cannot immediately reproduce this - see Contributing Bug Reports for some more information that can help you.

May 23 2016, 12:17 AM · Bug Report
epriestley added a comment to T11009: owners packages details screen throws exception.

Did you fully restart everything (see Restarting Phabricator) after upgrading? I can't reproduce this and it looks like the kind of thing that could happen if Phabricator was not fully restarted.

May 23 2016, 12:13 AM · Bug Report

May 22 2016

allixsenos created T11009: owners packages details screen throws exception.
May 22 2016, 11:59 PM · Bug Report
epriestley closed T10808: differential.querydiffs method does not provide "local:commits" as Resolved.

Presuming resolved by D15710.

May 22 2016, 1:11 PM · Differential, Bug Report