Skip to content

Config data patch: Adding facade_contributor_full_recollect#3303

Merged
sgoggins merged 2 commits intomainfrom
config-data-patch-33
Oct 16, 2025
Merged

Config data patch: Adding facade_contributor_full_recollect#3303
sgoggins merged 2 commits intomainfrom
config-data-patch-33

Conversation

@sgoggins
Copy link
Member

@sgoggins sgoggins commented Oct 9, 2025

Description

  • Adds facade_contributor_full_recollect parameter, defaulted to 0 to the augur_operations.config table.
  • Uses Alembic. We think there are other strategies more robust for config data items, but for now we also know this works and the config refactoring is down the road.

Signed-off-by: Sean P. Goggins <s@goggins.com>
Signed-off-by: Sean P. Goggins <s@goggins.com>
@sgoggins sgoggins requested review from MoralCode and Ulincsys October 9, 2025 15:30
Copy link
Contributor

@MoralCode MoralCode left a comment

Choose a reason for hiding this comment

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

While I generally disagree with using alembic structural migrations for modifying the database contents (when not related to a structural change), this does seem like a temporarily easier method to ensure non docker users of augur have correct configuration databases.

@sgoggins sgoggins added release Related to releasing a new version of Augur bug-fix Fixes a bug labels Oct 14, 2025
@MoralCode MoralCode added security Related to keeping Augur secure ready Items tested and seeking additional approvals or a merge. Usually for items under active development and removed security Related to keeping Augur secure labels Oct 16, 2025
@sgoggins sgoggins merged commit 8487825 into main Oct 16, 2025
15 checks passed
@sgoggins sgoggins deleted the config-data-patch-33 branch November 12, 2025 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Fixes a bug ready Items tested and seeking additional approvals or a merge. Usually for items under active development release Related to releasing a new version of Augur

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants