Skip to content

Conversation

kgiusti
Copy link
Contributor

@kgiusti kgiusti commented Feb 11, 2025

Include mechanism for propagating updates to affected connectors. Advertise the connection's ordinal to the peer router via the properties map in the Open performative.

@kgiusti kgiusti marked this pull request as ready for review February 13, 2025 14:23
@kgiusti kgiusti linked an issue Feb 13, 2025 that may be closed by this pull request
@kgiusti kgiusti marked this pull request as draft February 27, 2025 15:41
@kgiusti kgiusti marked this pull request as ready for review February 28, 2025 20:03
@kgiusti
Copy link
Contributor Author

kgiusti commented Mar 10, 2025

@ganeshmurthy I believe the latest push resolves all the outstanding issues. Please review and let me know thanks.

@kgiusti kgiusti requested a review from ganeshmurthy March 10, 2025 13:34
kgiusti added 5 commits March 10, 2025 11:42
Include mechanism for propagating updates to affected
connectors. Advertise the connection's ordinal to the peer router via
the properties map in the Open performative.
Based on what I've learned doing POC work in the router core and PR
request feedback.  The tls ordinal is leveraged as a group attribute
for the core. This will allow the core to identify which connection in
the group takes precedence. Also removed extra copies of the
correlator string and various code cleanups.
@kgiusti kgiusti merged commit f7160bf into skupperproject:main Mar 10, 2025
37 of 41 checks passed
@kgiusti kgiusti deleted the ISSUE-1741 branch March 10, 2025 16:20
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.

Add sslProfile ordinal and oldestValidOrdinal management attributes
2 participants