Page MenuHomePhabricator
Feed All Stories

Jul 24 2016

chad added a comment to T11368: Allow to watch only single column of the workboard.

I understand a high level problem of "i get too much mail when watching a project", but I don't really have a picture of the specific "root problem". We don't really want to guess why a specific column is more interesting. Knowing the background of the project and columns might provide some insight. Maybe they need to "sign off" for instance in anything that comes into a specific column, like "QA". We just don't know, so it's hard to understand what the correct upstream solution is without these details.

Jul 24 2016, 4:31 PM · Restricted Project, Maniphest, Workboards, Feature Request
Danny_B added a comment to T11368: Allow to watch only single column of the workboard.

I don't have any idea where further/deeper to go than to what I already described in the description:
User is not interested in being notified about all tasks of the given tag, but only on those which are in particular column.
If you ask "why", I can imagine perhaps "I don't want to be spammed by things I am not interested at".
Perhaps might yet be worth to note, that it obviously applies to boards sorted by categories/area, not progress-wise (Kanban).
I can also assume it is related to discouraging of "tracking tasks" (ie. due to many issues with task graphs) which are typically exactly equal to such workboard column - they "monitor" certain area of the project, so user is subscribed to them, but not watching the entire project.
I don't know what to add more. Better if you ask.

Jul 24 2016, 3:56 PM · Restricted Project, Maniphest, Workboards, Feature Request
chad added a comment to T11368: Allow to watch only single column of the workboard.

You should see this information about the feature request form:

Jul 24 2016, 3:14 PM · Restricted Project, Maniphest, Workboards, Feature Request
chad added a comment to T11368: Allow to watch only single column of the workboard.

@Danny_B Are you aware we don't take feature requests that omit root problems?

Jul 24 2016, 3:07 PM · Restricted Project, Maniphest, Workboards, Feature Request
Danny_B created T11368: Allow to watch only single column of the workboard.
Jul 24 2016, 3:00 PM · Restricted Project, Maniphest, Workboards, Feature Request
rabahmeradi added a comment to T5463: Could not stat AuthorizedKeysCommand .

I ran into the same issue and phabricator-ssh-hook.sh and its parent directory are owned by root and the script have 755 mode. The script was in /root/phabricator/ and vcs-user doesn't have access to that directory (because the /root directory is not accessible). I put the script in /etc/phabricator and it worked.

Jul 24 2016, 2:39 PM · Diffusion
epriestley added a comment to T10329: Implement internal workflows / a build engine in Arcanist.

Here's an effort to collect use cases for the hooks I plan to break or replace:

Jul 24 2016, 1:07 PM · Restricted Project, Arcanist
epriestley added a comment to T10329: Implement internal workflows / a build engine in Arcanist.

Concretely, I think this breaks down as a mixture of modernizations and new code:

Jul 24 2016, 12:46 PM · Restricted Project, Arcanist
epriestley added a comment to T5055: Distribution mechanism for arc extensions.

Changes connected to T8116 implement the initial server-side version of this. It's still very skeletal, but we probably need to make some client changes to move forward. In particular, the next object to implement is probably Signature, but signature algorithms should live in the client since the client will need to be able to verify signatures.

Jul 24 2016, 12:15 PM · Arcanist
epriestley merged T8115: arcanist plugin system into T5055: Distribution mechanism for arc extensions.
Jul 24 2016, 11:43 AM · Arcanist
epriestley merged task T8115: arcanist plugin system into T5055: Distribution mechanism for arc extensions.
Jul 24 2016, 11:43 AM · Arcanist
epriestley added a comment to T8115: arcanist plugin system.

I'm just going to consolidate this into T5055, I think they're pretty much the same task at this point.

Jul 24 2016, 11:42 AM · Arcanist
epriestley created Packages.
Jul 24 2016, 11:34 AM
epriestley edited the content of Roadmap.
Jul 24 2016, 11:25 AM
epriestley edited the content of Roadmap.
Jul 24 2016, 11:23 AM
epriestley edited the content of Starmap.
Jul 24 2016, 11:18 AM
epriestley changed the name of U30 Starmap from to Starmap.
Jul 24 2016, 11:17 AM
epriestley edited the content of Starmap.
Jul 24 2016, 11:14 AM
epriestley created T11367: Add directories to Files.
Jul 24 2016, 11:01 AM · Restricted Project, Phragment, Files
epriestley closed T11365: TOTP challenge entry not marked numeric-only on mobile as Resolved by committing Restricted Diffusion Commit.
Jul 24 2016, 4:13 AM · Mobile, Auth, Feature Request
chad edited the content of Starmap.
Jul 24 2016, 3:50 AM
chad edited the content of Roadmap.
Jul 24 2016, 3:31 AM
witrin added a comment to Q453: Sequentialization and parallelization of Harbormaster builds (Answer 424).

