Paths

Table of Contentst

Differential D12322

[terraform] MariaDB RDS config options
ClosedPublic
Actions

Authored by will on Jun 5 2024, 3:22 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Nov 15, 5:49 PM

	Unknown Object (File)
	Tue, Nov 12, 12:13 PM

	Unknown Object (File)
	Tue, Nov 12, 12:01 PM

	Unknown Object (File)
	Tue, Nov 12, 12:01 PM

	Unknown Object (File)
	Tue, Nov 12, 11:29 AM

	Unknown Object (File)
	Tue, Nov 12, 6:50 AM

	Unknown Object (File)
	Tue, Nov 12, 2:35 AM

	Unknown Object (File)
	Oct 20 2024, 1:26 PM

View All 91 Files

Subscribers

Details

Reviewers

ashoat
bartek
varun

Commits

rCOMMb2f66ef85324: [terraform] MariaDB RDS config options

Summary

This sets mostly the same parameter options as MariaDB besides innodb_buffer_pool_size which is set to {DBInstanceClassMemory*3/4} matching our phabricator database and general recommendations.

Here are the configuration options in our keyserver docker compose file:

--performance-schema
--max-allowed-packet=256M
--local-infile=0
--sql-mode=STRICT_ALL_TABLES
--innodb-buffer-pool-size=1600M
--innodb-ft-min-token-size=1
--innodb-ft-enable-stopword=0

Test Plan

successful terraform deploy. Ran transfer process and successfully accessed the rds instance with mysql

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

will created this revision.Jun 5 2024, 3:22 PM

Herald added a subscriber: tomek. · View Herald TranscriptJun 5 2024, 3:22 PM

will added inline comments.Jun 5 2024, 3:25 PM

services/terraform/self-host/aws_db.tf
60 ↗	(On Diff #40998)	This is representative of 256 MiB: (1024 * 1024 * 256).

will edited the test plan for this revision. (Show Details)Jun 5 2024, 3:27 PM

will edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B29463: Diff 40998.Jun 5 2024, 3:39 PM

will requested review of this revision.Jun 5 2024, 3:39 PM

will added a child revision: D12323: [terraform] remove usage of s3 bucket for terraform config.Jun 5 2024, 4:18 PM

I haven't dug deep into what these params mean but they match docker configuration so it should be okay

services/terraform/self-host/aws_db.tf
60 ↗	(On Diff #40998)	Comment would be good

This revision is now accepted and ready to land.Jun 6 2024, 12:34 AM

add comment: 256 MiB: (1024 * 1024 * 256)

Harbormaster completed remote builds in B29495: Diff 41068.Jun 6 2024, 12:00 PM

ashoat mentioned this in D12337: [keyserver] Bump MariaDB --innodb-buffer-pool-size config to 14000M in docker-compose.yml.Jun 6 2024, 8:02 PM

rebase

Harbormaster completed remote builds in B29506: Diff 41079.Jun 6 2024, 9:28 PM

ashoat mentioned this in rCOMMcdbbbc452d07: [keyserver] Bump MariaDB --innodb-buffer-pool-size config to 14000M in docker….Jun 7 2024, 5:55 AM

Closed by commit rCOMMb2f66ef85324: [terraform] MariaDB RDS config options (authored by will). · Explain WhyJun 7 2024, 10:55 AM

This revision was automatically updated to reflect the committed changes.

will added a commit: rCOMMb2f66ef85324: [terraform] MariaDB RDS config options.

Revision Contents
Changeset List

Path

Size

services/

terraform/

self-host/

51 lines

Diff 41112

services/terraform/self-host/aws_db.tf

Loading...