Skip to content

Conversation

@mnocon
Copy link
Contributor

@mnocon mnocon commented Dec 15, 2025

@mnocon mnocon force-pushed the ibea-cloud branch 2 times, most recently from df66a49 to f4bdd53 Compare December 16, 2025 12:23
@mnocon mnocon marked this pull request as ready for review December 16, 2025 12:34
@mnocon mnocon requested review from barw4 and micszo December 16, 2025 12:35

- `database` - main application database
- `dfs_database` - DFS database (required name for DFS functionality)
- `rediscache` - Redis for cache
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if the strict naming requirement makes sense now - Redis, Elasticsearch, Solr and databases don't have to be named in any specific way. The only required ones are dfs_database and redissession.

Developers can have multirepo setup with many services, and environmental variables are exposed accordingly. So maybe we shouldn't enforce the setup with word "use" - wdyt?

!!! note "[[= product_name_cloud =]]"

For [[= product_name_cloud =]]/Platform.sh: This is automatically configured in `vendor/ibexa/core/src/bundle/Core/DependencyInjection/IbexaCoreExtension.php` if you have enabled Redis as `rediscache` Platform.sh service.
For [[= product_name_cloud =]]/Platform.sh, it's configured based on the `.platform.app.yaml` file by the [`ibexa/cloud` package](install_on_ibexa_cloud.md).
Copy link
Contributor

@barw4 barw4 Dec 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we migrate naming to "Upsun"?

dsn: '%solr_dsn%'
core: '%solr_main_core%'
dsn: '%env(string:SOLR_DSN)%'
core: '%env(string:SOLR_MAIN_CORE)%'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think *_CORE are not exposed, it's rather *_HOST now, something to be verified

@mnocon
Copy link
Contributor Author

mnocon commented Jan 21, 2026

Thanky you @barw4 , I've added improvements in f8b7e1e

Two questions that came to my mind:

  • Should we still mention Memcached? It's not supported in v5
  • Should we somehow mention Valkey here? We probably should, but is it correctly detected etc.?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants