Page MenuHomePhabricator
Feed All Stories

Aug 26 2016

jonah214 added a comment to T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.

@michaeljs1990:

do you have any more information from the logs?

Should I post the output from bin/phd log? I can't even tell where the actual logfiles are.

Aug 26 2016, 10:05 PM · Documentation, Setup
epriestley added a comment to T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.

Does /usr/bin/env php -v also print 5.6, versus 7?

Aug 26 2016, 10:01 PM · Documentation, Setup
avivey added a comment to Q473: Arcanist 404 when installing certificate.

can you add --trace to arc, to see exactly what's failing? /api/ is 404ing for good installs too, so I'm guessing it's actually hitting a different uri.

Aug 26 2016, 9:57 PM
chad added a comment to T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.

I'm running 16.04 with PHP5.6 as well, haven't seen this issue. Here's my installed PHP info if that helps. Might want to verify what PHP your webserver is running, not just CLI.

Aug 26 2016, 9:55 PM · Documentation, Setup
michaeljs1990 added a comment to T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.

do you have any more information from the logs? This also seems like more a question than a bug report since daemons not working right is rather hard to debug.

Aug 26 2016, 9:49 PM · Documentation, Setup
michaeljs1990 updated the task description for T11543: Provide an on-page workflow for searching for objects to reference while typing a Remarkup comment.
Aug 26 2016, 9:46 PM · Restricted Project, Remarkup, Feature Request
jonah214 created T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.
Aug 26 2016, 9:46 PM · Documentation, Setup
chad added a comment to Q473: Arcanist 404 when installing certificate.

Server side routing would be my next guess. We regex all urls and map them to controllers internally. Though we should raise a setup warning if they're not set up correctly.

Aug 26 2016, 9:34 PM
n0nag0n added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

/var/log/httpd/error_log shows nothing with proc_open()

Aug 26 2016, 9:33 PM · Bug Report
gracicot added a comment to Q473: Arcanist 404 when installing certificate.

I noticed that my host regex matcher differs from the configuration tutorial found here: https://secure.phabricator.com/book/phabricator/article/configuration_guide/#webserver-configuring-li

Aug 26 2016, 9:26 PM
gracicot added a comment to Q473: Arcanist 404 when installing certificate.

I see. Good to know.

Aug 26 2016, 9:20 PM
jcox added a revision to T11532: Replace paste language selection dropdown with a typeahead: Unknown Object (Differential Revision).
Aug 26 2016, 9:09 PM · Paste, Contributor Onboarding
n0nag0n added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

Well I typed in one of those...

Aug 26 2016, 9:05 PM · Bug Report
chad added a comment to Q473: Arcanist 404 when installing certificate.

See T4200 for more details if you’re curious why we don’t currently support
OS packaging / install scripts.

Aug 26 2016, 9:03 PM
gracicot added a comment to Q473: Arcanist 404 when installing certificate.

The aur package actually clone from your git, so versioning should not be a problem.

Aug 26 2016, 8:59 PM
avivey added a project to T11389: Object links merged in plain text email: Remarkup.
Aug 26 2016, 8:50 PM · Remarkup, Bug Report, Mail
chad edited this Badge.
Aug 26 2016, 8:45 PM
• amu awarded T11520: Arc Lint - Severity configuration behaves differently for different linters a Dat Boi token.
Aug 26 2016, 8:41 PM · Bug Report
avivey added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

If you try to run /usr/bin/python /usr/bin/hg log --template={node} --rev 'some rev hash' manually in /projects/phabricator/2, does it break or do you get anything useful back?

Aug 26 2016, 8:40 PM · Bug Report
chad added a comment to T11523: Unhandled Exception ("AphrontSchemaQueryException").

How can we reproduce this error locally? Bug reports here require reproduction steps, not just the error message.

Aug 26 2016, 8:31 PM · Bug Report (Needs Information)
chad edited projects for T11523: Unhandled Exception ("AphrontSchemaQueryException"), added: Bug Report (Needs Information); removed Bug Report.
Aug 26 2016, 8:31 PM · Bug Report (Needs Information)
chad added a comment to T11543: Provide an on-page workflow for searching for objects to reference while typing a Remarkup comment.

It might be interesting to have slash-triggers for objects for searching. Right now we don't have a great way (since we have lots of objects) of providing a shortcut.

Aug 26 2016, 8:23 PM · Restricted Project, Remarkup, Feature Request
n0nag0n added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

Well scratch that last comment. It looks like it has pulled in commits from the last 3 days. It looks like under recent commits when I'm viewing the repo doesn't have the most recent commits from 3 days ago, but I just installed a panel on the dashboard showing the most recent commits, and all my commits area there. The panel runs off Audit, and the repo Diffusion.....maybe there's a disconnect? idk.

Aug 26 2016, 8:15 PM · Bug Report
nfirmani added a revision to T11520: Arc Lint - Severity configuration behaves differently for different linters: Unknown Object (Differential Revision).
Aug 26 2016, 8:05 PM · Bug Report
michaeljs1990 updated the task description for T11543: Provide an on-page workflow for searching for objects to reference while typing a Remarkup comment.
Aug 26 2016, 7:56 PM · Restricted Project, Remarkup, Feature Request
michaeljs1990 created T11543: Provide an on-page workflow for searching for objects to reference while typing a Remarkup comment.
Aug 26 2016, 7:51 PM · Restricted Project, Remarkup, Feature Request
chad added a revision to T11132: New Phabricator NUX: Unknown Object (Differential Revision).
Aug 26 2016, 7:42 PM · Design, Phacility, NUX
chad added a comment to Q473: Arcanist 404 when installing certificate.

Also, if you want you can try to install a cert from this server, or from a free instance on Phacility. Just to make sure it's not a server side issue.

Aug 26 2016, 7:34 PM
chad added a comment to Q473: Arcanist 404 when installing certificate.

https://secure.phabricator.com/book/phabricator/article/arcanist_quick_start/ is our guide, it's just cloning two repositorys and a PATH command, so installation should be simple. No magic.

Aug 26 2016, 7:08 PM
chad added a comment to Q473: Arcanist 404 when installing certificate.

(the same goes for Phabricator)

Aug 26 2016, 7:05 PM
chad added a comment to Q473: Arcanist 404 when installing certificate.

We don't maintain or provide any third party packages / installers. They are frequently not kept up to date. Can you install arcanist from our repositories?

Aug 26 2016, 7:04 PM
gracicot added a comment to Q473: Arcanist 404 when installing certificate.

I installed the latest arcanist through aur: https://aur.archlinux.org/packages/arcanist-git/

Aug 26 2016, 6:57 PM
epriestley added a comment to T11542: Make remarkup checklists interactive.

See some discussion in T3945.

Aug 26 2016, 6:56 PM · Remarkup, Feature Request
venky renamed T11542: Make remarkup checklists interactive from Make markdown checklists interactive to Make remarkup checklists interactive.
Aug 26 2016, 6:50 PM · Remarkup, Feature Request
venky updated the task description for T11542: Make remarkup checklists interactive.
Aug 26 2016, 6:49 PM · Remarkup, Feature Request
venky created T11542: Make remarkup checklists interactive.
Aug 26 2016, 6:47 PM · Remarkup, Feature Request
chad added a comment to Q473: Arcanist 404 when installing certificate.

What version of arcanist are you running?
If you manually put in the URI, does it work then?

Aug 26 2016, 6:42 PM
venky awarded T11540: Write something that is technically a game for Ludum Dare 36 a 100 token.
Aug 26 2016, 6:39 PM
gracicot added a comment to Q473: Arcanist 404 when installing certificate.

I edited the question to add my configuration of lighttpd. I also changed the address to reflect more the real one.

Aug 26 2016, 6:37 PM
gracicot updated the question details for Q473: Arcanist 404 when installing certificate.
Aug 26 2016, 6:37 PM
chad added a comment to Q473: Arcanist 404 when installing certificate.

Ah, true. A correct connection looks like this:

Aug 26 2016, 6:26 PM
jcox closed T11513: Convert the application selector in "pin applications" to a typeahead as Resolved by committing Restricted Diffusion Commit.
Aug 26 2016, 6:24 PM · Restricted Project, Contributor Onboarding, User Delight, Feature Request
gracicot added a comment to Q473: Arcanist 404 when installing certificate.

When I type http://myphabserver.com/api/ in the browser, I get the phabricator 404, but I assume that you shouldn't try to request the api in the browser directly.

Aug 26 2016, 6:12 PM
epriestley moved T11541: Remove getIconURI() from PhabricatorApplication from Backlog to Basic on the Contributor Onboarding board.
Aug 26 2016, 5:45 PM · Contributor Onboarding, Infrastructure, Design
epriestley created T11541: Remove getIconURI() from PhabricatorApplication.
Aug 26 2016, 5:45 PM · Contributor Onboarding, Infrastructure, Design
epriestley added a comment to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded.

