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

📖 Migrate jobs from github-actions-moj-CoFBZ3xV EKS Cluster #4622

Open
7 tasks
Gary-H9 opened this issue Jul 4, 2024 · 16 comments
Open
7 tasks

📖 Migrate jobs from github-actions-moj-CoFBZ3xV EKS Cluster #4622

Gary-H9 opened this issue Jul 4, 2024 · 16 comments
Assignees
Labels

Comments

@Gary-H9
Copy link
Contributor

Gary-H9 commented Jul 4, 2024

Proposal

Note

This ticket precedes this ticket to decommission the cluster.

As per this thread.

Migrate any workloads that run on the github-actions-moj-CoFBZ3xV cluster in analytical-platform-management-production.

Value / Purpose

Consolidate compute, reduce cost.

Definition of Done

  • Establish what workloads are running on the cluster, what they are doing (e.g. is it building a container?) and what role they are using
  • Liaise with owner to gather any requirements (e.g. can repos that interact with CT data run on GitHub hosted?) and inform of our intention to migrate and sunset offering
  • Identify which jobs can move to GitHub Actions hosted runners (GHA) vs. self hosted runners (SHR)
  • Create a role for each job, GitHub assumable for GHA and IRSA for SHR
  • Create/update any shared GitHub Actions actions utilised by workloads
  • Create any self hosted runners in APC
  • Migrate workloads to their new location
@Gary-H9 Gary-H9 added the story label Jul 4, 2024
@Gary-H9 Gary-H9 changed the title 📖 <title> 📖 Migrate jobs from github-actions-moj-CoFBZ3xV EKS Cluster Jul 4, 2024
@julialawrence
Copy link
Contributor

Split this ticket into 2:

  1. Identify and profile workloads (see proposal)
  2. Step to retire

@Emterry Emterry mentioned this issue Jul 12, 2024
4 tasks
@Emterry Emterry self-assigned this Nov 7, 2024
@Emterry Emterry moved this from 👀 TODO to 🚀 In Progress in Analytical Platform Nov 7, 2024
@Emterry
Copy link
Contributor

Emterry commented Nov 7, 2024

Spreadsheet for jobs using management-infrastructure and management-ecr https://docs.google.com/spreadsheets/d/1ZkTFF6MlbkZvk8sA7A9Cyd-JoJVcS6koE4FM2xuNZA4/edit?usp=sharing

Service accounts and roles on the cluster https://docs.google.com/document/d/10z0cGW0wTHP1rA0CPwKz5CcHobqceAD_s9FPdchKgBA/edit?usp=sharing

@Emterry
Copy link
Contributor

Emterry commented Nov 8, 2024

update: completed spreadsheet with job search as mentioned above - didnt include archived repos but can add them in if required, been thinking on how to progress with migration with this repo https://github.com/ministryofjustice/analytical-platform-nginx-jupyter as a test start point as its owned by us. Ive added some initial questions to the google doc linked above if anyone needs to look at it before im back in on tuesday 12/11

@BrianEllwood
Copy link
Contributor

BrianEllwood commented Nov 25, 2024

Update

Brian - Working on PR #6160 as changes requested

Emma - Working on PR #8739 for creating self hosted runner but currently blocked by another PR on APC

All the repositories in the management-infrastructure tab are part of a pr or do not require further action.

@BrianEllwood
Copy link
Contributor

Asked Francesca about the ct-tact-list work in this slack thread

@BrianEllwood
Copy link
Contributor

update from Francesca "I’m aware of the issue —
@Jacob Hamblin-Pyke and I were looking at it but we’re still stumped."

@BrianEllwood
Copy link
Contributor

@YvanMOJdigital YvanMOJdigital moved this from 🚀 In Progress to 🛂 In Review in Analytical Platform Dec 11, 2024
@YvanMOJdigital YvanMOJdigital moved this from 🛂 In Review to 🚀 In Progress in Analytical Platform Dec 11, 2024
@YvanMOJdigital YvanMOJdigital moved this from 🚀 In Progress to 🛂 In Review in Analytical Platform Dec 11, 2024
@BrianEllwood
Copy link
Contributor

Outstanding PR's for Management infrastructure migrations

#6160

#6117

Work on ct-tact-list is blocked by issues with code being addressed by the code owner.

Draft communication to the users here

@tom-webber tom-webber moved this from 🛂 In Review to 🚀 In Progress in Analytical Platform Dec 12, 2024
@Emterry
Copy link
Contributor

Emterry commented Dec 12, 2024

outstanding pr's to be reviewed merged :

https://github.com/ministryofjustice/analytics-platform-aws-federated-login/pull/1555 - failing due to needing to be made public. I havnt added devcontainer - does it need it? maybe?
ministryofjustice/analytics-platform-control-panel#1398 - control panel image
#6117 - our joint pr

Comms : https://docs.google.com/document/d/1bkC6IjvBQ3x_5LLOLEZCb5C-GbKx6I1DSab2rnqR17Y/edit?tab=t.0

@BrianEllwood
Copy link
Contributor

Just for clarity this is the roles and permissions PR that needs approval.

As there is quite a few its quite noisy

@BrianEllwood
Copy link
Contributor

Task - create PR's to implement Migrations to Github Actions for Repositories where new role has been created, then contact users for approval to implement after christmas.

@BrianEllwood
Copy link
Contributor

BrianEllwood commented Dec 18, 2024

This is the current state of play for the repositories and prs to make the changes to migrate to Github hosted runners

create-a-derived-table-infrastructure - changes made and merged
data-engineering-database-access - all checks passed on changes in draft pr
analytics-platform-helm-chart - draft changes still failing checks
airflow-contracts-etl - to do
data-engineering-cleanup - to do
data-discovery - to do
create-a-derived-table - to do

Agreed that we should create a issue on the repositories and post coms in the Support channel

@BrianEllwood
Copy link
Contributor

Comms sent out in slack

@YvanMOJdigital YvanMOJdigital moved this from 🚀 In Progress to 👀 TODO in Analytical Platform Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 👀 TODO
Development

No branches or pull requests

5 participants