Page MenuHomePhabricator
Feed All Stories

Thu, Nov 19

amckinley closed D21494: Change baseURI for Packages to avoid 404.
Thu, Nov 19, 9:48 PM
amckinley committed rPb2ab18f8f3d0: Change baseURI for Packages to avoid 404 (authored by amckinley).
Change baseURI for Packages to avoid 404
Thu, Nov 19, 9:48 PM
amckinley updated the diff for D21494: Change baseURI for Packages to avoid 404.

Trailing slash

Thu, Nov 19, 9:47 PM
epriestley requested review of D21495: When updating a Ferret search index document, reuse existing rows where possible.
Thu, Nov 19, 9:37 PM
epriestley added a revision to T13587: Ferret may exhaust AUTO_INCREMENT ID space of "ngrams" table after many reindexes: D21495: When updating a Ferret search index document, reuse existing rows where possible.
Thu, Nov 19, 9:36 PM · Search
epriestley accepted D21494: Change baseURI for Packages to avoid 404.

Seems reasonable, definitely the highest impact-per-character change I can come up with.

Thu, Nov 19, 9:35 PM
amckinley requested review of D21494: Change baseURI for Packages to avoid 404.
Thu, Nov 19, 9:33 PM
epriestley triaged T13587: Ferret may exhaust AUTO_INCREMENT ID space of "ngrams" table after many reindexes as Normal priority.
Thu, Nov 19, 8:24 PM · Search

Sat, Nov 7

epriestley requested review of D21493: Separate Herald transcripts into several different views.
Sat, Nov 7, 12:21 AM
epriestley added a revision to T13586: In Herald transcripts, surface exceptions encountered while evaluating fields: D21493: Separate Herald transcripts into several different views.
Sat, Nov 7, 12:20 AM · Herald
epriestley triaged T13586: In Herald transcripts, surface exceptions encountered while evaluating fields as Low priority.
Sat, Nov 7, 12:18 AM · Herald
epriestley closed T13298: Improve Herald profiling information available in transcripts as Resolved.

The profiling in D20566 seems satisfactory, and has been useful in resolving or understanding a few performance-related issuses.

Sat, Nov 7, 12:09 AM · Herald

Fri, Nov 6

artms requested review of D21492: Fix php code example for conduit calls.
Fri, Nov 6, 7:49 PM
artms added a comment to D19666: Improve Herald Rule execution speed for Personal Herald rules by avoiding expensive lookup for all adapters and instead use specific adapter check.

On my personal resource limited RasberryPI alike server (4x1.2ghz ARM A53, 2GB ram) running Phabricator - this changes saves ~15ms of runtime (180ms vs 167ms after) when running:

./bin/herald test --object T19 --type HeraldManiphestTaskAdapter
Fri, Nov 6, 7:38 PM
artms updated the diff for D19666: Improve Herald Rule execution speed for Personal Herald rules by avoiding expensive lookup for all adapters and instead use specific adapter check.

Rebase

Fri, Nov 6, 7:33 PM
artms requested review of D21049: Add Email command to mute notifications for object.
Fri, Nov 6, 7:11 PM
artms abandoned D20972: `arc patch --arcbundle` might require authentification if it can't find base commit.

This change is probably no longer relevant with recent large refactoring of arcanist

Fri, Nov 6, 7:10 PM
artms requested review of D21048: Do not count hidden changesets as changed changesets during diff comparison.
Fri, Nov 6, 7:07 PM
artms updated subscribers of D21491: Handle case when Differential Revision diff is an empty commit.
Fri, Nov 6, 7:03 PM
artms requested review of D21491: Handle case when Differential Revision diff is an empty commit.
Fri, Nov 6, 7:01 PM
artms requested review of D21490: Empty commit change.
Fri, Nov 6, 6:51 PM

Tue, Nov 3

epriestley closed D21488: Add a basic "harbormaster.step.search" API method.
Tue, Nov 3, 8:50 PM
epriestley committed rPbf8707d3a9bf: Add a basic "harbormaster.step.search" API method (authored by epriestley).
Add a basic "harbormaster.step.search" API method
Tue, Nov 3, 8:50 PM
epriestley closed D21489: Add a basic "harbormaster.step.edit" API method.
Tue, Nov 3, 8:50 PM
epriestley committed rP34082efb02c7: Add a basic "harbormaster.step.edit" API method (authored by epriestley).
Add a basic "harbormaster.step.edit" API method
Tue, Nov 3, 8:50 PM
epriestley added a revision to T13585: Expose "harbormaster.step.edit" and "harbormaster.step.search" over Conduit: D21488: Add a basic "harbormaster.step.search" API method.
Tue, Nov 3, 8:50 PM · Conduit, Harbormaster
epriestley updated the summary of D21488: Add a basic "harbormaster.step.search" API method.
Tue, Nov 3, 8:50 PM
epriestley added a comment to T13585: Expose "harbormaster.step.edit" and "harbormaster.step.search" over Conduit.
  • There is no way to pull custom field values with *.search. See above.
    • There is no way to edit non-custom field values with *.edit. The underlying transactions haven't been modularized yet; they should be, then EditEngine should get appropriate EditField definitions.
    • There's no way to identify which custom fields can be edited. This overlaps with T13248, but is also a Conduit API Console UI problem. The web UI should have some way to inspect parameter variations per object subtype for available subtypes.
      • As a workaround, you can try to edit an invalid field. The error message will identify valid fields.
    • There's no way to create steps with *.edit because you can't specify a step type. See T13449 for a general variation of the "type is required to initialize objects" problem.
Tue, Nov 3, 8:49 PM · Conduit, Harbormaster
epriestley requested review of D21489: Add a basic "harbormaster.step.edit" API method.
Tue, Nov 3, 8:47 PM
epriestley added a revision to T13585: Expose "harbormaster.step.edit" and "harbormaster.step.search" over Conduit: D21489: Add a basic "harbormaster.step.edit" API method.
Tue, Nov 3, 8:46 PM · Conduit, Harbormaster
epriestley renamed T13585: Expose "harbormaster.step.edit" and "harbormaster.step.search" over Conduit from Expose "harbormaster.buildstep.edit" and "harbormaster.buildstep.search" over Conduit to Expose "harbormaster.step.edit" and "harbormaster.step.search" over Conduit.
Tue, Nov 3, 8:45 PM · Conduit, Harbormaster
epriestley updated the diff for D21488: Add a basic "harbormaster.step.search" API method.
  • Use more consistent capitalization.
Tue, Nov 3, 8:38 PM
epriestley requested review of D21488: Add a basic "harbormaster.step.search" API method.
Tue, Nov 3, 8:36 PM
epriestley added a comment to T13585: Expose "harbormaster.step.edit" and "harbormaster.step.search" over Conduit.

HarbormasterBuildStepCoreCustomField emits no fields if a BuildStep has no attached Implementation object. This dates from D15352 and is conceptually reasonable ("don't crash if a build step exists but the implementation no longer does") but should probably be represented differently (e.g., an explicit "InvalidImplementation").

Tue, Nov 3, 8:18 PM · Conduit, Harbormaster
epriestley triaged T13585: Expose "harbormaster.step.edit" and "harbormaster.step.search" over Conduit as Normal priority.
Tue, Nov 3, 8:17 PM · Conduit, Harbormaster
epriestley renamed T13100: PCRE segfaults readily with default "pcre.backtrack_limit" and "pcre.recursion_limit" values from PCRE segfaults readily with default "pcre.backtrace_limit" and "pcre.recursion_limit" values to PCRE segfaults readily with default "pcre.backtrack_limit" and "pcre.recursion_limit" values.
Tue, Nov 3, 7:36 PM · Phacility
epriestley committed rPae5a38f3349f: Guarantee terms in PhabricatorAuthPasswordEngine are strings (authored by epriestley).
Guarantee terms in PhabricatorAuthPasswordEngine are strings
Tue, Nov 3, 7:05 PM
epriestley closed D21487: Guarantee terms in PhabricatorAuthPasswordEngine are strings.
Tue, Nov 3, 7:04 PM
epriestley closed T13584: Shallow Git clones fail under recent versions of Git as Resolved.

