CORE-7094 implement file handling - Githubissues

redpanda-data / redpanda

Redpanda is a streaming data platform for developers. Kafka API compatible. 10x faster. No ZooKeeper. No JVM!

https://redpanda.com

9.65k stars 589 forks source link

CORE-7094 implement file handling #23475

Closed michael-redpanda closed 1 month ago

michael-redpanda commented 1 month ago

Implements handling of files for debug bundle service

Backports Required

[X] none - not a bug fix
[ ] none - this is a backport
[ ] none - issue does not exist in previous branches
[ ] none - papercut/not impactful enough to backport
[ ] v24.2.x
[ ] v24.1.x
[ ] v23.3.x

Release Notes

None

vbotbuildovich commented 1 month ago

ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/55082#01922764-56af-450a-a02a-e87821168cd1

ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/55113#019229f0-1a0b-454c-973d-d751a8f7ccc0

ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/55113#01922aa0-eef9-410c-9481-e5826f085b17

ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/55113#01922aa0-eef5-4b5a-9cf1-e5cb75d47790

ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/55113#01922a87-98d5-4c67-b428-951ec8421d41

ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/55141#01922b2e-d7de-44ed-9e5c-71a1008b08f1

michael-redpanda commented 1 month ago

Do the tests need some cleanup of the bundle file?

I avoided cleaning up the test directories in case I wanted to inspect them post run. Please the 'files' are small (so far just contains the command line arguments). But I can do it in TearDown if you want

michael-redpanda commented 1 month ago

Force push 09cc8d3:

Made test error checking consistent
Addressed PR comments

michael-redpanda commented 1 month ago

Force push ad419e1:

Cleaned up from PR comments

michael-redpanda commented 1 month ago

Force push 9bfd199:

Added mutex locks for get file and delete file