Skip to content
This repository has been archived by the owner on Feb 4, 2025. It is now read-only.

Fix variable wrongly used #27

Merged
merged 2 commits into from
Feb 15, 2024
Merged

Conversation

juandspy
Copy link
Collaborator

Description

I found out this variable is never substituted because it doesn't use any brackets:

$ oc describe configmap dvo-extractor-config-map | grep "KAFKA_BOOTSTRAP_URL"
        bootstrap.servers: $KAFKA_BOOTSTRAP_URL

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Testing steps

Ephemeral.

Checklist

  • make before_commit passes
  • updated documentation wherever necessary
  • added or modified tests if necessary
  • updated schemas and validators in insights-data-schemas in case of input/output change

Sorry, something went wrong.

@juandspy juandspy merged commit 0d5c38e into RedHatInsights:master Feb 15, 2024
8 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants