Page MenuHomePhabricator
Feed Advanced Search

Aug 29 2016

swisspol updated the task description for T11469: Checking out from Phacility sometimes fails.
Aug 29 2016, 9:57 AM · Phacility, Bug Report
• szunyogh.tamas created T11549: sdefrgthzuioö.
Aug 29 2016, 8:30 AM · Bug Report
lucaro created T11548: 500 Internal Server Error when trying to clone HG repo via https.
Aug 29 2016, 7:41 AM · Bug Report

Aug 28 2016

michaeljs1990 triaged T11547: Paste query results do not display actual length of listed Pastes (but always "5 lines" if Paste is longer) as Wishlist priority.
Aug 28 2016, 6:44 PM · Contributor Onboarding, Paste, Bug Report
michaeljs1990 triaged T11517: arc lint missed some error reported from the golint as Low priority.
Aug 28 2016, 6:38 PM · Bug Report (Needs Information), Arcanist, Lint
michaeljs1990 claimed T11517: arc lint missed some error reported from the golint.
Aug 28 2016, 6:37 PM · Bug Report (Needs Information), Arcanist, Lint
michaeljs1990 added a comment to T11517: arc lint missed some error reported from the golint.

You are going to have to post more information. I am able to correctly see the error you posted following these steps.

Aug 28 2016, 6:37 PM · Bug Report (Needs Information), Arcanist, Lint
michaeljs1990 added a comment to T11517: arc lint missed some error reported from the golint.

Can you post your arclint file

Aug 28 2016, 5:46 PM · Bug Report (Needs Information), Arcanist, Lint
chad added a comment to T11517: arc lint missed some error reported from the golint.

See Planning for timelines on how we work. If this issue is urgent and blocking your company, see Support Resources for prioritization options.

Aug 28 2016, 4:22 PM · Bug Report (Needs Information), Arcanist, Lint
rayzyar added a comment to T11517: arc lint missed some error reported from the golint.

@chad could u take a look again

Aug 28 2016, 3:45 PM · Bug Report (Needs Information), Arcanist, Lint
swisspol updated the task description for T11469: Checking out from Phacility sometimes fails.
Aug 28 2016, 8:04 AM · Phacility, Bug Report
michaeljs1990 claimed T11547: Paste query results do not display actual length of listed Pastes (but always "5 lines" if Paste is longer).
Aug 28 2016, 6:50 AM · Contributor Onboarding, Paste, Bug Report
michaeljs1990 added a comment to T11547: Paste query results do not display actual length of listed Pastes (but always "5 lines" if Paste is longer).

I put up a diff that fixes this however I'm not sure if it's the correct solution.

Aug 28 2016, 6:34 AM · Contributor Onboarding, Paste, Bug Report
michaeljs1990 added a revision to T11547: Paste query results do not display actual length of listed Pastes (but always "5 lines" if Paste is longer): Restricted Differential Revision.
Aug 28 2016, 6:32 AM · Contributor Onboarding, Paste, Bug Report

Aug 27 2016

aklapper created T11547: Paste query results do not display actual length of listed Pastes (but always "5 lines" if Paste is longer).
Aug 27 2016, 5:26 PM · Contributor Onboarding, Paste, Bug Report
epriestley closed T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded as Resolved by committing Restricted Diffusion Commit.
Aug 27 2016, 4:11 PM · Phacility, Files, Diffusion, Bug Report
chad merged task T11545: Burnup rate looks messy and unreadable into T1562: Build "Facts", an ETL pipeline and charting application.
Aug 27 2016, 2:49 PM · Maniphest, Bug Report
shpytyack.artem added a project to T11545: Burnup rate looks messy and unreadable: Maniphest.
Aug 27 2016, 11:22 AM · Maniphest, Bug Report
shpytyack.artem created T11545: Burnup rate looks messy and unreadable.
Aug 27 2016, 11:16 AM · Maniphest, Bug Report
avivey added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

Do you have any other repositories to try this on? Smaller ones, or some public repo from BitBucket?

Aug 27 2016, 3:23 AM · Bug Report

Aug 26 2016

epriestley renamed T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS from Launching daemons kills the system to Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.
Aug 26 2016, 11:44 PM · Documentation, Setup
jonah214 added a comment to T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.

Thanks!

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

I'll retarget this to add a setup warning about micro instances, this isn't the first time they've done surprising things (T2727, T8204, T8210).

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

I did, and it certainly seems to match up. I'm surprised it's so dramatic. I guess since it was working fine before, the sensible thing is probably to just wait for the credits to recover and go back to things on the t2 micro.

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

If you fiddle with the dropdown in the upper right you should be able to get historic data and maybe see if the graph went not-so-good around then. Doesn't seem implausible.

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

My graph certainly seems to imply that this is the issue.

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

Here's the actual credit graph in the AWS web console, under the Monitoring tab (lower right):

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

One thing we could possibly do is try to hit the AWS meta-service during startup:

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

It's hard to say because workloads differ, but I'd steer away from the CPU credited "burstable" t2.* instances based on generally negative experiences with micros in the past.

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

Hm, that's interesting. I know basically nothing about AWS, but I'll try adjusting phd.taskmasters, and if that doesn't work, I'll try using a bigger instance. Is the next size up sufficient, in your experience?

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

In AWS, my understanding is that micro instances have CPU credits and their CPU is stopped or heavily throttled if you exceed the credit hours. Has the instance exhausted its CPU credits? I think there's a graph on the console, maybe in the "CloudWatch" tab or something like that? I don't actually have any micro instances up right now to check.

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

(Is your server an underclocked Raspberry Pi emulated on an Amiga?)

EC2 t2.micro

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

(Is your server an underclocked Raspberry Pi emulated on an Amiga?)

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

Not that was printed to the console.

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

That looks pretty normal to me.

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

P2005. This was all written to stderr, not stdout. I let it run for a little while and then interrupted.

Aug 26 2016, 10:13 PM · Documentation, Setup
n0nag0n added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

Yeah sorry a " " snuck in there while I was copy/pasting it.

Aug 26 2016, 10:11 PM · Bug Report
michaeljs1990 added a comment to T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.

@jonah214 I vote for see stuff that goes by. You can use the paste application https://secure.phabricator.com/paste/ so it's easier for us to look at.

Aug 26 2016, 10:08 PM · Documentation, Setup
avivey added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.
>>> [536] <exec> $ hg log --template='{node}' --rev ''\''caac1231de800ccaa18c96
98ae72f6cb7dfca637'\'''
Aug 26 2016, 10:07 PM · Bug Report
avivey added a comment to T11539: Phabricator Mercurial Repo “too many files open” error.

I don't know a lot about hg internals, but 1,000 branches and 10,000 commits doesn't sound like too much to process.

Aug 26 2016, 10:06 PM · Bug Report
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
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
jonah214 created T11544: Add a setup warning about using "Burstable CPU" (T2) instance classes in AWS.
Aug 26 2016, 9:46 PM · Documentation, Setup
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
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
avivey added a project to T11389: Object links merged in plain text email: Remarkup.
Aug 26 2016, 8:50 PM · Remarkup, Bug Report, Mail
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
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: Restricted Differential Revision.
Aug 26 2016, 8:05 PM · Bug Report
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 T10423: Huge diffs (>8MB) should stream directly into file storage in Diffusion when downloaded: Restricted Differential Revision.
Aug 26 2016, 5:28 PM · Phacility, Files, Diffusion, Bug Report
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
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
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
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: Restricted Differential Revision.
Aug 26 2016, 3:14 PM · Phacility, Files, Diffusion, Bug Report
n0nag0n created T11539: Phabricator Mercurial Repo “too many files open” error.
Aug 26 2016, 3:10 PM · Bug Report
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
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
jacob_matt created T11536: Conduit update of int-fields fails.
Aug 26 2016, 9:07 AM · Bug Report
avivey added a comment to T10746: Aborted build in Harbormaster still shows as building in Diffusion.

I've seen this happening here too (B13428)

Aug 26 2016, 12:06 AM · Restricted Project, Harbormaster, Bug Report
avivey merged task T11528: Revision build status doesn't status when one of the build step was failed into T10746: Aborted build in Harbormaster still shows as building in Diffusion.
Aug 26 2016, 12:04 AM · Bug Report
avivey merged T11528: Revision build status doesn't status when one of the build step was failed into T10746: Aborted build in Harbormaster still shows as building in Diffusion.
Aug 26 2016, 12:04 AM · Restricted Project, Harbormaster, Bug Report

Aug 25 2016

rabahmeradi created T11528: Revision build status doesn't status when one of the build step was failed.
Aug 25 2016, 7:15 AM · Bug Report
durin42 added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

So, yes, the bundle2 thing is definitely orthogonal to this. For now, I'd suggest that if phabricator breaks bundle2, filter the response to the capabilities command and omit bundle2=.* from the response.

Aug 25 2016, 4:06 AM · Mercurial, Bug Report
epriestley closed T11526: Differential Revision herald rule for Harbormaster build fired on commandeer + close as Resolved.

The action is indistinguishable from an update from Herald's perspective. Herald runs on the state of the object, not changes to it, so "Changed file content" will just match whatever the diff changes.

Aug 25 2016, 1:06 AM · Bug Report
venky added a comment to T11526: Differential Revision herald rule for Harbormaster build fired on commandeer + close.

Commandeer is a pretty infrequent action so this doesn't matter much, but out of curiosity could I avoid this with a better Herald rule? I guess I can try Changed file content matches regexp '.'.

Aug 25 2016, 12:50 AM · Bug Report

Aug 24 2016

epriestley added a comment to T11434: Arc patch fails: EXCEPTION: (ConduitClientException) ERR-INVALID-AUTH: Unknown Error..

T7385 is probably also approximately this.