I wasn't really sure. So I thought that "previous commit to build [...]" and "ancestors of that commit [...]" relates to the order in which Herald triggers, which must not be equal to the one in the repository. So Herald would trigger B-A-C and the build step Wait for Previous Commits to Build just ensures that they are processed sequentially. But regarding to your comment this was obviously a twisted thought of mine. :D

Jul 24 2016, 1:21 AM
epriestley added a comment to Q453: Sequentialization and parallelization of Harbormaster builds (Answer 424).

Given my description, what other behavior do you think it might have?

Jul 24 2016, 12:52 AM
witrin updated the question details for Q453: Sequentialization and parallelization of Harbormaster builds.
Jul 24 2016, 12:18 AM · Herald, Harbormaster
witrin added a comment to Q453: Sequentialization and parallelization of Harbormaster builds (Answer 424).

Thank you very much for your fast answer! What about the order? Does Herald/Harbormaster ensure in this case that the builds are always processed/triggered in the right order? I mean if there are three new commits A-B-C in branch develop pushed, is A build before B is build before C?

Jul 24 2016, 12:16 AM

Jul 23 2016

chad added a comment to T11366: "Edit Related Objects" is strange.

If you have the ability to edit a diff, you have the ability to edit a diff and it's relationships. That seems the simplest, most straightforward way to describe the way permissions work here.

Jul 23 2016, 11:33 PM · Maniphest, Bug Report
Luke081515.2 added a comment to T11366: "Edit Related Objects" is strange.

I expect, that I can a) don't add commits etc, no matter, if I press the button at the task nor at the commit for example, or b) I can do it at both ways.

Jul 23 2016, 11:28 PM · Maniphest, Bug Report
epriestley added a comment to T11351: Move Pholio to EditEngine and modern APIs.

(T11357 is a companion issue for Files.)

Jul 23 2016, 8:36 PM · Restricted Project, EditEngine, Conduit, Pholio, Feature Request
epriestley added a comment to T11365: TOTP challenge entry not marked numeric-only on mobile.

D16323 uses <input type="text" pattern="\d*" />, which appears to have the best behavior.

Jul 23 2016, 8:32 PM · Mobile, Auth, Feature Request
epriestley added a revision to T11365: TOTP challenge entry not marked numeric-only on mobile: Unknown Object (Differential Revision).
Jul 23 2016, 8:30 PM · Mobile, Auth, Feature Request
epriestley added a comment to T11365: TOTP challenge entry not marked numeric-only on mobile.

On iOS 9.2 in the simulator, <input type="number" /> gives you this keyboard:

Jul 23 2016, 8:27 PM · Mobile, Auth, Feature Request
epriestley added a comment to T11365: TOTP challenge entry not marked numeric-only on mobile.

Oh, there's a "Toggle Software Keyboard" option that I somehow disabled.

Jul 23 2016, 8:22 PM · Mobile, Auth, Feature Request
epriestley added a comment to T11365: TOTP challenge entry not marked numeric-only on mobile.

On iPhone Simulator, the behavior of <input type="number" /> appears to be "draw the keyboard for 1 frame and then immediately hide it".

Jul 23 2016, 8:19 PM · Mobile, Auth, Feature Request
chad added a comment to T11365: TOTP challenge entry not marked numeric-only on mobile.

Looks like an iPhone with Gboard keyboard.

Jul 23 2016, 8:18 PM · Mobile, Auth, Feature Request
epriestley added a comment to T11365: TOTP challenge entry not marked numeric-only on mobile.

Offhand, do you know if <input type="number" /> has the desired behavior on your device?

Jul 23 2016, 8:11 PM · Mobile, Auth, Feature Request
epriestley claimed T11365: TOTP challenge entry not marked numeric-only on mobile.
Jul 23 2016, 8:07 PM · Mobile, Auth, Feature Request
epriestley added Q453: Sequentialization and parallelization of Harbormaster builds (Answer 424).
Jul 23 2016, 8:06 PM
mariokri100 added Q454: Adding subscribes to a ponder question (Answer 423).
Jul 23 2016, 7:24 PM
• sshannin updated Q454: Adding subscribes to a ponder question from to Adding subscribes to a ponder question.
Jul 23 2016, 5:26 PM · Ponder
witrin updated Q453: Sequentialization and parallelization of Harbormaster builds from to Sequentialization and parallelization of Harbormaster builds.
Jul 23 2016, 2:53 PM · Herald, Harbormaster
kristo.mario added a comment to Q452: Can you be more regular in posting updates to your roadmap and starmap? (Answer 422).

