Skip to content

Commit

Permalink
Reorganise release and deployment process documentation (#4528)
Browse files Browse the repository at this point in the history
* Consolidate deployment documentation into a single shared space

* Remove misguided attempt at a unified deployment flowchart

* Fix redirects and page titles

* Use simpler document title

Co-authored-by: Madison Swain-Bowden <[email protected]>

* Address Madison's editorial feedback

* Cleanup, expand, edit, hopefully make things better but maybe not please let me know dear reviewer!

---------

Co-authored-by: Madison Swain-Bowden <[email protected]>
  • Loading branch information
sarayourfriend and AetherUnbound authored Aug 6, 2024
1 parent 5a6c120 commit 23467c8
Show file tree
Hide file tree
Showing 25 changed files with 530 additions and 434 deletions.
2 changes: 1 addition & 1 deletion .codespell/ignore_lines.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
;; Please include filenames and explanations for each ignored line.
;; See https://docs.openverse.org/meta/codespell.html for docs.

;; .github/workflows/release-app.yml
;; .github/workflows/draft_releases.yml
;; The release-drafter/release-drafter configuration variable name cannot be changed
commitish: main

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/draft_releases.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ jobs:
# Each time this runs, it should replace the previous drafted release matching the tag prefix set in the release drafter config
# This means there will always be a running drafted release for all apps
# Publishing the drafted release triggers the `release-app` workflow
# Publishing the drafted release triggers the "Tag docker image and trigger deployment" workflow
- uses: release-drafter/release-drafter@v6
id: release-drafter
with:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Release app
name: Tag Docker image and trigger deployment

on:
release:
Expand All @@ -11,7 +11,7 @@ concurrency:
cancel-in-progress: false

jobs:
release-app:
tag-docker-image-and-trigger-deployment:
name: Release app
runs-on: ubuntu-latest
# Prevent running this workflow on forks, it's unnecessary for external contributors
Expand Down
30 changes: 0 additions & 30 deletions documentation/api/guides/deploy.md

This file was deleted.

1 change: 0 additions & 1 deletion documentation/api/guides/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,5 @@
quickstart
test
deploy
documentation
```
84 changes: 0 additions & 84 deletions documentation/catalog/guides/deployment.md

This file was deleted.

2 changes: 0 additions & 2 deletions documentation/catalog/guides/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
:titlesonly:
quickstart
deploy
adding_a_new_provider
deployment
dag_testing
```
5 changes: 5 additions & 0 deletions documentation/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,11 @@ def add_ext_to_path():
"meta/contribution/good_first_and_help_wanted_issues": "/general/contribution/good_first_and_help_wanted_issues.html",
"frontend/reference/playwright_tests.html": "/frontend/reference/testing_guidelines.html#playwright-tests",
"frontend/reference/storybook_tests.html": "/frontend/reference/testing_guidelines.html#storybook-tests",
"frontend/guides/deploy": "/meta/release_and_deployment/runbooks/frontend.html",
"api/guides/deploy": "/meta/release_and_deployment/runbooks/api.html",
"catalog/guides/deploy": "/meta/release_and_deployment/runbooks/catalog.html",
"catalog/guides/deployment": "/meta/release_and_deployment/index.html#catalog",
"ingestion_server/guides/deploy": "/meta/release_and_deployment/runbooks/ingestion_server.html",
}

myst_enable_extensions = ["linkify"]
1 change: 0 additions & 1 deletion documentation/frontend/guides/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
:titlesonly:
quickstart
deploy
test
analytics
icons
Expand Down
Loading

0 comments on commit 23467c8

Please sign in to comment.