Skip to content

Update to final Connext 7.7.0 binaries - #879

Open
cottsay wants to merge 1 commit into
masterfrom
cottsay/connext-7.7.0-final
Open

Update to final Connext 7.7.0 binaries#879
cottsay wants to merge 1 commit into
masterfrom
cottsay/connext-7.7.0-final

Conversation

@cottsay

@cottsay cottsay commented Jun 17, 2026

Copy link
Copy Markdown
Member

Description

Update to final Connext 7.7.0 binaries

Is this user-facing behavior change?

No

Did you use Generative AI?

No

Additional Information

@cottsay cottsay self-assigned this Jun 17, 2026
@cottsay

cottsay commented Jun 17, 2026

Copy link
Copy Markdown
Member Author
  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@fgallegosalido

Copy link
Copy Markdown
Contributor

@cottsay it seems test issues in the Windows build are not related to the Connext RMW.

@fgallegosalido

Copy link
Copy Markdown
Contributor

@cottsay is this okay to get merged?

@cottsay

cottsay commented Jun 23, 2026

Copy link
Copy Markdown
Member Author

I've re-triggered the failing Windows build.

@cottsay
cottsay marked this pull request as ready for review June 23, 2026 21:26
@cottsay
cottsay requested a review from clalancette as a code owner June 23, 2026 21:26
@mergify

mergify Bot commented Jun 23, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@fgallegosalido

Copy link
Copy Markdown
Contributor

@cottsay the only failure in the Windows build is a test that normally doesn't fail (test_tutorial_parameter_events_async). The rest of them are not related to the Connext RMW.

@cottsay

cottsay commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

Here's a targeted build at demo_nodes_cpp. If it passes, we'll call it a flake.

  • Windows Build Status

@fgallegosalido

Copy link
Copy Markdown
Contributor

@cottsay it seems this test has already been reported: ros2/demos#693.

@fgallegosalido

Copy link
Copy Markdown
Contributor

@cottsay can we get this merged?

@fgallegosalido

Copy link
Copy Markdown
Contributor

@clalancette is it okay to merge this? This has been opened for quite some time already.

@fgallegosalido

Copy link
Copy Markdown
Contributor

@cottsay any updates on this? This is already looking good to be merged.

@fgallegosalido

Copy link
Copy Markdown
Contributor

@cottsay a gentle reminder to merge this.

@knmcguire

Copy link
Copy Markdown
Collaborator

Just another build here just in case as it has been a while:

Build Status

If succeeds I'll double check with scott if I can merge!

@fgallegosalido

fgallegosalido commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

@knmcguire the build failed very early on for reasons unrelated to the changes in this pull request. Can you take a look and launch a new validation? Also, can you just run the demo_nodes_cpp tests?

@knmcguire

Copy link
Copy Markdown
Collaborator

Ai that is a pretty hard failure actually.

Step 21/32 : RUN pixi --color never --no-progress -q install
 ---> Running in e58b9c8d3b41
Error: 
  × failed to parse project from C:\pixi_ws\pixi.toml: Unexpected keys,
  │ expected only 'name', 'version', 'description', 'authors', 'channels',
  │ 'channel-priority', 'platforms', 'license', 'license-file', 'readme',
  │ 'homepage', 'repository', 'documentation', 'conda-pypi-map', 'pypi-
  │ options', 'preview', 'target', 'build-variants'

but that doesnt have anything to do with this PR, and also demo_nodes_cpp will fail as well. Let me investigate but it would be good to update this branch.

@mjcarroll or anyone, I dont have the rights to update this branch to the latest master. Could anyone do that

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.

3 participants