Page MenuHomePhabricator
Feed All Stories

Apr 14 2016

epriestley added a comment to T10813: Maybe fix various statements that MySQL statement-based replication gets upset about.

It looks like the DELETE statements can not be expressed in a way which doesn't upset the "unsafeness detector", even when explicitly ordered by primary key. I think this is the relevant MySQL issue:

Apr 14 2016, 9:36 PM · Clusters
epriestley closed T10807: CI setup blocked by commit import bug? as Resolved.
Apr 14 2016, 9:22 PM · Bug Report
jaredbischof added a comment to T10807: CI setup blocked by commit import bug?.

I see, it makes sense now that I know what's going on. Not sure if there's a better way to write it honestly.

Apr 14 2016, 9:22 PM · Bug Report
epriestley added a comment to T10807: CI setup blocked by commit import bug?.

Here's the section that's intended to explain it. This applies to Mercurial, too (in theory) which is why we don't explicitly mention git fetch.

Apr 14 2016, 9:07 PM · Bug Report
CodeMouse92 updated the task description for T10779: Cannot manage base project milestones if base project has subprojects.
Apr 14 2016, 9:05 PM · Restricted Project, Projects, Bug Report
jaredbischof added a comment to T10807: CI setup blocked by commit import bug?.

Got it, thanks!

Apr 14 2016, 9:05 PM · Bug Report
CodeMouse92 added a comment to T10779: Cannot manage base project milestones if base project has subprojects.

@chad, I've updated the title to reflect the updated (and confirmed) duplication instructions. The specific problem is that, if a project has subprojects, then any milestones belonging to the base project cannot be modified.

Apr 14 2016, 9:05 PM · Restricted Project, Projects, Bug Report
epriestley added a comment to T10813: Maybe fix various statements that MySQL statement-based replication gets upset about.

Alternatively it seems like we can switch to MIXED mode and recommend that, but some installs probably won't be able to and I'm not sure what the tradeoffs are.

Apr 14 2016, 9:05 PM · Clusters
CodeMouse92 renamed T10779: Cannot manage base project milestones if base project has subprojects from Cannot Manage Milestones to Cannot manage base project milestones if base project has subprojects.
Apr 14 2016, 9:04 PM · Restricted Project, Projects, Bug Report
epriestley removed a project from T10813: Maybe fix various statements that MySQL statement-based replication gets upset about: Phacility.
Apr 14 2016, 8:55 PM · Clusters
epriestley updated subscribers of T10813: Maybe fix various statements that MySQL statement-based replication gets upset about.
Apr 14 2016, 8:55 PM · Clusters
epriestley merged task T6453: MySQL warnings after storage adjustment into T10813: Maybe fix various statements that MySQL statement-based replication gets upset about.
Apr 14 2016, 8:55 PM · Database
epriestley created T10813: Maybe fix various statements that MySQL statement-based replication gets upset about.
Apr 14 2016, 8:55 PM · Clusters
epriestley added a comment to T10794: Upgrading: Changes to Aphlict (Real-Time Notifications) Configuration.

It should be fixed in HEAD, it just only regenerates once every 24 hours or something I think.

Apr 14 2016, 8:45 PM · Installing & Upgrading, Aphlict
eadler added a comment to T10794: Upgrading: Changes to Aphlict (Real-Time Notifications) Configuration.

https://secure.phabricator.com/book/phabricator/article/notifications/ needs to be modified to fit the new world.

Apr 14 2016, 8:44 PM · Installing & Upgrading, Aphlict
epriestley closed T10758: Make `bin/storage dump` replica-aware as Resolved by committing Restricted Diffusion Commit.
Apr 14 2016, 8:23 PM · Restricted Project
epriestley closed T10758: Make `bin/storage dump` replica-aware, a subtask of T4571: Allow Phabricator to run in Read-Only Mode, as Resolved.
Apr 14 2016, 8:23 PM · Phacility
epriestley added a comment to T10812: Use monospaced fonts when displaying credentials in passphrase.

Should be fixed in HEAD of master. Thanks for the report!

Apr 14 2016, 8:10 PM · Passphrase, Feature Request
epriestley closed T10812: Use monospaced fonts when displaying credentials in passphrase as Resolved by committing Restricted Diffusion Commit.
Apr 14 2016, 8:09 PM · Passphrase, Feature Request
epriestley added a comment to T10807: CI setup blocked by commit import bug?.

The staging repository allows you to git fetch the change to the build server (which is easier and more reliable), instead of using arc patch.

Apr 14 2016, 8:08 PM · Bug Report
epriestley added a revision to T10812: Use monospaced fonts when displaying credentials in passphrase: Unknown Object (Differential Revision).
Apr 14 2016, 8:08 PM · Passphrase, Feature Request
epriestley claimed T10812: Use monospaced fonts when displaying credentials in passphrase.
Apr 14 2016, 8:04 PM · Passphrase, Feature Request
yelirekim created T10812: Use monospaced fonts when displaying credentials in passphrase.
Apr 14 2016, 8:01 PM · Passphrase, Feature Request
epriestley added a comment to T10811: `phd-daemon` crash after automatic restart following config change.

The diagnostically useful information from the trace is that this:

Apr 14 2016, 7:58 PM · Daemons, Bug Report
joshma added a comment to T7907: Ability to filter/search Maniphest tasks based on Workboard column position.

We're currently using columns as lightweight "subprojects" - i.e. our Bugs project has a FIXME column for higher-priority bugs. Would the recommendation here be to just use a subproject?

Apr 14 2016, 7:55 PM · Maniphest, Workboards
epriestley added a comment to T10811: `phd-daemon` crash after automatic restart following config change.

In https://<your.install.com>/config/history/, is there a configuration change of any kind shortly before 17:19 UTC?

Apr 14 2016, 7:50 PM · Daemons, Bug Report
jaredbischof added a comment to T10807: CI setup blocked by commit import bug?.

I see. The rule you showed was actually exactly how I set this up before adding a staging repo.

Apr 14 2016, 7:46 PM · Bug Report
areitz created T10811: `phd-daemon` crash after automatic restart following config change.
Apr 14 2016, 7:41 PM · Daemons, Bug Report
swisspol added a comment to T10810: Add Renaming Column command to column contextual menu in workboards.

If you have this in mind, I'm not looking for batches renaming or anything like this.

Apr 14 2016, 7:26 PM · Feature Request
swisspol added a comment to T10810: Add Renaming Column command to column contextual menu in workboards.

That's already the root problem 😉

Apr 14 2016, 7:25 PM · Feature Request
epriestley added a comment to T10694: Improve behavior of context for inline comments in mail.

For design reference, here's the GitHub version of this mail (each comment arrives in a separate mail):

Apr 14 2016, 7:08 PM · Prioritized, Restricted Project, Differential, Mail
epriestley added a revision to T10758: Make `bin/storage dump` replica-aware: Unknown Object (Differential Revision).
Apr 14 2016, 6:44 PM · Restricted Project
slashsBin awarded T7924: Unprototype Calendar (v1) a Love token.
Apr 14 2016, 6:38 PM · Restricted Project, Prioritized, Calendar
• denispoperechny closed Q366: Disable HostName Restrictions as resolved.
Apr 14 2016, 6:36 PM
• denispoperechny added a comment to Q366: Disable HostName Restrictions (Answer 361).

Thanks Evan!

Apr 14 2016, 6:33 PM
epriestley added Q366: Disable HostName Restrictions (Answer 361).
Apr 14 2016, 6:27 PM
epriestley added a comment to T10810: Add Renaming Column command to column contextual menu in workboards.

Why are you renaming columns?

Apr 14 2016, 6:23 PM · Feature Request
areitz added a comment to T10786: Support for process supervision for phabricator daemons.

Yeah, I'm familiar with monit, and I think it will work decently well for this situation. Unfortunately, the current environment standardized on daemontools instead of monit.

Apr 14 2016, 6:20 PM · Feature Request
• denispoperechny updated Q366: Disable HostName Restrictions from to Disable HostName Restrictions.
Apr 14 2016, 6:14 PM
swisspol created T10810: Add Renaming Column command to column contextual menu in workboards.
Apr 14 2016, 6:07 PM · Feature Request
epriestley added a comment to T10751: Make Phabricator Highly Available.

