kata-irc-bot | <derlee> Regarding why cargo-deny is taking so long on kata-containers vs dragonflyoss/image-service: First, I did go a roundabout way of getting all the Cargo.toml files scanned. Normally when cargo-deny runs, it works by scanning either the root directory or a provided directory for a cargo.toml file. However, the kata project does not have a single "root" cargo file. Instead, it has a bunch of components, each with their own files, each file | 16:31 |
---|---|---|
kata-irc-bot | potential having its own workspace. I made a composite action to find all the Cargo.toml files (that were modified) and then run the action on each one individually. In contrast, the image-service does have a "root" Cargo file that lists all other cargo.toml files in its workspace. Rust also does not support nested workspaces as far as I can tell, which prevents me from easily just merging all our exisitng files under one large global workspace. | 16:31 |
kata-irc-bot | There is overhead here being incurred as well as likely some redundancy (files belonging to a workspace may be double counted, but I cannot think of a better way to do this). Secondly, there are currently 30 Cargo.tomls in kata-containers, 20 of which are being caught by cargo-deny in my PR right now. dragonflyoss/image-service has 9. This probably has some effect as well. Thirdly, I did split the advisories and license checks into two separate | 16:31 |
kata-irc-bot | jobs. I did this when I started because it was the recommended workflow on the cargo-deny-action page, but is unnecessary if we are just keeping everything as non-blocking anyways. I can change the last one and see how it affects performance. The first issue I struggled with for a while and did not see any other obvious solution, so if anyone has any suggestions that would be appreciated. The second point is basically unavoidable. I'll post | 16:31 |
kata-irc-bot | this in the slack and the PR for visibility. | 16:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!