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 #14586: Update relay component #2175
Fixes #14586: Update relay component #2175
Conversation
a313238
to
621b2f5
Compare
Commit modified |
logs: log_entries >> | ||
rudder_report_begin >> | ||
policy: take_until_and_consume_s!("@@") >> | ||
event_type: take_until_and_consume_s!("@@") >> | ||
rule_id: take_until_and_consume_s!("@@") >> | ||
directive_id: take_until_and_consume_s!("@@") >> | ||
serial: take_until_and_consume_s!("@@") >> | ||
serial: map_res!(take_until_and_consume_s!("@@"), parse_i32) >> |
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.
serial should always be 0
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.
meaning: there is no way the serial can be set to something else than 0 in Rudder 4.3 and more
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.
The idea is to allow reusing this field if we want it in the future, without having to upgrade all relays.
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.
(specially as it is cheap and just as simple to implement)
621b2f5
to
3027802
Compare
Commit modified |
3027802
to
c0c9ac3
Compare
Commit modified |
1 similar comment
Commit modified |
bf7e401
to
1271c36
Compare
Commit modified |
1271c36
to
30e70de
Compare
Commit modified |
1 similar comment
Commit modified |
aed0582
to
49babf5
Compare
Commit modified |
1 similar comment
Commit modified |
Commit modified |
976eafb
to
33fe570
Compare
Commit modified |
33fe570
to
465e764
Compare
PR rebased |
465e764
to
85daa29
Compare
Commit modified |
85daa29
to
c483f14
Compare
Commit modified |
c483f14
to
66d4f4b
Compare
Commit modified |
Fixes #14586: Update relay component
Commit modified |
1 similar comment
Commit modified |
130bc60
to
92177be
Compare
Commit modified |
Fixes #14586: Update relay component
Commit modified |
OK, squash merging this PR |
1 similar comment
OK, squash merging this PR |
Commit modified |
1 similar comment
Commit modified |
https://issues.rudder.io/issues/14586