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
dd-agent: fix compatibility issue with iostat #41035
Conversation
Applies a patch to the dd-agent derivation that fixes a compatibility issue with the current version of iostat, which no longer contains a colon after its table headers. This patch is applied in order for the fix to be backportable to existing stable releases. A final "proper" fix will be an upgrade to a newer version of dd-agent, but that requires several other changes. This fixes NixOS#40103.
@@ -38,6 +38,8 @@ in stdenv.mkDerivation rec { | |||
sha256 = "1iqxvgpsqibqw3vk79158l2pnb6y4pjhjp2d6724lm5rpz4825lx"; | |||
}; | |||
|
|||
patches = [ ./40103-iostat-fix.patch ]; |
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.
Can you use fetchpatch of this commit instead? DataDog/dd-agent@bfc3057
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.
I'll test that
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.
@Mic92 The diff of the test does not apply cleanly (dd-agent
is at a much newer version upstream than what we have packaged). Not sure if it's worth pursuing that vs. having the patch in nixpkgs for a short while (I'll follow up with a proper fix by updating the package version eventually, see #39808)
@GrahamcOfBorg build dd-agent |
Success on x86_64-darwin (full log) Attempted: dd-agent Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: dd-agent Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: dd-agent Partial log (click to expand)
|
@xeji Thanks! Is there some kind of "process" for getting the fix released into 18.03? |
No, just mention in the PR that you'd like this backported to 18.03 (which you just did). |
Applies a patch to the dd-agent derivation that fixes a compatibility issue with the current version of iostat, which no longer contains a colon after its table headers. This patch is applied in order for the fix to be backportable to existing stable releases. A final "proper" fix will be an upgrade to a newer version of dd-agent, but that requires several other changes. This fixes #40103. (cherry picked from commit aee19ca)
@xeji Thanks again :) |
backported in c11a4a5 |
Fix for NixOS/nixpkgs#41035 (dd-agent) is backported now.
Fix for NixOS/nixpkgs#41035 (dd-agent) is backported now.
Motivation for this change
Applies a patch to the dd-agent derivation that fixes a compatibility
issue with the current version of iostat, which no longer contains a
colon after its table headers.
This patch is applied in order for the fix to be backportable to
existing stable releases. A final "proper" fix will be an upgrade to a
newer version of dd-agent, but that requires several other changes.
This fixes #40103.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)cc: @bflyblue