Skip to content

Conversation

stevenhorsman
Copy link
Member

Since #10780 the dbs crates are managed as members of the dragonball workspace, so we can remove the lockfile as it's now workspace managed now

@katacontainersbot katacontainersbot added the size/huge Largest and most complex task (probably needs breaking into small pieces) label Apr 29, 2025
Copy link
Member

@lifupan lifupan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's reasonable, thanks @stevenhorsman.

Copy link
Member

@justxuewei justxuewei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, thanks!

Copy link
Member

@RuoqingHe RuoqingHe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, thanks for taking care of this @stevenhorsman ❤️, clearly this is something should be done along with the workspace change

@stevenhorsman
Copy link
Member Author

It looks like some genpolicy PRs have come in, which is causing the SNP test failures, so I will try rebasing

Since kata-containers#10780 the dbs crates are managed as members
of the dragonball workspace, so we can remove the lockfile
as it's now workspace managed now

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
When a PR has no new files the cargo deny runner fails with:
```
[cargo-deny-generator.sh:17] ERROR: changed_files_status=
```
so add `|| true` to try and help this

Co-authored-by: Ruoqing He <heruoqing@iscas.ac.cn>
Signed-off-by: stevenhorsman <steven@uk.ibm.com>
@stevenhorsman stevenhorsman merged commit b97bc03 into kata-containers:main Apr 30, 2025
326 of 346 checks passed
@stevenhorsman stevenhorsman deleted the dragonball-lockfiles branch April 30, 2025 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/huge Largest and most complex task (probably needs breaking into small pieces)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants