Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix 2 issues related to DataSource and remove PostGIS comment #60

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

wmuldergov
Copy link

To be able to restore from the BC Gov S3 service we need to set repo2-s3-uri-style to 'path'. This adds that to the PostgresCluster.yaml and values.yaml file. It also corrects the source for 'stanza'

I also removed the comment around setting the CrunchyImage variable when enabling POSTGIS as that is not required and will cause issues.

wmuldergov and others added 3 commits June 19, 2024 15:48
To restore from a OCIO s3 bucket, this must be set to path otherwise it will use host by default.
@wmuldergov
Copy link
Author

@caggles are you able to take a look at this PR?

@mishraomp
Copy link
Collaborator

@wmuldergov , I deployed a crunchy chat into one of my namespace without giving the image of postgis and postgis version in the chart. It does not enable postgis. I would be curious how you enabled postgis :).

without explicit instructions in helm chart.
image

with explicit instructions in helm chart.
image

@wmuldergov
Copy link
Author

Hello @mishraomp
I set it to:

postgresVersion: 16
postGISVersion: '3.4'

and it worked as expected in Gold DR:

postgres=# SELECT PostGIS_Version();
            postgis_version            
---------------------------------------
 3.4 USE_GEOS=1 USE_PROJ=1 USE_STATS=1
(1 row)

It looks like it pulled:
Image
artifacts.developer.gov.bc.ca/bcgov-docker-local/crunchy-postgres-gis
Image version/tag
ubi8-16.2-3.4-0

@mishraomp
Copy link
Collaborator

Hello @mishraomp I set it to:

postgresVersion: 16
postGISVersion: '3.4'

and it worked as expected in Gold DR:

postgres=# SELECT PostGIS_Version();
            postgis_version            
---------------------------------------
 3.4 USE_GEOS=1 USE_PROJ=1 USE_STATS=1
(1 row)

It looks like it pulled: Image artifacts.developer.gov.bc.ca/bcgov-docker-local/crunchy-postgres-gis Image version/tag ubi8-16.2-3.4-0

Ahh I see, nice, so may be thats the way to go, we just need to mention that, if postgis is needed, mention the postgis version explicitly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants