-
Notifications
You must be signed in to change notification settings - Fork 593
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
application: print command line #22826
application: print command line #22826
Conversation
2d532e9
to
b3a980e
Compare
new failures in https://buildkite.com/redpanda/redpanda/builds/52732#01913948-b5c3-4ca7-b1c4-4e88b72b638b:
|
ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/52732#0191394a-b602-4a29-a271-15d4462e41b9 ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/52732#0191394a-b5fe-4700-b05f-9921cd3d5fa8 ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/52852#01914c9b-a840-462b-8fba-41bf658691da ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/52869#01914da6-2d8a-4382-b5cb-43665c19724b |
/ci-repeat 1 |
Somehow we've gotten away without printing the command line all this time. In some contexts it is printed out by some other component but we should print it out ourselves in Redpanda too. Ref CORE-6746.
327a8cf
to
119312b
Compare
Applied suggestion to use |
apparently for the bazel build you need to add
|
Weird, because |
It's because of https://redpandadata.atlassian.net/wiki/spaces/CORE/pages/714670307/Bazel#Why-does-Bazel-complain-about-so-many-missing-dependencies%3F I think |
Effectively, it always was a dependency, but now that we've added the header we need to make it explicit, because Bazel.
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.
🔥
Somehow we've gotten away without printing the command line all this time. In some contexts it is printed out by some other component but we should print it out ourselves in Redpanda too.
Ref CORE-6746.
Backports Required
Release Notes
Improvements