Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

corrected typo in the nifi configmap for bootstrap-notification-servi… #134

Merged
merged 1 commit into from
Oct 12, 2021
Merged

Conversation

mh013370
Copy link
Contributor

@mh013370 mh013370 commented Oct 7, 2021

…ces.xml

Q A
Bug fix? yes
New feature? no
API breaks? no, this configmap key is never looked up by its value
Deprecations? no
Related tickets fixes #127
License Apache 2.0

What's in this PR?

Simple one-liner to correct the name of the bootstrap-notification-services.xml file that gets laid down into the running nifi pod so nifi correctly picks it up.

Why?

NiFi isn't currently reading this file since it doesn't match the file name in the related configuration property.

Additional context

Checklist

  • Implementation tested
  • Error handling code meets the guideline
  • Logging code meets the guideline
  • User guide and development docs updated (if needed)
  • Append changelog with changes

@erdrix erdrix added bug Something isn't working community labels Oct 12, 2021
@erdrix erdrix self-assigned this Oct 12, 2021
@erdrix erdrix merged commit 17bed7f into Orange-OpenSource:master Oct 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nifikop creates conf/bootstrap-notification-servces instead of conf/bootstrap-notification-services
2 participants