Aug 24 2016, 11:45 PM · Restricted Project, Arcanist, Bug Report
epriestley merged T7385: arc feature D123 fails with ERR-INVALID-SESSION into T11434: Arc patch fails: EXCEPTION: (ConduitClientException) ERR-INVALID-AUTH: Unknown Error..
Aug 24 2016, 11:45 PM · Restricted Project, Arcanist, Bug Report
epriestley added a comment to T11526: Differential Revision herald rule for Harbormaster build fired on commandeer + close.

Commandeer fires Herald rules so stuff like "Require Legalpad Signature" or "Add Reviewers: that intern's wrangler" work.

Aug 24 2016, 11:44 PM · Bug Report
epriestley added a comment to T11434: Arc patch fails: EXCEPTION: (ConduitClientException) ERR-INVALID-AUTH: Unknown Error..

T7074 may not be this, exactly, but they're close enough for government work.

Aug 24 2016, 11:29 PM · Restricted Project, Arcanist, Bug Report
epriestley merged T7074: `arc patch Dx` shouldn't ask for authentication if Dx is public into T11434: Arc patch fails: EXCEPTION: (ConduitClientException) ERR-INVALID-AUTH: Unknown Error..
Aug 24 2016, 11:28 PM · Restricted Project, Arcanist, Bug Report
venky updated the task description for T11526: Differential Revision herald rule for Harbormaster build fired on commandeer + close.
Aug 24 2016, 11:20 PM · Bug Report
venky created T11526: Differential Revision herald rule for Harbormaster build fired on commandeer + close.
Aug 24 2016, 11:19 PM · Bug Report
epriestley moved T11434: Arc patch fails: EXCEPTION: (ConduitClientException) ERR-INVALID-AUTH: Unknown Error. from Backlog to arc patch on the Arcanist board.
Aug 24 2016, 11:16 PM · Restricted Project, Arcanist, Bug Report
jcox added a revision to T10951: Search shows document types corresponding to disabled apps: Restricted Differential Revision.
Aug 24 2016, 10:11 PM · Contributor Onboarding, Search, Bug Report
jcox added a revision to T10951: Search shows document types corresponding to disabled apps: Restricted Differential Revision.
Aug 24 2016, 9:44 PM · Contributor Onboarding, Search, Bug Report
epriestley merged T5312: Error raised when updating a revision which has non-permitted reviewers because of policy constraints into T11388: Running "arc diff" on this instance, with a restricted owners package fails (after "arc patch").
Aug 24 2016, 7:02 PM · Bug Report
avivey renamed T11521: Request Review action should clear "accepted older" status from [Differential] "Ready to review" list doesn't include NEEDS_REVIEW revisions with reviewer status ACCEPTED_OLDER to Request Review action should clear "accepted older" status.
Aug 24 2016, 5:00 PM · Restricted Project, Differential, Bug Report
epriestley added a comment to T10252: Username autocompletion doesn't work with some non-US keyboard layouts.

Ah, thanks! I think that should be relatively easy to fix.

Aug 24 2016, 12:43 PM · Restricted Project, Bug Report
de_jean_7777 added a comment to T10252: Username autocompletion doesn't work with some non-US keyboard layouts.

This happens for everyone here in the company on windows chrome with the HRV (Croatian) layout, where the @ symbol is achieved with the right side alt (ALTGR) + "V". Think that registers ALT + CTRL + V.

Aug 24 2016, 12:37 PM · Restricted Project, Bug Report
gokulpd added a comment to T11523: Unhandled Exception ("AphrontSchemaQueryException").

Yes, I have tried to do storage upgrade but it ends up with an error.

Aug 24 2016, 11:32 AM · Bug Report (Needs Information)
epriestley added a comment to T11523: Unhandled Exception ("AphrontSchemaQueryException").

Have you tried following the instructions given in the error message?

Aug 24 2016, 11:13 AM · Bug Report (Needs Information)
gokulpd updated subscribers of T11523: Unhandled Exception ("AphrontSchemaQueryException").
Aug 24 2016, 9:55 AM · Bug Report (Needs Information)
gokulpd created T11523: Unhandled Exception ("AphrontSchemaQueryException").
Aug 24 2016, 9:55 AM · Bug Report (Needs Information)

Aug 23 2016

nornagon created T11521: Request Review action should clear "accepted older" status.
Aug 23 2016, 6:41 PM · Restricted Project, Differential, Bug Report
nfirmani created T11520: Arc Lint - Severity configuration behaves differently for different linters.
Aug 23 2016, 6:39 PM · Bug Report
jasonfsmitty added a comment to T11519: Herald unable to email the owners of a package.

Forgot to mention we're running stable week 34.

Aug 23 2016, 6:25 PM · Bug Report
epriestley added a comment to T11519: Herald unable to email the owners of a package.

Let me see if I can reproduce this, but this is probably straightforward to fix -- I suspect some code just didn't get updated when Owners packages became mailable.

Aug 23 2016, 6:25 PM · Bug Report
jasonfsmitty created T11519: Herald unable to email the owners of a package.
Aug 23 2016, 6:19 PM · Bug Report
rayzyar updated the task description for T11517: arc lint missed some error reported from the golint.
Aug 23 2016, 5:42 PM · Bug Report (Needs Information), Arcanist, Lint