api interop layer deploy #1825
Merged
api interop layer deploy #1825
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
eric-gade
approved these changes
Sep 26, 2024
eric-gade
force-pushed
the
api-interop-layer
branch
from
61bdf9e
to
9a65274
Compare
jamestranovich-noaa
force-pushed
the
jt/api-interop-layer-deploy
branch
from
00bc002
to
1fd69f7
Compare
jamestranovich-noaa
force-pushed
the
api-interop-layer
branch
from
b686bc3
to
4c0a9bf
Compare
jamestranovich-noaa
force-pushed
the
jt/api-interop-layer-deploy
branch
from
a0c190e
to
30d9136
Compare
|
@greg-does-weather @eric-gade i've made some changes to this PR, mostly making sure all of the env variables are properly propagated. please take another look 👀 |
Yes. cloud.gov apps don't need that much memory because (ostensibly) it's running only that one app. The good thing is that NR will track app utilization for us and we can set warnings if we are running out of memory or disk or whatever. It looks like we currently have 5G across all environments/instances. (To check the quota, you can get the quota id from |
|
Note that this also closes the specific issue outlined in #1786 |
1 task
greg-does-weather
approved these changes
Oct 1, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do? 🛠️
Addresses #1791 -- except, in james, and not sandbox, since I don't think we're quite set up on that front (yet)
What does the reviewer need to know? 🤔
This only works in
jamescurrently. Visit https://api-weathergov-james.app.cloud.gov/ (by default the app will return{ ok: true }for/)Also, https://weathergov-james.app.cloud.gov/ uses the interop layer by default now. Feel free to poke around.
Also, NR logs are enabled: