Expose dva-processing in test-env - #71
Merged
bzp99 merged 1 commit intoJul 29, 2026
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the local test docker-compose environment to publish the dva-processing provider/consumer services on dedicated host ports so companion orchestrator flows can reach them directly.
Changes:
- Expose
dva-processing-provideron host port 5007 (container port 5000). - Expose
dva-processing-consumeron host port 5008 (container port 5000).
Comments suppressed due to low confidence (1)
test-env/compose.yml:80
- The PR description says these services should bind on localhost, but this port mapping publishes on all interfaces by default. Prefix the mapping with 127.0.0.1 to restrict exposure to the local machine (or adjust the PR description if external exposure is intended).
ports: [5008:5000]
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| interval: 5s | ||
| timeout: 5s | ||
| retries: 3 | ||
| ports: [5007:5000] |
| retries: 3 | ||
| ports: [5007:5000] | ||
| depends_on: | ||
| rabbit-consumer: {condition: service_healthy} |
bzp99
self-requested a review
July 29, 2026 20:16
bzp99
approved these changes
Jul 29, 2026
bzp99
left a comment
bzp99
left a comment
Collaborator
There was a problem hiding this comment.
Sure, if I undestand correctly this just exposes the ports in the Compose network in preparation of giving dva-processing an (internal) REST API.
The commit message is of the wrong category though but I'll amend that.
bzp99
force-pushed
the
build/test-env
branch
2 times, most recently
from
July 29, 2026 20:27
75eb856 to
372986f
Compare
Expose provider- and consumer-side processing on ports 5007 and 5008 respectively.
bzp99
force-pushed
the
build/test-env
branch
2 times, most recently
from
July 29, 2026 20:29
cdc670e to
e303687
Compare
bzp99
merged commit Jul 29, 2026
e303687
into
Prometheus-X-association:yassine-refactor
1 check passed
bzp99
changed the title
This file contains hidden or 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
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.
Adjusts
test-env/compose.ymlso the dva-processing provider and consumer services bind to host ports 5007 and 5008 respectively, matching the endpoints expected by the slimmed-down dva-api orchestrator in companion PRs.Depends on: none
@bzp99