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

task/WG-196: add react-based hazmapper service #171

Merged
merged 6 commits into from
Jan 18, 2024

Conversation

nathanfranklin
Copy link
Collaborator

@nathanfranklin nathanfranklin commented Jan 9, 2024

Overview:

This PR

In TACC-Cloud/hazmapper#189, a new image for react-based hazmapper is added. This serves that service alongside hazmapper and taggit. Our VM hazmapper.tacc.utexas.edu has been updated to have:

  • dev-react point to dev.geoapi-services:82
  • staging-react point to staging.geoapi-services:82
  • hazmapper-react point to prod.geoapi-services:82

Related Jira tickets:

Summary of Changes:

Testing Steps:

  1. Check that https://hazmapper.tacc.utexas.edu/dev-react is working as expected.
  2. To see watchtower working you can see the logs of that container on portal@dev.geoapi-services.tacc.utexas.edu. You can see it checking every 300s and also it updating containers if it finds something new.

@nathanfranklin nathanfranklin marked this pull request as draft January 9, 2024 20:28
@nathanfranklin nathanfranklin marked this pull request as ready for review January 11, 2024 16:19
Copy link
Contributor

@sophia-massie sophia-massie left a comment

Choose a reason for hiding this comment

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

In step 1 of the testing instructions, is it supposed to be hazmapper.tacc.utexas.edu/dev-react rather than /react-dev? I'm getting a 404 on react-dev but the hazmapper PR directions worked for me.

@nathanfranklin
Copy link
Collaborator Author

In step 1 of the testing instructions, is it supposed to be hazmapper.tacc.utexas.edu/dev-react rather than /react-dev? I'm getting a 404 on react-dev but the hazmapper PR directions worked for me.

yes, hazmapper.tacc.utexas.edu/dev-react 👍

@tjgrafft tjgrafft self-requested a review January 18, 2024 15:41
Copy link
Contributor

@tjgrafft tjgrafft left a comment

Choose a reason for hiding this comment

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

dev-react worked as expected for me. ssh'ed into dev.geoapi-services and checked watchtower logs. LGTM!

Copy link
Contributor

@taoteg taoteg left a comment

Choose a reason for hiding this comment

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

LGTM!

@nathanfranklin nathanfranklin merged commit 5d5a593 into master Jan 18, 2024
3 checks passed
@nathanfranklin nathanfranklin deleted the task/WG-196-handle-various-environments branch January 18, 2024 19:38
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.

None yet

4 participants