Skip to content

Conversation

@aarongable
Copy link
Contributor

@aarongable aarongable commented Dec 4, 2025

The new "created" column is always populated by the current time, indicating when the row was created. We do this directly inside the database, rather than in the Go data model, because Boulder doesn't actually care when these rows were created. Instead, we care about the creation timestamp for the purpose of creating and dropping partitions to manage our total data size.

Previously we've been partitioning these tables based on their expires column. But that column is no longer a good proxy for when the row was created, because not all certificates/orders/authzs have the same lifetime anymore.

Example after an integration test run:

MariaDB [boulder_sa_integration]> select id,created,expires from orderFqdnSets limit 5;
+----+---------------------+---------------------+
| id | created             | expires             |
+----+---------------------+---------------------+
|  1 | 2025-12-04 00:49:10 | 2025-12-11 00:49:10 |
|  3 | 2025-12-04 00:49:12 | 2025-12-11 00:49:12 |
|  6 | 2025-12-04 00:49:17 | 2025-12-11 00:49:17 |
|  9 | 2025-12-04 00:49:23 | 2025-12-11 00:49:23 |
| 10 | 2025-12-04 00:49:24 | 2025-12-11 00:49:24 |
+----+---------------------+---------------------+
5 rows in set (0.000 sec)

IN-12028 tracks the corresponding SRE-side changes

@aarongable aarongable marked this pull request as ready for review December 4, 2025 01:01
@aarongable aarongable requested a review from a team as a code owner December 4, 2025 01:01
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

@aarongable, this PR appears to contain configuration and/or SQL schema changes. Please ensure that a corresponding deployment ticket has been filed with the new values.

@aarongable aarongable requested review from jprenken, jsha and lenaunderwood22 and removed request for beautifulentropy December 4, 2025 01:01
@aarongable aarongable merged commit 16cfacd into main Dec 5, 2025
17 checks passed
@aarongable aarongable deleted the auto-created-dt branch December 5, 2025 20:59
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.

5 participants