Bug ReportBugs
ActivePublic

Description

This task was filed via the "New Bug Report" form.

Recent Activity

Today

chad added a project to T12098: Create Revision from FavoritesMenuItem doesn't work: Profile Menu Item.
Sat, Jan 21, 3:16 AM · Profile Menu Item, Differential, Bug Report
ftdysa added a comment to T12137: Stemming + MySQL Fulltext search can lead to unexpected results.

I'll make sure to update this task with full reproduction steps on Monday.

Sat, Jan 21, 1:14 AM · Bug Report
cspeckmim added a comment to T11521: Request Review action should clear "accepted older" status.

I appreciate the details and pointers - we may look at a custom reject transaction and/or required action bucket. I just went digging here for related discussion to see if others may have run across similar scenarios.

If you miss a comment and then forget about a revision you authored for a while, it probably wasn't that critical?

I can investigate further what issues people are having with the current solutions - I'm not sure being critical is the concern but more-so there being a delay for the change to land/verify without it being clear who should be the next step (presumably from the dashboard/bucket?).

Sat, Jan 21, 12:45 AM · Restricted Project, Differential, Bug Report

Yesterday

epriestley added a comment to T12137: Stemming + MySQL Fulltext search can lead to unexpected results.

How do I reproduce unexpected results at the application level (for example, what's something I can search for that doesn't match but should match)?

Fri, Jan 20, 11:18 PM · Bug Report
epriestley added a comment to T11521: Request Review action should clear "accepted older" status.

Yes and no. So: maybe?

Fri, Jan 20, 11:12 PM · Restricted Project, Differential, Bug Report
ftdysa created T12137: Stemming + MySQL Fulltext search can lead to unexpected results.
Fri, Jan 20, 10:56 PM · Bug Report
cspeckmim added a comment to T11521: Request Review action should clear "accepted older" status.

I want to resist adding additional actions here if they would be rarely used and reasonable workarounds already exist. After EditEngine (T11114), third-party extensions will also be able to provide supplemental actions.

Fri, Jan 20, 9:24 PM · Restricted Project, Differential, Bug Report
epriestley renamed T12135: When editing bot account "Pronoun" setting, the example strings use the viewer's username, not the bot's username from "when editing bot account settings, the default values suggest I'm editing my own settings" to "When editing bot account "Pronoun" setting, the example strings use the viewer's username, not the bot's username".
Fri, Jan 20, 6:20 PM · Bug Report
epriestley triaged T12135: When editing bot account "Pronoun" setting, the example strings use the viewer's username, not the bot's username as "Wishlist" priority.

There's a TODO about this in the code:

Fri, Jan 20, 6:19 PM · Bug Report
allixsenos created T12135: When editing bot account "Pronoun" setting, the example strings use the viewer's username, not the bot's username.
Fri, Jan 20, 6:15 PM · Bug Report
chad added a comment to T12133: Manually Closing Revisions not possible in Audit.

@Ondrej I was mostly trying to understand if you were seeing a regression, a mis-understanding of how Audit currently works, or missing some setting we should be trying against a default install. There were a number of variables still open based on your explanation to me at least, sounds like @epriestley has plans forward here though already.

Fri, Jan 20, 5:06 PM · Bug Report
chad closed T12131: Edit History inserts itself wherever it feels like as "Resolved" by committing rP98a29f3de90a: Put "View Edit History" above "Remove Comment" in timeline comment action….
Fri, Jan 20, 4:56 PM · Bug Report
Ondrej added a comment to T12133: Manually Closing Revisions not possible in Audit.
  • Chad, lets say I would rise a concern on your commit and I would never come back to Phabricator's website. You cannot mark your revision as closed. You need to remove me as an auditor and accept your commit (what is little bit complicated). What's more, there's no functionality to help me to follow this concern. When you commit a fix, how can you let me know about this fix?
  • Epriestley, T2393 would fix the second issue, that would be great to have it today:-)!
Fri, Jan 20, 3:44 PM · Bug Report
epriestley added a revision to T12131: Edit History inserts itself wherever it feels like: D17234: Put "View Edit History" above "Remove Comment" in timeline comment action dropdown.
Fri, Jan 20, 3:40 PM · Bug Report
epriestley added a comment to T12133: Manually Closing Revisions not possible in Audit.

(This change was half-accidental, and then I just figured I'd roll it into T2393 when I realized I'd made it.)

Fri, Jan 20, 3:30 PM · Bug Report
epriestley merged task T12133: Manually Closing Revisions not possible in Audit into T2393: Adds a "Needs Verification" state to Audits.
Fri, Jan 20, 3:29 PM · Bug Report
epriestley added a comment to T12133: Manually Closing Revisions not possible in Audit.

This did actually change in connection with T10978, but only if audit.can-author-close-audit is configured. I plan to resolve T2393 (maybe today?) and hopefully get rid of "Close" and this option entirely. I'm just going to merge this there.

Fri, Jan 20, 3:29 PM · Bug Report
chad added a comment to T12133: Manually Closing Revisions not possible in Audit.
if a concern is raised; no one except this auditor can close the audit.
Fri, Jan 20, 3:01 PM · Bug Report
Ondrej created T12133: Manually Closing Revisions not possible in Audit.
Fri, Jan 20, 9:52 AM · Bug Report
chad added a comment to T12130: DateTime parsing error in phd log.

We will also need reproduction steps to accept this as a valid bug report. See Contributing Bug Reports.

Fri, Jan 20, 2:39 AM · Bug Report (Needs Information)
chad created T12131: Edit History inserts itself wherever it feels like.
Fri, Jan 20, 2:36 AM · Bug Report
chad added a comment to T12130: DateTime parsing error in phd log.

You're at least 6 weeks out of date. Can you update Phabricator and verify the bug at HEAD before filing a bug, please?

Fri, Jan 20, 2:34 AM · Bug Report (Needs Information)
RuralHunter created T12130: DateTime parsing error in phd log.
Fri, Jan 20, 2:31 AM · Bug Report (Needs Information)

Thu, Jan 19

epriestley added a comment to T12129: Mercurial repository updates cause high CPU usage.

I am unable to reproduce the issue you described by following the steps provided. To move forward:

Thu, Jan 19, 10:06 PM · Bug Report (Needs Information)
epriestley added a comment to T12129: Mercurial repository updates cause high CPU usage.

Here's what I did to try to reproduce this:

Thu, Jan 19, 10:04 PM · Bug Report (Needs Information)
LeCoyote added a comment to T12129: Mercurial repository updates cause high CPU usage.

The repository at https://bitbucket.org/LeCoyote/brokentest is enough to see unusual CPU usage. Of course, it is nowhere near as bad as the other one, but the CPU is still coasting a noticeable 2 or 3% above average, which seems disproportionate for such a small repo. Also, when calling the update manually:

$ time ./bin/repository update 3
Updated repository "cieltest".
Thu, Jan 19, 9:37 PM · Bug Report (Needs Information)
LeCoyote added a comment to T12129: Mercurial repository updates cause high CPU usage.

Noted. I'll try and reproduce the problem with a different repository.

Thu, Jan 19, 9:04 PM · Bug Report (Needs Information)
epriestley added a comment to T12129: Mercurial repository updates cause high CPU usage.

(A synthetic repository which you create by running hg commit in a loop, or instructions which allow us to build such a synthetic repository, are also fine.)

Thu, Jan 19, 8:57 PM · Bug Report (Needs Information)
epriestley added a comment to T12129: Mercurial repository updates cause high CPU usage.

An existing, open source project is perfectly fine, provided you first verify that it reproduces the problem.

Thu, Jan 19, 8:57 PM · Bug Report (Needs Information)
LeCoyote added a comment to T12129: Mercurial repository updates cause high CPU usage.

Unless you can make this task private to a group of people who would be prepared to sign a NDA in French, that's not going to happen.

Thu, Jan 19, 8:55 PM · Bug Report (Needs Information)
epriestley added a comment to T12129: Mercurial repository updates cause high CPU usage.

We need specific reproduction steps to move forward, and "a repository with about 2800 commits" isn't sufficiently specific.

Thu, Jan 19, 8:35 PM · Bug Report (Needs Information)
LeCoyote created T12129: Mercurial repository updates cause high CPU usage.
Thu, Jan 19, 8:31 PM · Bug Report (Needs Information)
epriestley edited projects for T12098: Create Revision from FavoritesMenuItem doesn't work, added: Differential; removed Diffusion.
Thu, Jan 19, 4:46 PM · Profile Menu Item, Differential, Bug Report
ioeric added a comment to T12118: Differential email does not contain reviewers/subscribers information anymore.

Thanks for the detailed explanation!

Thu, Jan 19, 1:19 PM · Bug Report
aurelijus created T12124: Counterintuitive priority setting via maniphest.edit conduit call.
Thu, Jan 19, 1:04 AM · Bug Report
aurelijus created T12123: Setting 2 columns in one transaction breaks the Task UI.
Thu, Jan 19, 12:30 AM · Bug Report

Wed, Jan 18

epriestley closed T12122: Cannot login to Phabricator as "Resolved".
Wed, Jan 18, 10:59 PM · Bug Report
john.snow added a comment to T12122: Cannot login to Phabricator.

Yes, I've updated to 269dd81f91784d40abdd9af2d95061cdb3d2c394 and the issue is resolved.

Wed, Jan 18, 10:58 PM · Bug Report
epriestley added a comment to T12122: Cannot login to Phabricator.

Does upgrading past b21cd24341c6552a3fbd21305ca682a161129a6b resolve it?

Wed, Jan 18, 10:43 PM · Bug Report
john.snow created T12122: Cannot login to Phabricator.
Wed, Jan 18, 10:38 PM · Bug Report
epriestley moved T10900: Grep file content returns matches found in old versions of files from Backlog to Mercurial on the Diffusion board.
Wed, Jan 18, 7:01 PM · Mercurial, Search, Diffusion, Bug Report
epriestley moved T11173: User unable to create repository if not in the "Default Edit Policy" of Diffusion Application from Backlog to v3 on the Diffusion board.
Wed, Jan 18, 7:01 PM · Diffusion (v3), Restricted Project, Policy, Bug Report
epriestley moved T11205: Hard/Impossible to disable SSH built-in URIs from Backlog to v3 on the Diffusion board.
Wed, Jan 18, 7:01 PM · Diffusion (v3), Bug Report
epriestley moved T11314: Observe Mode + GitHub has weird mail interactions due to pull request refs from Backlog to v3 on the Diffusion board.
Wed, Jan 18, 7:01 PM · Diffusion (v3), Restricted Project, Mail, Haskell.org, Bug Report
epriestley moved T11165: Changing a callsign throws a 404 from Backlog to v3 on the Diffusion board.
Wed, Jan 18, 7:00 PM · Diffusion (v3), Bug Report
epriestley moved T12033: Large diffs can still repeatedly fail to insert from Backlog to v3 on the Diffusion board.
Wed, Jan 18, 6:59 PM · Diffusion (v3), Restricted Project, Mail, Differential, Bug Report
epriestley moved T11953: Herald rule for commits on autoclose branch didn't trigger when a commit was first pushed to a branch, then to master (fast-forward) from Backlog to v3 on the Diffusion board.
Wed, Jan 18, 6:59 PM · Diffusion (v3), Herald, Bug Report
epriestley closed T12119: `arc diff --reviewers foo` puts reviewers into Summary field as "Invalid".

making the old version of arc continue working as it did, if trivial, with new versions of the backend

Wed, Jan 18, 5:31 PM · Bug Report
allixsenos added a comment to T12119: `arc diff --reviewers foo` puts reviewers into Summary field.

based on the fact that 6 month old code worked fine until a backend upgrade last week, I'm guessing the breakage happened because of server-side changes

Wed, Jan 18, 5:06 PM · Bug Report
epriestley added a comment to T12119: `arc diff --reviewers foo` puts reviewers into Summary field.

goes away after upgrading both to latest

Wed, Jan 18, 4:14 PM · Bug Report