D16460 is marked as fixing this, and we always send data via Files afterwards. This is different from the approach I'd outlined above (no mode where we continue to return text directly), but two additional considerations have arisen:

Aug 26 2016, 5:32 PM · Phacility, Files, Diffusion, Bug Report
epriestley added a revision to T11524: Import-time diff generation fails for particular commits in cluster mode: Unknown Object (Differential Revision).
Aug 26 2016, 5:28 PM · Phacility, Clusters, Daemons, Diffusion
epriestley added a revision to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded: Unknown Object (Differential Revision).
Aug 26 2016, 5:28 PM · Phacility, Files, Diffusion, Bug Report
jcox added a revision to T11513: Convert the application selector in "pin applications" to a typeahead: Unknown Object (Differential Revision).
Aug 26 2016, 5:14 PM · Restricted Project, Contributor Onboarding, User Delight, Feature Request
n0nag0n added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

Yeah I just double checked, it is a full clone of the original repo at the time it was at, but looking at it now, it hasn't picked up a new commit for 3 days.

Aug 26 2016, 4:31 PM · Bug Report
epriestley added a comment to T11336: Provide a stable address range for Phacility outbound requests.

Yes, that's correct.

Aug 26 2016, 4:22 PM · Customer Impact
• rileytg added a comment to T11336: Provide a stable address range for Phacility outbound requests.

This would be for whitelist/whitelisting IPs. For example creating a security group on AWS that only allows 443 (https) traffic from a phabricator cluster to a jenkins instance.

Aug 26 2016, 4:21 PM · Customer Impact
avivey added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

Than probably not a reasonable case of "too many open files".

Aug 26 2016, 4:21 PM · Bug Report
chad added Q474: Can chatrooms be configured to send message upon RETURN press? (Answer 443).
Aug 26 2016, 4:19 PM
ReynoldsCahoon updated Q474: Can chatrooms be configured to send message upon RETURN press? from to Can chatrooms be configured to send message upon RETURN press?.
Aug 26 2016, 4:16 PM · Conpherence
ReynoldsCahoon added a comment to T11540: Write something that is technically a game for Ludum Dare 36.

Also you can sell No Man's Sky for $60 nowadays so I think the bar has dropped a lot.

Aug 26 2016, 4:11 PM
epriestley added a comment to T11513: Convert the application selector in "pin applications" to a typeahead.

Oh! var_dump(...) or print_r(...) should send stuff to a big pink header on the page.

Aug 26 2016, 3:59 PM · Restricted Project, Contributor Onboarding, User Delight, Feature Request
chad added a comment to T11513: Convert the application selector in "pin applications" to a typeahead.

print_r($variable);
look at web page.

Aug 26 2016, 3:54 PM · Restricted Project, Contributor Onboarding, User Delight, Feature Request
jcox added a comment to T11513: Convert the application selector in "pin applications" to a typeahead.

@epriestley PHP noob question: what do you do for debugging? I'm having a hard time just looking at the values of the variables involved here:

$pins = $request->getStrList('pin');
$pin = head($pins);
$app = id(new PhabricatorApplicationQuery())
  ->setViewer($viewer)
  ->withPHIDs(array($phid))
  ->executeOne();
if ($app) {
  $pin = get_class($app);
}
Aug 26 2016, 3:45 PM · Restricted Project, Contributor Onboarding, User Delight, Feature Request
n0nag0n added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.
cat /proc/sys/fs/file-max
1620069
Aug 26 2016, 3:38 PM · Bug Report
simevo added a comment to T11538: the conduit API is awkward to use when one must send arrays.

we'll discuss internally next week and let you know. thanks for now

Aug 26 2016, 3:31 PM · Bug Report
epriestley created T11540: Write something that is technically a game for Ludum Dare 36.
Aug 26 2016, 3:27 PM
epriestley created an object: C147: Ludum Dare 36.
Aug 26 2016, 3:19 PM
avivey added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

You're work-around is fine.
What does cat /proc/sys/fs/file-max say? Is it possible you just have too many open files?

Aug 26 2016, 3:15 PM · Bug Report
epriestley added a revision to T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded: Unknown Object (Differential Revision).
Aug 26 2016, 3:14 PM · Phacility, Files, Diffusion, Bug Report
epriestley added a revision to T11524: Import-time diff generation fails for particular commits in cluster mode: Unknown Object (Differential Revision).
Aug 26 2016, 3:14 PM · Phacility, Clusters, Daemons, Diffusion
n0nag0n created T11539: Phabricator Mercurial Repo “too many files open” error.
Aug 26 2016, 3:10 PM · Bug Report
epriestley added a comment to T11513: Convert the application selector in "pin applications" to a typeahead.

Ah, right -- I think those are both expected and that you're on the right track. You should be able to use $request->getStrList(...) instead of $request->getStr(...) to read the list, then head() to grab the first (item if it exists).

Aug 26 2016, 3:09 PM · Restricted Project, Contributor Onboarding, User Delight, Feature Request
jcox added a comment to T11513: Convert the application selector in "pin applications" to a typeahead.

Alright I'm making a bit of progress here. Posting this update on the chance that there's something obvious I'm missing. The AphrontFormTokenizerControl behaves a little bit differently than the AphrontFormSelectControl. Specifically, this is what the form post data looks like the old way (SelectControl):

Aug 26 2016, 3:03 PM · Restricted Project, Contributor Onboarding, User Delight, Feature Request
epriestley closed T11537: Survive import of artificially manufactured Git commits with 64-bit timestamps as Resolved by committing Restricted Diffusion Commit.
Aug 26 2016, 2:39 PM · Diffusion
epriestley closed T9235: Add a warning about "always_populate_raw_post_data" being misconfigured as Resolved by committing Restricted Diffusion Commit.
Aug 26 2016, 2:38 PM · Contributor Onboarding, Setup
epriestley added a comment to T11538: the conduit API is awkward to use when one must send arrays.

I'm happy to rewrite your script so it works, but we charge $1,500/hr for custom development (see Consulting) because time spent helping individual users with unique problems takes away from developing and improving Phabricator. Are you interested in this?

Aug 26 2016, 2:29 PM · Bug Report
simevo added a comment to T11538: the conduit API is awkward to use when one must send arrays.

this is fine since the way python serializes over HTTP is certainly peculiar; but what about JSON ?

Aug 26 2016, 2:00 PM · Bug Report
epriestley closed T11538: the conduit API is awkward to use when one must send arrays as Invalid.

This is not a bug.

Aug 26 2016, 1:46 PM · Bug Report
chad added a comment to T11538: the conduit API is awkward to use when one must send arrays.

This doesn't look like a bug report nor a feature request. I don't know what you want us to do with this.

Aug 26 2016, 1:43 PM · Bug Report
avivey added a comment to T11538: the conduit API is awkward to use when one must send arrays.
Aug 26 2016, 1:43 PM · Bug Report
chad added a comment to Q473: Arcanist 404 when installing certificate.

That doesn't look like a Phabricator 404 page.

Aug 26 2016, 1:39 PM
simevo created T11538: the conduit API is awkward to use when one must send arrays.
Aug 26 2016, 1:36 PM · Bug Report
avivey merged T11536: Conduit update of int-fields fails into T10456: Conduit interprets all arguments from curl requests as strings.
Aug 26 2016, 1:26 PM · Restricted Project, Conduit, Bug Report
avivey merged task T11536: Conduit update of int-fields fails into T10456: Conduit interprets all arguments from curl requests as strings.
Aug 26 2016, 1:26 PM · Bug Report
epriestley added a revision to T11524: Import-time diff generation fails for particular commits in cluster mode: Unknown Object (Differential Revision).
Aug 26 2016, 12:36 PM · Phacility, Clusters, Daemons, Diffusion
epriestley added a revision to T11537: Survive import of artificially manufactured Git commits with 64-bit timestamps: Unknown Object (Differential Revision).
Aug 26 2016, 12:24 PM · Diffusion
epriestley added a comment to T11537: Survive import of artificially manufactured Git commits with 64-bit timestamps.

Introducing an epoch64 type ends up being pretty messy:

Aug 26 2016, 12:13 PM · Diffusion
gracicot updated Q473: Arcanist 404 when installing certificate from to Arcanist 404 when installing certificate.
Aug 26 2016, 12:12 PM
epriestley added a comment to T11537: Survive import of artificially manufactured Git commits with 64-bit timestamps.

Perhaps a better solution is to introduce an epoch64 datatype and store these timestamps faithfully, since they're at least sort of possible to write and do have a relatively unambiguous meaning. Introducing a new type instead of changing the meaning of the existing epoch would let us accommodate this without requiring tons and tons of migrations or exposing us to a lot of weird fallout.

Aug 26 2016, 12:01 PM · Diffusion
epriestley added a comment to T11537: Survive import of artificially manufactured Git commits with 64-bit timestamps.
$ git cat-file commit HEAD > message.original
$ nano message.original # Adjust timestamps to crazy values.
$ git hash-object -t commit --stdin -w < message.original
23c3584bd6030c4f8b9ad9ce5419d218990b75a5
$ git reset --hard 23c3584bd6030c4f8b9ad9ce5419d218990b75a5
HEAD is now at 23c3584 Time travel! Spooky!
$ git show
commit 23c3584bd6030c4f8b9ad9ce5419d218990b75a5
Author: epriestley <git@epriestley.com>
Date:   Mon Jul 22 06:20:00 48622 -0700
Aug 26 2016, 11:59 AM · Diffusion
epriestley added a comment to T11537: Survive import of artificially manufactured Git commits with 64-bit timestamps.

After some effort, I'm unable to figure out how to generate these commits using vanilla git. I can't figure out how get them with --date or GIT_COMMITTER_DATE to git commit: dates after 2100 are treated like the current date.

Aug 26 2016, 11:53 AM · Diffusion
epriestley created T11537: Survive import of artificially manufactured Git commits with 64-bit timestamps.
Aug 26 2016, 11:39 AM · Diffusion
epriestley moved T4369: Phabricator HTTP repository hosting has fairly severe scalability limits from Backlog to v3 on the Diffusion board.
Aug 26 2016, 11:10 AM · Diffusion (v3), Restricted Project
oskari.petas2 added a comment to T10004: ApplicationEditor Errata.

I'd be interested in the Refine the workboard/form interaction idea.

Aug 26 2016, 10:50 AM · ApplicationEditor
jacob_matt created T11536: Conduit update of int-fields fails.
Aug 26 2016, 9:07 AM · Bug Report
scott.horsley awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Aug 26 2016, 3:32 AM · Workboards (v3), Restricted Project
chad added a comment to T1895: Disable live previews .

Full screen disables live previews in any remarkup box.

Aug 26 2016, 2:40 AM · User Preferences
gslin added a comment to T1895: Disable live previews .

I think this task should be reopened, because there is no way to disable live preview in desktop currently.

Aug 26 2016, 2:38 AM · User Preferences
samking added a comment to T11535: Make it possible for arc land to disable upstream branch tracking.

Developers want to push their local branches to GitHub for a variety of reasons:

  • so that if their computer dies, their work doesn't disappear
  • so that they can work across multiple machines (eg, if they have a separate computer at home vs at work, or if they have a cloud development environment separate from their local workstation)
  • to show their work to other people before sending out a Phabricator revision (which they could still do with Phabricator, but some folks feel like they should only use Phabricator once something is ready for formal review)
  • because at a previous organization, they always pushed their branches, so they're in the habit of doing it
Aug 26 2016, 1:57 AM · Arcanist, Restricted Project
epriestley added a comment to T11535: Make it possible for arc land to disable upstream branch tracking.

Why are you tracking branches you don't want to land onto?

Aug 26 2016, 1:28 AM · Arcanist, Restricted Project
epriestley added a comment to T11524: Import-time diff generation fails for particular commits in cluster mode.

The particular issue is that that the change removed a file containing the bytes ... 0xE5 0x76 .... The 0xE5 is probably an å in Latin 1.

Aug 26 2016, 1:25 AM · Phacility, Clusters, Daemons, Diffusion
epriestley added a comment to T11524: Import-time diff generation fails for particular commits in cluster mode.

I extracted the response body prior to encoding as a serialized()'d object so I can narrow down what's causing the encoding to fail and it looks reasonable, looking like it's almost certainly a UTF8 issue now.

Aug 26 2016, 1:12 AM · Phacility, Clusters, Daemons, Diffusion
epriestley added a comment to T11524: Import-time diff generation fails for particular commits in cluster mode.

Not too surprisingly, the response is empty. AphrontResponse->encodeJSONForHTTPResponse() uses json_encode() and does not detect UTF8-related encoding failures, so the chance that this is a UTF8 issue is pretty high. We should also probably fix that so fail earlier and more explicitly (and don't emit HTTP 200).

Aug 26 2016, 1:04 AM · Phacility, Clusters, Daemons, Diffusion
hach-que closed T11531: Support http.cookiefile for Diffusion mirrors as Invalid.

Oh, so for the hell of it I tried using the cookie data (it's in a username=token format) as the basic HTTP authentication credentials, and it appears to work! I have no idea why Google tries to get you to set up cookies when basic HTTP authentication works just fine.... :/

Aug 26 2016, 1:02 AM · Feature Request