I slightly reduced the amount of header magic, and the notification stuff is now cluster-ready (in theory, at least). I'm planning to get notifications, web nodes, and databases redundant in production today, and maybe repositories/daemons if nothing else comes up. No issues with repo bookkeeping so far but even if it's smooth sailing it needs a fair bit of UI work, and same with the daemons.

Apr 14 2016, 5:50 PM · Restricted Project, Infrastructure, Prioritized
epriestley closed T10806: Support "ssl.chain" in Aphlict configuration as Resolved by committing Restricted Diffusion Commit.
Apr 14 2016, 5:41 PM · Aphlict
epriestley added a comment to T10809: SVN service proxying in Phacility cluster can fail in a couple of ways.

rCOREf9b858d57460 gives web nodes sudo access to svnserve when they're proxying SSH connections.

Apr 14 2016, 5:14 PM · Diffusion, Subversion, Phacility
epriestley added a revision to T10809: SVN service proxying in Phacility cluster can fail in a couple of ways: Unknown Object (Differential Revision).
Apr 14 2016, 5:10 PM · Diffusion, Subversion, Phacility
eadler moved T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 14 2016, 5:08 PM · Prioritized, Restricted Project, Diffusion (v3), Feature Request
eadler moved T10694: Improve behavior of context for inline comments in mail from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 14 2016, 5:08 PM · Prioritized, Restricted Project, Differential, Mail
epriestley added projects to T10809: SVN service proxying in Phacility cluster can fail in a couple of ways: Subversion, Diffusion.
Apr 14 2016, 5:07 PM · Diffusion, Subversion, Phacility
epriestley created T10809: SVN service proxying in Phacility cluster can fail in a couple of ways.
Apr 14 2016, 5:06 PM · Diffusion, Subversion, Phacility
epriestley moved T9790: HTML email doesn't syntax highlight code blocks, inline diffs, or diff context above inline comments from Backlog to The Queue on the Prioritized board.
Apr 14 2016, 4:43 PM · Prioritized, Restricted Project, Remarkup, Mail
epriestley moved T10694: Improve behavior of context for inline comments in mail from Backlog to The Queue on the Prioritized board.
Apr 14 2016, 4:43 PM · Prioritized, Restricted Project, Differential, Mail
epriestley added a project to T9790: HTML email doesn't syntax highlight code blocks, inline diffs, or diff context above inline comments: Prioritized.
Apr 14 2016, 4:43 PM · Prioritized, Restricted Project, Remarkup, Mail
epriestley added a project to T10694: Improve behavior of context for inline comments in mail: Prioritized.
Apr 14 2016, 4:43 PM · Prioritized, Restricted Project, Differential, Mail
chad changed the name of U23 Phacility Consulting from to Phacility Consulting.
Apr 14 2016, 4:31 PM
epriestley raised the priority of T6915: Allow the Aphlict server to scale across multiple hosts from Low to Normal.
Apr 14 2016, 4:13 PM · Restricted Project, Infrastructure, Aphlict
jdickson added a comment to Q365: What are Mercurial shops doing to avoid the bundle2 issue? External repositories?.

Tried running 2.6.2 on the server side, and that indeed does fix the bundle2 issue, but creates more issues when trying to push from a 3.7.3 hg client.. :-(

Apr 14 2016, 4:12 PM · Mercurial, Diffusion
epriestley added a revision to T6915: Allow the Aphlict server to scale across multiple hosts: Unknown Object (Differential Revision).
Apr 14 2016, 4:10 PM · Restricted Project, Infrastructure, Aphlict
overlordtm awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Love token.
Apr 14 2016, 3:50 PM · Workboards (v3), Restricted Project
overlordtm added a member for Workboards: overlordtm.
Apr 14 2016, 3:49 PM
epriestley added a comment to T10807: CI setup blocked by commit import bug?.

(Per T10799, T6878 is the underlying issue with why they don't import, but it's currently never materially useful to import them so this hasn't been a priority to fix. Empirically, they do not import on this install but builds work fine -- except see also T10667 for a weird special case mostly unique to this install.)

Apr 14 2016, 3:10 PM · Bug Report
epriestley added a comment to T10807: CI setup blocked by commit import bug?.

Staging is required (well, not strictly required, but strongly recommended) to setup CI.

Apr 14 2016, 3:08 PM · Bug Report
jaredbischof added a comment to T10807: CI setup blocked by commit import bug?.

Thanks for the reply. I may be confused then. Based on the Harbormaster documentation I understood that staging was required to setup CI. Please correct me if I'm wrong. This is my first time setting this up.

Apr 14 2016, 3:03 PM · Bug Report
Pawka added a revision to T10808: differential.querydiffs method does not provide "local:commits": Unknown Object (Differential Revision).
Apr 14 2016, 2:59 PM · Differential, Bug Report
jdickson added a comment to Q365: What are Mercurial shops doing to avoid the bundle2 issue? External repositories?.

If I host externally, I basically lose the access control and herald rules, right?

Apr 14 2016, 2:56 PM · Mercurial, Diffusion
Pawka updated the task description for T10808: differential.querydiffs method does not provide "local:commits".
Apr 14 2016, 2:49 PM · Differential, Bug Report
Pawka created T10808: differential.querydiffs method does not provide "local:commits".
Apr 14 2016, 2:43 PM · Differential, Bug Report
cspeckmim added a comment to Q365: What are Mercurial shops doing to avoid the bundle2 issue? External repositories?.

We host most of our Mercurial repositories externally - this is primarily the infrastructure we had prior to adopting Phabricator. We have a few repositories hosted by Diffusion, however our server runs 2.6.2 which does not utilize the bundle2 protocol.

Apr 14 2016, 2:39 PM · Mercurial, Diffusion
cspeckmim added a project to Q365: What are Mercurial shops doing to avoid the bundle2 issue? External repositories?: Mercurial.
Apr 14 2016, 2:37 PM · Mercurial, Diffusion
epriestley edited this Badge.
Apr 14 2016, 2:36 PM
jdickson updated Q365: What are Mercurial shops doing to avoid the bundle2 issue? External repositories? from to What are Mercurial shops doing to avoid the bundle2 issue? External repositories?.
Apr 14 2016, 2:35 PM · Mercurial, Diffusion
spawnlt backed I2 Donate to Phacility with $15.00 USD.
Apr 14 2016, 2:27 PM · Phacility
benjumanji added a comment to T10807: CI setup blocked by commit import bug?.

I am not sure exactly what you are trying to achieve, but if you want to do CI with differentials, just set up a herald rule to trigger a build plan when a diff is created / updated. This will trigger whether the staging commits are imported by phabricator or not.

Apr 14 2016, 2:23 PM · Bug Report
benjumanji updated Q364: Is it possible to create a herald to reject non-fastforward pushes? from Is it possible to teach herald to reject non-fastforward pushes? to Is it possible to create a herald to reject non-fastforward pushes?.
Apr 14 2016, 2:16 PM · Herald
benjumanji updated Q364: Is it possible to create a herald to reject non-fastforward pushes? from to Is it possible to teach herald to reject non-fastforward pushes?.
Apr 14 2016, 2:16 PM · Herald
jaredbischof created T10807: CI setup blocked by commit import bug?.
Apr 14 2016, 2:09 PM · Bug Report
jaredbischof added a comment to T10799: Commits pushed to staging repo stuck in "Commit Still Parsing".

Since the commit is not imported, I believe this is currently preventing me from setting up CI with Herald/Harbormaster/Jenkins.

Apr 14 2016, 1:40 PM · Bug Report
epriestley added a revision to T10806: Support "ssl.chain" in Aphlict configuration: Unknown Object (Differential Revision).
Apr 14 2016, 12:51 PM · Aphlict
epriestley added a comment to T10794: Upgrading: Changes to Aphlict (Real-Time Notifications) Configuration.

Here's how I modified the configuration on this host (secure.phabricator.com) to account for this change.

Apr 14 2016, 12:28 PM · Installing & Upgrading, Aphlict
epriestley moved T10794: Upgrading: Changes to Aphlict (Real-Time Notifications) Configuration from Backlog to vNext on the Aphlict board.
Apr 14 2016, 12:22 PM · Installing & Upgrading, Aphlict
epriestley created T10806: Support "ssl.chain" in Aphlict configuration.
Apr 14 2016, 12:22 PM · Aphlict
epriestley added a comment to T10802: Unable to create new forms, even though I am an administrator.

I can not reproduce this issue by following your reproduction steps. Here's what I see on this install:

Apr 14 2016, 12:21 PM · Bug Report
sdarshanam updated the task description for T10802: Unable to create new forms, even though I am an administrator.
Apr 14 2016, 12:18 PM · Bug Report
thuffir closed Q363: Phabricator is not configured to serve cluster requests. as resolved.
Apr 14 2016, 12:11 PM
thuffir added Q363: Phabricator is not configured to serve cluster requests. (Answer 360).
Apr 14 2016, 12:11 PM
epriestley closed T10783: Live notifications do not work behind apache SSL reverse proxy as Resolved by committing Restricted Diffusion Commit.
Apr 14 2016, 11:57 AM · Bug Report
epriestley closed T10697: Move Aphlict server configuration to a separate configuration file as Resolved by committing Restricted Diffusion Commit.
Apr 14 2016, 11:57 AM · Restricted Project, Aphlict
epriestley closed T10697: Move Aphlict server configuration to a separate configuration file, a subtask of T6915: Allow the Aphlict server to scale across multiple hosts, as Resolved.
Apr 14 2016, 11:57 AM · Restricted Project, Infrastructure, Aphlict
Judge added a comment to T10805: Exception in master (related to cluster functioning).

I can confirm this is fixed now.
Thanks, @vinzent @epriestley , for this ultra quick fix!

Apr 14 2016, 11:49 AM · Bug Report
epriestley added a comment to T10783: Live notifications do not work behind apache SSL reverse proxy.

I've marked D15705 as fixing this, since it makes our interpretation of the path unambiguous: we would no longer interpret /ws/ as a request for namespace "ws".

Apr 14 2016, 11:40 AM · Bug Report
3c7 added a comment to T10805: Exception in master (related to cluster functioning).

@epriestley Thanks for the quick fix!

Apr 14 2016, 11:39 AM · Bug Report
epriestley added a comment to T10805: Exception in master (related to cluster functioning).

Sorry about breaking this -- there are a lot of large changes going in this week related to T10751, so master might be a little less stable than usual until the dust settles there.

Apr 14 2016, 11:37 AM · Bug Report
vinzent added a comment to T10805: Exception in master (related to cluster functioning).

just updated, it works now without cluster.addresses set.

Apr 14 2016, 11:36 AM · Bug Report
epriestley added a comment to T10805: Exception in master (related to cluster functioning).

This should now be fixed in HEAD.

Apr 14 2016, 11:33 AM · Bug Report
epriestley closed T10805: Exception in master (related to cluster functioning) as Resolved by committing Restricted Diffusion Commit.
Apr 14 2016, 11:32 AM · Bug Report
epriestley added a comment to T10802: Unable to create new forms, even though I am an administrator.

This report does not include reproduction steps. See Providing Reproduction Steps for help.

Apr 14 2016, 11:31 AM · Bug Report
epriestley updated subscribers of T6878: Tagged commits which are not ancestors of any branch head don't get imported.
Apr 14 2016, 11:29 AM · Wikimedia, Restricted Project, Diffusion
epriestley merged task T10799: Commits pushed to staging repo stuck in "Commit Still Parsing" into T6878: Tagged commits which are not ancestors of any branch head don't get imported.
Apr 14 2016, 11:29 AM · Bug Report
epriestley closed T10800: Facing exception while setting up password on credential menu as Invalid.

Upgrade Phabricator. This is fixed in HEAD.

Apr 14 2016, 11:27 AM · Bug Report
epriestley merged task T10801: Unhandled Exception into T10805: Exception in master (related to cluster functioning).
Apr 14 2016, 11:27 AM · Bug Report
epriestley updated subscribers of T10805: Exception in master (related to cluster functioning).
Apr 14 2016, 11:27 AM · Bug Report
epriestley updated subscribers of T10805: Exception in master (related to cluster functioning).
Apr 14 2016, 11:24 AM · Bug Report