Skip to content

Sync controller authorization on deploy - #51

Merged
gabotechs merged 1 commit into
mainfrom
codex/sync-controller-authorized-logins
Aug 25, 2026
Merged

Sync controller authorization on deploy#51
gabotechs merged 1 commit into
mainfrom
codex/sync-controller-authorized-logins

Conversation

@gabotechs

Copy link
Copy Markdown
Owner

Summary

  • pass the committed authorized GitHub login list through normal controller releases
  • update only AUTHORIZED_GITHUB_LOGINS in the protected controller environment, preserving GH_TOKEN
  • ignore upstream AMI drift so application deploys do not replace the persistent controller

Validation

  • npm run build
  • npm test (106 tests)
  • deployed in place to the existing controller
  • verified live allowlist contains all three committed users
  • replayed PR 639 request accepted as benchmark job 44
  • final Pulumi preview: 16 resources unchanged

@gabotechs
gabotechs merged commit f666ab4 into main Aug 25, 2026
2 checks passed
@gabotechs
gabotechs deleted the codex/sync-controller-authorized-logins branch August 25, 2026 06:31
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.

1 participant