Skip to content

fix: only run CDN purge for production storybook #4321

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.

Sign up for GitHub

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

Merged
pomfrida merged 6 commits into from
Dec 9, 2025
Merged

fix: only run CDN purge for production storybook #4321

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
6ec1343
fix: move env vars to job level for environment context
pomfrida Dec 9, 2025
ef97365
fix: only run CDN purge for production storybook
pomfrida Dec 9, 2025
dbfe215
refactor: make publish_storybook manual-only and default to production
pomfrida Dec 9, 2025
26f1291
fix: keep push trigger for dev storybook, only purge CDN for production
pomfrida Dec 9, 2025
b8cb67d
docs: add dev storybook to readme
pomfrida Dec 9, 2025
6088864
docs: add comment explaining CDN purge condition
pomfrida Dec 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 6 additions & 8 deletions .github/workflows/_purge_cdn.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,19 +8,17 @@ on:
endpoint:
type: string
default: artefacts
# Because of limitations in how to set environment context when calling reusable workflows,
# we are using ENV variables directly in this one. Not pretty, but it works.
env:
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
AZURE_CDN_PROFILE_NAME: ${{ vars.AZURE_CDN_PROFILE_NAME }}
AZURE_CDN_ENDPOINT_NAME: ${{ vars.AZURE_CDN_ENDPOINT_NAME }}
AZURE_STORYBOOK_ENDPOINT_NAME: ${{ vars.AZURE_STORYBOOK_ENDPOINT_NAME }}
AZURE_RESOURCE_GROUP: ${{ vars.AZURE_RESOURCE_GROUP }}
jobs:
purge:
name: Purge the cdn to refresh cache
runs-on: ubuntu-latest
environment: ${{ inputs.environment }}
env:
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
AZURE_CDN_PROFILE_NAME: ${{ vars.AZURE_CDN_PROFILE_NAME }}
AZURE_CDN_ENDPOINT_NAME: ${{ vars.AZURE_CDN_ENDPOINT_NAME }}
AZURE_STORYBOOK_ENDPOINT_NAME: ${{ vars.AZURE_STORYBOOK_ENDPOINT_NAME }}
AZURE_RESOURCE_GROUP: ${{ vars.AZURE_RESOURCE_GROUP }}
steps:
- name: Az CLI login 🔑 # Using federated credential
uses: azure/login@v1
Expand Down
7 changes: 4 additions & 3 deletions .github/workflows/publish_storybook.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,9 @@ on:
required: false
type: choice
options:
- development
- production
default: 'development'
- development
default: 'production'
push:
branches:
- main
Expand Down Expand Up @@ -177,9 +177,10 @@ jobs:

purge-cdn:
needs: [publish-storybook]
if: github.event.inputs.environment == 'production'
Comment thread
pomfrida marked this conversation as resolved.
name: Purge cdn
uses: ./.github/workflows/_purge_cdn.yaml
secrets: inherit
with:
environment: ${{ github.event.inputs.environment || 'development' }}
environment: production
endpoint: storybook
3 changes: 2 additions & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ While the design system itself lives in [Figma][], this repository contains impl

## Quick links

- [Storybook](https://storybook.eds.equinor.com/)
- [Storybook (prod)](https://storybook.eds.equinor.com/)
- [Storybook (dev)](https://s478stedsstorybookdev.z16.web.core.windows.net/) – latest from `main`
- [Figma](https://www.figma.com/files/682286909510540417/team/590517879490131675/EDS---Equinor-Design-System?fuid=677437722215124736)
- [Storefront](https://eds.equinor.com)
- [Slack](https://equinor.slack.com/archives/CJT20H1B9)
Expand Down
Loading