Skip to content

feat: enable daily safe-settings sync and clean up documentation - #216

Merged
marcusburghardt merged 2 commits into
complytime:mainfrom
marcusburghardt:chore/enable-safe-settings-schedule
Sep 4, 2026
Merged

feat: enable daily safe-settings sync and clean up documentation#216
marcusburghardt merged 2 commits into
complytime:mainfrom
marcusburghardt:chore/enable-safe-settings-schedule

Conversation

@marcusburghardt

Copy link
Copy Markdown
Member

Summary

Enable the Safe Settings Sync workflow to run daily and clean up all
outdated migration-era comments and documentation.

Changes

Commit 1: Enable daily schedule

  • Add schedule trigger at 07:00 UTC (90-minute window after
    peribolos apply at 05:30 UTC, which can take ~40 minutes)
  • Scheduled runs apply changes automatically (dry-run=false)
  • Manual dispatch retains dry-run=true as default for safety
  • Log level set to info for scheduled runs (not debug)

Commit 2: Clean up outdated comments and documentation

safe-settings config files:

  • Remove future-tense comments ("will enable", "this enables it
    org-wide", "Currently only on") that described planned changes
    which are now the current state
  • Remove gemara-content-service references (no longer in the org)
  • Remove manual ruleset ID references from repo override files
  • Fix inaccurate suborg description (non-code repos do require
    code owner review)

MAINTAINING.md:

  • Replace "Future automation" section with "Automation" (schedule
    is now live)
  • Update troubleshooting to reference daily sync
  • Add deployment-settings.yml step to "Add a New Repository" workflow
  • Remove completed "Migration Notes" section
  • Fix restrictedRepos description (include-based allowlist, not
    exclude)

gxmiranda
gxmiranda previously approved these changes Sep 4, 2026

@gxmiranda gxmiranda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Add a daily schedule trigger at 07:00 UTC, after peribolos apply
(05:30 UTC, ~40 min runtime) to ensure org membership is current
before safe-settings applies repository settings.

Scheduled runs apply changes (dry-run=false) against all managed
repos. Manual dispatch retains dry-run=true as default for safety.

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: Marcus Burghardt <maburgha@redhat.com>
Remove migration-era comments from safe-settings config files:
- settings.yml: remove future-tense comments, gemara reference
- repos/complyctl.yml: remove manual ruleset ID reference
- repos/complytime.yml: remove manual ruleset ID reference
- suborgs/code-repos.yml: remove gemara reference
- suborgs/non-code-repos.yml: fix inaccurate description

Update MAINTAINING.md to reflect current state:
- Replace "Future automation" with "Automation" (schedule is live)
- Update troubleshooting to reference daily sync instead of manual-only
- Add deployment-settings.yml step to new repo workflow
- Remove gemara-content-service from excluded repos
- Fix restrictedRepos description (include-based, not exclude)
- Remove completed Migration Notes section

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: Marcus Burghardt <maburgha@redhat.com>
@marcusburghardt

Copy link
Copy Markdown
Member Author

@gxmiranda I had to rebase in order to resolve the conflict.

@marcusburghardt
marcusburghardt enabled auto-merge (rebase) September 4, 2026 14:53

@gxmiranda gxmiranda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM after rebase

@marcusburghardt
marcusburghardt merged commit c8cdfc7 into complytime:main Sep 4, 2026
14 checks passed
@marcusburghardt
marcusburghardt deleted the chore/enable-safe-settings-schedule branch September 4, 2026 14:53
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.

3 participants