-
Notifications
You must be signed in to change notification settings - Fork 464
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
beta - Java 17 required for 7.5.165 #673
Comments
I forked and created a working proof-of-concept supporting v7.5. There's a delicate line to walk here - the base image has to be Ubuntu 20.04 (aka Focal). Focal has Java 17 and MongoDB 3.6. The 18.04/Bionic release doesn't seem to have Java 17, so that's out. The more recent LTS, 22.04/Jammy has Java 17, but no Mongo packages at all. You can of course add the mongodb.org repo for Jammy to the image, but the furthest back they support is v4.x, which reportedly works, though isn't supported by Ubiquiti. There seems to be a minor thing introduced by Java 17 here, where the controller tosses its cookies because of a permissions problem with java.time. I found a workaround though and added that to the JVM options in my proof-of-concept. The differences here are not massive. It looks possible to support v7.5 without a ton of pain. Here's the diff between where I landed and the current release. IMHO, this whole thing would get a LOT easier if Ubiquiti would support MongoDB 6. |
@jacobalberty any news on this? |
FYI - I just updated to 7.5.174 and everything works in my testing. That link I posted above still goes to the diff's between this repo and my fork. I'm happy to file a pull request once 7.5 goes GA if folks would like. |
Any ideas @jacobalberty when this can be added to the official release? Would love to get 7.5 going :) Thanks again for all your hard work |
yeah that would be nice! i see that https://community.ui.com/releases/UniFi-Network-Application-7-5-174/d05b091f-f00c-4ebb-8f42-b77e0adac78b is a RC now |
Now that it's official, want to do that pull request now :) |
I'm testing. I don't expect any problems, but wanted to check first. |
Taking a look at the pr right now, though its currently failing the build though, my guess is its failing on the 32 bit arm support. iirc 20.04 does not have any path to supporting arm32 which is why we stayed on 18.04 for so long. It might be worth bumping mongodb soon, the release notes for 7.5.x specify |
I suspect you're correct. I don't have anything arm32 handy to test with, unfortunately. I didn't catch the mongodb 3.6 or newer line either. It would be nice if it worked on 6.x, which would enable going to the Ubuntu Jammy base image. I'll see if I get some time to try that out this week. |
Taking a peek at the .deb file, the dependencies show...
If I'm reading that right, 3.6 - 4.4.24 would now be acceptable versions. The earliest version available for Ubuntu Jammy in their repositories seems to be 5.x, so that still leaves us stuck on Focal/20.04. Definitely progress though! |
I also just removed arm32 from the automatic builds, committed, and now this pull request passes testing, albeit without arm32. I think you're right - arm32 may be over. The good news is arm64 devices are pretty cheap now. I've got a couple of Orange Pi 4 LTS's around the house. 6 cores, 4GB RAM, 16GB eMMC onboard, and they're like $75-ish on Ali Express. Armbian arm64 builds are solid on them. |
Thanks! |
This is in the latest image, not beta. The release is GA. |
Saw that, but my initial question/report was for the beta image, so question is what's with the beta image :-) |
Operating system
Debian bullseye, docker beta image
UniFi Tag
EA
Docker run
docker run -d --init --restart=unless-stopped -p 8080:8080 -p 8443:8443 -p 3478:3478/udp -e TZ='Europe/Berlin' -v unifi-data:/unifi --name unifi -e https://dl.ui.com/unifi/7.5.165-9316de8cde/unifi_sysvinit_all.deb jacobalberty/unifi:beta
Bug description
Controller won't start and errors out because of incompatible java version.
Would be nice if the beta image could get some love again.
Probably changing the java version should do the trick already, if AFAIK the beta image uses bullseye and the package openjdk-17-jre-headless is available there.
Thanks in advance!
Steps to reproduce
No response
Relevant log output
No response
The text was updated successfully, but these errors were encountered: