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

[20021] TCP support for Discovery server CLI and env. var. #4097

Merged
merged 14 commits into from
Dec 14, 2023

Conversation

cferreiragonz
Copy link
Contributor

@cferreiragonz cferreiragonz commented Dec 4, 2023

Description

This PR adds TCPv4/6 support for both the Fast DDS CLI and the ROS_DISCOVERY_SERVER environment variable.

@Mergifyio backport 2.12.x 2.11.x 2.10.x 2.6.x

Contributor Checklist

Reviewer Checklist

  • The PR has a milestone assigned.
  • Check contributor checklist is correct.
  • Check CI results: changes do not issue any warning.
  • Check CI results: failing tests are unrelated with the changes.

src/cpp/rtps/RTPSDomain.cpp Outdated Show resolved Hide resolved
src/cpp/rtps/RTPSDomain.cpp Outdated Show resolved Hide resolved
src/cpp/rtps/RTPSDomain.cpp Outdated Show resolved Hide resolved
tools/fds/server.cpp Outdated Show resolved Hide resolved
tools/fds/server.cpp Outdated Show resolved Hide resolved
tools/fds/server.h Outdated Show resolved Hide resolved
tools/fds/server.cpp Outdated Show resolved Hide resolved
tools/fds/server.cpp Show resolved Hide resolved
test/blackbox/common/BlackboxTestsDiscovery.cpp Outdated Show resolved Hide resolved
@EduPonz EduPonz added this to the v2.13.0 milestone Dec 8, 2023
@cferreiragonz
Copy link
Contributor Author

@richiprosima please test this

@cferreiragonz
Copy link
Contributor Author

@richiprosima please test this

1 similar comment
@cferreiragonz
Copy link
Contributor Author

@richiprosima please test this

@cferreiragonz cferreiragonz added needs rebase ci-pending PR which CI is running labels Dec 12, 2023
@cferreiragonz
Copy link
Contributor Author

@Mergifyio rebase

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Copy link
Contributor

mergify bot commented Dec 12, 2023

rebase

✅ Branch has been successfully rebased

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
elianalf
elianalf previously approved these changes Dec 12, 2023
@cferreiragonz
Copy link
Contributor Author

@richiprosima please test this

@cferreiragonz
Copy link
Contributor Author

@Mergifyio backport 2.12.x 2.11.x 2.10.x 2.6.x

Copy link
Contributor

mergify bot commented Dec 13, 2023

backport 2.12.x 2.11.x 2.10.x 2.6.x

✅ Backports have been created

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
@cferreiragonz
Copy link
Contributor Author

@richiprosima please test this

@cferreiragonz cferreiragonz added ready-to-merge Ready to be merged. CI and changes have been reviewed and approved. and removed ci-pending PR which CI is running labels Dec 14, 2023
@EduPonz EduPonz merged commit 2653efb into master Dec 14, 2023
12 of 14 checks passed
@EduPonz EduPonz deleted the feature/tcp_disc_server branch December 14, 2023 07:37
mergify bot pushed a commit that referenced this pull request Dec 14, 2023
* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit 2653efb)

# Conflicts:
#	versions.md
mergify bot pushed a commit that referenced this pull request Dec 14, 2023
* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit 2653efb)

# Conflicts:
#	versions.md
mergify bot pushed a commit that referenced this pull request Dec 14, 2023
* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit 2653efb)

# Conflicts:
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServerPublisher.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServerServer.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServerSubscriber.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServer_main.cpp
#	examples/cpp/dds/DiscoveryServerExample/README.md
#	examples/cpp/dds/DiscoveryServerExample/arg_configuration.h
#	examples/cpp/dds/DiscoveryServerExample/common.h
#	versions.md
mergify bot pushed a commit that referenced this pull request Dec 14, 2023
* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit 2653efb)

# Conflicts:
#	examples/C++/DDS/BasicConfigurationExample/BasicConfigurationPublisher.cpp
#	examples/C++/DDS/BasicConfigurationExample/BasicConfigurationSubscriber.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServerServer.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServer_main.cpp
#	examples/cpp/dds/DiscoveryServerExample/README.md
#	examples/cpp/dds/DiscoveryServerExample/arg_configuration.h
#	examples/cpp/dds/DiscoveryServerExample/common.h
#	include/fastdds/rtps/attributes/ServerAttributes.h
#	src/cpp/rtps/RTPSDomain.cpp
#	src/cpp/rtps/builtin/discovery/participant/PDPClient.cpp
#	test/blackbox/common/BlackboxTestsDiscovery.cpp
#	tools/fds/server.cpp
#	tools/fds/server.h
#	versions.md
EduPonz pushed a commit that referenced this pull request Dec 16, 2023
* TCP support for Discovery server CLI and env var (#4097)

* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit 2653efb)

# Conflicts:
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServerPublisher.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServerServer.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServerSubscriber.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServer_main.cpp
#	examples/cpp/dds/DiscoveryServerExample/README.md
#	examples/cpp/dds/DiscoveryServerExample/arg_configuration.h
#	examples/cpp/dds/DiscoveryServerExample/common.h
#	versions.md

* Fix Conflicts

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Co-authored-by: Carlos Ferreira González <carlosferreira@eprosima.com>
EduPonz pushed a commit that referenced this pull request Dec 18, 2023
* TCP support for Discovery server CLI and env var (#4097)

* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit 2653efb)

# Conflicts:
#	versions.md

* Fix Conflicts

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Co-authored-by: Carlos Ferreira González <carlosferreira@eprosima.com>
EduPonz pushed a commit that referenced this pull request Dec 20, 2023
* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
MiguelCompany pushed a commit that referenced this pull request Dec 21, 2023
* TCP support for Discovery server CLI and env var (#4097)

* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit 2653efb)

# Conflicts:
#	examples/C++/DDS/BasicConfigurationExample/BasicConfigurationPublisher.cpp
#	examples/C++/DDS/BasicConfigurationExample/BasicConfigurationSubscriber.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServerServer.cpp
#	examples/cpp/dds/DiscoveryServerExample/DiscoveryServer_main.cpp
#	examples/cpp/dds/DiscoveryServerExample/README.md
#	examples/cpp/dds/DiscoveryServerExample/arg_configuration.h
#	examples/cpp/dds/DiscoveryServerExample/common.h
#	include/fastdds/rtps/attributes/ServerAttributes.h
#	src/cpp/rtps/RTPSDomain.cpp
#	src/cpp/rtps/builtin/discovery/participant/PDPClient.cpp
#	test/blackbox/common/BlackboxTestsDiscovery.cpp
#	tools/fds/server.cpp
#	tools/fds/server.h
#	versions.md

* Fix Conflicts

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Revert "Fix Conflicts"

This reverts commit b79bbe9.

* Fix building incompatibilities

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Remove Discovery Server example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Apply changes & Fix env var reading

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Fix CLI tool

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Co-authored-by: Carlos Ferreira González <carlosferreira@eprosima.com>
EduPonz pushed a commit that referenced this pull request Dec 22, 2023
* TCP support for Discovery server CLI and env var (#4097)

* Refs #20021: TCP support for CLI Discovery server

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP to Discovery Server Example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: CLI: Add argument for TCP port and example

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add TCP support to environment variable

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Use autofill feature & check existing transports

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update BlackboxTestsDiscovery for TCPv4/6 dns support

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Apply Revision's Suggestions

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add test for TCP single IP address & Fix DNS test

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Add default TCP interface to CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Uncrustify

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update tests with new locator syntax & fix CLI

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Update Readme

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs #20021: Fix warnings

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Refs 20021: Update versions.md

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit 2653efb)

# Conflicts:
#	versions.md

* Fix Conflicts

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>

* Uncrustify

Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com>

---------

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Signed-off-by: elianalf <62831776+elianalf@users.noreply.github.com>
Co-authored-by: Carlos Ferreira González <carlosferreira@eprosima.com>
Co-authored-by: elianalf <62831776+elianalf@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Ready to be merged. CI and changes have been reviewed and approved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants