-
Notifications
You must be signed in to change notification settings - Fork 38
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
Fixes #7436: Techniques that call _logger, _logger_default or logger_rudder will break because these methods take 4 arguments now and no upgrade path has been planned #269
Conversation
…ck we never use logger* again
…rudder will break because these methods take 4 arguments now and no upgrade path has been planned
@@ -31,9 +31,13 @@ | |||
|
|||
bundle agent _logger(message, old_class_prefix, origin_class_prefix, args) |
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.
shouldn't you change the signature of the method ?
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.
Oops. Well spotted, thanks. Fixed.
8ef9127
to
4e3ee64
Compare
PR updated |
4e3ee64
to
4f5f0ea
Compare
# @parameter origin_class_prefix The prefix of the class for different states (1.x version) | ||
# @parameter args The arguments used to call the generic method (slist) | ||
# | ||
# @class_prefix _logger_default |
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.
why the class prefix ? i'm pretty sure there's none of them
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.
why the class prefix ? i'm pretty sure there's none of them
All bundles must have a class_prefix. It's our internal style tests' policy. If we don't have one, tests will fail.
Also, there has always been one in this file, I just copied it without changing this line (see the individual commits, rather than the general overview)
4f5f0ea
to
d84d81b
Compare
d84d81b
to
984f42d
Compare
…logger_logger_default_or_logger_rudder_will_break_because_these_methods_take_4_arguments_now_and_no_upgrade_path_has_been_planned_2 Fixes #7436: Techniques that call _logger, _logger_default or logger_rudder will break because these methods take 4 arguments now and no upgrade path has been planned
https://www.rudder-project.org/redmine/issues/7436