Skip to content

subset.ipynb: allow to test GeoServer on new production server#232

Merged
tlvu merged 1 commit intomasterfrom
allow-to-test-geoserver-on-new-production-server
Apr 5, 2022
Merged

subset.ipynb: allow to test GeoServer on new production server#232
tlvu merged 1 commit intomasterfrom
allow-to-test-geoserver-on-new-production-server

Conversation

@tlvu
Copy link
Collaborator

@tlvu tlvu commented Apr 4, 2022

Switch from boreas.ouranos.ca to pavics.ouranos.ca will allow us to
conditionally regex replace pavics.ouranos.ca to the server under test
so we can actually test the Geoserver on that new server.

Everywhere under this finch repo, we use pavics.ouranos.ca. This
is the only locations that hardcode boreas.ouranos.ca because
geoserver data are not replicated to standard test servers so hardcoding
it this way will use the data from the production server.

This patch achieve the same "data from prod" but also allow override if
We need to actually test the future production server.

Related to Ouranosinc/pavics-sdi#183 and Ouranosinc/PAVICS-e2e-workflow-tests#104.

Switch from 'boreas.ouranos.ca' to 'pavics.ouranos.ca' will allow us to
conditionally regex replace 'pavics.ouranos.ca' to the server under test
so we can actually test the Geoserver on that new server.

Everywhere under this pavics-sdi repo, we use 'pavics.ouranos.ca'. This
is the only locations that hardcode 'boreas.ouranos.ca' because
geoserver data are not replicated to standard test servers so hardcoding
it this way will use the data from the production server.

This patch achieve the same "data from prod" but also allow override if
We need to actually test the future production server.

Related to Ouranosinc/pavics-sdi#183
@tlvu tlvu requested review from huard and tlogan2000 April 4, 2022 19:52
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@tlvu tlvu merged commit 56ab163 into master Apr 5, 2022
@tlvu tlvu deleted the allow-to-test-geoserver-on-new-production-server branch April 5, 2022 15:35
tlvu added a commit to Ouranosinc/PAVICS-e2e-workflow-tests that referenced this pull request Apr 7, 2022
…from-test-server

test framework: allow testing using all test data from system under test

Set `TEST_NO_USE_PROD_DATA` to use all test data from system under test.

Useful to test future production server that will have all the huge datas.

Fixes #103.

Relate to bird-house/finch#232 and Ouranosinc/pavics-sdi#254.
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.

3 participants