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

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

Merged
merged 1 commit into from
Apr 5, 2022

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