This appears fixed by D21484 (and @20after4 confirmed the fix on rPc0414732 -- thanks!) so I'm going to mark it as resolved.

Tue, Nov 3, 7:04 PM · Diffusion
epriestley requested review of D21487: Guarantee terms in PhabricatorAuthPasswordEngine are strings.
Tue, Nov 3, 6:56 PM
epriestley added a revision to T2312: Modifying PHP: D21487: Guarantee terms in PhabricatorAuthPasswordEngine are strings.
Tue, Nov 3, 6:37 PM · Infrastructure
epriestley added a comment to T2312: Modifying PHP.

See https://discourse.phabricator-community.org/t/strpos-throwing-in-phabricatorauthpasswordengine/4335, and D21477.

Tue, Nov 3, 6:19 PM · Infrastructure

Mon, Nov 2

20after4 accepted rPc04147328fa3: Fix isValidGitShallowCloneResponse.

This has been tested in wmf production and the fix is working.

Mon, Nov 2, 10:21 PM

Oct 30 2020

epriestley added a comment to D21484: Fix isValidGitShallowCloneResponse.

Ah, thanks for the pointers!

Oct 30 2020, 11:30 PM · Diffusion
dzduvall added a comment to D21484: Fix isValidGitShallowCloneResponse.

Minor nitpicks inlines, I'll just tweak them locally as I land this.

(Since this wasn't submitted with arc, the Git "Author" information has been lost and the commit will be incorrectly attributed to me as an author. If you submit further patches, you can use arc to retain authorship information if you'd like.)

Oct 30 2020, 10:48 PM · Diffusion
epriestley committed rPc04147328fa3: Fix isValidGitShallowCloneResponse (authored by epriestley).
Fix isValidGitShallowCloneResponse
Oct 30 2020, 8:46 PM
epriestley closed D21484: Fix isValidGitShallowCloneResponse.
Oct 30 2020, 8:46 PM · Diffusion
epriestley accepted D21484: Fix isValidGitShallowCloneResponse.

Minor nitpicks inlines, I'll just tweak them locally as I land this.

Oct 30 2020, 8:32 PM · Diffusion
epriestley committed rP671986592baf: Add a missing "GROUP BY" to MailQuery when querying for multiple recipients (authored by epriestley).
Add a missing "GROUP BY" to MailQuery when querying for multiple recipients
Oct 30 2020, 8:02 PM
epriestley closed D21486: Add a missing "GROUP BY" to MailQuery when querying for multiple recipients.
Oct 30 2020, 8:02 PM
epriestley requested review of D21486: Add a missing "GROUP BY" to MailQuery when querying for multiple recipients.
Oct 30 2020, 7:58 PM
jcox updated jcox.
Oct 30 2020, 5:31 PM
avivey published D21485: Packages: Load'em from .cache for review.
Oct 30 2020, 4:54 PM
avivey added a revision to T5055: Distribution mechanism for arc extensions: D21485: Packages: Load'em from .cache.
Oct 30 2020, 4:50 PM · Arcanist

Oct 26 2020

sshannin added a comment to T4627: PhabricatorTaskmasterDaemon 100% CPU load.

I know it's an old thread, but I am encountering what is, as far as I can tell, exactly the same issue. I'm on the latest version of trunk (~Oct 26) with php 5.5.9 on Linux 3.13.0. I also observed the issue on the May 30 version.

Oct 26 2020, 11:17 PM · Daemons
altendky added a comment to T5508: Add branch name parameter to Harbormaster "Make HTTP Request" build step.

