Skip to content

pass in pubsub filestream ordered subscriber through env variable to filestream service #529

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.

Sign up for GitHub

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

Jump to bottom
Open
estellazx wants to merge 3 commits into
base: main
Choose a base branch
from
Open

pass in pubsub filestream ordered subscriber through env variable to filestream service #529

estellazx wants to merge 3 commits into from

Conversation

@estellazx
Copy link
Contributor

@estellazx estellazx commented Nov 19, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Chores
    • Added a new environment variable for the filestream component to support sourcing runtime configuration from the run update settings.
    • Bumped the chart version (release metadata updated) with no functional behavior changes.

✏️ Tip: You can customize this high-level summary in your review settings.

@estellazx estellazx requested a review from a team as a code owner November 19, 2025 22:32
@estellazx estellazx changed the title pass in pubsub filestream ordered subscriber through env variable pass in pubsub filestream ordered subscriber through env variable to filestream service Jan 14, 2026
@estellazx estellazx requested a review from dacbd January 14, 2026 17:01
@estellazx
Copy link
Contributor Author

estellazx commented Jan 14, 2026

@dacbd I ended up reopening this PR as I tested things on FMB test and i need this PR so that filestream knows where to find the address of the new subscriber via env variable
Screenshot 2026-01-14 at 12 04 32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dacbd dacbd Awaiting requested review from dacbd

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@estellazx @dacbd