Page MenuHomePhabricator
Feed All Stories

Jun 16 2016

epriestley added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

I wonder if the actual authoritative repository has a refs/remotes/origin/XYZ ref which we're mirroring?

Jun 16 2016, 10:50 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

We fetch bare working copies with git fetch origin +refs/*:refs/* --prune.

Jun 16 2016, 10:45 PM · Restricted Project, Prioritized, Audit, Restricted Project
joshuaspence added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

Hrrm -- is the repository a non-bare working copy (e.g., has files and folders in it instead of just config/, hooks/, info/, etc)?

Jun 16 2016, 10:44 PM · Restricted Project, Prioritized, Audit, Restricted Project
scode added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

Is there a lack of a git remote prune origin or similar?

Jun 16 2016, 10:43 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

Hrrm -- is the repository a non-bare working copy (e.g., has files and folders in it instead of just config/, hooks/, info/, etc)?

Jun 16 2016, 10:43 PM · Restricted Project, Prioritized, Audit, Restricted Project
joshuaspence added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

OK, running ./bin/repository discover --verbose rX gives me a bunch of output including this:

Jun 16 2016, 10:40 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

We have Diffusion configured to track only the master branch, but now Diffusion is importing commits from all branches, it seems.

Jun 16 2016, 10:31 PM · Restricted Project, Prioritized, Audit, Restricted Project
joshuaspence added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

Updated our Phabricator install just now. One issue I discovered is that we are having a bunch of audits triggered for commits that are newly discovered. We have Diffusion configured to track only the master branch, but now Diffusion is importing commits from all branches, it seems.

Jun 16 2016, 10:28 PM · Restricted Project, Prioritized, Audit, Restricted Project
shpytyack.artem updated the task description for T11163: Speed up wiki documents creation.
Jun 16 2016, 10:26 PM · Phriction, Feature Request
shpytyack.artem created T11163: Speed up wiki documents creation.
Jun 16 2016, 10:24 PM · Phriction, Feature Request
• medemi68 created T11162: Unable to Edit Projects as Administrator, Showing Restricted.
Jun 16 2016, 10:10 PM · Projects, Bug Report (Needs Information)
thoughtpolice added a comment to Q426: How can I trigger a harbormaster build when a differential review is accepted and ready to land (Answer 398).

I updated my original answer (since I can't post a new one; I meant to add it as a comment on your original Q) with instructions on how to write the Herald rule you need, so you can accept that answer as "the accepted answer" for this Ponder.

Jun 16 2016, 8:26 PM
thoughtpolice updated the answer details for Q426: How can I trigger a harbormaster build when a differential review is accepted and ready to land (Answer 397).
Jun 16 2016, 8:25 PM
avivey renamed T11152: Some workflows of arc are broken on Windows from EXCEPTION: The current working directory is not part of a working copy for a supported version control system to Some workflows of arc are broken on Windows.
Jun 16 2016, 8:20 PM · Arcanist, Windows, Bug Report
avivey added a comment to T11152: Some workflows of arc are broken on Windows.

We're basically having lots of problems with Windows; There are a few reports about arc version just not working on Windows (I'm a little surprised @hach-que doesn't run into this; Can you maybe do a quick phriction page about your Windows setup, as in "this setup works, do this"?).

Jun 16 2016, 8:17 PM · Arcanist, Windows, Bug Report
scode added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

Yeah. I just wanted to make sure whether mark-reachable was safe to run without fear of accidentally marking something unreachable that isn't (i.e., if the command were to imply "mark everything in the import queue as unreachable no matter what"). Sorry it was probably a bit of a lazy question since a more careful re-reading of the rest of the ticket would probably have answered it for me.

Jun 16 2016, 8:11 PM · Restricted Project, Prioritized, Audit, Restricted Project
avivey added Q426: How can I trigger a harbormaster build when a differential review is accepted and ready to land (Answer 399).
Jun 16 2016, 8:09 PM
epriestley added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

It basically runs git log --all, then checks every commit in the database against that list and marks the missing ones as unreachable.

Jun 16 2016, 7:44 PM · Restricted Project, Prioritized, Audit, Restricted Project
jsturgeon added Q426: How can I trigger a harbormaster build when a differential review is accepted and ready to land (Answer 398).
Jun 16 2016, 7:41 PM
scode added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

If the mark-reachable command is used, is it subject to races in the sense that in-flight legit newly importing commits may be marked unreachable, or does it just trigger the process of going back and re-evaluating true reachability of all commits?

Jun 16 2016, 7:35 PM · Restricted Project, Prioritized, Audit, Restricted Project
thoughtpolice added Q426: How can I trigger a harbormaster build when a differential review is accepted and ready to land (Answer 397).
Jun 16 2016, 7:33 PM
jsturgeon updated Q426: How can I trigger a harbormaster build when a differential review is accepted and ready to land from to How can I trigger a harbormaster build when a differential review is accepted and ready to land.
Jun 16 2016, 7:17 PM
• rmaspoch updated Q425: Any way to select the "correct answer" from to Any way to select the "correct answer".
Jun 16 2016, 7:09 PM · Ponder Moderators, Ponder
epriestley added a revision to T9028: Phabricator handles commits which are published and later deleted poorly: Unknown Object (Differential Revision).
Jun 16 2016, 6:52 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley moved T9028: Phabricator handles commits which are published and later deleted poorly from The Queue to Paused on the Prioritized board.

I've pushed this here, and enabled dangerous changes on rGITTEST if anyone wants to try anything.

Jun 16 2016, 6:49 PM · Restricted Project, Prioritized, Audit, Restricted Project
chad added a comment to T11160: View Wiki Docs that only pertain to a certain group.

Your best workaround for now is multiple text panels for each organization.

Jun 16 2016, 6:41 PM · Dashboards, Phriction, Bug Report
chad added a comment to T11160: View Wiki Docs that only pertain to a certain group.

T11161 is the follow up for Dashboards

Jun 16 2016, 6:40 PM · Dashboards, Phriction, Bug Report
chad created T11161: Panel creation can be confusing.
Jun 16 2016, 6:36 PM · Design, Dashboards (v2)
epriestley added a project to T11085: Prevent users from generating an ambiguous commit message by editing revisions with the web UI: Prioritized.
Jun 16 2016, 6:32 PM · Prioritized, Restricted Project, Arcanist, Bug Report
chad added a comment to T11160: View Wiki Docs that only pertain to a certain group.

I'll create a separate task to re-think Panel creation UI.

Jun 16 2016, 6:31 PM · Dashboards, Phriction, Bug Report
chad merged T11160: View Wiki Docs that only pertain to a certain group into T9622: Allow Phriction pages to have Project tags.
Jun 16 2016, 6:31 PM · Projects, Phriction, Restricted Project
chad merged task T11160: View Wiki Docs that only pertain to a certain group into T9622: Allow Phriction pages to have Project tags.
Jun 16 2016, 6:31 PM · Dashboards, Phriction, Bug Report
eadler moved T10967: Move Differential reviewers back to a dedicated database storage table from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 16 2016, 6:28 PM · Restricted Project, Infrastructure, Differential
eadler moved T11145: Dialog submit buttons don't disable on click from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 16 2016, 6:28 PM · Restricted Project, PHUI, Bug Report
eadler moved T11150: ArcanistTextLinter: Support UTF8 from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 16 2016, 6:28 PM · Restricted Project, Lint, Feature Request
eadler moved T11153: Provide better tools for preventing deadlocks in build plans which acquire multiple resources from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 16 2016, 6:28 PM · Restricted Project, Drydock, Harbormaster, Feature Request
eadler moved T11154: Build results don't always get conferred to their buildable from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 16 2016, 6:28 PM · Restricted Project, Harbormaster, Bug Report
eadler moved T11085: Prevent users from generating an ambiguous commit message by editing revisions with the web UI from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 16 2016, 6:28 PM · Prioritized, Restricted Project, Arcanist, Bug Report
eadler moved T3875: Allow arc to perform cascading rebases, maybe during or after `arc land` from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 16 2016, 6:28 PM · Prioritized, Restricted Project, Arcanist
chad added a comment to T11160: View Wiki Docs that only pertain to a certain group.

You're not creating a custom query, at least in that screenshot. The query is "All Documents", which is what you're getting.

Jun 16 2016, 6:27 PM · Dashboards, Phriction, Bug Report
eadler added a project to T11154: Build results don't always get conferred to their buildable: Restricted Project.
Jun 16 2016, 6:26 PM · Restricted Project, Harbormaster, Bug Report
eadler added a project to T11153: Provide better tools for preventing deadlocks in build plans which acquire multiple resources: Restricted Project.
Jun 16 2016, 6:26 PM · Restricted Project, Drydock, Harbormaster, Feature Request
eadler added a project to T11145: Dialog submit buttons don't disable on click: Restricted Project.
Jun 16 2016, 6:26 PM · Restricted Project, PHUI, Bug Report
eadler added a project to T11150: ArcanistTextLinter: Support UTF8: Restricted Project.
Jun 16 2016, 6:26 PM · Restricted Project, Lint, Feature Request
epriestley added a comment to T6878: Tagged commits which are not ancestors of any branch head don't get imported.

I think this is resolved, but see T9028 for related followups.

Jun 16 2016, 6:22 PM · Wikimedia, Restricted Project, Diffusion
epriestley closed T6878: Tagged commits which are not ancestors of any branch head don't get imported as Resolved by committing Restricted Diffusion Commit.
Jun 16 2016, 6:20 PM · Wikimedia, Restricted Project, Diffusion
epriestley closed T6878: Tagged commits which are not ancestors of any branch head don't get imported, a subtask of T9028: Phabricator handles commits which are published and later deleted poorly, as Resolved.
Jun 16 2016, 6:20 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley added a comment to T9028: Phabricator handles commits which are published and later deleted poorly.

I'm vaguely suspicious that I may have missed some weird stuff here since this pipeline is complicated, but I'm going to land this stuff now and just plan to hold the release a bit if it feels shaky tomorrow.

Jun 16 2016, 6:19 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley added a comment to T7047: Add user preference for diffs in emails.

Can anyone interested in this explain the use case for making it per-Herald or per-user instead of global? Are half your users bitterly opposed to inline diffs, even after we fixed the Differential ones to not be terrible (T10694)?

Jun 16 2016, 6:17 PM · Restricted Project, Differential, Mail, Diffusion
epriestley merged T10742: PhabricatorRepositoryCommitParserWorker fails on deleted commits into T9028: Phabricator handles commits which are published and later deleted poorly.
Jun 16 2016, 6:08 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley merged task T10742: PhabricatorRepositoryCommitParserWorker fails on deleted commits into T9028: Phabricator handles commits which are published and later deleted poorly.
Jun 16 2016, 6:08 PM · Diffusion, Bug Report
epriestley added a comment to T10742: PhabricatorRepositoryCommitParserWorker fails on deleted commits.

I'm going to merge this into T9028. Specifically, daemons should be able to permanently fail operations on unreachable commits after D16131. That task discusses deleted/unreachable commits more broadly.

Jun 16 2016, 6:08 PM · Diffusion, Bug Report
epriestley closed T10933: You Shall Not Pass: Unknown Object (Repository) as Invalid.

We can't reproduce this, so we can't move forward. Feel free to open a new issue if you are able to provide reproduction steps.

Jun 16 2016, 6:05 PM · Diffusion, Bug Report
epriestley closed T10922: Upgrading: New Repository UI, APIs, and URI management as Resolved.

There don't appear to be any major outstanding issues here.

Jun 16 2016, 6:03 PM · Diffusion (v3), Installing & Upgrading
epriestley closed T10923: Diffusion API/URI Errata as Resolved.

This has seemed pretty stable for about a month. Future work is tracked in more specific tasks.

Jun 16 2016, 6:03 PM · Diffusion (v3)
epriestley closed T11137: Unable to update Git repositories with latest stable as Resolved.

This fix appears to be holding.

Jun 16 2016, 6:02 PM · Diffusion, Bug Report
epriestley merged T5715: Diffusion commit count is incorrect after forced push into T9028: Phabricator handles commits which are published and later deleted poorly.
Jun 16 2016, 6:01 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley merged task T5715: Diffusion commit count is incorrect after forced push into T9028: Phabricator handles commits which are published and later deleted poorly.
Jun 16 2016, 6:01 PM · Restricted Project, Diffusion
epriestley added a comment to T5715: Diffusion commit count is incorrect after forced push.

I'm going to merge this into T9028. Specifically, D16133 rebuilds the summary table after any commit becomes unreachable, which should produce the correct commit count and other summary details.

Jun 16 2016, 6:01 PM · Restricted Project, Diffusion
avivey added a comment to T6878: Tagged commits which are not ancestors of any branch head don't get imported.

black-listing phabricator/ sounds good to me :)

Jun 16 2016, 6:00 PM · Wikimedia, Restricted Project, Diffusion
epriestley added a comment to T6878: Tagged commits which are not ancestors of any branch head don't get imported.

D16134 just hard-codes phabricator/ tags as untracked, which should minimize collateral damage.

Jun 16 2016, 5:59 PM · Wikimedia, Restricted Project, Diffusion
epriestley added a revision to T6878: Tagged commits which are not ancestors of any branch head don't get imported: Unknown Object (Differential Revision).
Jun 16 2016, 5:57 PM · Wikimedia, Restricted Project, Diffusion
epriestley added a revision to T9028: Phabricator handles commits which are published and later deleted poorly: Unknown Object (Differential Revision).
Jun 16 2016, 5:57 PM · Restricted Project, Prioritized, Audit, Restricted Project
• rmaspoch created T11160: View Wiki Docs that only pertain to a certain group.
Jun 16 2016, 5:53 PM · Dashboards, Phriction, Bug Report
epriestley created T11159: Cluster detection of "replication OK" is unreasonably optimistic.
Jun 16 2016, 5:50 PM · Prioritized, Clusters
epriestley added a revision to T9028: Phabricator handles commits which are published and later deleted poorly: Unknown Object (Differential Revision).
Jun 16 2016, 5:48 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley added a comment to T6878: Tagged commits which are not ancestors of any branch head don't get imported.

Yeah, the actual rule D16129 implements is this:

Jun 16 2016, 5:33 PM · Wikimedia, Restricted Project, Diffusion
avivey added a comment to T6878: Tagged commits which are not ancestors of any branch head don't get imported.

For now, I think we can assume tags are always autoclose/track, but may need to refine this in the future.

Jun 16 2016, 5:29 PM · Wikimedia, Restricted Project, Diffusion
epriestley merged T11158: Allow arbitrary status updates to harbormaster via harbormaster.sendmessage into T9124: Support uploading build log data via the Harbormaster API.
Jun 16 2016, 4:43 PM · Restricted Project, Harbormaster
epriestley merged task T11158: Allow arbitrary status updates to harbormaster via harbormaster.sendmessage into T9124: Support uploading build log data via the Harbormaster API.
Jun 16 2016, 4:43 PM · Harbormaster, Feature Request
chad added a comment to T10901: Support title art for Phame blogs.

oh wow

Jun 16 2016, 4:41 PM · Phacility, Phame
epriestley added a comment to T10901: Support title art for Phame blogs.

high-horse-soapbox.jpg (1×1 px, 330 KB)

Jun 16 2016, 4:40 PM · Phacility, Phame
epriestley added a comment to T10901: Support title art for Phame blogs.

they are mine T.T

Jun 16 2016, 4:39 PM · Phacility, Phame
chad added a comment to T10901: Support title art for Phame blogs.

i can has arts

Jun 16 2016, 4:39 PM · Phacility, Phame
epriestley added a comment to T10901: Support title art for Phame blogs.

We could do posts too, but I only have 3 arts so far, so blogs seemed easier to start with.

Jun 16 2016, 4:38 PM · Phacility, Phame
chad added a comment to T10901: Support title art for Phame blogs.

Oh for blogs, not posts.

Jun 16 2016, 4:36 PM · Phacility, Phame
chad added a comment to T10901: Support title art for Phame blogs.

Is this an example of "title art" or is there something better. I seem to think there was an actual header, but maybe not.

Jun 16 2016, 4:36 PM · Phacility, Phame
epriestley added a project to T3875: Allow arc to perform cascading rebases, maybe during or after `arc land`: Prioritized.
Jun 16 2016, 4:07 PM · Prioritized, Restricted Project, Arcanist
epriestley added a revision to T9028: Phabricator handles commits which are published and later deleted poorly: Unknown Object (Differential Revision).
Jun 16 2016, 4:04 PM · Restricted Project, Prioritized, Audit, Restricted Project
jsturgeon added a comment to Q423: How do I send detail to harbormaster.sendmessage 'work' message (Answer 396).

OK, Feature request filed here: T11158

Jun 16 2016, 3:53 PM
jsturgeon updated the task description for T11158: Allow arbitrary status updates to harbormaster via harbormaster.sendmessage.
Jun 16 2016, 3:52 PM · Harbormaster, Feature Request
jsturgeon created T11158: Allow arbitrary status updates to harbormaster via harbormaster.sendmessage.
Jun 16 2016, 3:50 PM · Harbormaster, Feature Request
epriestley moved T11140: Support at-rest encryption in Files from The Queue to Paused on the Prioritized board.

I've deployed this and configured this server to use AES-256-CBC, although I haven't retroactively migrated existing data.

Jun 16 2016, 3:27 PM · Prioritized, Files
epriestley added a revision to T9028: Phabricator handles commits which are published and later deleted poorly: Unknown Object (Differential Revision).
Jun 16 2016, 3:05 PM · Restricted Project, Prioritized, Audit, Restricted Project
epriestley added a revision to T9028: Phabricator handles commits which are published and later deleted poorly: Unknown Object (Differential Revision).
Jun 16 2016, 2:30 PM · Restricted Project, Prioritized, Audit, Restricted Project
allixsenos added a comment to T11157: ability to disable support for certain VCS.

fair enough

Jun 16 2016, 2:12 PM · Diffusion, Feature Request
chad added a comment to T11157: ability to disable support for certain VCS.

We don't feel it's a good use of time to build out hypothetical features. The main reason is you're asking us to build something for a case that's never happened, and may never happen. We'll always likely have something more impactful to spend our development time on. Even taking this as "Wishlist" essentially means we'll never get to it in a reasonable timeframe.

Jun 16 2016, 2:02 PM · Diffusion, Feature Request
allixsenos added a comment to T11157: ability to disable support for certain VCS.

I'd like the UI to not offer things which don't work - I have not tested Mercurial and SVN hosting and given that I've done nothing to make them work I imagine they don't

Jun 16 2016, 1:57 PM · Diffusion, Feature Request
chad added a comment to T11157: ability to disable support for certain VCS.

Is this actually a problem or are you just anticipating users will do this?

Jun 16 2016, 1:50 PM · Diffusion, Feature Request
epriestley closed T8588: One instance has a severe, global performance issue, a subtask of T7664: Moving a task on a workboard is very slow (and sometimes times out with a fatal error), as Wontfix.
Jun 16 2016, 1:29 PM · Restricted Project, Workboards
epriestley closed T8588: One instance has a severe, global performance issue as Wontfix.

We can't reproduce this and haven't seen other reports of similar issues (including from other installs of greater size and complexity), so we have no way to move forward on this.

Jun 16 2016, 1:29 PM · Restricted Project
epriestley closed T8588: One instance has a severe, global performance issue, a subtask of T8444: Increase typeahead speed, as Wontfix.
Jun 16 2016, 1:29 PM · Typeahead, PHUI, Restricted Project
allixsenos created T11157: ability to disable support for certain VCS.
Jun 16 2016, 1:26 PM · Diffusion, Feature Request
epriestley removed a project from T5791: Write Herald rules for outbound mail: Prioritized.

Bookkeeping; there's no longer an external driver for this.

Jun 16 2016, 1:19 PM · Restricted Project, Restricted Project, FreeBSD, Mail
epriestley removed a project from T8646: Provide more context for search results, particularly wiki documents: Prioritized.

Just bookkeeping; this doesn't have an active external priority driver.

Jun 16 2016, 1:18 PM · Design, Search, Restricted Project
epriestley removed a project from T182: Commit into repository directly from differential: Prioritized.

Just bookkeeping, this doesn't currently have external priority.

Jun 16 2016, 1:17 PM · Haskell.org, Restricted Project, Differential
angel added a comment to T11156: Exception on auth configuration.

Pulled and working! Thanks a lot!

Jun 16 2016, 1:12 PM · Auth, Bug Report
epriestley edited this Badge.
Jun 16 2016, 1:06 PM
epriestley added a comment to T11156: Exception on auth configuration.

This should be fixed at HEAD of master. Let us know if you're still seeing issues after updating.

Jun 16 2016, 1:06 PM · Auth, Bug Report
epriestley closed T11156: Exception on auth configuration as Resolved by committing Restricted Diffusion Commit.
Jun 16 2016, 1:04 PM · Auth, Bug Report