Skip to content

Fix SUPASS environment variable in docker-compose-dl-streamer-example#596

Merged
saratpoluri merged 2 commits intorelease-2025.2from
fix/example-SUPASS-web-environment
Nov 20, 2025
Merged

Fix SUPASS environment variable in docker-compose-dl-streamer-example#596
saratpoluri merged 2 commits intorelease-2025.2from
fix/example-SUPASS-web-environment

Conversation

@rawatts10
Copy link
Copy Markdown
Contributor

Change from SUPASS to SUPASS=${SUPASS} to properly pass through the environment variable.

📝 Description

Adding additional environment variables after this line can be problematic since the SUPASS wasn't set to any value. This fix correctly sets the SUPASS to the host environment variable for SUPASS (if used).

✨ Type of Change

Select the type of change your PR introduces:

  • [ x ] 🐞 Bug fix – Non-breaking change which fixes an issue
  • 🚀 New feature – Non-breaking change which adds functionality
  • 🔨 Refactor – Non-breaking change which refactors the code base
  • 💥 Breaking change – Changes that break existing functionality
  • 📚 Documentation update
  • 🔒 Security update
  • 🧪 Tests
  • 🚂 CI

🧪 Testing Scenarios

Describe how the changes were tested and how reviewers can test them too:

  • [ x ] ✅ Tested manually
  • 🤖 Ran automated end-to-end tests

✅ Checklist

Before submitting the PR, ensure the following:

  • 🔍 PR title is clear and descriptive
  • 📝 For internal contributors: If applicable, include the JIRA ticket number (e.g., ITEP-123456) in the PR title. Do not include full URLs
  • 💬 I have commented my code, especially in hard-to-understand areas
  • 📄 I have made corresponding changes to the documentation
  • ✅ I have added tests that prove my fix is effective or my feature works

….yml

Change from SUPASS to SUPASS=${SUPASS} to properly pass through the environment variable.
@saratpoluri saratpoluri enabled auto-merge (squash) November 18, 2025 18:31
@saratpoluri saratpoluri merged commit f0669ad into release-2025.2 Nov 20, 2025
28 of 29 checks passed
@saratpoluri saratpoluri deleted the fix/example-SUPASS-web-environment branch November 20, 2025 01:24
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

Successfully merging this pull request may close these issues.

2 participants