Page MenuHomePhabricator
Feed Advanced Search

Jul 5 2016

epriestley added a comment to T10745: Emails are sent out about restricted changes.

The support cost for the current behavior is very small. The behavior of the system is obvious. Users can file precise tasks like this one which clearly describe the problem (or find this task):

Jul 5 2016, 1:45 PM · Restricted Project, Bug Report
• Maxim moved T3569: Make Phrequent somewhat usable from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Phrequent
• Maxim moved T5151: Allow to export query results from Phrequent from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Phrequent
• Maxim moved T2928: "arc start" / "arc feature" / "arc push" Discussion from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Arcanist, Maniphest, Phrequent
• Maxim moved T3999: Support Harvest integration for Phrequent via Doorkeeper from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Doorkeeper, Phrequent
• Maxim moved T5149: Allow to add and edit Phrequent entries manually from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Phrequent
• Maxim moved T8480: Support querying for tasks where a user has tracked time against them from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:14 PM · Phrequent, Maniphest
• Maxim moved T5150: Extend query by time restrictions from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:14 PM · Phrequent
• Maxim moved T8666: When a task is resolved with "Resolves T123", Phrequent should stop tracking time from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:14 PM · Maniphest, Phrequent
epriestley added a project to T4706: Maybe make `arc land D123` go look up the branch?: Prioritized.
Jul 5 2016, 12:37 PM · Prioritized, Restricted Project, Arcanist
epriestley added a project to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Prioritized.
Jul 5 2016, 12:37 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley merged T11272: Passphrases requires Editable By status for Read access into T4721: Improve user-facing documentation for Passphrase.
Jul 5 2016, 12:36 PM · Documentation, Passphrase
epriestley merged task T11272: Passphrases requires Editable By status for Read access into T4721: Improve user-facing documentation for Passphrase.
Jul 5 2016, 12:36 PM · Bug Report
epriestley added a comment to T11272: Passphrases requires Editable By status for Read access.

This behavior is intentional; it allows users to use a credential to configure other applications without actually having access to the plaintext.

Jul 5 2016, 12:36 PM · Bug Report
epriestley closed T9888: Provide guidance in the documentation about PHP7, a subtask of T9640: Make Phabricator compatible with PHP7, as Resolved.
Jul 5 2016, 12:33 PM · Prioritized, Restricted Project, Infrastructure
epriestley closed T9888: Provide guidance in the documentation about PHP7 as Resolved by committing Restricted Diffusion Commit.
Jul 5 2016, 12:33 PM · Documentation
epriestley closed T11250: New Herald rule: "Differential revision accepted by" as Resolved.

Great! I think the Herald rule is also reasonable to build down the line if some future user arrives here via search and has a different use case for it in the future.

Jul 5 2016, 11:44 AM · Feature Request
• minishine updated the task description for T11273: Update error while Repository Updates .
Jul 5 2016, 11:34 AM · Bug Report
• minishine updated the task description for T11273: Update error while Repository Updates .
Jul 5 2016, 11:28 AM · Bug Report
• minishine updated the task description for T11273: Update error while Repository Updates .
Jul 5 2016, 11:28 AM · Bug Report
• minishine created T11273: Update error while Repository Updates .
Jul 5 2016, 11:25 AM · Bug Report
vpetersson created T11272: Passphrases requires Editable By status for Read access.
Jul 5 2016, 11:24 AM · Bug Report
vit0p updated Q439: There is a conduit method to retrieve differentials attached to a maniphest task? from to There is a conduit method to retrieve differentials attached to a maniphest task?.
Jul 5 2016, 10:51 AM · Differential, Maniphest, Conduit
• emtized awarded T11269: Repository update command fails on empty Git repositories a Like token.
Jul 5 2016, 9:52 AM · Diffusion, Bug Report
HuMMeL621 awarded T11250: New Herald rule: "Differential revision accepted by" a Love token.
Jul 5 2016, 9:17 AM · Feature Request
benjumanji added a comment to Q437: How do I debug an ldap issue? (Answer 414).

That did the trick, thanks!

Jul 5 2016, 9:13 AM
benjumanji closed Q437: How do I debug an ldap issue? as resolved.
Jul 5 2016, 9:13 AM
claassen added a comment to T11250: New Herald rule: "Differential revision accepted by".

This is awesome! I would've never found the ability to ignore emails from a project on my own. That's just the feature I needed to set this up correctly. Thanks guys for pointing me in the right direction.

Jul 5 2016, 7:28 AM · Feature Request
lucaro added a comment to T11267: Commit graph broken for Mercurial repositories.

Thanks for the quick response. I'll let you know as soon as the change gets merged into stable.

Jul 5 2016, 6:54 AM · Diffusion, Bug Report
Mnkras added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Of course! Thanks for poking around!

Jul 5 2016, 5:55 AM · Daemons, Bug Report
hach-que added a comment to Q438: Is there a concrete relationship between harbormaster artifact types and drydock resource types? (Answer 415).

The constants aren't interchangeable no; one is a constant for the Harbormaster artifact type and one is a constant for a Drydock resource type. They're two different sets of types.

Jul 5 2016, 5:48 AM
yelirekim added a comment to Q438: Is there a concrete relationship between harbormaster artifact types and drydock resource types? (Answer 415).

See edit for clarification.

Jul 5 2016, 5:21 AM
yelirekim updated the question details for Q438: Is there a concrete relationship between harbormaster artifact types and drydock resource types?.
Jul 5 2016, 5:20 AM · Drydock, Harbormaster
hach-que updated the answer details for Q438: Is there a concrete relationship between harbormaster artifact types and drydock resource types? (Answer 415).
Jul 5 2016, 5:16 AM
hach-que added Q438: Is there a concrete relationship between harbormaster artifact types and drydock resource types? (Answer 415).
Jul 5 2016, 5:15 AM
yelirekim updated Q438: Is there a concrete relationship between harbormaster artifact types and drydock resource types? from to Is there a concrete relationship between harbormaster artifact types and drydock resource types?.
Jul 5 2016, 5:10 AM · Drydock, Harbormaster
hach-que awarded T11244: Identify additional tokens to add to Phabricator a Y So Serious token.
Jul 5 2016, 4:49 AM · Celerity, Tokens
MZMcBride added a comment to T10745: Emails are sent out about restricted changes.

I received this e-mail recently:

Jul 5 2016, 4:23 AM · Restricted Project, Bug Report
epriestley awarded T11244: Identify additional tokens to add to Phabricator a Dat Boi token.
Jul 5 2016, 3:38 AM · Celerity, Tokens
thoughtpolice awarded T11244: Identify additional tokens to add to Phabricator a 100 token.
Jul 5 2016, 3:36 AM · Celerity, Tokens
ofbeaton added a comment to T10745: Emails are sent out about restricted changes.

We have started to encounter this as we use phabricator in a devops environment with many teams from a large organisation.

Jul 5 2016, 3:09 AM · Restricted Project, Bug Report
epriestley added a comment to T10538: Bridge GitHub to Phabricator.

Two-way.

Jul 5 2016, 1:52 AM · Nuance, GitHub
adrelanos added a comment to T10538: Bridge GitHub to Phabricator.

Will this be a one way or two way bridge?

Jul 5 2016, 1:48 AM · Nuance, GitHub
epriestley added a comment to T10745: Emails are sent out about restricted changes.

There are several classes of transaction like this. One is that all actions are restricted:

Jul 5 2016, 1:44 AM · Restricted Project, Bug Report
Danny_B added a comment to T11271: Inconsistency in project membership displaying.

Will try to get more details for you from colleagues.

Jul 5 2016, 1:32 AM · Projects, People, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

(And thanks for providing access to an environment where the issue reproduces!)

Jul 5 2016, 1:02 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

@Mnkras, I think I've undone everything I did and terminated all sessions on the host. You can revoke my key at your convenience.

Jul 5 2016, 1:00 AM · Daemons, Bug Report
epriestley added a comment to T11271: Inconsistency in project membership displaying.

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

Jul 5 2016, 12:59 AM · Projects, People, Bug Report
epriestley added a revision to T9640: Make Phabricator compatible with PHP7: Unknown Object (Differential Revision).
Jul 5 2016, 12:56 AM · Prioritized, Restricted Project, Infrastructure
epriestley added a revision to T9888: Provide guidance in the documentation about PHP7: Unknown Object (Differential Revision).
Jul 5 2016, 12:56 AM · Documentation
epriestley changed the name of U28 PHP7 from to PHP7.
Jul 5 2016, 12:50 AM
epriestley renamed T9640: Make Phabricator compatible with PHP7 from Ensure PHP7 compatiblity to Phabricator PHP 7 Compatibility.
Jul 5 2016, 12:49 AM · Prioritized, Restricted Project, Infrastructure
epriestley added a comment to T9640: Make Phabricator compatible with PHP7.

T11270 describes a major issue with asynchronous signal handling in PHP 7 which I don't believe we have a reasonable workaround for.

Jul 5 2016, 12:43 AM · Prioritized, Restricted Project, Infrastructure
eadler added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I was writing my reply while you were writing the above. Just figured I'd mention it.

Jul 5 2016, 12:42 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Yes, that's synchronous signal handling.

Jul 5 2016, 12:42 AM · Daemons, Bug Report
eadler added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

note that in the above we're explicitly handling signals. The signal_dispatch call must be placed inside of some activity loop.

Jul 5 2016, 12:41 AM · Daemons, Bug Report
eadler added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

function onsignal($signal) {

echo "Got SIGINT.\n";
  exit;

}

Jul 5 2016, 12:40 AM · Daemons, Bug Report
epriestley added a subtask for T9640: Make Phabricator compatible with PHP7: T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.
Jul 5 2016, 12:39 AM · Prioritized, Restricted Project, Infrastructure
epriestley added a parent task for T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands: T9640: Make Phabricator compatible with PHP7.
Jul 5 2016, 12:39 AM · Daemons, Bug Report
epriestley renamed T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands from Daemons fail to stop and must be terminated to PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.
Jul 5 2016, 12:38 AM · Daemons, Bug Report
Danny_B added projects to T11271: Inconsistency in project membership displaying: People, Projects.

Just found another account disabled couple days ago which also says that doesn't belong to any projects, but some projects list him as a member. So the issue is in current version still present.

Jul 5 2016, 12:38 AM · Projects, People, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Yes, this reproduces on a local build of PHP 7.0.8. Here's what I did:

Jul 5 2016, 12:38 AM · Daemons, Bug Report
Danny_B created T11271: Inconsistency in project membership displaying.
Jul 5 2016, 12:36 AM · Projects, People, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I'm building PHP 7.0.8 locally. In the meantime, I found these possibly similar reports:

Jul 5 2016, 12:27 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Here's a smaller, self-contained test case:

Jul 5 2016, 12:15 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I produced this test script to try to see about version issues via 3v4l.org, but you can't write files there. Requiring C as part of a different file is important, so I don't see a way to use that.

Jul 5 2016, 12:06 AM · Daemons, Bug Report

Jul 4 2016

epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I put the global signal handler in run(), unresponsive.

Jul 4 2016, 11:52 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I no-op'd the constructor and put the logic in run(), still unresponsive.

Jul 4 2016, 11:44 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I disabled the entire actual daemon behavior, still unresponsive.

Jul 4 2016, 11:40 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Removing the SIGINT handler entirely made it responsive.

Jul 4 2016, 11:34 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I changed the SIGINT handler to exit immediately, still unresponsive to SIGINT:

Jul 4 2016, 11:32 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

This trivial script works correctly:

Jul 4 2016, 11:29 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

The PhutilNiceDaemon daemon also does not respond to ^C.

Jul 4 2016, 11:24 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

The debug daemon does not appear to respond to an external SIGTERM (via kill <pid> either).

Jul 4 2016, 11:21 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

With --trace, the daemon isn't doing any work, and also isn't hung (just normal queries to look at the queue).

Jul 4 2016, 11:20 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I ran bin/phd stop and saw similar behavior:

Jul 4 2016, 11:17 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

@Mnkras gave me access to the host.

Jul 4 2016, 11:03 PM · Daemons, Bug Report
eadler moved T4706: Maybe make `arc land D123` go look up the branch? from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 10:50 PM · Prioritized, Restricted Project, Arcanist
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I can't reproduce that either:

Jul 4 2016, 10:49 PM · Daemons, Bug Report
Mnkras added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Im really not sure, when I run restart this is what my daemon page looks like:

Jul 4 2016, 10:48 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

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

Jul 4 2016, 10:43 PM · Daemons, Bug Report
chad awarded T10745: Emails are sent out about restricted changes a Burninate token.
Jul 4 2016, 9:27 PM · Restricted Project, Bug Report
rodbegbie awarded T10745: Emails are sent out about restricted changes a Like token.
Jul 4 2016, 9:26 PM · Restricted Project, Bug Report
rodbegbie added a comment to T9161: How can we fix "too much mail"?.

Given that I just received a useless eadler moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board. email notification for this ticket, I suggest that T10745 be considered as part of email reduction.

Jul 4 2016, 9:26 PM · Restricted Project, User Preferences, Mail
eadler moved T4631: Allow Differential to raise warnings on the server side via Conduit from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:21 PM · Restricted Project, Differential, Arcanist
eadler moved T8090: Allow Harbormaster to perform change handoff in a defensible way from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:20 PM · Haskell.org, Restricted Project, Harbormaster
eadler moved T10429: Projects query shows projects and subprojects as unrelated from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Feature Request
eadler moved T10429: Projects query shows projects and subprojects as unrelated from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Feature Request
eadler moved T1084: Preview what's new in a diff when attaching it to an existing revision from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Arcanist, Differential
eadler moved T10622: Auto add reviewers based on custom logic from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Arcanist, Feature Request
eadler moved T6072: Improve usability of `arc diff` from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Documentation, Arcanist
eadler moved T10681: Add Phurl Conduit API methods from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Contributor Onboarding, Restricted Project, Conduit, Phurl
eadler moved T10622: Auto add reviewers based on custom logic from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Arcanist, Feature Request
eadler moved T10671: Implement `badges.search` and `badges.edit` Conduit API methods from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Conduit, Badges (v0)
eadler moved T7488: Add shell completion support to PhutilArgumentWorkflow from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, libphutil
eadler moved T10650: `arc land` can raise misleading error message if a branch exists in the remote but has never been fetched from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Arcanist
eadler moved T2549: Support linking multiple external accounts from the same provider with one Phabricator account from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Restricted Project, LDAP, Auth, OAuthServer
eadler moved T5791: Write Herald rules for outbound mail from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Restricted Project, Restricted Project, FreeBSD, Mail
eadler moved T10225: arc patch --nobranch leaves submodules in the wrong state from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Haskell.org, Restricted Project, Arcanist, Bug Report
eadler moved T10517: Make Harbormaster build variables extensible from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:19 PM · Harbormaster, Restricted Project, Feature Request