openhab / openhabian

openHABian - empowering the smart home, for Raspberry Pi and Debian systems
https://community.openhab.org/t/13379
ISC License
820 stars 251 forks source link

New image #946

Closed mstormi closed 4 years ago

mstormi commented 4 years ago

With the new Raspbian image out we should compile, test for compatibility and finally release a new image.

DONE

update first-boot.bash (done in #890), rc.local / error output #840 / #890, add Oracle OpenJDK to menu #948,first-boot.bash#L132 halts install if no Internet => #950, git,make,gcc packages must be preloaded with the image, frontail install hangs in "npm install" if server unreachable use --force => #950, preload/include default Java (Zulu 8) with image, openHABian repo itself => #950, caching in build.bash, adaptions in first-boot #984, offline install capability #825 (to test block inet for box and use "openhabian-config unattended")

TODO

Test issues

holgerfriedrich commented 4 years ago

Test images (32 and 64bit) including PR #890 and #931 available from: https://github.com/holgerfriedrich/openhabian/actions/runs/121959703

mstormi commented 4 years ago

These images are xz-encoded TWICE so Etcher does not want them unless unzipped once.

The 64bit boots into desktop mode. Maybe you didn't download the lite version there ?

You also need to set debugmode=off in openhabian.conf (instead of the "on" default used in Travis)

mstormi commented 4 years ago

@holgerfriedrich are you done with the new image? Else reopen this please and update the list in post 1 what's left to do.

ecdye commented 4 years ago

Do we merge #976 before we finalize 1.6?

mstormi commented 4 years ago

yes why do you ask?

ecdye commented 4 years ago

Just making sure. I was forgetting about that part.

mstormi commented 4 years ago

I think we should hurry up. Current code has broken installation because helpers.bash and openhabian.bash are not copied over to /boot.

ecdye commented 4 years ago

What else needs to be done before we finalize and start testing a new image?

mstormi commented 4 years ago

Holger has to build an image that pre-loads the prerequisites. That's about it I would think.

ecdye commented 4 years ago

So, essentially we are waiting for #984 / #985 to be merged then built?

mstormi commented 4 years ago

yes unless I'm missing something

ecdye commented 4 years ago

I have created a new build image action for my own branch, should we just start building images as part of CI in master? It might be easier, I can create a PR with a bunch of CI updates that I have made as a part of #990 and include it.

Edit: see #1008

Additionally this allows for checking that the image builds properly and we can also test an image of a PRs changes. When making releases we need to make sure to save the artifacts as they expire after 90 days.

ecdye commented 4 years ago

Random thought, has anyone tested an online install with any of the recent images? I am about to, but we should not forget to test that online install still works after finishing offline install.

mstormi commented 4 years ago

yes I just did, it's ok (due to most recent fixes). It wasn't for some days I think Fortunately it only affects people to install from scratch AND use master.

ecdye commented 4 years ago

Make sure that we build the image from the stable branch as otherwise we will have issue with incompatible changes. That also means we need to forward the master branch to stable.

mstormi commented 4 years ago

Holger your most recent image builds still use your repo. Could you fix that so we can start testing the image for release ?

holgerfriedrich commented 4 years ago

@mstormi did you try https://github.com/openhab/openhabian/actions/runs/166692092 on master branch?

Actually using the current PR repo and branch is set in ci-setup.bash, see the last 2 lines of the code. @ecdye introduced that, and I have no strong opinion if this is good or bad. I can think of situations, where you want the image to refer to your own branch, but we should mark this as "-custom-" in the image filename to avoid publishing.... Once you merged to master it should behave as you expect.

mstormi commented 4 years ago

ah no will try I'd think ci-setup.bash is mostly ok since master is what we want to use for building any official images. For development purposes you can of course change ci-setup.sh (you need to as well for e.g. debugmode pre-set)

ecdye commented 4 years ago

@mstormi Are we ready to do a final sync to stable of master and release the image on commit 7a30199?

mstormi commented 4 years ago

Technically speaking I believe yes we're done, but I think we should take the time to double-check these questions before: 1) eventually rework license to include what raspberrypi.org has on their download site (Oracle JDK et al) 2) re-evaluate the state of offline install. Based on https://github.com/openhab/openhabian/pull/985#issuecomment-657387276 , now that the image was grown to provide enough storage space and now that Holger should be back (@holgerfriedrich are you? Please check back with us), it might be just a little more work to include it, too. 3) what about #1055? Yes it's mostly unrelated but if in doubt, I'd prefer to have a single big, potentially breaking change rather than two one after another. I expect (temporary) major support efforts once the image is out, distracting us from development. 4) more testing needed ?

ecdye commented 4 years ago

I agree with all that however we need to release the image ASAP. We should probably need to just speed up the plans for release a bit.

mstormi commented 4 years ago

do we? Given the alpha release is available we can direct people in (immediate) need there

ecdye commented 4 years ago

Perhaps, to give a more complete response to your list earlier, #1 I think is less of an issue as Oracle JDK is not even included in our project. #2 may need to take the back burner as it is not one of my priorities right now and Holger seems to have lost the drive to continue for the moment. #3, yes it would be nice to have that in but I am not sure when I will complete it the last one took me almost a month and IDK how much more I will change before I am satisfied especially as I have barely even touched the surface for that at the moment. #4 yes and no, the only changes I see that are breaking are FIND3 which are labeled as alpha anyway.

ecdye commented 4 years ago

@mstormi I don't think there is any way for me to preinstall frontail as npm will not work in the environment that we use for offline prefetch. I can install nodeJS (already am doing so) but cannot ensure that Frontail will properly install in a offline environment.

Frontail in offline may be a good topic for future development.

mstormi commented 4 years ago

fine for me. But we need to fix it currently doesn't install (at least not in CI)

Is gdisk prefetched? if so please tick in post 1

ecdye commented 4 years ago

Wow, for the first time since I started working on this project, we don't have any open PR's.

This has been a long and winding road to 1.6, great job @openhab/openhabian-maintainers @holgerfriedrich the end is finally in sight! Now that this is done, onto the next big project: openHAB3!

mstormi commented 4 years ago

Indeed, great job of yours Ethan, too !

Any pathetic wording you would like to be part of the announcement ? Any particular feature you would want to point out ?

mstormi commented 4 years ago

We're done, v1.6 is released. Thanks again everybody and let's continue with adding OH3.