Merge branch 'main' into patch-1

Author: Sarah Edwards

.github/actions-scripts/purge-fastly-edge-cache-per-language.js

@@ -5,6 +5,25 @@ import { languageKeys } from '../../lib/languages.js'
 import { makeLanguageSurrogateKey } from '../../middleware/set-fastly-surrogate-key.js'
 import purgeEdgeCache from '../../script/deployment/purge-edge-cache.js'
 
+/**
+ * In simple terms, this script sends purge commands for...
+ *
+ * 1. no-language
+ * 2. 'en'
+ * 3. 'ja'
+ * 4. 'pt'
+ * ...
+ *
+ * ...and so on for all languages.
+ * But to avoid a stampeding herd after each purge, and to avoid unnecessary
+ * load on the backend, put a slight delay between each language.
+ * This gives the backend a chance to finish processing all the now stale
+ * URLs, for that language, before tackling the next.
+ */
+const DELAY_BETWEEN_LANGUAGES = 10 * 1000
+
+const sleep = (ms) => new Promise((resolve) => setTimeout(resolve, ms))
+
 // This covers things like `/api/webhooks` which isn't language specific.
 await purgeEdgeCache(makeLanguageSurrogateKey())
 
@@ -16,6 +35,10 @@ const languages = process.env.LANGUAGES
     languageKeys.sort((a) => (a === 'en' ? -1 : 1))
 
 for (const language of languages) {
+  console.log(
+    `Sleeping ${DELAY_BETWEEN_LANGUAGES / 1000} seconds before purging for '${language}'...`
+  )
+  await sleep(DELAY_BETWEEN_LANGUAGES)
   await purgeEdgeCache(makeLanguageSurrogateKey(language))
 }
 

.github/workflows/browser-test.yml

@@ -8,6 +8,12 @@ name: Codeowners - Docs Engineering
 
 on:
   pull_request:
+    types:
+      - edited
+      - opened
+      - ready_for_review
+      - reopened
+      - synchronize
     paths:
       - '**.js'
       - '**.ts'
@@ -20,7 +26,7 @@ on:
 
 jobs:
   codeowners-docs-engineering:
-    if: ${{ github.repository == 'github/docs-internal' }}
+    if: ${{ github.repository == 'github/docs-internal' && !github.event.pull_request.draft }}
     runs-on: ubuntu-latest
     env:
       GH_TOKEN: ${{ secrets.DOCS_BOT_FR }}