In broad terms, well, you've got a good product and vibe going on and I guess I'm not the only one asking myself "when will phab have feature x and be able to do feature y". Being more transparent in that way could help better promote phab and even win more users over just by seeing what you're up to. It's easy to get addicted to your progress and to see exciting new stuff regularly.

Jul 23 2016, 2:30 PM
epriestley added Q452: Can you be more regular in posting updates to your roadmap and starmap? (Answer 422).
Jul 23 2016, 2:21 PM
nemobis added a comment to T5267: Localize Phabricator.

I've not looked into the details of the implementation you started, but because it's gettext I thought maybe some links wouldn't harm:

Jul 23 2016, 2:17 PM · Prioritized, Localization
epriestley added a comment to T11366: "Edit Related Objects" is strange.

What behavior do you expect?

Jul 23 2016, 2:05 PM · Maniphest, Bug Report
Luke081515.2 added a comment to T11366: "Edit Related Objects" is strange.

@epriestley Why? Actually I think this is confusing.

Jul 23 2016, 2:02 PM · Maniphest, Bug Report
kristo.mario updated Q452: Can you be more regular in posting updates to your roadmap and starmap? from to Can you be more regular in posting updates to your roadmap and starmap?.
Jul 23 2016, 1:31 PM · Public & Media Relations
enckse added a comment to T9640: Make Phabricator compatible with PHP7.

I would like to add, to anyone browsing this and trying to get a new instance of phabricator working on php7, to not do that (since it isn't supported and is now blocked by the software itself).

Jul 23 2016, 1:19 PM · Prioritized, Restricted Project, Infrastructure
enckse added a comment to T9640: Make Phabricator compatible with PHP7.

I recognize as a user (and a developer myself) the situation and that, as a user, I'm just complaining and not giving solutions. I also recognize you don't want to be dual-developing phabricator on multiple fronts. We're running on arch so I'm extremely biased AND recognize that saying this isn't a workable solution but because of the "bleeding edge" php levels on arch, I would prefer focus on php7 (again, super biased and I know that isn't just going to happen).

Jul 23 2016, 12:36 PM · Prioritized, Restricted Project, Infrastructure
epriestley added a comment to T9640: Make Phabricator compatible with PHP7.

What would you like us to do differently?

Jul 23 2016, 12:30 PM · Prioritized, Restricted Project, Infrastructure
enckse added a comment to T9640: Make Phabricator compatible with PHP7.

I think what is most frustrating about this is, having fought through php 7 configuration (without opening tickets, just trying to follow other tickets and doc), after an update today I'm greeted with having to edit source now to get php 7 (unsupported, I know) to continue working...

Jul 23 2016, 12:22 PM · Prioritized, Restricted Project, Infrastructure
epriestley edited the content of Changelog.
Jul 23 2016, 11:49 AM
epriestley created 2016 Week 30 (Late July).
Jul 23 2016, 11:48 AM
epriestley closed T11366: "Edit Related Objects" is strange as Invalid.

This is working as intended.

Jul 23 2016, 12:49 AM · Maniphest, Bug Report
chad added a comment to T11366: "Edit Related Objects" is strange.

Did you run into this on a local install or this install?

Jul 23 2016, 12:22 AM · Maniphest, Bug Report
Luke081515.2 created T11366: "Edit Related Objects" is strange.
Jul 23 2016, 12:01 AM · Maniphest, Bug Report

Jul 22 2016

Luke081515.2 added a comment to T11339: Calendar: Inviting / uninviting transactions omit key information.

Thanks :).

Jul 22 2016, 11:55 PM · Calendar, Bug Report
chad added a comment to T10246: Deploy Drydock in the Phacility cluster.

File a bug report please.

Jul 22 2016, 10:15 PM · Harbormaster (v3), Drydock (v3), Almanac (v2), Phacility
rabahmeradi added a comment to T10246: Deploy Drydock in the Phacility cluster.

It didn't work for me.

