New Docker dev image: I need your feedback! :)

Development
1

Note: This message is for experienced users, the devs of this forum :smiley:

Hello everyone!

Today I merged the PR from @VonOx on the new CI process. We have switched from CircleCI to GitHub Actions so far, and especially the dev and prod builds are done with the same Dockerfile and built with buildx.

I would like to have your feedback on this dev image, and especially on the native integrations (Z-Wave, Bluetooth, camera), is everything working well?

The image is available on all platforms with:

docker pull gladysassistant/gladys:dev

And then to run it, it’s the same commands as usual in the documentation, you just need to change the tag to gladys:dev.

2

Well, I had tested but you never know:

  • Bluetooth ok (I don’t have a peripheral but the scan is ok)
  • Z-Wave ok

image
image914×428 15.8 KB

3

Hello!!

We tried with @damalgos to push our changes to Netatmo, but it asked us to update from the Master, which we did. However, the installation of dependencies no longer works. We also see that there is a red cross on your migration. Could something have broken on the PRs? I thought there were some in green, but now there’s only red and orange.

Perhaps it’s unrelated??

4

I just checked the netatmo PR (very quickly)
But if you push a modification of the package.json, you also need to push the package-lock.json

Yes, we fixed it directly (the fail was not on the npm dependencies)

There’s something wrong with the npm package sqlite3 in version 4.2++

node-pre-gyp WARN Pre-built binaries not installable for sqlite3@4.2.0 and node@12.20.0 (node-v72 ABI, glibc) (falling back to source compile with node-gyp)
node-pre-gyp WARN Hit error bad download

Is the update vital?

I went to see their github (sqlite3) but it’s a mess.

5

Thanks @VonOx for your feedback, I’ve passed it on to @damalgos. Let’s look into it!!

Oh dear!! Troublesome
!!

6

Oh well if it’s @damalgos’ fault 
 :stuck_out_tongue:

7

Yes, I was also wondering, why are you updating SQLite in a Netatmo PR?

As much as possible, avoid changes to the core in PRs on the services :folded_hands:

8

Oh yes of course!! It’s a gross error, a handling error, certainly on my side after all, trying to go through wsl2 / ubuntu. It was not planned at all. I recovered the modified files, and @damalgos will do a reverse. I tried something on my side, but still struggling with github ^^

9

Oops, I wasn’t paying attention!

I’ll revert :smiley:

11

Anyone to start a small container :sweat_smile:

12

@VonOx

Hello :slight_smile:

Small issue on the netatmo branch, do you want to take a look? I removed the sqlite3 upgrade but it still doesn’t work :confused:

13

For information, I have launched a build of a dev image that includes:

  • Sequelize v6
  • Node 14
  • « Sunrise/sunset Â» trigger
  • @AlexTrovato’s bluetooth fix

I’m open to feedback once it’s ready!

( https://github.com/GladysAssistant/Gladys/actions/runs/667565020 )