-
-
Notifications
You must be signed in to change notification settings - Fork 9
fix: Disable create-reporting-task for NiFi 2.x and document regression #708
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
Conversation
|
Currently testing this PR on openshift and |
sbernauer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM when tests pass
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>
Good catch, the upgrade test probably also needs the same if clause |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only small doc nits, LGTM otherwise (I can't approve)
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>
maltesander
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
|
Please include a link to the docs |
Description
Please add a description here. This will become the commit message of the merge request later.
Definition of Done Checklist