Page MenuHomePhabricator
Feed All Stories

Jan 11 2016

eadler moved T4994: Support reporting code coverage information into Diffusion from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Restricted Project, Diffusion
eadler moved T3626: Detection of language for some file types isn't good from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Customer Impact, Restricted Project, Differential
eadler moved T9410: Policy behavior for Herald rules is murky and inconsistent from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Herald, Restricted Project
eadler moved T10109: Differental does not trigger Herald for mundane updates from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Differential, Restricted Project, Herald, Bug Report
eadler moved T10105: Disabled account should disable all outbound mail from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · llvm, Restricted Project, Mail, Bug Report
eadler moved T10096: Incorrect remarkup rendering of bold text, that contains hashtag (#) from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Remarkup, Bug Report
eadler moved T9957: Arcanist cpplint install information is out of date from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Lint, Bug Report
eadler moved T9951: Store/display more information for Unit tests from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Restricted Project, Harbormaster, Feature Request
eadler moved T9892: I want to be sent email when a build fails for any commit in a repository from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Harbormaster
eadler moved T9886: Add Java linters, checkstyle and PMD from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Arcanist, Lint
eadler moved T9870: Include clang-format-lint as a linter from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Lint, Arcanist
eadler moved T9846: Diff on `arc lint` misleading when the replacement has newlines from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Arcanist, Lint
eadler moved T9764: Lint star colors have different meaning for revisions and diffs from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Differential
eadler moved T9768: `arc diff` blows up if providing bad JIRA ticket name from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Arcanist
eadler moved T9749: `arc lint --only-new` doesn't work as expected from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Lint, Arcanist
eadler moved T9652: Request for test case management application from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Unnamed Manual Test Case Application
eadler moved T9482: Problems in the "Commits in this Package that Need Attention" view from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jan 11 2016, 9:42 PM · Restricted Project, Restricted Project, Audit
eadler added a project to T9482: Problems in the "Commits in this Package that Need Attention" view: Restricted Project.
Jan 11 2016, 9:40 PM · Restricted Project, Restricted Project, Audit
eadler added a project to T9652: Request for test case management application: Restricted Project.
Jan 11 2016, 9:40 PM · Restricted Project, Unnamed Manual Test Case Application
eadler added a project to T9749: `arc lint --only-new` doesn't work as expected: Restricted Project.
Jan 11 2016, 9:40 PM · Restricted Project, Lint, Arcanist
eadler renamed T9768: `arc diff` blows up if providing bad JIRA ticket name from `arc diff` blows up if providing bad JIRA ticket name to `arc diff` blows up if providing bad JIRA ticket name.
Jan 11 2016, 9:39 PM · Restricted Project, Arcanist
eadler added a project to T9764: Lint star colors have different meaning for revisions and diffs: Restricted Project.
Jan 11 2016, 9:39 PM · Restricted Project, Differential
eadler added a project to T9846: Diff on `arc lint` misleading when the replacement has newlines: Restricted Project.
Jan 11 2016, 9:39 PM · Restricted Project, Arcanist, Lint
eadler added a project to T9870: Include clang-format-lint as a linter: Restricted Project.
Jan 11 2016, 9:39 PM · Restricted Project, Lint, Arcanist
eadler added a project to T9886: Add Java linters, checkstyle and PMD: Restricted Project.
Jan 11 2016, 9:39 PM · Restricted Project, Arcanist, Lint
eadler added a project to T9892: I want to be sent email when a build fails for any commit in a repository: Restricted Project.
Jan 11 2016, 9:39 PM · Restricted Project, Harbormaster
eadler added a project to T9951: Store/display more information for Unit tests: Restricted Project.
Jan 11 2016, 9:39 PM · Restricted Project, Restricted Project, Harbormaster, Feature Request
eadler added a project to T9957: Arcanist cpplint install information is out of date: Restricted Project.
Jan 11 2016, 9:39 PM · Restricted Project, Lint, Bug Report
eadler added a project to T10096: Incorrect remarkup rendering of bold text, that contains hashtag (#): Restricted Project.
Jan 11 2016, 9:38 PM · Restricted Project, Remarkup, Bug Report
eadler added a project to T10105: Disabled account should disable all outbound mail: Restricted Project.
Jan 11 2016, 9:38 PM · llvm, Restricted Project, Mail, Bug Report
eadler added a project to T10109: Differental does not trigger Herald for mundane updates: Restricted Project.
Jan 11 2016, 9:38 PM · Differential, Restricted Project, Herald, Bug Report
eadler added a project to T9410: Policy behavior for Herald rules is murky and inconsistent: Restricted Project.
Jan 11 2016, 9:38 PM · Restricted Project, Herald, Restricted Project
eadler added a project to T3626: Detection of language for some file types isn't good: Restricted Project.
Jan 11 2016, 9:38 PM · Customer Impact, Restricted Project, Differential
eadler added a project to T4994: Support reporting code coverage information into Diffusion: Restricted Project.
Jan 11 2016, 9:38 PM · Restricted Project, Restricted Project, Diffusion
eadler updated subscribers of T8856: Show a summary of failed unit tests at the end of `arc unit` or `arc diff` if there are failures.
Jan 11 2016, 9:37 PM · Restricted Project, Arcanist
eadler added a project to T9299: Send code reviews to non-Phabricator users: Restricted Project.
Jan 11 2016, 9:36 PM · Phacility, People, Differential, Phacility Support
eadler added a project to T9363: improve handling of panels that do not always return results: Restricted Project.
Jan 11 2016, 9:36 PM · Dashboards (v2), ApplicationSearch, Restricted Project, Restricted Project
eadler added a project to T9365: Harbormaster results are hard to aggregate : Restricted Project.
Jan 11 2016, 9:36 PM · Harbormaster, Restricted Project, Restricted Project
eadler added a project to T10123: Allow short "result" message for CI builds: Restricted Project.
Jan 11 2016, 9:27 PM · Restricted Project, Restricted Project, Harbormaster, Feature Request
avivey added a comment to T10118: regex for parsing severity codes in ArcanistCppLinter is wrong.

Looks like you are already aware of T8404, and this is a dup; Why did you submit another ticket?

Jan 11 2016, 8:59 PM · Lint, Arcanist, Bug Report
avivey created T10123: Allow short "result" message for CI builds.
Jan 11 2016, 7:16 PM · Restricted Project, Restricted Project, Harbormaster, Feature Request
jonah214 added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

Thanks so much.

Jan 11 2016, 6:47 PM · Maniphest, Bug Report
carlokok added a comment to T10122: ctrl+enter to commit in all fields.

Right. This was a feature request to have "ctrl+enter" (or cmd enter on osx) to work in all fields (in addition to enter in the fields where it makes sense), so that there's 1 key that always works to submit the current form.

Jan 11 2016, 6:43 PM · Restricted Project, ApplicationEditor, Feature Request
epriestley added a comment to T10122: ctrl+enter to commit in all fields.

I largely can't reproduce this. Here's what I did:

Jan 11 2016, 6:41 PM · Restricted Project, ApplicationEditor, Feature Request
tbrown_SFC added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

Also for posterity, I discovered that I was able to take a task in which this link works as expected and break it simply by assigning it to myself. Removing the assignment to myself and forcing uncached page refresh did not unbreak the task (so now I seem to have another one in permanent workaround state). This is still not a very detailed reproduction scenario, but maybe if someone hits this, it will help hone in on the problem.

Jan 11 2016, 6:38 PM · Bug Report
carlokok created T10122: ctrl+enter to commit in all fields.
Jan 11 2016, 6:34 PM · Restricted Project, ApplicationEditor, Feature Request
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

Alright, I think you should get different behavior at HEAD for both of those issues. No promises it will be correct, but let me know if those changes just uncover a second series of errors.

Jan 11 2016, 6:33 PM · Maniphest, Bug Report
epriestley added a revision to T10121: Command died with status 255: ERROR 8: Array to string conversion: Unknown Object (Differential Revision).
Jan 11 2016, 6:31 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

I also don't think it's wholly unreasonable to change the prefix rule to be stricter, but I think getting rid of the username@ stuff will moot it in all realistic cases. We could look at the rules after the next Conpherence update if we want to reintroduce the username@ rule.

Jan 11 2016, 6:27 PM · Maniphest, Bug Report
jonah214 added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

Ah, that makes perfect sense. Thanks!

Jan 11 2016, 6:26 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

The mail looks like this:

Jan 11 2016, 6:23 PM · Maniphest, Bug Report
jonah214 added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

How is it even getting mail addressed to, say, jonah@company.com if the system's mail is handled by Postfix and the aliases file only pipes phabricator@ to the Phabricator script?

Jan 11 2016, 6:17 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

Phabricator strips the prefix if it is present, but does not require it to be present, and attempts to route all mail for the domain it is given.

Jan 11 2016, 6:16 PM · Maniphest, Bug Report
jonah214 added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

The way I thought it was supposed to work is that instead of addresses like D832@company.com, we'd use phabricator+D832@company.com and thus only have to reserve the phabricator@ address. Is that not viable?

Jan 11 2016, 6:12 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

I intend to remove username@domain.com causing interactions with Conpherence.

Jan 11 2016, 6:12 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

(Allowing Phabricator to receive mail at domain.com and also having real mailing lists and users at that domain is still a bad idea, because the two namespaces will collide, but few normal addresses resemble Phabricator object addresses so the practical badness is low -- almost no one has D832@mycompany.com as a real address.)

Jan 11 2016, 6:11 PM · Maniphest, Bug Report
jonah214 added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

Which feature do you mean to remove?

Jan 11 2016, 6:11 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

The second (or third or fourth or whatever, depending on how we're counting) Conpherence issue I'm unsure about.

Jan 11 2016, 6:09 PM · Maniphest, Bug Report
jonah214 added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

Great, thanks. I've been putting off opening a task about the various errors we get when trying to comment on tasks by email when they involve aliases and replying all, but I will do it next time.

Jan 11 2016, 6:04 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

"Array to string conversion" is a bug. Once that's fixed (by D14996), the user will start getting some kind of error email instead (the bad conversion was occurring while trying to compose an email to notify the user about an error). That error email may describe an upstream bug, or may describe user error. The stack trace doesn't have enough information for me to guess.

Jan 11 2016, 6:03 PM · Maniphest, Bug Report
epriestley added a revision to T10121: Command died with status 255: ERROR 8: Array to string conversion: Unknown Object (Differential Revision).
Jan 11 2016, 6:01 PM · Maniphest, Bug Report
jonah214 added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

In the sense that I'm doing something wrong, or that it's a real bug but probably easy to fix?

Jan 11 2016, 5:56 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

The first error is happening while we're trying to generate an error mail, so this maybe quite a deep rabbit hole, but that first surface error looks straightforward.

Jan 11 2016, 5:49 PM · Maniphest, Bug Report
jonah214 added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

I (user B) also got this error message:

Jan 11 2016, 5:45 PM · Maniphest, Bug Report
jonah214 created T10121: Command died with status 255: ERROR 8: Array to string conversion.
Jan 11 2016, 5:40 PM · Maniphest, Bug Report
epriestley closed T8826: Add authored date to commit properties as Resolved by committing Restricted Diffusion Commit.
Jan 11 2016, 5:32 PM · Diffusion
epriestley closed T9523: Make search autocomplete show repository names as Resolved by committing Restricted Diffusion Commit.
Jan 11 2016, 5:32 PM · Restricted Project, Diffusion, Search
epriestley closed T9523: Make search autocomplete show repository names, a subtask of T9287: Working with many repositories, as Resolved.
Jan 11 2016, 5:32 PM · Restricted Project, Restricted Project, Diffusion
epriestley added a comment to T8826: Add authored date to commit properties.

I've marked D14995 as fixing this, but only commits parsed after the upgrade will have author dates shown.

Jan 11 2016, 5:21 PM · Diffusion
epriestley closed T10117: Exception when trying to unassign owner via conduit maniphest.edit endpoint as Resolved by committing Restricted Diffusion Commit.
Jan 11 2016, 5:19 PM · Maniphest, Conduit, Bug Report
chad added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

Sorry we don't have more ideas for you.

Jan 11 2016, 5:18 PM · Bug Report
tbrown_SFC added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

For posterity, opening the "Edit Blocking Tasks" link in a new tab did work, and that's good enough for me.

Jan 11 2016, 5:17 PM · Bug Report
epriestley closed T10116: ctrl+enter in comment field in maniphest issues with new editor as Resolved by committing Restricted Diffusion Commit.
Jan 11 2016, 5:16 PM · ApplicationEditor, Bug Report
epriestley closed T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector as Invalid.

There's currently no way to create those relationships via the API (see T5873; see also T10034).

Jan 11 2016, 5:15 PM · Bug Report
tbrown_SFC added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

The server side of this is a fairly boring RHEL 7 setup. It's doing nothing other than running Phabricator, and so it has no other server side modules installed in Apache other than the minimums required to run Phabricator. My browser is also pretty boring; nothing extra added except for Postman. I've set up some creative and full featured web stacks in my day, and this isn't one of them.

Jan 11 2016, 5:10 PM · Bug Report
chad added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

Ok, thought I missed some magic strings or a trace posted elsewhere.

Jan 11 2016, 5:09 PM · Bug Report
jbinney added a comment to T10050: Differential-to-Asana integration, like with JIRA.

Got it - those are fair points. We'll maintain a fork of Phabricator with this Asana integration, so please ping me if you'd like to reference it at some point in the future!

Jan 11 2016, 5:08 PM · Doorkeeper, Differential, Asana, Feature Request
epriestley added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

Oh, sorry, I was agreeing with you ("points to something environmental").

Jan 11 2016, 5:07 PM · Bug Report
chad added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

Oh, where does the trace say that?

Jan 11 2016, 5:05 PM · Bug Report
epriestley added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

Yes, the trace implies your entire environment is unsound. RocketLoader (T9716), mod_pagespeed (T4854) and other technologies which rewrite page content and Javascript can sometimes cause this, if you have any of that stuff enabled.

Jan 11 2016, 5:02 PM · Bug Report
chad added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

I don't know offhand what debugging information you can collect. No one else is reporting it and we can't reproduce it. This generally points to something environmental on your server (not Phabricator itself).

Jan 11 2016, 5:02 PM · Bug Report
tbrown_SFC added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

I don't know what about these tasks is unusual. The tasks I noticed this on were created two months ago and only today did I want to connect them together. From a user standpoint all I'm doing is opening the tasks and clicking on the edit blocking tasks link, and then I see only this in the javascript console:

Jan 11 2016, 4:58 PM · Bug Report
chad added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

I've also noticed this does not happen on every task (though it is certainly more than a few)

Jan 11 2016, 4:44 PM · Bug Report
epriestley added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

I can't reproduce this. Here's what I did:

Jan 11 2016, 4:43 PM · Bug Report
tbrown_SFC updated the task description for T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.
Jan 11 2016, 4:41 PM · Bug Report
tbrown_SFC created T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.
Jan 11 2016, 4:38 PM · Bug Report
• bitglue added a comment to T5155: Evaluate support for AWS IAM Roles in S3 Client.

For me, the big advantage to using IAM roles is that the credentials rotate automatically, every hour.

Jan 11 2016, 4:05 PM · Files
epriestley closed T10119: This is a test as Spite.
Jan 11 2016, 3:14 PM · Bug Report
• niczak awarded T10119: This is a test a The World Burns token.
Jan 11 2016, 3:14 PM · Bug Report
• niczak created T10119: This is a test.
Jan 11 2016, 3:13 PM · Bug Report
epriestley added a revision to T8826: Add authored date to commit properties: Unknown Object (Differential Revision).
Jan 11 2016, 1:20 PM · Diffusion
epriestley added a revision to T9523: Make search autocomplete show repository names: Unknown Object (Differential Revision).
Jan 11 2016, 1:09 PM · Restricted Project, Diffusion, Search
michaeloa edited the content of Community Resources.
Jan 11 2016, 12:18 PM
michaeloa edited the content of Community Resources.
Jan 11 2016, 12:11 PM
michaeloa added a comment to T10038: Plan the mid-term pathway for unit-test/linter bindings.

This sounds awesome.

Jan 11 2016, 12:04 PM · Lint, Arcanist
michaeloa created T10118: regex for parsing severity codes in ArcanistCppLinter is wrong.
Jan 11 2016, 11:59 AM · Lint, Arcanist, Bug Report
epriestley added a comment to T4705: Add an "Optional Display URI" for remote repositories in Diffusion.

Yes, this would really be a useful feature. Especially as CentOS still uses a 5.3 version of sshd and on production machines, I would like to keep the "normal" sshd on the standard port. Using a different ssh port for all other tools require a lot of extra config.

Jan 11 2016, 11:59 AM · Restricted Project, Diffusion (v3)
epriestley moved T4705: Add an "Optional Display URI" for remote repositories in Diffusion from Backlog to vNext on the Diffusion board.
Jan 11 2016, 11:57 AM · Restricted Project, Diffusion (v3)
epriestley moved T5406: Offer read-only copy of a externally hosted repository via Diffusion clone url (https/ssh) from Backlog to vNext on the Diffusion board.
Jan 11 2016, 11:57 AM · Diffusion (v3)
epriestley renamed T5406: Offer read-only copy of a externally hosted repository via Diffusion clone url (https/ssh) from Feature Request: Offer read-only copy of a externally hosted repository via Diffusion clone url (https/ssh) to Offer read-only copy of a externally hosted repository via Diffusion clone url (https/ssh).
Jan 11 2016, 11:56 AM · Diffusion (v3)