Jul 22 2016, 10:12 PM · Harbormaster (v3), Drydock (v3), Almanac (v2), Phacility
avivey added a revision to T10856: Small Revision page enhanchments: Unknown Object (Differential Revision).
Jul 22 2016, 10:11 PM · Restricted Project, Feature Request, Differential
• rhettelliot23 awarded F1732535: PIC a Pirate Logo token.
Jul 22 2016, 8:04 PM
chad edited projects for T11361: View/Edit Policy Problem of Parent Project Members , added: Projects, Policy; removed Maniphest.
Jul 22 2016, 6:36 PM · Policy, Projects, Bug Report
chad added a project to T11365: TOTP challenge entry not marked numeric-only on mobile: Mobile.
Jul 22 2016, 6:34 PM · Mobile, Auth, Feature Request
chad edited projects for T11365: TOTP challenge entry not marked numeric-only on mobile, added: Feature Request, Auth; removed Bug Report.
Jul 22 2016, 6:31 PM · Mobile, Auth, Feature Request
fooishbar updated the task description for T11365: TOTP challenge entry not marked numeric-only on mobile.
Jul 22 2016, 6:23 PM · Mobile, Auth, Feature Request
fooishbar created T11365: TOTP challenge entry not marked numeric-only on mobile.
Jul 22 2016, 6:22 PM · Mobile, Auth, Feature Request
avivey added a comment to T8115: arcanist plugin system.

Isn't this more or less the same issue as arc set-config needs to update either local or global configs?

Jul 22 2016, 6:15 PM · Arcanist
avivey closed T11339: Calendar: Inviting / uninviting transactions omit key information as Resolved by committing Restricted Diffusion Commit.
Jul 22 2016, 6:03 PM · Calendar, Bug Report
epriestley added a project to T4964: Windows installer for Arcanist: Windows.
Jul 22 2016, 5:34 PM · Windows, Wikimedia, Setup, Arcanist
fooishbar added a comment to T11364: arc confused by HTTP->HTTPS redirects.

Specifically, I'd like to understand what concrete pathway led you to encounter this. A list of hypothetical situations isn't useful in anticipating or recovering from broader problems.

Jul 22 2016, 5:29 PM · Arcanist, Bug Report
epriestley added a comment to T11364: arc confused by HTTP->HTTPS redirects.

Specifically, I'd like to understand what concrete pathway led you to encounter this. A list of hypothetical situations isn't useful in anticipating or recovering from broader problems.

Jul 22 2016, 5:23 PM · Arcanist, Bug Report
fooishbar added a comment to T11364: arc confused by HTTP->HTTPS redirects.

@epriestley Because it allows the reader of this report to reproduce the underlying issue without cloning anything or clobbering their .arcrc. Here are some less contrived scenarios, which are not mutually exclusive:

  • the user makes a typo (project lacking .arcconfig, or trying to use Phabricator for the first time themselves)
  • the .arcconfig comes from a time before Let's Encrypt (e.g. if you're in the middle of a git bisect that takes you back some distance)
  • the admin is just attempting to migrate people to more secure services and does not expect that services will fail (or at least not fail in such an opaque and confusing manner) in the face of a redirect
Jul 22 2016, 5:16 PM · Arcanist, Bug Report
michaeljs1990 added a comment to T11364: arc confused by HTTP->HTTPS redirects.

