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

Skipping flaky tests #413

Merged
merged 4 commits into from
Dec 6, 2024
Merged

Skipping flaky tests #413

merged 4 commits into from
Dec 6, 2024

Conversation

ct2034
Copy link
Collaborator

@ct2034 ct2034 commented Dec 6, 2024

No description provided.

Signed-off-by: Christian Henkel <[email protected]>
Signed-off-by: Christian Henkel <[email protected]>
Signed-off-by: Christian Henkel <[email protected]>
@ct2034 ct2034 merged commit beb9935 into ros2 Dec 6, 2024
12 checks passed
@ct2034 ct2034 deleted the fix/flaky_tests branch December 6, 2024 09:59
ct2034 added a commit to ct2034/diagnostics that referenced this pull request Dec 6, 2024
* skipping flaky ntp test

Signed-off-by: Christian Henkel <[email protected]>

* Also skipping tests in diagnostic_aggregator

Signed-off-by: Christian Henkel <[email protected]>

* also the test_critical_pub skipped

Signed-off-by: Christian Henkel <[email protected]>

* oh no ..

Signed-off-by: Christian Henkel <[email protected]>

---------

Signed-off-by: Christian Henkel <[email protected]>
(cherry picked from commit beb9935)

# Conflicts:
#	diagnostic_updater/CMakeLists.txt
ct2034 added a commit to ct2034/diagnostics that referenced this pull request Dec 6, 2024
* skipping flaky ntp test

Signed-off-by: Christian Henkel <[email protected]>

* Also skipping tests in diagnostic_aggregator

Signed-off-by: Christian Henkel <[email protected]>

* also the test_critical_pub skipped

Signed-off-by: Christian Henkel <[email protected]>

* oh no ..

Signed-off-by: Christian Henkel <[email protected]>

---------

Signed-off-by: Christian Henkel <[email protected]>
(cherry picked from commit beb9935)

# Conflicts:
#	diagnostic_updater/CMakeLists.txt
ct2034 added a commit to ct2034/diagnostics that referenced this pull request Dec 6, 2024
* skipping flaky ntp test

Signed-off-by: Christian Henkel <[email protected]>

* Also skipping tests in diagnostic_aggregator

Signed-off-by: Christian Henkel <[email protected]>

* also the test_critical_pub skipped

Signed-off-by: Christian Henkel <[email protected]>

* oh no ..

Signed-off-by: Christian Henkel <[email protected]>

---------

Signed-off-by: Christian Henkel <[email protected]>
(cherry picked from commit beb9935)
@ct2034
Copy link
Collaborator Author

ct2034 commented Dec 6, 2024

💚 All backports created successfully

Status Branch Result
ros2-humble
ros2-iron
ros2-jazzy

Questions ?

Please refer to the Backport tool documentation

ct2034 added a commit that referenced this pull request Dec 6, 2024
* skipping flaky ntp test

Signed-off-by: Christian Henkel <[email protected]>

* Also skipping tests in diagnostic_aggregator

Signed-off-by: Christian Henkel <[email protected]>

* also the test_critical_pub skipped

Signed-off-by: Christian Henkel <[email protected]>

* oh no ..

Signed-off-by: Christian Henkel <[email protected]>

---------

Signed-off-by: Christian Henkel <[email protected]>
(cherry picked from commit beb9935)
ct2034 added a commit that referenced this pull request Dec 6, 2024
* skipping flaky ntp test

Signed-off-by: Christian Henkel <[email protected]>

* Also skipping tests in diagnostic_aggregator

Signed-off-by: Christian Henkel <[email protected]>

* also the test_critical_pub skipped

Signed-off-by: Christian Henkel <[email protected]>

* oh no ..

Signed-off-by: Christian Henkel <[email protected]>

---------

Signed-off-by: Christian Henkel <[email protected]>
(cherry picked from commit beb9935)

# Conflicts:
#	diagnostic_updater/CMakeLists.txt
ct2034 added a commit that referenced this pull request Dec 6, 2024
* skipping flaky ntp test

Signed-off-by: Christian Henkel <[email protected]>

* Also skipping tests in diagnostic_aggregator

Signed-off-by: Christian Henkel <[email protected]>

* also the test_critical_pub skipped

Signed-off-by: Christian Henkel <[email protected]>

* oh no ..

Signed-off-by: Christian Henkel <[email protected]>

---------

Signed-off-by: Christian Henkel <[email protected]>
(cherry picked from commit beb9935)

# Conflicts:
#	diagnostic_updater/CMakeLists.txt
MrBlenny added a commit to Greenroom-Robotics/diagnostics that referenced this pull request Dec 12, 2024
* Add missing rclpy dependency to common_diagnostics to fix rosdoc2 output (ros#402)

* Skipping flaky ntp test (ros#409)

* skipping flaky ntp test

Signed-off-by: Christian Henkel <[email protected]>

* Also skipping tests in diagnostic_aggregator

Signed-off-by: Christian Henkel <[email protected]>

---------

Signed-off-by: Christian Henkel <[email protected]>

* Skipping flaky tests (ros#413)

* skipping flaky ntp test

Signed-off-by: Christian Henkel <[email protected]>

* Also skipping tests in diagnostic_aggregator

Signed-off-by: Christian Henkel <[email protected]>

* also the test_critical_pub skipped

Signed-off-by: Christian Henkel <[email protected]>

* oh no ..

Signed-off-by: Christian Henkel <[email protected]>

---------

Signed-off-by: Christian Henkel <[email protected]>

* feat: add publish_values parameter

---------

Signed-off-by: Christian Henkel <[email protected]>
Co-authored-by: R Kent James <[email protected]>
Co-authored-by: Christian Henkel <[email protected]>
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.

1 participant