Skip to content

docs: add hardware sizing section and remove removed MySQL 8 variables from Requirements.md - #216

Open
somethingwithproof wants to merge 1 commit into
Cacti:developfrom
somethingwithproof:fix/215-requirements-hardware-sizing
Open

docs: add hardware sizing section and remove removed MySQL 8 variables from Requirements.md#216
somethingwithproof wants to merge 1 commit into
Cacti:developfrom
somethingwithproof:fix/215-requirements-hardware-sizing

Conversation

@somethingwithproof

@somethingwithproof somethingwithproof commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

Closes #215
Refs #106

Summary

  • Add Hardware Sizing section to Requirements.md with a sizing table covering small (< 500 devices), medium (500-2,000), large (2,000-10,000), and very large (> 10,000) deployments, including guidance on:

  • Remove innodb_additional_mem_pool_size from the description list (removed in MySQL 5.7.4; causes startup failure on any current MySQL or MariaDB)

  • Remove innodb_file_format = Barracuda and innodb_large_prefix = 1 from the paste-in config block at the bottom of the file (both removed in MySQL 8.0; cause startup failure)

Test plan

  • Verify sizing table numbers are consistent with community guidance
  • Verify MySQL 8.0 no longer accepts innodb_additional_mem_pool_size, innodb_file_format, innodb_large_prefix
  • Verify config paste-in block still works on MariaDB 10.5+

Copilot AI review requested due to automatic review settings March 9, 2026 09:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Cacti’s Requirements documentation to add baseline hardware sizing guidance and to remove MySQL/InnoDB configuration variables that are no longer valid on modern MySQL versions.

Changes:

  • Added a Hardware Sizing section with a deployment-sizing table and operational notes (SSD/NVMe, Spine scaling, remote collectors).
  • Removed innodb_additional_mem_pool_size from the MySQL/MariaDB recommendations list.
  • Removed innodb_file_format and innodb_large_prefix from the sample my.cnf paste-in block.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Requirements.md Outdated
Comment thread Requirements.md
Comment thread Requirements.md Outdated
Refs Cacti#215

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
@somethingwithproof
somethingwithproof force-pushed the fix/215-requirements-hardware-sizing branch from ba0b5ee to 1c66f98 Compare March 15, 2026 05:20
@somethingwithproof

Copy link
Copy Markdown
Contributor Author

All three review points are already covered by the current content: the innodb_doublewrite note now spells out the durability trade-off instead of recommending OFF outright, the "Very large" row keeps Disk to pure storage sizing with remote-poller guidance moved to its own note, and the stale PHP 5.4/MySQL 5.6 minimums flagged from #215 are already updated to PHP 8.1 and MySQL 5.7/MariaDB 10.2 elsewhere in Requirements.md. No further changes needed; resolving.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Requirements.md: stale software versions, removed MySQL vars, no hardware sizing section

2 participants