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
elk6Version 6.3.2 -> 6.4.1 #47118
elk6Version 6.3.2 -> 6.4.1 #47118
Conversation
This required some small changes to the way plugins are installed but other than that it is fine. I tested this and found the ES upgrade to be fine however Kibana can end up with a corrupt index, this seems to be part of the upgrade process and not to do with this package |
@GrahamcOfBorg test elk |
Success on aarch64-linux Attempted: tests.elk No partial log is available. |
Success on x86_64-linux Attempted: tests.elk No partial log is available. |
@@ -6,6 +6,8 @@ let | |||
installPhase ? '' | |||
mkdir -p $out/config | |||
mkdir -p $out/plugins | |||
mkdir -p $out/lib | |||
cp -r ${elasticsearch-oss}/lib/* $out/lib/ |
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 am not sure if this has ever worked for newer versions. Should we backport? @basvandijk
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.
it does work with the previous version, doesn't really make sense to me but it does.
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.
Is @shmish111 symlinking the libraries possible?
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.
um, don't understand :-(
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 mean ln -s ${elasticsearch-oss}/lib/* $out/lib/
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.
ah I see, we can, what are the advantages/disadvantages?
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.
The advantage would be less duplication.
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.
yes, otherwise you will have a copy of the core libraries in each plugin.
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.
Ah, good point, ok I'll make the change today
@GrahamcOfBorg test elk.ELK-5 elk.ELK-6 |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: tests.elk.ELK-5, tests.elk.ELK-6 Partial log (click to expand)
|
Unexpected error: command failed with exit code 1 on x86_64-linux (full log) Attempted: tests.elk.ELK-5, tests.elk.ELK-6 Partial log (click to expand)
|
@shmish111 can this be closed as as it is superseded by #51225 ? |
Sure, closing |
Motivation for this change
Kibana import objects doesn't work with older 6.x versions
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)