Page MenuHomePhabricator

jessjohnson (Jess Johnson)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Aug 19 2016, 5:35 PM (306 w, 2 d)
Availability
Available

Recent Activity

Aug 1 2017

jessjohnson added a comment to T10598: Options to display Real Name instead of username everywhere.

@ofbeaton Thanks for the tips. Sounds better than the approach we are using to slightly modify the code base.

Aug 1 2017, 6:10 PM · Feature Request
jessjohnson added a comment to T10598: Options to display Real Name instead of username everywhere.

@ofbeaton Just curious what's your strategy on applying patches as part of your deployment for phabricator? Do you use a find/replace strategy? Apply diffs?

Aug 1 2017, 3:03 AM · Feature Request

Apr 20 2017

jessjohnson added a comment to T12609: Prevent arc land when unit tests or build fails.

Yes, it would. We haven't fully integrated the build system with Phabricator, but I'll review the docs and see how much more work we need to do.

Apr 20 2017, 10:25 PM · Feature Request
jessjohnson added a comment to T12609: Prevent arc land when unit tests or build fails.

Engineers are not hitting a prompt when landing, they are disregarding the unit test failure message or not reviewing why the build failed. Since the build failure is on an external system, the engineer needs to view this information separately (the system posts a comment why the build failed). The overall goal of a herald rule blocking the land is so that owner can communicate that checking in code without running or successful unit tests is not cool.

Apr 20 2017, 10:17 PM · Feature Request
jessjohnson created T12609: Prevent arc land when unit tests or build fails.
Apr 20 2017, 8:35 PM · Feature Request

Dec 22 2016

jessjohnson added a comment to T12039: Emails from Diffusion contain space or new line character in link making them not clickable.

Thanks I made the smtp-encoding change. We're just about ready to update to a vanilla install. If it doesn't fix it and I can provide reproducible steps, I'll update this bug report.

Dec 22 2016, 5:31 PM · Bug Report
jessjohnson added a comment to T12039: Emails from Diffusion contain space or new line character in link making them not clickable.

We're running a fork and sorry, I know it's a little behind stable. We're working on getting back to a vanilla install to make updates easier.

$ git merge-base HEAD origin/master
33bf2a79def2e03c2248730afa080ead489942be
Dec 22 2016, 4:32 PM · Bug Report
jessjohnson created T12039: Emails from Diffusion contain space or new line character in link making them not clickable.
Dec 22 2016, 4:22 PM · Bug Report

Oct 12 2016

jessjohnson added a comment to T11708: Daemons erroneously saying "Diff does not exist".

@jcox I just noticed this line from you:

Oct 12 2016, 11:29 PM · Daemons, Differential, Bug Report
jessjohnson added a comment to T11708: Daemons erroneously saying "Diff does not exist".

@jcox, I dove into this a bit, but like you, I didn't come up with much. We have not used bin/remove destroy for any of our repos and there doesn't appear to be any meaningful relationship between the failing tasks that I have found yet. The script that @epriestley kindly provided was able to load any diff I threw at it. The script was helpful in determining the right queries to make to investigate further.

Oct 12 2016, 11:05 PM · Daemons, Differential, Bug Report
jessjohnson added a comment to T11708: Daemons erroneously saying "Diff does not exist".

Just checked this morning and many tasks have returned and started to fail again. I'm sure there's a way to reproduce this, but our setup is a bit complex and I'm not sure where to start. @epriestley Could you tell me how to determine what these tasks are even doing? Is there a way to see what action triggered them and why they will fail indefinitely?

Oct 12 2016, 5:34 PM · Daemons, Differential, Bug Report

Oct 11 2016

jessjohnson added a comment to T11708: Daemons erroneously saying "Diff does not exist".

Also, like @jcox, the diffs always load in that advanced space-age script like there's no problem. Perhaps these exceptions are aware of our attempts to end them. @epriestley Do you have an idea of something else we could try to see why so many exceptions are occurring?

Oct 11 2016, 11:07 PM · Daemons, Differential, Bug Report
jessjohnson added a comment to T11708: Daemons erroneously saying "Diff does not exist".

We're also seeing about 100k of these failures in our logs over the past few weeks. I clear out all the tasks and they quickly return. Have you found the root cause?

Oct 11 2016, 10:50 PM · Daemons, Differential, Bug Report

Sep 1 2016

jessjohnson added a comment to T11573: Phabricator goes down with minimal load due to asset delivery issue for some users.

Just some quick googling revealed this issue with a prior version of APCu.

Sep 1 2016, 9:35 PM · Bug Report
jessjohnson added a comment to T11573: Phabricator goes down with minimal load due to asset delivery issue for some users.

I turned it back on and ran additional load tests but Apache was able to handle everything. To be honest, I was surprised how well the site held up with APCu turned off. The numbers are about the same.

Sep 1 2016, 9:32 PM · Bug Report
jessjohnson added a comment to T11573: Phabricator goes down with minimal load due to asset delivery issue for some users.

I've been running load tests for the past 2-3 hours and for some reason, I can't bring it down anymore. You can probably mark this bug as 'unable to reproduce'. I'll continue looking into the issue and see if there's a definitive way to reproduce.

Sep 1 2016, 8:36 PM · Bug Report
jessjohnson added a comment to T11573: Phabricator goes down with minimal load due to asset delivery issue for some users.

I've started the load tests without APC. One odd item I didn't notice before that may or may not be related is that Phabricator is having a hard time knowing that daemons are running (e.g. you have 2 unresolved setup issues instead of just 1 because APC is turned off). I checked the daemon logs and see they are flooded with this with one entry occurring a lot. I can't track down the source because I don't know what address it's trying to reach out to (is this a repository pull or a herald rule?). Again, this may be completely unrelated and a different issue all together.

Sep 1 2016, 6:59 PM · Bug Report
jessjohnson added a comment to T11573: Phabricator goes down with minimal load due to asset delivery issue for some users.

Are you running APC (ConfigCache Status shows APC as the active "Data Cache")?

If so, can you reproduce it if you disable APC (apc.enabled=0 in php.ini), restart apache, verify that no "Data Cache" is available any more in ConfigCache Status, and then use the takedown script?

Sep 1 2016, 6:36 PM · Bug Report
jessjohnson added a comment to T11573: Phabricator goes down with minimal load due to asset delivery issue for some users.

Here's the output. I removed "events" from some of them to improve formatting.

Sep 1 2016, 6:33 PM · Bug Report
jessjohnson added a comment to T11573: Phabricator goes down with minimal load due to asset delivery issue for some users.

I've added a simple javascript file on the installation I grabbed this report from. As part of the troubleshooting process, I installed a vanilla installation of Phabricator on a separate machine with no modifications (version referenced in the bug report) and was able to reproduce the problem.

Sep 1 2016, 6:24 PM · Bug Report
jessjohnson created T11573: Phabricator goes down with minimal load due to asset delivery issue for some users.
Sep 1 2016, 6:12 PM · Bug Report

Aug 22 2016

jessjohnson added a comment to T11188: Updating to latest Phabricator caused feed corruption.

To build on what was originally suggested, you can have all your repos stop publishing events by running the following which will list all repos and put them back to importing mode so they don't publish events:

Aug 22 2016, 4:45 PM · Bug Report

Aug 20 2016

jessjohnson added a comment to T11503: Notifications for past events sent after Phabricator upgrade.

Yes, this seems to be exactly what the other issue was referring to. We've just recently upgraded (previous codebase was 2+ years old) and massive amounts of commit email are getting sent out.

Aug 20 2016, 4:14 AM · Diffusion, Bug Report (Needs Information)
jessjohnson updated the task description for T11503: Notifications for past events sent after Phabricator upgrade.
Aug 20 2016, 1:15 AM · Diffusion, Bug Report (Needs Information)
jessjohnson created T11503: Notifications for past events sent after Phabricator upgrade.
Aug 20 2016, 1:14 AM · Diffusion, Bug Report (Needs Information)