This looks closer to a server configuration issue to me and is actually a pretty strange behavior (mostly because i'm not sure how the redirect after it looks like a request is served works). If I go to http://secure.phabricator.com/api/user.whoami in the browser i get the following

{
result: null,
error_code: "ERR-CONDUIT-CORE",
error_info: "You are trying to save some data to Phabricator, but the request your browser made included an incorrect token. Reload the page and try again. You may need to clear your cookies. This was a Web request. This request had no CSRF token."
}

as expected or maybe not? However if i run curl http://secure.phabricator.com/api/user.whoami i get

{"result":null,"error_code":"ERR-INVALID-SESSION","error_info":"Session key is not present."}
Jul 22 2016, 5:15 PM · Arcanist, Bug Report
epriestley added a comment to T11364: arc confused by HTTP->HTTPS redirects.

Why are you running arc install-certificate with a manually entered URL?

Jul 22 2016, 5:10 PM · Arcanist, Bug Report
fooishbar added a project to T11364: arc confused by HTTP->HTTPS redirects: Arcanist.
Jul 22 2016, 5:00 PM · Arcanist, Bug Report
fooishbar created T11364: arc confused by HTTP->HTTPS redirects.
Jul 22 2016, 4:54 PM · Arcanist, Bug Report
csilvers added a comment to T8115: arcanist plugin system.

arc upgrade: Upgrade everything? This feels sort of bad/dangerous, maybe? But maybe fine? We could prompt?

Jul 22 2016, 4:41 PM · Arcanist
ivo added a comment to T11362: Arcanist tries to land onto identical remote branch instead of 'master'.

I guess you're right. We've never really questioned the behaviour of these GUI tools. We assumed that because they do this automatically and provide no way to disable it "they'll probably know what they're doing".

Jul 22 2016, 4:20 PM · Arcanist, Bug Report
epriestley added a comment to T8115: arcanist plugin system.

Part of the tension here is that any package-related command potentially has two targets: arc itself, or the current working directory (Phabricator, some third-party app you're developing, etc).

Jul 22 2016, 3:44 PM · Arcanist
epriestley added a project to T11363: Differential "revision updated" mail preference is pretty useless: Phacility.
Jul 22 2016, 3:38 PM · Customer Impact, Restricted Project, Differential, Mail
epriestley added a subtask for T10448: Modularize mail tags: T11363: Differential "revision updated" mail preference is pretty useless.
Jul 22 2016, 3:24 PM · Prioritized, Restricted Project, Mail, User Preferences, Owners, Feature Request
epriestley added a parent task for T11363: Differential "revision updated" mail preference is pretty useless: T10448: Modularize mail tags.
Jul 22 2016, 3:24 PM · Customer Impact, Restricted Project, Differential, Mail
epriestley created T11363: Differential "revision updated" mail preference is pretty useless.
Jul 22 2016, 3:24 PM · Customer Impact, Restricted Project, Differential, Mail
epriestley closed T8116: Prototype a package management application as Resolved.

None of that really does anything yet, but the next steps (signing, installing) need to have major components in arc, so I'm going to consolidate this into T8115 / T5055 and other related arc work.

Jul 22 2016, 2:53 PM
epriestley added a comment to T11362: Arcanist tries to land onto identical remote branch instead of 'master'.

(We could detect this case and fail with an error, but the root issue here seems to be that tracking is being set on a remote branch which the local branch really does not track.)

Jul 22 2016, 2:41 PM · Arcanist, Bug Report
epriestley added a comment to T11362: Arcanist tries to land onto identical remote branch instead of 'master'.

Generally, I'm very hesitant to use "will the push change anything" as a signal, because I think this is unintuitive and will create confusing false negatives. Specifically, if you do this:

Jul 22 2016, 2:38 PM · Arcanist, Bug Report
epriestley added a comment to T11362: Arcanist tries to land onto identical remote branch instead of 'master'.

Have you asked Sourcetree or Git Tower to change their behavior instead? They can compute ahead/behind without setting tracking, and it seems like setting tracking is not really correct here (the local branches do not really "track" temporary remote CI branches in the traditional sense).

Jul 22 2016, 2:36 PM · Arcanist, Bug Report
ivo added a comment to T11362: Arcanist tries to land onto identical remote branch instead of 'master'.

I'm not sure actually. Most of us use GUI tools like Sourcetree or Git Tower, which automatically set up tracking when a branch is pushed. I guess they do this to be able to show the "Commits behind / ahead" badges on branches.

Jul 22 2016, 2:33 PM · Arcanist, Bug Report
epriestley added a revision to T8116: Prototype a package management application: Unknown Object (Differential Revision).
Jul 22 2016, 2:11 PM
epriestley added a revision to T8116: Prototype a package management application: Unknown Object (Differential Revision).
Jul 22 2016, 1:37 PM
epriestley added a revision to T8116: Prototype a package management application: Unknown Object (Differential Revision).
Jul 22 2016, 1:12 PM
epriestley added a comment to T11362: Arcanist tries to land onto identical remote branch instead of 'master'.

Why does the local branch track the remote branch? Do you push using --set-upstream? If so, why?

Jul 22 2016, 12:52 PM · Arcanist, Bug Report
ivo created T11362: Arcanist tries to land onto identical remote branch instead of 'master'.
Jul 22 2016, 12:21 PM · Arcanist, Bug Report
RuralHunter created T11361: View/Edit Policy Problem of Parent Project Members .
Jul 22 2016, 6:44 AM · Policy, Projects, Bug Report
lewellyn added a comment to T9042: Diviner fails to create all books, path is error.

I see the same behavior noted by @lianghu in T9042#129510.

Jul 22 2016, 5:15 AM · Diviner
epriestley created T11360: Detect MySQL `read_only` and degrade masters to replicas.
Jul 22 2016, 3:21 AM · Contributor Onboarding, Restricted Project, Clusters
epriestley changed the name of U29 httpoxy Vulnerability Information from to httpoxy Vulnerability Information.
Jul 22 2016, 12:36 AM
epriestley created T11359: "httpoxy" HTTP Proxy Environmental Vulnerability.
Jul 22 2016, 12:36 AM · Security

Jul 21 2016

avivey edited P2001 AddSubscribersFromJIRAIssuesHeraldCustomAction.php.
Jul 21 2016, 9:54 PM · Herald
Shredder121 created an object: C145: End of UNIX time.
Jul 21 2016, 9:19 PM