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

Support to add fieldRef fieldPath for environment variable for stream apps #5849

Open
dineshkumarsarangapani opened this issue Jun 26, 2024 · 2 comments
Milestone

Comments

@dineshkumarsarangapani
Copy link

Problem description:
The current system does not support passing the status.hostIP as an environment variable to the pods in stream apps. This limitation restricts our ability to configure and manage environment variables dynamically based on the host IP, which is critical for certain operational scenarios and integrations.

Solution description:
The solution involves adding support for fieldRef and fieldPath for environment variables specifically for stream applications. This would allow us to set the status.hostIP as an environment variable within the pod specifications. In edge cases where the status.hostIP is not available or accessible, the system should log an appropriate error message and possibly fall back to a default IP or handle the absence gracefully without crashing the application.

Description of alternatives:
Alternatives considered include:

Manually injecting the status.hostIP via init containers or sidecar containers, which adds complexity and overhead.
Using a custom script to fetch and set the status.hostIP as an environment variable at runtime, which is less efficient and prone to errors.
Configuring the status.hostIP via a ConfigMap or Secret, which is not dynamic and requires frequent updates and maintenance.

@github-actions github-actions bot added the status/need-triage Team needs to triage and take a first look label Jun 26, 2024
@cppwfs cppwfs added for/team-attention For team attention and removed status/need-triage Team needs to triage and take a first look labels Jul 1, 2024
@cppwfs cppwfs added this to the 3.0.x milestone Jul 3, 2024
@cppwfs cppwfs removed the for/team-attention For team attention label Jul 3, 2024
@dineshkumarsarangapani
Copy link
Author

dineshkumarsarangapani commented Jul 18, 2024

@cppwfs Let me know if I can contribute to this issue.

@dineshkumarsarangapani
Copy link
Author

I have created this issue in the spring cloud deployer repo as well; I think the dependency needs to be fixed first.

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

No branches or pull requests

2 participants