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

[WIP] Add SITL integration tests in CI #499

Closed
wants to merge 8 commits into from
Closed

Conversation

julianoes
Copy link
Collaborator

This is work in progress to have SITL integration tests running in CI

This adds a Dockerfile (and a patch) to build a docker image which
contains the PX4 Firmware v1.8.0 as well as Gazebo.
This is just to check the environment.
@dagar
Copy link
Member

dagar commented Aug 18, 2018

Awesome!

You should take a look at the "package" we're using to do this in PX4.
http://ci.px4.io:8080/blue/organizations/jenkins/PX4_misc%2FFirmware-SITL_tests/detail/master/57/pipeline
image

  1. Build: make posix_sitl_default sitl_gazebo
  2. Build: make posix_sitl_default package
    • this creates an archive with PX4 + sitl_gazebo (libraries and models)
    • stash (pipeline temp storage)
  3. ROS Tests: separate jobs for every test that run in parallel
    • unstash px4 and run

For every successful build (master/beta/stable) we could archive the package so that you could use it directly.

FYI @lamping7

@dagar
Copy link
Member

dagar commented Aug 18, 2018

Another interesting idea might be to have PX4 master/beta/stable create and push docker containers to docker hub.

You could then use docker-compose to drive this testing (PX4 container + Dronecode SDK container driving it).

@lamping7
Copy link
Contributor

lamping7 commented Aug 19, 2018

Pushing a stable px4 firmware docker image is a good idea. The SDK shouldn't have to build the the firmware.

We could also just push the px4 package to s3. This would make it available outside Jenkins too, for users.

I can help out here. Ping me on slack or comment here if you want to bounce this of someone.

@JonasVautherin JonasVautherin changed the title [WIP] Add SITL integration tests in CI Add SITL integration tests in CI Sep 12, 2018
@JonasVautherin JonasVautherin changed the title Add SITL integration tests in CI [WIP] Add SITL integration tests in CI Sep 12, 2018
@julianoes
Copy link
Collaborator Author

That has been done in #642.

@julianoes julianoes closed this Jun 12, 2019
@julianoes julianoes deleted the add-sitl-ci branch June 12, 2019 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants