Change container metadata to use CONTAINER_APP_REPLICA_NAME #2099
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.
Replace environment variable HOSTNAME with CONTAINER_APP_REPLICA_NAME.
Description
In new ACA Environment with workload profiles, using the Consumption profile, HOSTNAME does not appear to exist as an environment variable anymore - thus the application cannot start. However, the env variable CONTAINER_APP_REPLICA_NAME exists in both classic ACA Environment (Consumption) and in new workload profiles (Dedicated + Consumption). So this change should be backwards compatible as well.
Also, it makes more sense in this context and uses the same standard as the other Container App metadata env variables that are fetched.
Purpose
This pull request is a:
Checklist