Skip to content
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

Unit tests for code under src/gc/ and ./src/pal/src/misc/ ? #13156

Closed
omajid opened this issue Jul 25, 2019 · 5 comments
Closed

Unit tests for code under src/gc/ and ./src/pal/src/misc/ ? #13156

omajid opened this issue Jul 25, 2019 · 5 comments
Milestone

Comments

@omajid
Copy link
Member

omajid commented Jul 25, 2019

I am looking to port the changes in dotnet/corefx#39686 to coreclr too. The coreclr code is getting a bit hairy and adding unit tests would give me confidence that everything is working as designed.

I am modifying these two files:

./src/gc/unix/cgroup.cpp
./src/pal/src/misc/cgroup.cpp

Are there existing unit tests (as in, tests that always run, irrespective of the environment) for these two files? If not, where would be the best place to add them?

@omajid
Copy link
Member Author

omajid commented Jul 25, 2019

Additional question: there's a file src/pal/tests/palsuite/miscellaneous/CGroup/test1/test.cpp. This isn't a unit test, is it? It requires some setup which means it's too environment dependent to be a useful unit test.

@Maoni0
Copy link
Member

Maoni0 commented Jul 26, 2019

so recently we've been working on open sourcing our perf test suite that should be used for something like this. @andy-ms is the main contact on that. it will include container testing. he's out sick today but I will check with him where he's at wrt that when he's in.
CC-ing @sergiy-k.

@omajid
Copy link
Member Author

omajid commented Oct 24, 2019

Hey folks! It's been a few months and I want to circle back here.

I am looking for any pointers/docs/examples for testing cgroupv2 support. I am happy to implement both unit and performance tests to make sure cgroupv2 is support by coreclr.

@andy-ms, Are there any open source-perf tests I can base off of? Could you point me to any examples/docs for these perf tests?

Also, I think we are getting late with this, because Fedora 31 is about to be released. Fedora 31 only supports cgroupv2 out of the box and makes using coreclr in a container context harder than it needs to be.

@ghost
Copy link

ghost commented Oct 24, 2019

There are GC performance tests at https://github.com/dotnet/performance/tree/master/src/benchmarks/gc . There is a tutorial in the README that can help get you started. Let me know if you need any help using this.

@msftgits msftgits transferred this issue from dotnet/coreclr Jan 31, 2020
@msftgits msftgits added this to the 5.0 milestone Jan 31, 2020
@Maoni0
Copy link
Member

Maoni0 commented Jul 14, 2020

this was open sourced so closing this...if you have problems running the perf suite (explained in the readme from the link @andy-ms mentioned above please feel free to open an issue at the dotnet/performance repo.

@Maoni0 Maoni0 closed this as completed Jul 14, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants