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

Release 1.9.3 #800

Closed
18 tasks done
MiguelCompany opened this issue Oct 18, 2019 · 7 comments
Closed
18 tasks done

Release 1.9.3 #800

MiguelCompany opened this issue Oct 18, 2019 · 7 comments
Milestone

Comments

@MiguelCompany
Copy link
Member

MiguelCompany commented Oct 18, 2019

Bugfixes

Improvements

Features

Backports

@nuclearsandwich
Copy link
Contributor

#819 re-introduced the communication failures between Fast-RTPS and RTI Connext in the ROS 2 communication tests. The test failures are still present on the latest 1.9.x branch as well.

Example CI build: https://ci.ros2.org/job/ci_linux/8586/
Example exact repos file: https://gist.githubusercontent.com/nuclearsandwich/9cf29e1add9721ab6454b505bea433c4/raw/6e1464600505ed16fa030bc3f1577ed000453edf/fastrtps.repos

@nuclearsandwich
Copy link
Contributor

@MiguelCompany it looks like @laura-eprosima pushed the 1.9.3 bump a few hours ago but there are still regressions on this branch compared to 1.9.2. Is there a plan to address those before the release is cut?

@MiguelCompany
Copy link
Member Author

it looks like @laura-eprosima pushed the 1.9.3 bump a few hours ago but there are still regressions on this branch compared to 1.9.2.

In order to generate the documentation with the correct version number, we need it to be changed. We have done it so we can start preparing the docs and accelerate the release procedure.

Is there a plan to address those before the release is cut?

Yes, it should have been addressed on #846. It seems that Connext is not respecting point 8.3.7.3.3 item 5 of the RTPS standard. We added some checks on #819 that have been relaxed on #846

@nuclearsandwich
Copy link
Contributor

Thanks for the clarification. I'll give #846 a try and post results here.

@nuclearsandwich
Copy link
Contributor

Our cross-vendor tests do pass with #846. Thanks.

@MiguelCompany
Copy link
Member Author

@nuclearsandwich @dirk-thomas v1.9.3 is finally out. We would like to have it on Eloquent as soon as possible, at least on the testing packages.

We have run ROS2 tests successfully

@nuclearsandwich
Copy link
Contributor

I've triggered CI for the v1.9.3 tag and it's running here

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

Yesterday I released a 1.9.2 extension based on 141b778 and that's what is currently released. I'll put another release out later today once CI comes back.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants