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

💔 Split out Github Component to Separate Repo #3044

Closed
5 of 6 tasks
Tracked by #1827
julialawrence opened this issue Jan 22, 2024 · 7 comments
Closed
5 of 6 tasks
Tracked by #1827

💔 Split out Github Component to Separate Repo #3044

julialawrence opened this issue Jan 22, 2024 · 7 comments
Assignees
Labels
data-platform-apps-and-tools This issue is owned by Data Platform Apps and Tools 🧑‍💻 Apps & Tools BAU (Epic #1827)

Comments

@julialawrence
Copy link
Contributor

julialawrence commented Jan 22, 2024

We would like to move our Github configuration into a separate repo to make it easier for self-serve requests.

As platform engineers, we've encountered reluctance on the part of our users to raise PRs and otherwise self-serve requests for team membership because the github component of a rather large monorepo. Although the process is isolated, there is fear that improperly made request will have ripple effects on other infra. Therefore, we would like to split out GH component into its own repo, potentially expanding it later to include other self-serve functionality.

Proposal:

Split off GH team management component into its own repository.

Definition of Done:

  • Repository created
  • Dependencies to other part of the code identified and remediated
  • Workflows created
  • Provisioning roles updated
  • Code tested and reviewed by other team members
  • Documentation for self-servicing requests updated
@julialawrence julialawrence converted this from a draft issue Jan 22, 2024
@julialawrence julialawrence added the data-platform-apps-and-tools This issue is owned by Data Platform Apps and Tools label Jan 22, 2024
@julialawrence julialawrence changed the title Split out Github Component to Internal Repo Split out Github Component to Separate Repo Jan 22, 2024
@julialawrence julialawrence moved this to 🧐 To Do in Analytical Platform Jan 22, 2024
@jacobwoffenden jacobwoffenden changed the title Split out Github Component to Separate Repo 💔 Split out Github Component to Separate Repo Feb 12, 2024
@BrianEllwood
Copy link
Contributor

BrianEllwood commented Mar 20, 2024

Confirm that the repo in question is data_platform
Confirm the name of the new repository
identify the GH team management components to be moved
need more clarification about Provisioning roles updated

@BrianEllwood
Copy link
Contributor

confirmed
source data-platform
new data-platform-github-access - internal

this is the component to move https://github.com/ministryofjustice/data-platform/tree/main/terraform/github
start with teams using import to create.

@murad-ali-MoJ murad-ali-MoJ moved this from 👀 TODO to 🚀 In Progress in Analytical Platform Mar 21, 2024
@AntFMoJ
Copy link
Contributor

AntFMoJ commented Mar 21, 2024

New repo created in this PR.
Repo located here.

Copied terraform/github directory to new repo, terraform plan is showing no change as expected.

@AntFMoJ
Copy link
Contributor

AntFMoJ commented Mar 28, 2024

Tasks left to do:

  • Create workflows
  • Add devcontainer
  • Add documentation for self-servicing requests updated

@BrianEllwood
Copy link
Contributor

draft pr to add workflow for repository pr

@BrianEllwood
Copy link
Contributor

We had been working in two branches one for the workflow and one for the terraform we had issues with the later and the workflow failed as it was still an empty repo so we started a new branch git_hub_workflow.

in the new branch we found out the terraform plan which had previously had no actions had multiple creates and destroys so we copied the github dir from data-platform again and the plans were then identical 2 to change ? -- remember we will have to do this again to keep the new repo in sync with data-platform until the switch over

The workflow had issues with the AWS credentials in the draft pr so had to manually amend the role GlobalGitHubActionAccess to include the new repo "repo:ministryofjustice/data-platform-github-access:*". this enabled the workflow to create the terraform plan

Lots of issues with the super linter that we will have to address

@github-project-automation github-project-automation bot moved this from 🚀 In Progress to 🎉 Done in Analytical Platform Apr 8, 2024
@BrianEllwood
Copy link
Contributor

In the end we merged Jacob W's version after a session where we went through his version compared to the work we had done. We had carried forward methods/code from the mono repo which were not required in the new stand alone repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data-platform-apps-and-tools This issue is owned by Data Platform Apps and Tools 🧑‍💻 Apps & Tools BAU (Epic #1827)
Projects
Archived in project
Development

No branches or pull requests

4 participants