I think I've got a small tidbit to add on a specific use-case with Jenkins. I had been using pipeline builds which are working nicely but I just tried to setup a multi-branch pipeline build and this requires the branch name in the Jenkins URL of the HTTP request triggering the build. I suspect in this case the proper thing to do would be to trigger and build twice. I guess that's more of an on-branch-change sort of situation than on-commit and using those triggers in Herald (they look to be present, I haven't used them) and having that information passed on to Harbormaster as mentioned could fill the need.

Oct 26 2020, 9:40 PM · Restricted Project, Harbormaster

Oct 24 2020

epriestley triaged T13584: Shallow Git clones fail under recent versions of Git as Low priority.
Oct 24 2020, 11:05 PM · Diffusion
dzduvall requested review of D21484: Fix isValidGitShallowCloneResponse.
Oct 24 2020, 10:43 PM · Diffusion

Oct 22 2020

artms added a comment to D21287: Fix an issue where inline comments with only edit suggestions are considered empty.

@epriestley - empty inline comment with suggestion renders rather peculiar transaction which is somewhat confusing and odd... Any plans on tackling it? Like See context greyed out...

Oct 22 2020, 9:02 AM

Oct 19 2020

bastiao added a comment to T13576: Certain "arc land" merges fail when they should succeed.

Same issue here.

Oct 19 2020, 10:32 PM · Arcanist
epriestley committed rP86ad69863930: (stable) When a new, deleted, draft inline is revived with "Undo", undelete it (authored by epriestley).
(stable) When a new, deleted, draft inline is revived with "Undo", undelete it
Oct 19 2020, 7:37 PM
epriestley committed rPbc4f86d27996: When a new, deleted, draft inline is revived with "Undo", undelete it (authored by epriestley).
When a new, deleted, draft inline is revived with "Undo", undelete it
Oct 19 2020, 7:34 PM
epriestley closed D21483: When a new, deleted, draft inline is revived with "Undo", undelete it.
Oct 19 2020, 7:34 PM
epriestley requested review of D21483: When a new, deleted, draft inline is revived with "Undo", undelete it.
Oct 19 2020, 7:32 PM
epriestley edited the content of Changelog.
Oct 19 2020, 7:05 PM
epriestley created an object: 2020 Week 42.
Oct 19 2020, 7:04 PM
epriestley committed rARCac54d61d7af2: (stable) Promote 2020 Week 42 (authored by epriestley).
(stable) Promote 2020 Week 42
Oct 19 2020, 7:04 PM
epriestley committed rP916bce119a2b: (stable) Promote 2020 Week 42 (authored by epriestley).
(stable) Promote 2020 Week 42
Oct 19 2020, 7:04 PM
epriestley committed rPb2e96df3a3be: Update "arc call-conduit" instructions in Conduit API console for required "--" (authored by epriestley).
Update "arc call-conduit" instructions in Conduit API console for required "--"
Oct 19 2020, 7:02 PM
epriestley closed D21482: Update "arc call-conduit" instructions in Conduit API console for required "--".
Oct 19 2020, 7:02 PM
epriestley requested review of D21482: Update "arc call-conduit" instructions in Conduit API console for required "--".
Oct 19 2020, 6:55 PM
epriestley added a revision to T13491: Why does Arcanist require "--"?: D21482: Update "arc call-conduit" instructions in Conduit API console for required "--".
Oct 19 2020, 6:54 PM · Guides

Oct 16 2020

epriestley committed rARC4b3baca999a4: Fix a typo of "previously" in FutureIterator (authored by epriestley).
Fix a typo of "previously" in FutureIterator
Oct 16 2020, 9:23 PM
epriestley closed D21478: Fix a typo of "previously" in FutureIterator.
Oct 16 2020, 9:23 PM
epriestley committed rP2b8bbae5fb27: Set an explicit height when drawing the dependent revision graph (authored by epriestley).
Set an explicit height when drawing the dependent revision graph
Oct 16 2020, 9:10 PM
epriestley closed D21481: Set an explicit height when drawing the dependent revision graph.
Oct 16 2020, 9:10 PM
epriestley requested review of D21481: Set an explicit height when drawing the dependent revision graph.
Oct 16 2020, 9:06 PM
epriestley committed rP058d2489e7a3: Expose the "file attached to object" and "object attached to file" edges via… (authored by epriestley).
Expose the "file attached to object" and "object attached to file" edges via…
Oct 16 2020, 8:45 PM
epriestley closed D21480: Expose the "file attached to object" and "object attached to file" edges via "edge.search".
Oct 16 2020, 8:45 PM
epriestley requested review of D21480: Expose the "file attached to object" and "object attached to file" edges via "edge.search".
Oct 16 2020, 8:43 PM
epriestley committed rP1f7c736f9ab6: Add a "Comment content" field to Herald (authored by epriestley).
Add a "Comment content" field to Herald
Oct 16 2020, 8:43 PM
epriestley closed D21479: Add a "Comment content" field to Herald.
Oct 16 2020, 8:43 PM
epriestley requested review of D21479: Add a "Comment content" field to Herald.
Oct 16 2020, 8:30 PM
epriestley added a revision to T13583: Support reasonable guardrails against mismatched data retention policies in comment content: D21479: Add a "Comment content" field to Herald.
Oct 16 2020, 8:29 PM · Herald
epriestley triaged T13583: Support reasonable guardrails against mismatched data retention policies in comment content as Normal priority.
Oct 16 2020, 8:27 PM · Herald
epriestley requested review of D21478: Fix a typo of "previously" in FutureIterator.
Oct 16 2020, 7:31 PM
epriestley added a revision to T13572: Daemon overseers may have a slow memory leak, possibly in FuturePool: D21478: Fix a typo of "previously" in FutureIterator.
Oct 16 2020, 7:31 PM · Daemons
epriestley committed rARCccf74a40dd3b: Fix an issue where "phutil_utf8v()" could fatal when passed an integer (authored by epriestley).
Fix an issue where "phutil_utf8v()" could fatal when passed an integer
Oct 16 2020, 4:22 PM
epriestley closed D21477: Fix an issue where "phutil_utf8v()" could fatal when passed an integer.
Oct 16 2020, 4:22 PM
epriestley requested review of D21477: Fix an issue where "phutil_utf8v()" could fatal when passed an integer.
Oct 16 2020, 4:20 PM

Oct 9 2020

Netrics updated Netrics.
Oct 9 2020, 8:47 AM
Netrics updated Netrics.
Oct 9 2020, 8:45 AM
Netrics updated Netrics.
Oct 9 2020, 8:44 AM

Oct 2 2020

epriestley committed rP0f27cd46ccad: Never render "Show More Context" inside an inline comment suggestion diff (authored by epriestley).
Never render "Show More Context" inside an inline comment suggestion diff
Oct 2 2020, 4:47 PM
epriestley closed D21476: Never render "Show More Context" inside an inline comment suggestion diff.
Oct 2 2020, 4:47 PM
epriestley requested review of D21476: Never render "Show More Context" inside an inline comment suggestion diff.
Oct 2 2020, 4:46 PM
epriestley committed rP0f0e94ca71fe: Use "getInlines()", not "_inlines", to access inlines on client Changeset… (authored by epriestley).
Use "getInlines()", not "_inlines", to access inlines on client Changeset…
Oct 2 2020, 4:19 PM
epriestley closed D21475: Use "getInlines()", not "_inlines", to access inlines on client Changeset objects.
Oct 2 2020, 4:19 PM
epriestley updated the diff for D21475: Use "getInlines()", not "_inlines", to access inlines on client Changeset objects.
  • Fix a possible call reentrancy issue, where "getInlines()" and "getInlineForRow()" could call one another.
Oct 2 2020, 4:18 PM
epriestley requested review of D21475: Use "getInlines()", not "_inlines", to access inlines on client Changeset objects.
Oct 2 2020, 4:12 PM