generated from ministryofjustice/template-repository
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Move back to the standard github workflow for modernisation platform #60
Comments
moj-data-platform-robot
transferred this issue from ministryofjustice/analytical-platform
Apr 25, 2024
MatMoore
added a commit
to ministryofjustice/modernisation-platform-environments
that referenced
this issue
May 8, 2024
We are no longer using any of this. It also came with a customisation to the standard github actions workflow which we wanted to remove. ministryofjustice/data-catalogue#60
MatMoore
added a commit
to ministryofjustice/modernisation-platform-environments
that referenced
this issue
May 8, 2024
We are no longer using any of this. It also came with a customisation to the standard github actions workflow which we wanted to remove. ministryofjustice/data-catalogue#60
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Background
We have diverged from the standard workflow by creating our own workflow:
.github/workflows/reusable_terraform_plan_apply_test.yml
We did this as part of PoC for testing with terratest and smoke testing, but we probably shouldn't have let this become a permanent solution.
Task
Temporarily remove the smoke tests, and/or find a way to trigger them from the main workflow in a way that is reusable by other teams.
If we remove it temporarily, we can raise an issue with modernisation platform to add this functionality back in a way they can support.
Don't wait until we have finalised the testing work: we will likely improve tests bit by bit as our focus now is on the catalogue.
Stretch goal: David Eliot requested a demo for the MP of the tests working https://mojdt.slack.com/archives/C01A7QK5VM1/p1704463604108639?thread_ts=1704455070.725129&cid=C01A7QK5VM1
Value / Purpose
The current approach is creating friction for both us and modernisation platform team
Example thread where this happened in January https://mojdt.slack.com/archives/C01A7QK5VM1/p1704455070725129
Response from Ewa Stempel https://mojdt.slack.com/archives/C01A7QK5VM1/p1704462864082089?thread_ts=1704455070.725129&cid=C01A7QK5VM1
Definition of Done
The text was updated successfully, but these errors were encountered: