Skip to content

K8SPG-857 Fixed image for PostGIS doc#335

Merged
nastena1606 merged 4 commits into2.0from
K8SPG-857-Fix-Postgis-doc
Nov 13, 2025
Merged

K8SPG-857 Fixed image for PostGIS doc#335
nastena1606 merged 4 commits into2.0from
K8SPG-857-Fix-Postgis-doc

Conversation

@nastena1606
Copy link
Collaborator

modified: docs/postgis.md
modified: variables.yml

modified:   docs/postgis.md
	modified:   variables.yml
Copy link

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 fixes the PostGIS image reference in the documentation by adding the PostGIS version tag to the image name.

Key Changes:

  • Updated the PostGIS image name format to include the PostGIS version variable
  • Added a duplicate postgisrecommended variable definition in the variables file

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docs/postgis.md Updated PostGIS image reference to include version tag using the postgisrecommended variable
variables.yml Added duplicate definition of postgisrecommended variable (appears to be unintentional)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
docs/postgis.md Outdated
spec:
...
image: percona/percona-postgresql-operator:{{ release }}-ppg{{ postgresrecommended }}-postgres-gis
image: docker.io/percona/percona-postgresql-operator:{{ release }}-ppg{{ postgresrecommended }}-postgres-gis
Copy link
Contributor

Choose a reason for hiding this comment

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

add s{{postgisrecommended}} here also

@it-percona it-percona temporarily deployed to K8SPG-857-Fix-Postgis-doc - percona-postgresql-operator-2.0 PR #335 November 8, 2025 18:26 — with Render Destroyed
@nastena1606 nastena1606 requested a review from eleo007 November 8, 2025 18:26
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.

4 participants