-
Notifications
You must be signed in to change notification settings - Fork 19
ci: update nightly in CI to 2026-02-08 #1539
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,10 +10,12 @@ use std::process; | |
| use std::{fs, path::PathBuf}; | ||
|
|
||
| use anyhow::Context; | ||
| #[cfg(all(target_os = "linux", not(target_env = "musl")))] | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is only used in tests that have the same gate to not run on our alpine builds at the moment and I was getting unused import warnings. |
||
| use bin_tests::test_runner::run_crash_no_op; | ||
| use bin_tests::{ | ||
| artifacts::{self, StandardArtifacts}, | ||
| build_artifacts, | ||
| test_runner::{run_crash_no_op, run_crash_test_with_artifacts, CrashTestConfig, ValidatorFn}, | ||
| test_runner::{run_crash_test_with_artifacts, CrashTestConfig, ValidatorFn}, | ||
| test_types::{CrashType, TestMode}, | ||
| validation::PayloadValidator, | ||
| ArtifactsBuild, BuildProfile, | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -17,6 +17,7 @@ pub fn default_signals() -> Vec<libc::c_int> { | |
| Vec::from(DEFAULT_SYMBOLS) | ||
| } | ||
|
|
||
| #[cfg(target_os = "linux")] | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is only called by tests that have the same cfg targets. It should resolve dead code warnings on other envs. |
||
| pub(super) fn mark_preload_logger_collector() { | ||
| // This function is specific only for LD_PRELOAD testing | ||
| // Best effort; this symbol exists only when the preload logger preload is present. | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,7 +15,6 @@ use libdd_telemetry::data::LogLevel; | |
| use serde::{Deserialize, Serialize}; | ||
| use std::sync::Arc; | ||
| use std::time::{Duration, Instant}; | ||
| use std::{fs, path::PathBuf}; | ||
| use tokio::io::AsyncBufReadExt; | ||
|
|
||
| #[derive(Debug)] | ||
|
|
@@ -524,6 +523,8 @@ pub(crate) async fn receive_report_from_stream( | |
|
|
||
| #[cfg(target_os = "linux")] | ||
| fn enrich_thread_name(builder: &mut CrashInfoBuilder) -> anyhow::Result<()> { | ||
| use std::{fs, path::PathBuf}; | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. moved the import since it's only used for linux with this function. It should resolve unused import warnings in other envs. |
||
|
|
||
| if builder.error.thread_name.is_some() { | ||
| return Ok(()); | ||
| } | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are these CFGs for, in relation to updating nightly? New warnings or..?