Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Make it clear that new tests should go to the kata-containers repo #5718

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

fidencio
Copy link
Member

Let's make it as explicit as possible that new tests should be developed
for the kata-containers repo instead of this one.

Fixes: #5717

@fidencio fidencio requested a review from a team as a code owner July 18, 2023 14:55
@fidencio
Copy link
Member Author

/test

@katacontainersbot katacontainersbot added the size/small Small and simple task label Jul 18, 2023
It's been deprecated for some time already.

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
Let's make it as explicit as possible that new tests should be developed
for the `kata-containers` repo instead of this one.

Fixes: kata-containers#5717

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
@fidencio fidencio force-pushed the topic/make-it-clear-that-new-tests-should-go-to-kata-containers branch from 48de655 to 39eebfb Compare July 18, 2023 16:00
@fidencio
Copy link
Member Author

/test

Copy link
Member

@stevenhorsman stevenhorsman 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 for the go get fix whilst you were in the README

Copy link
Member

@studychao studychao left a comment

Choose a reason for hiding this comment

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

Thanks, a simple comment.
And also, do we plan to migrate all tests into Kata main repo (including tests like spelling check, license check)?

@@ -4,10 +4,26 @@ This repository contains various types of tests and utilities (called
"content" from now on) for testing the [Kata Containers](https://github.com/kata-containers)
code repositories.

**NOTE:** If you're working on adding new tests to Kata Containers, **please**,
keep in mind that we're in the process of migrating the tests from this repo to
the kata-containers one.
Copy link
Member

Choose a reason for hiding this comment

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

maybe a link here would be better?
[Kata Containers](https://github.com/kata-containers)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/small Small and simple task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

README: Add note about moving the tests to the kata-containers repo
4 participants