-
Woodstock
is gerrit broken again?
-
andyf
I can get to it ok
-
tsoome
I have seen few messages but it appears to be ok still
-
sommerfeld
illumos.org/issues/16880#change-47687 is a post from someone who just registered on illumos.org which has a final sentence that looks like SEO link spam
-
fenix
→
BUG 16880: pkglint fails on OpenIndiana with openssl mediator set to 3.1 (New)
-
bahamat
ptribble: Are you around, by chance?
-
ptribble
hi bahamat!
-
bahamat
Hi!
-
bahamat
Ok, so we're using your JDK17 for Jenkins. The latest update requires at least JDK 17.0.11, and the binaries you have are 17+35.
-
bahamat
I was hoping you'd be able to build a 17.0.11-ga for us fairly easily?
-
bahamat
And to complicate things, it needs to be able to run on 20181206T011455Z.
-
bahamat
I'll check what the illumos symbol version is on that right now, but I don't remember what it is...
-
danmcd
I'll figure that out bahamat
-
bahamat
thanks
-
ptribble
I presume 17+35 actually works? (Other than age.)
-
danmcd
illumos-gate commit for the above SmartOS is 2f3d18f2c9
-
bahamat
Yeah, it seemed to. The error I was getting is "sun.security.pkcs11.wrapper.PKCS11Exception: CKR_GENERAL_ERROR"
-
danmcd
Libc was SYMBOL_VERSION ILLUMOS_0.28 {
-
bahamat
When I asked in jenkins on matrix they said that there's been a bunch of updates and mine is too old.
-
bahamat
Yeah, I can confirm that there were at least some of our agents running on your jdk17+35 that were working until this latest Jenkins upgrade.
-
ptribble
So I'm building on something a bit newer, I think with ILLUMOS_0.40, but java is very conservative about the libc functionality it requires
-
bahamat
Actually, I retract my confirmation...I'm not completely sure.
-
bahamat
A couple I checked were actually on java11...
-
bahamat
danmcd: What libc version is 20210826T002459Z?
-
rmustacc
If you look at the java binary with pvs it'll tell you what it requires.
-
danmcd
Hang on...
-
rmustacc
pvs -d /usr/lib/libc.so.1 will get you that, probably pipe to head.
-
danmcd
SYMBOL_VERSION ILLUMOS_0.39 {
-
bahamat
[root@jenkins-462d490c-21-4-0 /opt/tools/java/openjdk17/bin]# pvs java
-
bahamat
libc.so.1 (ILLUMOS_0.26, SUNW_0.7);
-
rmustacc
Yeah, makes sense that it has 0.26 given that was an accidental weak version.
-
danmcd
( 20210826 libc is 0.39 )
-
bahamat
OK, I actually have some with 17-internal+0-adhoc.ptribble.jdk17-jdk-17-35 and some with 17.0.11-internal+0-adhoc..jdk-il-jdk-17.0.11-ga
-
bahamat
And pvs reports ILLUMOS_0.26 for both of them
-
rmustacc
Which means it's probably less than that but oops accidental weak version.
-
bahamat
But maybe I can just copy it over and use it?
-
rmustacc
Should, yes.
-
rmustacc
At least from a libc perspective.
-
rmustacc
There are probably other shared objects outside of illumos it links against you may need to consider.
-
bahamat
Ok, let's give that a shot and see what happens...
-
ptribble
My 17+35 was really intended for people to build their own
-
ptribble
I've put a copy of 17.0.13 at
pkgs.tribblix.org/openjdk
-
bahamat
Damn, no, that doesn't work.
-
bahamat
ld.so.1: java: fatal: libc.so.1: version 'ILLUMOS_0.37' not found (required by file /opt/tools/java/jdk17-jdk-17.0.11-ga/bin/../lib/pkgsrc/libiconv.so.2)
-
bahamat
Yeah, the thing is I need to be able to run it on ILLUMOS_0.28
-
ptribble
And as far as I can tell (looking at all the objects) that 17.0.13 build only needs ILLUMOS_0.26
-
bahamat
Ok, let me try that then
-
rmustacc
It's the libiconv that's pulling it in, not Java.
-
rmustacc
Beacuse that was built with stack protection.
-
rmustacc
So where did you get that libiconv bahamat?
-
bahamat
Either the PI, pkgsrc-tools, or statically linked with java
-
rmustacc
Not statically linked.
-
rmustacc
The path says it's /opt/tools, so probalby tools pkgsrc.
-
rmustacc
Which to me says that you're trying to use too new a tools pkgsrc.
-
rmustacc
If it was staically linked, it wouldn't be calling out a different shared object.
-
ptribble
That looks like a pkgsrc build shipped with some extra local copies of the 3rd-party libraries from pkgsrc
-
bahamat
rmustacc: But libiconv hasn't changed version
-
rmustacc
I do'nt understand what you mean?
-
rmustacc
ld.so.1 says libiconv.so.2 requires 0.37 which is the version that introduced the stack protector in libc.
-
rmustacc
If you look at what java depends on, it depends on libiconv.so.2 and the libiconv.so.2 is what's too new to run on your platform.
-
bahamat
OK, so that jdk17 came from pkgsrc
-
bahamat
That's what you meant.
-
bahamat
ptribble: that version of jdk17.0.13 works for me!
-
bahamat
Awesome, thank you!!
-
danmcd
Thanks ptribble, rmustacc, and bahamat!
-
ptribble
Excellent!
-
bahamat
And that was definitely faster than me picking apart dobuild until I got it to work on SmartOS :-D
-
jperkin
bahamat: I assume you were using the 17.0.11 from
us-central.manta.mnx.io/pkgsrc/public/pkg-bootstraps/index.html which was built on 20210826, and presumably on a platform older than that?
-
jperkin
or did I somehow bundle the wrong libs?
-
jperkin
I use that as the bootstrap for the openjdk21 build on the same OS so it should work..
-
richlowe
so I have a set of changes that make java optional that come from arm64-gate, and just optionally amputate (because who's going to port java for me?)
-
richlowe
would people like those upstream?
-
richlowe
I don't like flags that slice off bits of build, but the java situation seems special, except for Peter being a super hero
-
richlowe
-
richlowe
why that's showing some of sommerfeld's changes I don't know :\
-
ptribble
That neatly identifies our dependencies on java, too
-
danmcd
@richlowe our use of java isn't for SmartOS per se... it's for Jenkins agents.
-
danmcd
All of this was because bahamat couldn't get one of her Triton component builds to work.
-
ptribble
Although it may be worth asking (again) how much of the java stuff in gate is actually useful
-
danmcd
Oh yes... SmartOS jettisoned that WELL before I starting working there.
-
ptribble
I *thought* I had removed it all from Tribblix, besides resource pools, but looking at that change I missed some of the bits in the dtrace tests
-
ptribble
As for illumos on arm64, I would like to have java there too.
-
ptribble
And it's probably easier to support than sparc, because openjdk already have a fully supported aarch64 implementation
-
tsoome
btw, we do have rising problem with java in illumos-gate, some of the code is already deprecated and once they stop supporting those functions.... I have been looking on some cases and those are definitely fixable, but require time (as usual).
-
tsoome
and of course testing those bits is even more interesting.
-
ptribble
I wouldn't worry about deprecated methods too much, unless they're actually deprecated for removal
-
ptribble
What will hit eventually is the phasing out of JNI
-
rmustacc
If we look at the java usage the bigger things will be where do we actually rely on it from the core implementation ala poold.
-
rmustacc
I suspect the use of dtrace_jni and similar is minimal in practice.
-
richlowe
ptribble: the real crime is availdevs, if someone with the energy just wanted to delete it
-
richlowe
as best as we can tell, it's only used by installers that don't exist (and perhaps never existed, really)
-
richlowe
(I filed the bug already at least once)
-
richlowe
it's just always having something else to do in the way
-
ptribble
And it can't possibly work on any current distro, because it's 32-bit and no distro ships a 32-bit java
-
tsoome
then it is axe time:D
-
richlowe
the problem with gardening, is sometimes you have to tend to what's alive before you can dead head
-
jclulow
-
fenix
→
BUG 15147: remove /usr/lib/zfs/availdevs (In Progress)
-
jclulow
It's assigned to you richlowe lol
-
jclulow
ptribble: It doesn't actually use Java; it's so much worse than that.
-
richlowe
jclulow: I can drop that for anyone who wants to delete the damn thing before I get to it!
-
ptribble
I just looked at the code, it's ... a terrible concept
-
jclulow
Yes
-
sommerfeld
richlowe: huh? (didn't see anything of mine in that commit link you posted)
-
bahamat
jperkin: Circling back...your build is fine. I copied it from a 2022 PI to a 2018 PI
-
richlowe
sommerfeld: in my viewing it, (parts of) your ipv6 scope changes appeared in the diffs
-
sommerfeld
Didn't see that when I looked at. Huh.