Skip to content

Conversation

@stuzart
Copy link
Member

@stuzart stuzart commented Oct 20, 2025

Environment variables to define the public and private graphs used with virtuoso when running within a docker container, including disabling the private graph. Also updated the example docker-compose for virtuoso to include these variables, and also added them to the standard seek container as they are needed for the sparql view. Since duplicated, moved these variables to a virtuoso.env file.

@stuzart stuzart added this to the 1.17.2 milestone Oct 20, 2025
@stuzart stuzart moved this to In review in SEEK 1.17.x Oct 20, 2025
@stuzart stuzart requested a review from Copilot October 20, 2025 12:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces more flexible configuration for RDF graphs in Docker deployments by adding environment variables to control public and private graph URIs for Virtuoso. It consolidates virtuoso-related environment variables into a dedicated configuration file and updates the docker-compose setup to use these new variables.

  • Replaced hardcoded RDF graph URIs with configurable environment variables
  • Created a centralized virtuoso.env file to manage Virtuoso-specific configuration
  • Updated docker-compose services to use the new environment file structure

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
docker/virtuoso_settings.docker.yml Replaced hardcoded graph URIs with environment variables
docker/virtuoso.env New environment file containing Virtuoso configuration variables
docker-compose-virtuoso.yml Updated services to use virtuoso.env file and pinned Virtuoso version

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@stuzart stuzart requested a review from kdp-cloud October 20, 2025 13:13
@stuzart stuzart merged commit 8e1d415 into seek-1.17 Oct 22, 2025
19 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in SEEK 1.17.x Oct 22, 2025
@stuzart stuzart deleted the docker-virtuoso-graphs branch October 22, 2025 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants