Bug 566716 - Docker support needs integration testing
Summary: Docker support needs integration testing
Status: NEW
Alias: None
Product: CDT
Classification: Tools
Component: cdt-docker (show other bugs)
Version: Next   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact: Jonah Graham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-09-07 02:49 EDT by Moritz "morty" Strübe CLA
Modified: 2020-09-11 02:41 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Moritz "morty" Strübe CLA 2020-09-07 02:49:39 EDT
The interactions with linuxtools and docker have a high coupling, that need testing.

I found plenty of issues cross-compiling a Yocto Kernel module. Mostly in linux-tools, though. Compiling a "hello world" kernel module using a current Debian/Ubuntu Kernel should do the job and should also be quite easy to set up and maintain docker-wise. Not sure about writing the actual test. ;)
Comment 1 Jonah Graham CLA 2020-09-07 08:45:38 EDT
I'm not sure where the API/testable boundaries are in the docker/cdt integration are that can be unit tested. Adding integration testing would be a good first step.

There is one challenge with writing integration tests - we* don't have a way to test docker stuff as we don't have machines where we can launch docker.

Is it possible and/or useful to mock the actual docker part?


* The Eclipse CDT project running on Eclipse Foundation servers
Comment 2 Jonah Graham CLA 2020-09-07 08:57:12 EDT
This recent conversation is on topic about infrastructure question of using docker in build: https://www.eclipse.org/lists/cbi-dev/msg02225.html
Comment 3 Moritz "morty" Strübe CLA 2020-09-09 03:43:10 EDT
Even if those tests do not run via the CI*, I think it would be a good idea to have them in place for manual testing. Writing a wrapper-script that copies everything into a VM without public / with limited access would at least allow for nightly tests on some private system.