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

DOCS: Updated docker instructions, removed obsolete variables #2217

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

waldbauer-certat
Copy link
Contributor

Fixes #2195

-e INTELMQ_SOURCE_PIPELINE_HOST: redis \
-e INTELMQ_DESTINATION_PIPELINE_HOST: redis \
-e INTELMQ_REDIS_CACHE_HOST: redis \
sudo docker run -e INTELMQ_PIPELINE_HOST: redis \
Copy link
Member

@sebix sebix Jul 15, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The environment variable INTELMQ_PIPELINE_HOST is only used in the tests, but not in the normal pipeline/bot code. Should be INTELMQ_SOURCE_PIPELINE_HOST and INTELMQ_DESTINATION_PIPELINE host IMO.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To clarify what I meant: Setting INTELMQ_PIPELINE_HOST does not set the pipeline host to redis for the bots at runtime and is thus ineffective.

@sebix sebix added bug Indicates an unexpected problem or unintended behavior documentation Indicates a need for improvements or additions to documentation labels Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior documentation Indicates a need for improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Installation documentation: obsolete docker environment variables?
2 participants