Page MenuHomePhabricator

Provide some hints for Amazon RDS configuration
ClosedPublic

Authored by epriestley on Aug 25 2014, 2:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 11, 7:14 PM
Unknown Object (File)
Mon, Dec 9, 4:02 PM
Unknown Object (File)
Wed, Nov 20, 5:28 PM
Unknown Object (File)
Nov 15 2024, 9:59 PM
Unknown Object (File)
Nov 11 2024, 8:27 PM
Unknown Object (File)
Nov 9 2024, 8:39 AM
Unknown Object (File)
Nov 8 2024, 12:51 AM
Unknown Object (File)
Oct 25 2024, 10:09 AM
Subscribers

Details

Summary

Fixes T2605. Provide some instructions on configuring RDS properly. The "DB Parameter Group" thing in the web UI seems pretty easy to use, it's just not obvious that it's what you should be using.

Test Plan

Jiggled these warnings to trigger them, viewed the output, saw a table of values and a hint about RDS.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Provide some hints for Amazon RDS configuration.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
src/applications/config/check/PhabricatorSetupCheckMySQL.php
145

I made this change because a user in IRC is reporting that their setting is set to 6, apparently (??). I figure users who legitimately want to have this at 6 can just ignore the warning. This seems unusual.

btrahan edited edge metadata.

Nice, I love a good doc.

src/applications/config/check/PhabricatorSetupCheckMySQL.php
149–150

might as well use $min_len here instead of 4

src/docs/user/userguide/amazon_rds.diviner
10

support meaning our ability to help you? (or said differently, all features should work?)

This revision is now accepted and ready to land.Aug 25 2014, 6:13 PM
src/docs/user/userguide/amazon_rds.diviner
10

Our ability, yeah. I'll clarify this.

epriestley edited edge metadata.
  • Reuse variable.
  • Clarify meaning of "support".
epriestley updated this revision to Diff 24908.

Closed by commit rP6dd82d86a2b7 (authored by @epriestley).