Page MenuHomePhabricator
Feed All Stories

Apr 26 2016

ofbeaton added a comment to T10878: workboard horizontal scroll wonky on ie11.

I should add that when I am "stuck" in drag mode (no mouse buttons held down, but moving it around scroll me around like I am click-dragging), I can do a fast left-click on an empty spot on the workboard and it becomes unstuck.

Apr 26 2016, 5:49 PM · Workboards, Windows, Bug Report (Needs Information)
ofbeaton added a comment to T10878: workboard horizontal scroll wonky on ie11.

Added time markers as it isn't clear what I'm doing with mouse/keyboard without a visual cue.

Apr 26 2016, 5:48 PM · Workboards, Windows, Bug Report (Needs Information)
chad added a comment to T10878: workboard horizontal scroll wonky on ie11.

Thanks for the video, unfortunately I still can't reproduce the issue locally. Everything works for me.

Apr 26 2016, 5:47 PM · Workboards, Windows, Bug Report (Needs Information)
ofbeaton added a comment to T10878: workboard horizontal scroll wonky on ie11.

Please see attached .wmv file. I used a technet video capture utility as it was quick and did the job, sorry the quality is not higher but it should do the job.

Apr 26 2016, 5:42 PM · Workboards, Windows, Bug Report (Needs Information)
epriestley closed T10386: HTTP instructions in "Diffusion User Guide: Repository Hosting" are incomplete as Resolved by committing Restricted Diffusion Commit.
Apr 26 2016, 5:07 PM · Documentation, Bug Report
epriestley closed T10876: Make the setup warning count cache database-backed, a subtask of T10751: Make Phabricator Highly Available, as Resolved.
Apr 26 2016, 5:03 PM · Restricted Project, Infrastructure, Prioritized
epriestley closed T10876: Make the setup warning count cache database-backed as Resolved by committing Restricted Diffusion Commit.
Apr 26 2016, 5:03 PM · Clusters
chad added a comment to T10878: workboard horizontal scroll wonky on ie11.

This the browser I tested:

pasted_file (347×424 px, 54 KB)

Apr 26 2016, 4:23 PM · Workboards, Windows, Bug Report (Needs Information)
chad added a comment to T10878: workboard horizontal scroll wonky on ie11.

I can't reproduce the issue you're describing. Maybe make a video to clarify the specific behaviour?

Apr 26 2016, 4:22 PM · Workboards, Windows, Bug Report (Needs Information)
ofbeaton added a comment to T10878: workboard horizontal scroll wonky on ie11.

We were able to consistently replicate it on multiple Windows 7 64-bit IE 11.0.9600.17843 (11.0.20) and Windows 7 32-bit IE 11.0.9600.18204 (11.0.28) workstations.

Apr 26 2016, 4:11 PM · Workboards, Windows, Bug Report (Needs Information)
chad added a comment to T10878: workboard horizontal scroll wonky on ie11.

What OS should I be testing?

Apr 26 2016, 3:42 PM · Workboards, Windows, Bug Report (Needs Information)
remusvrm closed Q378: Read-only access on hosted Phabricator for non-login users as resolved.
Apr 26 2016, 3:31 PM · Auth
remusvrm added a comment to Q378: Read-only access on hosted Phabricator for non-login users (Answer 369).

This is what I was looking for!
Thanks! Works!

Apr 26 2016, 3:30 PM
chad added a comment to T10879: Daily TODO list.

Tasks are already re-orderable (to just you) in Maniphest. Not sure what else you want? T5793 might also be of interest.

Apr 26 2016, 3:26 PM · Feature Request
benjumanji added a comment to T10879: Daily TODO list.
  1. make a private project
  2. add tasks you like to it
  3. ???
  4. profit
Apr 26 2016, 3:19 PM · Feature Request
theguild created T10879: Daily TODO list.
Apr 26 2016, 3:07 PM · Feature Request
epriestley added Q378: Read-only access on hosted Phabricator for non-login users (Answer 369).
Apr 26 2016, 3:05 PM
seungrye added a comment to T10877: Large sized GIF is not animated properly..

@epriestley // Oh, I don't know that. Cuz, OS X preview shows repeating animation.
Thanks Very much : )

Apr 26 2016, 2:59 PM · Bug Report
epriestley added a comment to T10877: Large sized GIF is not animated properly..

The GIF data is set to repeat the animation only once (see circled control in footer):

Apr 26 2016, 2:57 PM · Bug Report
chad added a comment to T10877: Large sized GIF is not animated properly..

I don't have access to this server. How your server is set up is more important. What php version and what image library are you using? Is it only this gif or other large gifs as well?

Apr 26 2016, 2:49 PM · Bug Report
ofbeaton created T10878: workboard horizontal scroll wonky on ie11.
Apr 26 2016, 2:46 PM · Workboards, Windows, Bug Report (Needs Information)
epriestley added a revision to T10876: Make the setup warning count cache database-backed: Unknown Object (Differential Revision).
Apr 26 2016, 2:44 PM · Clusters
seungrye added a comment to T10877: Large sized GIF is not animated properly..

@chad // I update the description, but I can't reopen this issue. Should I make a new bug report?

Apr 26 2016, 2:43 PM · Bug Report
seungrye updated the task description for T10877: Large sized GIF is not animated properly..
Apr 26 2016, 2:41 PM · Bug Report
chad closed T10877: Large sized GIF is not animated properly. as Invalid.

Please fill in all required information for a bug report. Specifically missing are steps to reproduce and what version of Phabricator you are using.

Apr 26 2016, 2:33 PM · Bug Report
seungrye created T10877: Large sized GIF is not animated properly..
Apr 26 2016, 2:28 PM · Bug Report
urzds added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

@cspeckmim Sadly I don't have any Mercurial host that is not Phabricator.

Apr 26 2016, 2:27 PM · Mercurial, Bug Report
epriestley added a revision to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API: Unknown Object (Differential Revision).
Apr 26 2016, 2:00 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
cspeckmim added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

@fcoelho oops sorry about the mixup. I've previously had success by downgrading the client to 3.4 or less.

Apr 26 2016, 1:45 PM · Mercurial, Bug Report
epriestley created T10876: Make the setup warning count cache database-backed.
Apr 26 2016, 1:32 PM · Clusters
epriestley added a revision to T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API: Unknown Object (Differential Revision).
Apr 26 2016, 12:45 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
epriestley added a comment to T10868: Make a line-linkable view for individual build logs.
  • (T9516) Use the same copy/paste line number stuff that Paste, Diffusion, etc use.
    • (T10179) Make formatting smarter. For example, the "Run CircleCI Build" plan produces logs that we know to be HTTP headers (which we could format as a table) and (almost always) a JSON blob (which we could format as human-readable JSON).
    • (T8656) Make one of the smart formatters "ANSI".
    • I'd like to style this like other stuff (Paste, Diffusion) and let users pick dark themes globally with T5701 if they prefer.
    • Logic for loading the data is clunky, and fetches much more data than it needs.
    • Clicking pager links affects all build logs on the page, this should probably be JS-ish at some point.
    • Pager logic often feels clunky to me. For example, if a log is exactly 21 lines long, we still show pager controls. If a log is 27 lines long, we should probably show the whole thing. "Show more" would be more consistent with how other pagers work, etc.
    • (T6139) Not sure if this is really the right way to frame it, but we often have information about how useful a build log is and can do a better job of organizing them (e.g., successful build logs are usually uninteresting and could be folded by default).
Apr 26 2016, 12:32 PM · Restricted Project, Harbormaster, Feature Request
epriestley added a comment to T10868: Make a line-linkable view for individual build logs.

@hach-que, that plan sounds good to me.

Apr 26 2016, 12:23 PM · Restricted Project, Harbormaster, Feature Request
epriestley added a revision to T10849: Support query timeouts via `max_statement_time` or MYSLQI_ASYNC: Unknown Object (Differential Revision).
Apr 26 2016, 12:16 PM · Infrastructure, Phacility
eadler updated the task description for T10874: Add repository or project to email subject.
Apr 26 2016, 12:15 PM · Restricted Project, Mail, Feature Request
epriestley merged T10875: Browse in Diffusion doesn't take branch into account into T3216: "Browse in Diffusion" always going to default branch.
Apr 26 2016, 11:56 AM · Differential
epriestley merged task T10875: Browse in Diffusion doesn't take branch into account into T3216: "Browse in Diffusion" always going to default branch.
Apr 26 2016, 11:56 AM · Bug Report
epriestley added a comment to T10874: Add repository or project to email subject.

Not directly to your point, but you can remove or shorten the [Differential] token by adjusting metamta.differential.subject-prefix, if you didn't stumble across that already.

Apr 26 2016, 11:28 AM · Restricted Project, Mail, Feature Request
epriestley closed T10873: New installation fails to create/setup database as Resolved.
Apr 26 2016, 11:27 AM · Bug Report
remusvrm updated Q378: Read-only access on hosted Phabricator for non-login users from to Read-only access on hosted Phabricator for non-login users.
Apr 26 2016, 11:20 AM · Auth
aubort created T10875: Browse in Diffusion doesn't take branch into account.
Apr 26 2016, 10:47 AM · Bug Report
Twilight awarded T9031: When I ignore subscription changes, I'm not notified of being cc'ed a Like token.
Apr 26 2016, 9:54 AM · Mail, Notifications
dfaure-kdab created T10874: Add repository or project to email subject.
Apr 26 2016, 9:36 AM · Restricted Project, Mail, Feature Request
toelke added a comment to T10873: New installation fails to create/setup database.

Ok, sorry for the noise, a complete re-install of mysql fixed the issue; I assume that I had not removed all traces of a previous installation as thoroughly as I hoped.

Apr 26 2016, 9:08 AM · Bug Report
toelke renamed T10873: New installation fails to create/setup database from New installation fails to create database to New installation fails to create/setup database.
Apr 26 2016, 7:26 AM · Bug Report
toelke created T10873: New installation fails to create/setup database.
Apr 26 2016, 7:26 AM · Bug Report
florianb added a comment to T10866: Missing Information in Diffusion User Guide: Repo Hosting.

Interesting, thanks. I just followed the "Diffusion User Guide" which states that the git user must have access to the repos as the phd-user.

Apr 26 2016, 7:08 AM · Diffusion, Documentation, Bug Report
epriestley added a comment to T10751: Make Phabricator Highly Available.

I'm still not really thrilled with where the documentation situation on repositories is, but it's getting a little more manageable.

Apr 26 2016, 3:18 AM · Restricted Project, Infrastructure, Prioritized
epriestley added a revision to T4292: Implement repository replication: Unknown Object (Differential Revision).
Apr 26 2016, 2:47 AM · Restricted Project, Diffusion
epriestley added a comment to T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to.

Here are a few ideas that might shed some light on what's happening:

Apr 26 2016, 1:59 AM · Arcanist, Restricted Project, Bug Report
epriestley added a comment to T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to.

I can't reproduce this. Here's what I did:

Apr 26 2016, 12:49 AM · Arcanist, Restricted Project, Bug Report
epriestley added a comment to T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to.

arc diff is supported, but most of these reproduction instructions don't make much sense to me in the context of arc diff -- for example, arc diff does not apply patches.

Apr 26 2016, 12:42 AM · Arcanist, Restricted Project, Bug Report
angie added a comment to T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to.

As I mentioned, we also experience this with arc diff. Is that not supported?

Apr 26 2016, 12:35 AM · Arcanist, Restricted Project, Bug Report
epriestley added a comment to T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to.

Specifically, any effort to reproduce this in the upstream will meet this grizzly fate on the arc test step:

Apr 26 2016, 12:35 AM · Arcanist, Restricted Project, Bug Report
epriestley closed T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to as Invalid.

Yeah, arc test is not part of the upstream.

Apr 26 2016, 12:33 AM · Arcanist, Restricted Project, Bug Report
epriestley closed T10871: Aphlict does not properly guess root path of installation as Invalid.

This is a Bitnami issue, and you should report it to them. See Contributing Bug Reports:

Apr 26 2016, 12:30 AM · Bug Report
cteffetalor created T10872: any(Project) does not include Milestones of Project.
Apr 26 2016, 12:13 AM · Restricted Project, ApplicationSearch, Projects, Bug Report
cteffetalor renamed T10871: Aphlict does not properly guess root path of installation from Aphlict does not properly guess root path of installation correctly to Aphlict does not properly guess root path of installation.
Apr 26 2016, 12:09 AM · Bug Report
cteffetalor created T10871: Aphlict does not properly guess root path of installation.
Apr 26 2016, 12:08 AM · Bug Report

Apr 25 2016

hach-que added a comment to T10866: Missing Information in Diffusion User Guide: Repo Hosting.

Just throwing my input in here:

Apr 25 2016, 11:44 PM · Diffusion, Documentation, Bug Report
hach-que added a comment to T10870: Allow Harbormaster builds to run another build plan on the same buildable.

(Those two things seem reasonably separate, so this might be better split into two issues, but I'll leave it up to @epriestley to decide that; I just wanted to get better visibility on the discussions that happened around those original code reviews in case any community member wanted to tackle implementing these features with upstream guidance)

Apr 25 2016, 11:31 PM · Restricted Project, Harbormaster
avivey added a comment to T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to.
  • arc patch attempts to apply the patch to the commit the diff was based off; Is that what you were expecting?
  • arc test is not a command I have; What is it?
Apr 25 2016, 11:24 PM · Arcanist, Restricted Project, Bug Report
hach-que added a comment to T10870: Allow Harbormaster builds to run another build plan on the same buildable.

I managed to find the original restart target feedback that @epriestley wrote.

Apr 25 2016, 11:22 PM · Restricted Project, Harbormaster
angie awarded T10323: "Action With No Effect" when marking inline comments as done a Love token.
Apr 25 2016, 11:20 PM · Restricted Project, Restricted Project, Bug Report, Differential
hach-que closed T5542: Allow individual Harbormaster targets to be restarted, a subtask of T1049: Implement Harbormaster, as Invalid.
Apr 25 2016, 11:18 PM · Harbormaster
hach-que closed T5542: Allow individual Harbormaster targets to be restarted as Invalid.

The goals here don't align with upstream, so I've consolidated the general feedback from @epriestley into T10870.

Apr 25 2016, 11:18 PM · Harbormaster
angie added a project to T10323: "Action With No Effect" when marking inline comments as done: Restricted Project.
Apr 25 2016, 11:18 PM · Restricted Project, Restricted Project, Bug Report, Differential
hach-que updated the task description for T10870: Allow Harbormaster builds to run another build plan on the same buildable.
Apr 25 2016, 11:16 PM · Restricted Project, Harbormaster
hach-que updated the task description for T10870: Allow Harbormaster builds to run another build plan on the same buildable.
Apr 25 2016, 11:14 PM · Restricted Project, Harbormaster
chad added a project to T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to: Arcanist.
Apr 25 2016, 11:13 PM · Arcanist, Restricted Project, Bug Report
hach-que added a comment to T10870: Allow Harbormaster builds to run another build plan on the same buildable.

There were also some comments around embedding build plans (with individual build steps being restartable), but I can't find them now. I'd like to pursue these features in the near future, as we're finding Jenkins "multi-configuration projects" don't actually allow you to restart individual builds (which we rely on because AWS is inherently unreliable in it's operations).

Apr 25 2016, 11:11 PM · Restricted Project, Harbormaster
hach-que created T10870: Allow Harbormaster builds to run another build plan on the same buildable.
Apr 25 2016, 11:11 PM · Restricted Project, Harbormaster
angie created T10869: `arc test` sometimes picks an older (incorrect) parent to apply patch to.
Apr 25 2016, 11:07 PM · Arcanist, Restricted Project, Bug Report
avivey awarded T10868: Make a line-linkable view for individual build logs a Like token.
Apr 25 2016, 10:46 PM · Restricted Project, Harbormaster, Feature Request
hach-que updated subscribers of T10868: Make a line-linkable view for individual build logs.

This seems reasonably easy to tackle. Roughly:

Apr 25 2016, 10:41 PM · Restricted Project, Harbormaster, Feature Request
hach-que added a project to T10868: Make a line-linkable view for individual build logs: Harbormaster.
Apr 25 2016, 10:30 PM · Restricted Project, Harbormaster, Feature Request
epriestley added a comment to T10867: Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect.

Maybe also reasonable would be something like this:

Apr 25 2016, 10:08 PM · Daemons, Feature Request
epriestley renamed T10867: Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect from Gracefully handle builds when restarting daemons to Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect.
Apr 25 2016, 9:53 PM · Daemons, Feature Request
epriestley added a comment to T10867: Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect.

One thing we could do to improve this is version daemons in /daemon/, so your "go to sleep" routine could be more like:

Apr 25 2016, 9:52 PM · Daemons, Feature Request
yelirekim added a comment to T10867: Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect.

I'll start doing this, I guess it makes me a little uncomfortable since the first thing I do is typically make sure nothing is broken once the daemons restart, but then again, nothing has ever been broken during a deployment. So maybe I will assume I am as smart as our past deployments have indicated I am, issue the reload, go to sleep, and then check in the morning.

Apr 25 2016, 9:47 PM · Daemons, Feature Request
yelirekim created T10868: Make a line-linkable view for individual build logs.
Apr 25 2016, 9:45 PM · Restricted Project, Harbormaster, Feature Request
epriestley added a comment to T10867: Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect.

You can use bin/phd reload -- instead of restart -- to instruct daemons to exit and restart only after completing their current tasks, with no time limit on task completion. The semantics on this are necessarily a little fuzzy, but basically:

Apr 25 2016, 9:39 PM · Daemons, Feature Request
yelirekim created T10867: Version daemons more clearly in daemon console so it's clear when `phd reload` has taken effect.
Apr 25 2016, 9:24 PM · Daemons, Feature Request
fcoelho added a comment to T10382: Transaction abort when pushing via SSH to a Mercurial repository.

@cspeckmim I think you probably confused me with @urzds, I didn't post any logs here yet.

Apr 25 2016, 8:23 PM · Mercurial, Bug Report
jdickson added Q365: What are Mercurial shops doing to avoid the bundle2 issue? External repositories? (Answer 368).
Apr 25 2016, 7:07 PM
Pawka edited P1974 Masterwork From Distant Lands.
Apr 25 2016, 6:48 PM
eadler moved T5029: Fill in missing context for low-context diffs against tracked repositories from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 25 2016, 6:02 PM · KDE, Restricted Project, Diffusion, Differential
eadler moved T10855: Allow external systems to edit the content of dashboard panels so they can publish information from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 25 2016, 6:02 PM · Dashboards, Restricted Project, Conduit, Feature Request
eadler added a project to T10855: Allow external systems to edit the content of dashboard panels so they can publish information: Restricted Project.
Apr 25 2016, 6:02 PM · Dashboards, Restricted Project, Conduit, Feature Request
• sketchbookgames added a comment to Blog Post: Updates to Workboards: Dragging, Cover Photos and Points.

Looks nice! Looking forward to the future plans for it, in the meantime i have a question:
Feature currently seems to assume a certain width for images, and scales images up or down to fit. In order to help my cover-photos avoid scaling and look their best I would like to know what that width is.

Apr 25 2016, 4:54 PM · Workboards, Maniphest
GHowe added a comment to T6609: Support math (LaTeX) equation in Remarkup.

Throwing a vote in that this feature would definitely be of great use to us as well.

Apr 25 2016, 4:29 PM · Restricted Project, Remarkup
epriestley added a revision to T10386: HTTP instructions in "Diffusion User Guide: Repository Hosting" are incomplete: Unknown Object (Differential Revision).
Apr 25 2016, 3:32 PM · Documentation, Bug Report
epriestley added a revision to T10866: Missing Information in Diffusion User Guide: Repo Hosting: Unknown Object (Differential Revision).
Apr 25 2016, 3:32 PM · Diffusion, Documentation, Bug Report
epriestley closed T10602: Adding client side private key addition information to Diffusion Repo Hosting Guide. as Wontfix.

This document is already large and complex and the source of many configuration mistakes. I worry including basic information like this could cause more harm than good, by making the document even larger and making it easy to miss something important.

Apr 25 2016, 3:23 PM · Feature Request
epriestley closed T10865: Remarkup is not rendered in Pholio Application as Resolved by committing Restricted Diffusion Commit.
Apr 25 2016, 3:16 PM · Remarkup, Pholio, Bug Report
epriestley closed T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too as Resolved by committing Restricted Diffusion Commit.
Apr 25 2016, 3:16 PM · Diffusion, Documentation
florianb added a comment to T10866: Missing Information in Diffusion User Guide: Repo Hosting.

I understand - i didn't wanted you to fix my configuration. I think as soon as i have some experience running phabricator in this setup i can try to help improving the setup, if even possible.

Apr 25 2016, 2:47 PM · Diffusion, Documentation, Bug Report
epriestley added a comment to T10866: Missing Information in Diffusion User Guide: Repo Hosting.

That is, we normally navigate complex configuration by verifying every step and just telling you what's wrong and how to fix it (e.g., moving all the MySQL, etc., configuration from documentation to automatic setup issues was a huge step forward), but there's basically no test we can perform on most of this chain.

Apr 25 2016, 2:41 PM · Diffusion, Documentation, Bug Report
epriestley added a comment to T10866: Missing Information in Diffusion User Guide: Repo Hosting.

In general, this configuration is a huge horrible mess but I'm having a hard time coming up with ways to really fix it that don't just tell you to run everything as root.

Apr 25 2016, 2:39 PM · Diffusion, Documentation, Bug Report
florianb added a comment to T10866: Missing Information in Diffusion User Guide: Repo Hosting.

However, i wouldn't exclude that i misunderstood something about the moving parts of Phabricator.

Apr 25 2016, 2:38 PM · Diffusion, Documentation, Bug Report