Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove access from Jamie and Malcolm #741

Merged
merged 6 commits into from
Mar 25, 2024

Conversation

monotypical
Copy link
Contributor

@monotypical monotypical commented Mar 18, 2024

What

Remove access to concourse for Jamie and Malcolm, and remove their GPG keys from those allowed to sign merge commits, as they have left the team

How to review

  • Set a dev concourse environment to use this branch
  • Upload the trusted people file using the remove-feb-leavers branch: gds aws paas-dev-admin -- make <dev-env-name> upload-paas-trusted-people PAAS_TRUSTED_PEOPLE_BRANCH=remove-feb-leavers
  • run the create-bosh-concourse pipeline and confirm that access for Jamie and Malcolm has been removed.
  • Ensure that the GPG keys belonging to Jamie van Dyke and Malcolm Saunders have been removed from .gpg-id

Who can review

Existing PaaS team members with access to dev environments


🚨⚠️ Please do not merge this pull request via the GitHub UI ⚠️🚨

@monotypical monotypical changed the title Remove concourse access from Jamie and Malcolm Remove access from Jamie and Malcolm Mar 18, 2024
Copy link
Contributor

@dark5un dark5un left a comment

Choose a reason for hiding this comment

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

LGTM

@monotypical monotypical merged commit c43d111 into main Mar 25, 2024
10 checks passed
@monotypical monotypical deleted the 187103644-jamie-malcolm-leavers branch March 25, 2024 10:51
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.

2 participants