gaughenhey tdaitx - checking in on the open items for openjdk11 - what's left to do?17:33
* sbeattie was trying to figure out the same17:43
tdaitxI was reviewing and testing the uploads we just did18:42
tdaitxthere was a small difference in tomcat8 that I hadn't realized before, but I believe I actually got it right18:45
tdaitxthe diff I had done when writing the document was based on the versions that balint used for the "bb to bb-proposed" u-u test18:45
tdaitxbut what we probably missed is that we actually cared about "bb-security to bb-proposed"18:45
tdaitxso while the initial diff suggested that tomcat8 lacked proper support for openjdk 11 (and 10) in the init file, that was fixed on a security update18:46
tdaitxfortunately I reverted tomcat8's initd script to the one from bionic-security, not bb, so we have the sane update18:46
sbeattieoh cool18:48
* sbeattie was playing around with u-u last night, too, but is not sure he was driving it correctly.18:48
tdaitxat the same time it means that 8.5.30-1ubuntu1.3 was not upgraded for users that had local changes to its initd18:49
tdaitx* automatically upgraded by u-u18:49
tdaitxanyhow, now that is done, I'm looking over a few autopkgtests that were marked as failed and a couple updates from debian18:50
tdaitxsbeattie: meanwhile, how long do you want to let the new uploads bake in -proposed?18:51
sbeattiegiven that the recent changes should not have touched code, I don't think they need to do the 7 day wait.18:53
tdaitxthe updates on debian that I am looking into are https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=925071 and https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=92558618:55
tdaitx* am planning to look into18:55
tdaitxthe first bug would affect xenial to bionic updates, but I haven't tested that18:56
tdaitxthe second is reported against android-sdk-helper which we did not rebuild, but could possibly cause other packages to fail as well (the ones we uploaded were not affected by that issue)18:59
tdaitxso neither feel like an actual blocker19:00
tdaitxdoko: sbeattie: vorlon: let me know if any of you disagree with that assumption ^19:01
vorlontdaitx: so one last question, do the new binary packages from openjdk-lts declare Breaks: on older versions of the packages that are runtime-incompatible with the new jre?19:28
tdaitxvorlon: we only have that "Breaks: clojure1.8 (<= 1.8.0-7ubuntu1~)"19:29
tdaitxin openjdk-11-jre-headless19:29
vorlontdaitx: I was just chatting w/ sarnold and it occurred to me that this might matter, in particular for packages that could be held back in some cases by u-u (AIUI: netbeans, visualvm, jetty9, tomcat8)19:29
tdaitxhmm, yeah, we didn't update it to add the "Breaks" clause for those19:31
tdaitxnetbeans never worked afaik, so we could ignore that one19:31
tdaitxoh well, maybe somebody got netbeans working with _some_ jdk by playing around the conffiles, so yeah, we might want to add it anyway19:34
vorlonsbeattie: ^^ do you think we should do a rebuild of openjdk-lts to add these breaks before pushing out, or do you think we should bundle that fix (which is correct but probably only helps a very small number of people) with the subsequent openjdk-lts security upload?19:38
* sbeattie hrms.19:40
tdaitxwth, why does openjdk in bionic-proposed (11.0.2+9-3ubuntu1~18.04.2) shows the armhf as failed while the armhf build page says all was fine?19:44
vorlontdaitx: https://launchpad.net/ubuntu/+source/openjdk-lts/11.0.2+9-3ubuntu1~18.04.2/+publishinghistory shows doko had to delete it and recopy it because it was "copied without armhf binaries" the first tiem19:46
vorlonso the binaries made it across the second time, but unfortunately the web ui gives you links to the build record for this archive instead of for the source archive, for armhf19:47
tdaitxwe could rebuild it, just consider that armhf takes ~12 hours (and arm64 about 11h)19:47
vorlontdaitx: it doesn't need rebuilt19:47
vorlonfor this19:47
tdaitxI meant for the breaks =)19:47
sbeattieI'm less concerned about netbeans, visualvm, and jetty9, but potentially breaking people's tomcat8 in the middle of the night seems ill-advised.19:50
sbeattieso my inclination is to lean towards adding the breaks19:52
tdaitxsbeattie: vorlon: is this good enough: "Breaks: clojure1.8 (<= 1.8.0-7ubuntu1~), jetty9 (<< 9.4.15-1~), netbeans (<< 10.0-3~), tomcat8 (<< 8.5.39-1ubuntu1~), visualvm (<< 1.4.2-2~)"?21:28
tdaitxdo we want it for cosmic as well? disco?21:28
vorlontdaitx: are those the correct binary package names for each case?21:28
vorlonand is it the case that each of those 4 packages is included in the update due to runtime breakage?21:29
vorlonbut e.g. maybe you should have libjetty9-java instead of jetty921:29
tdaitxvorlon: yes they are, I think libjetty9-java does not ship any of the conf files21:30
tdaitxthey were all in jetty9 iirc, let me double check that21:30
tdaitxlibjetty9-java only has the jar and maven files21:32
tdaitxvorlon: is my assumption correct that we want the binary that carries the actual conffiles?21:33
vorlontdaitx: the breaks: should be against whichever packages are actually broken at runtime, and *might* be held back by u-u21:34
tdaitxso, libjetty9-java does not depend on the conffiles afaik, it is just the shared classes21:34
tdaitxjetty9 does use those shared classes to get the server setup and then uses the conffiles for configuration21:35
vorlontdaitx: right, so that makes sense for jetty921:49
vorlonif the other packages fit in the same category, then all good21:50
tdaitxyep, all of the same21:50
tdaitxoh lintian, oh joy21:59
tdaitxuploaded new openjdk-lts into stage5 for bionic22:05
tdaitxsame for cosmic, if we don't want the breaks in cosmic for any reason feel free to stop the builders and delete it22:16
tdaitxafk, back in ~2.5h22:17

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!