Sync Upsun skill with latest configuration guidance (fba1ad70a503)#24
Draft
pjcdawkins wants to merge 1 commit intomainfrom
Draft
Sync Upsun skill with latest configuration guidance (fba1ad70a503)#24pjcdawkins wants to merge 1 commit intomainfrom
pjcdawkins wants to merge 1 commit intomainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR synchronizes the Upsun skill’s configuration reference documentation with the latest internal configuration guidance, regenerating per-language and per-framework templates and updating the chapter index.
Changes:
- Replaces the main
references/config.mdcontent with an index of per-language and per-framework guides. - Adds/regenerates reference configuration templates for common languages (Go, Node.js, PHP, Python, Ruby, static sites) and frameworks (e.g., Drupal, Symfony, Next.js, Nuxt, Strapi, WordPress).
- Updates version pointers and guidance notes to align with the canonical Upsun image registry.
Reviewed changes
Copilot reviewed 29 out of 29 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| plugins/upsun/skills/upsun/references/config.md | Reworked into an index page linking per-language/framework reference docs. |
| plugins/upsun/skills/upsun/references/config/directus.md | Adds Directus reference configuration template and operational notes. |
| plugins/upsun/skills/upsun/references/config/django.md | Adds Django reference configuration template (uv-based) and deployment notes. |
| plugins/upsun/skills/upsun/references/config/drupal.md | Adds Drupal reference configuration template including Drush + mounts + caching. |
| plugins/upsun/skills/upsun/references/config/echo.md | Adds Echo (Go) reference configuration template and DB options guidance. |
| plugins/upsun/skills/upsun/references/config/express.md | Adds Express reference configuration template for Node.js apps. |
| plugins/upsun/skills/upsun/references/config/flask.md | Adds Flask reference configuration template (uv + gunicorn) and options. |
| plugins/upsun/skills/upsun/references/config/gatsby.md | Adds Gatsby reference configuration template and cache mount guidance. |
| plugins/upsun/skills/upsun/references/config/gin.md | Adds Gin reference guidance pointing to Go baseline patterns. |
| plugins/upsun/skills/upsun/references/config/go.md | Adds Go baseline reference template plus DB and worker patterns. |
| plugins/upsun/skills/upsun/references/config/hugo.md | Adds Hugo static-site reference configs (Nix/composable + pinned binary). |
| plugins/upsun/skills/upsun/references/config/jekyll.md | Adds Jekyll static-site reference config (composable image). |
| plugins/upsun/skills/upsun/references/config/js.md | Adds Node.js baseline reference config template and package manager notes. |
| plugins/upsun/skills/upsun/references/config/laravel.md | Adds Laravel reference config template (migrations, mounts, env setup). |
| plugins/upsun/skills/upsun/references/config/nextjs.md | Adds Next.js reference template including writable .next handling. |
| plugins/upsun/skills/upsun/references/config/nuxt.md | Adds Nuxt reference template including writable .nuxt handling. |
| plugins/upsun/skills/upsun/references/config/php.md | Adds PHP baseline guidance for build/runtime/extension configuration. |
| plugins/upsun/skills/upsun/references/config/python.md | Adds Python baseline guidance for uv/poetry/pipenv/pip. |
| plugins/upsun/skills/upsun/references/config/rails.md | Adds Rails reference template including DB and worker notes. |
| plugins/upsun/skills/upsun/references/config/reactjs.md | Adds high-level React guidance pointing to Node.js baseline patterns. |
| plugins/upsun/skills/upsun/references/config/ruby.md | Adds Ruby baseline guidance for JS deps, DB URL, workers, mounts. |
| plugins/upsun/skills/upsun/references/config/sinatra.md | Adds Sinatra reference template and DB option notes. |
| plugins/upsun/skills/upsun/references/config/static.md | Adds static site guidance including composable image recommendation. |
| plugins/upsun/skills/upsun/references/config/strapi.md | Adds Strapi reference template including DB service and secrets guidance. |
| plugins/upsun/skills/upsun/references/config/sylius.md | Adds Sylius reference template including frontend build + mounts + cron guidance. |
| plugins/upsun/skills/upsun/references/config/symfony.md | Adds Symfony reference template including database and messenger worker notes. |
| plugins/upsun/skills/upsun/references/config/vite.md | Adds Vite static-site reference template and caching guidance. |
| plugins/upsun/skills/upsun/references/config/vuejs.md | Adds Vue.js static-site reference template and caching guidance. |
| plugins/upsun/skills/upsun/references/config/wordpress.md | Adds WordPress reference template (Bedrock + adjustments) and caching notes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+46
to
+48
| export PROJECT_NAME='Directus on Upsun' | ||
| export ADMIN_EMAIL='admin@example.com' | ||
| export ADMIN_PASSWORD='password' |
|
|
||
| **Configuration Details**: | ||
|
|
||
| - **Database Service**: PostgreSQL 16 service. Service named "db" exposes standard DB_ environment variables. |
Comment on lines
+59
to
+66
| - **Database Client**: Set automatically based on relationship service type | ||
| - **Admin Panel**: Accessible through Strapi's built-in interface | ||
| - **API Access**: All routes pass through to Strapi application | ||
|
|
||
| **Database Setup**: | ||
| - Strapi automatically configures database connection from `DATBASE_` variables Upsun provides for a relationship named `database` | ||
| - No manual environment variable configuration required | ||
| - Database migrations run during application startup |
Comment on lines
+33
to
+35
| commands: | ||
| start: bundle exec rails server | ||
|
|
| expires: 1w | ||
|
|
||
| mounts: | ||
| web/app/wp-content/cache: |
| - **API Access**: All routes pass through to Strapi application | ||
|
|
||
| **Database Setup**: | ||
| - Strapi automatically configures database connection from `DATBASE_` variables Upsun provides for a relationship named `database` |
Comment on lines
+38
to
+39
| ^/(api|health): | ||
| passthru: true |
Comment on lines
+35
to
+39
| echo >> "$settings_dir"/settings.py "\n# Upsun configuration" | ||
| echo >> "$settings_dir"/settings.py 'ALLOWED_HOSTS = ["*"]' | ||
| if ! grep -q STATIC_ROOT "$settings_dir"/settings.py; then | ||
| echo >> "$settings_dir"/settings.py 'STATIC_ROOT = BASE_DIR / "static"' | ||
| echo >> "$settings_dir"/settings.py 'STATIC_URL = "/static/"' |
|
|
||
| # Set up environment variables for use at runtime. | ||
| echo >> .environment 'export DATABASE_URL="${DB_SCHEME}://${DB_USERNAME}:${DB_PASSWORD}@${DB_HOST}:${DB_PORT}/${DB_PATH}"' | ||
| echo >> .environment 'export WP_HOME=$(echo $PLATFORM_ROUTES | base64 --decode | jq -r "to_entries[] | select(.value.primary == true) | .key")' |
Comment on lines
+29
to
+38
| CACHE_ENABLED=true | ||
| CACHE_STORE=redis | ||
| REDIS_HOST=\$REDIS_HOST | ||
| REDIS_PORT=\$REDIS_PORT | ||
| RATE_LIMITER_ENABLED=true | ||
| RATE_LIMITER_STORE=redis | ||
| RATE_LIMITER_REDIS_HOST=\$REDIS_HOST | ||
| RATE_LIMITER_REDIS_PORT=\$REDIS_PORT | ||
| KEY=\$PLATFORM_PROJECT_ENTROPY | ||
| SECRET=\$PLATFORM_PROJECT_ENTROPY |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated sync of the Upsun skill with configuration guidance in the
internal
platformsh/airepository.Signature:
fba1ad70a503Files updated (regenerated deterministically from the Jinja guides):
plugins/upsun/skills/upsun/references/config.mdplugins/upsun/skills/upsun/references/config/directus.mdplugins/upsun/skills/upsun/references/config/django.mdplugins/upsun/skills/upsun/references/config/drupal.mdplugins/upsun/skills/upsun/references/config/echo.mdplugins/upsun/skills/upsun/references/config/express.mdplugins/upsun/skills/upsun/references/config/flask.mdplugins/upsun/skills/upsun/references/config/gatsby.mdplugins/upsun/skills/upsun/references/config/gin.mdplugins/upsun/skills/upsun/references/config/go.mdplugins/upsun/skills/upsun/references/config/hugo.mdplugins/upsun/skills/upsun/references/config/jekyll.mdplugins/upsun/skills/upsun/references/config/js.mdplugins/upsun/skills/upsun/references/config/laravel.mdplugins/upsun/skills/upsun/references/config/nextjs.mdplugins/upsun/skills/upsun/references/config/nuxt.mdplugins/upsun/skills/upsun/references/config/php.mdplugins/upsun/skills/upsun/references/config/python.mdplugins/upsun/skills/upsun/references/config/rails.mdplugins/upsun/skills/upsun/references/config/reactjs.mdplugins/upsun/skills/upsun/references/config/ruby.mdplugins/upsun/skills/upsun/references/config/sinatra.mdplugins/upsun/skills/upsun/references/config/static.mdplugins/upsun/skills/upsun/references/config/strapi.mdplugins/upsun/skills/upsun/references/config/sylius.mdplugins/upsun/skills/upsun/references/config/symfony.mdplugins/upsun/skills/upsun/references/config/vite.mdplugins/upsun/skills/upsun/references/config/vuejs.mdplugins/upsun/skills/upsun/references/config/wordpress.mdSource locations in
platformsh/ai:guides/per-group/<lang>.md.jinja— per-language references.guides/per-result/<framework>.md.jinja— per-framework references.references/config.md) is built from the file list.Rendering reuses
internal/registryinplatformsh/ai, so versions herematch what the configuration-generation API recommends.
If these changes aren't wanted, close the PR; a subsequent run will open a
fresh PR on a new branch (the closed PR stays closed). A new PR is opened
automatically when the source guidance changes.