Pin Fast-RTPS and Fast-CDR before releasing Eloquent #806
Conversation
Pin Fast-CDR to v1.0.11 Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
|
Please reference the CI build results here. |
|
Sorry, that was a bit trigger happy, @ivanpauno can you include the CI results? |
|
This was the next rerun, pinned back to eProsima/Fast-DDS@9d56202 based on #801 (comment) and FastCDR at v1.0.11 |
Completing the others: |
|
@richiware I thought that we should bring this to your attention. It seems that using your 1.9.2 tag currently causes our cross-vendor tests (specifically between FastRTPS and Connext) to fail. Our current workaround is to pin to the master branch at eProsima/Fast-DDS@9d56202 Can the 1.9.x branch be patched so that these tests will continue to pass, or is a new release (1.9.3?) coming that will fix this issue? |
Even with an upcoming patch release it would be good to get the patch backported to the |
|
@mjcarroll @dirk-thomas We would start with a patch for the |
|
@mjcarroll @dirk-thomas We found the issue (it only happened with versions 5.x of connext, 6.x was working correctly). The patch is going to |
|
@mjcarroll @ivanpauno @dirk-thomas branch |
I propose temporally pinning both, to decouple from their changes.
Particularly, there are a lot of changes going on in
Fast-RTPSand cross-vendor tests started failing.Pin Fast-RTPS to v1.9.2
Pin Fast-CDR to v1.0.11