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

Dome version update #262

Merged
merged 11 commits into from
Aug 3, 2020
Merged

Dome version update #262

merged 11 commits into from
Aug 3, 2020

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Jul 27, 2020

This updates required versions for Dome.

@nkoenig nkoenig requested review from j-rivero and chapulina July 27, 2020 18:24
@nkoenig nkoenig requested a review from maryaB-osr as a code owner July 27, 2020 18:24
@github-actions github-actions bot added the 🔮 dome Ignition Dome label Jul 27, 2020
@chapulina
Copy link
Contributor

chapulina commented Jul 27, 2020

This requires:

@j-rivero
Copy link
Contributor

This requires:

we might need to update to ign-physics3 before merging

2020-07-29T15:57:55.0423009Z  libignition-physics2-dev : Depends: libignition-physics2-dartsim-dev but it is not going to be installed
2020-07-29T15:57:55.0423472Z                             Depends: libignition-physics2-sdf-dev but it is not going to be installed
2020-07-29T15:57:55.0423965Z  libsdformat10-dev : Depends: libsdformat10 (= 10.0.0~pre1+nightly+git20200728+1r0f926078fd82ff34087f29350c387f999ac8a50d-1~bionic) but it is not going to be installed
2020-07-29T15:57:55.0542525Z E: Unable to correct problems, you have held broken packages.

chapulina and others added 6 commits July 29, 2020 23:06
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
Signed-off-by: Jose Luis Rivero <[email protected]>
@j-rivero j-rivero force-pushed the dome_version_update branch from eb13e4d to a21ac7f Compare July 29, 2020 21:07
@j-rivero
Copy link
Contributor

we might need to update to ign-physics3 before merging

physics3 problem is solved. I just add support for bionic in fuel-tools5, waiting in Jenkins to be built, that should make github actions happy (pending on X support for tests or disable them).

Signed-off-by: Louise Poubel <[email protected]>
@j-rivero
Copy link
Contributor

action is failing for tests:

2020-07-30T19:24:32.6186403Z The following tests FAILED:
2020-07-30T19:24:32.6186676Z 	 23 - UNIT_Server_TEST (Failed)
2020-07-30T19:24:32.6186953Z 	 39 - INTEGRATION_air_pressure_system (Failed)
2020-07-30T19:24:32.6187228Z 	 41 - INTEGRATION_altimeter_system (SEGFAULT)
2020-07-30T19:24:32.6187564Z 	 43 - INTEGRATION_apply_joint_force_system (SEGFAULT)
2020-07-30T19:24:32.6187855Z 	 45 - INTEGRATION_battery_plugin (SEGFAULT)
2020-07-30T19:24:32.6188119Z 	 47 - INTEGRATION_breadcrumbs (Failed)
2020-07-30T19:24:32.6188390Z 	 51 - INTEGRATION_contact_system (Failed)
2020-07-30T19:24:32.6188664Z 	 53 - INTEGRATION_detachable_joint (Failed)
2020-07-30T19:24:32.6188937Z 	 55 - INTEGRATION_diff_drive_system (Failed)
2020-07-30T19:24:32.6189205Z 	 63 - INTEGRATION_examples_build (Failed)
2020-07-30T19:24:32.6189479Z 	 65 - INTEGRATION_follow_actor_system (Failed)
2020-07-30T19:24:32.6189735Z 	 67 - INTEGRATION_imu_system (SEGFAULT)
2020-07-30T19:24:32.6190020Z 	 69 - INTEGRATION_joint_controller_system (Failed)
2020-07-30T19:24:32.6190293Z 	 71 - INTEGRATION_lift_drag_system (Failed)
2020-07-30T19:24:32.6190571Z 	 79 - INTEGRATION_logical_camera_system (SEGFAULT)
2020-07-30T19:24:32.6190849Z 	 81 - INTEGRATION_magnetometer_system (SEGFAULT)
2020-07-30T19:24:32.6191124Z 	 85 - INTEGRATION_multicopter (SEGFAULT)
2020-07-30T19:24:32.6191399Z 	 87 - INTEGRATION_network_handshake (Failed)
2020-07-30T19:24:32.6191671Z 	 89 - INTEGRATION_performer_detector (Failed)
2020-07-30T19:24:32.6191943Z 	 91 - INTEGRATION_physics_system (SEGFAULT)
2020-07-30T19:24:32.6192219Z 	 95 - INTEGRATION_pose_publisher_system (Failed)
2020-07-30T19:24:32.6192485Z 	 97 - INTEGRATION_save_world (Failed)
2020-07-30T19:24:32.6192769Z 	 99 - INTEGRATION_scene_broadcaster_system (Failed)
2020-07-30T19:24:32.6193043Z 	105 - INTEGRATION_thermal_system (Failed)
2020-07-30T19:24:32.6193308Z 	107 - INTEGRATION_touch_plugin (Failed)
2020-07-30T19:24:32.6193583Z 	109 - INTEGRATION_triggered_publisher (Failed)
2020-07-30T19:24:32.6193855Z 	111 - INTEGRATION_user_commands (SEGFAULT)
2020-07-30T19:24:32.6194108Z 	113 - INTEGRATION_log_system (SEGFAULT)
2020-07-30T19:24:32.6194375Z 	115 - INTEGRATION_wind_effects (Failed)
2020-07-30T19:24:32.6194640Z 	119 - PERFORMANCE_level_manager (Failed)

@chapulina do we want to add xfvb support or just disable them in github actions? There are many errors coming from the fact of not being able to find plugins.

@j-rivero
Copy link
Contributor

@osrf-jenkins run tests please

@j-rivero
Copy link
Contributor

@chapulina do we want to add xfvb support or just disable them in github actions? There are many errors coming from the fact of not being able to find plugins.

testing it here 1734811

@codecov
Copy link

codecov bot commented Jul 31, 2020

Codecov Report

Merging #262 into master will increase coverage by 15.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #262       +/-   ##
===========================================
+ Coverage   65.78%   80.79%   +15.01%     
===========================================
  Files         128      176       +48     
  Lines        6307     9229     +2922     
===========================================
+ Hits         4149     7457     +3308     
+ Misses       2158     1772      -386     
Impacted Files Coverage Δ
include/ignition/gazebo/components/Actor.hh 27.27% <ø> (ø)
...nclude/ignition/gazebo/components/ChildLinkName.hh 100.00% <ø> (ø)
include/ignition/gazebo/components/Name.hh 100.00% <ø> (ø)
...clude/ignition/gazebo/components/ParentLinkName.hh 100.00% <ø> (ø)
...de/ignition/gazebo/components/PerformerAffinity.hh 100.00% <ø> (ø)
.../ignition/gazebo/components/PhysicsEnginePlugin.hh 100.00% <ø> (ø)
...clude/ignition/gazebo/components/SourceFilePath.hh 100.00% <ø> (ø)
...ude/ignition/gazebo/components/LevelEntityNames.hh 100.00% <100.00%> (ø)
...nclude/ignition/gazebo/components/Serialization.hh 100.00% <100.00%> (ø)
src/systems/apply_joint_force/ApplyJointForce.hh 100.00% <0.00%> (ø)
... and 64 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aea1acc...2155d62. Read the comment docs.

@j-rivero
Copy link
Contributor

Looking into the failures for linux and brew.

@j-rivero
Copy link
Contributor

@osrf-jenkins run tests

@j-rivero
Copy link
Contributor

j-rivero commented Aug 3, 2020

@osrf-jenkins run tests

@j-rivero
Copy link
Contributor

j-rivero commented Aug 3, 2020

the fail on Linux is expected (docker cache not being updated correctly). I'm testing a permanent fix in next builds.

@j-rivero
Copy link
Contributor

j-rivero commented Aug 3, 2020

wew! finally get all the CI mechanisms working. Merging and going for launch.

Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

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

🚢 it

@j-rivero j-rivero merged commit 8ec8d66 into master Aug 3, 2020
@j-rivero j-rivero deleted the dome_version_update branch August 3, 2020 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 dome Ignition Dome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants