Skip to content
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

Incorrect System Information (Distribution) for Armbian #1385

Closed
halfmanbear opened this issue Mar 5, 2024 · 7 comments · Fixed by #1388
Closed

Incorrect System Information (Distribution) for Armbian #1385

halfmanbear opened this issue Mar 5, 2024 · 7 comments · Fixed by #1388
Assignees
Labels
GH - Bug Something isn't working Upstream - Moonraker For everything API related
Milestone

Comments

@halfmanbear
Copy link

Fluidd Version

v1.28.1

Browser

Chrome

Device

Laptop

Operating System

macOS

What happened

Distribution value incorrect from DistroInfo,
Screenshot 2024-03-05 at 12 12 58

What did you expect to happen

Screenshot 2024-03-05 at 12 22 28

How to reproduce

armbian-release files always start with # PLEASE DO NOT EDIT THIS FILE

mks@znp-k1:~$ cat /etc/armbian-release
# PLEASE DO NOT EDIT THIS FILE
BOARD=znp-k1
BOARD_NAME="ZNP-K1"
BOARDFAMILY=rockchip64
BUILD_REPOSITORY_URL=https://github.com/halfmanbear/Armbian-ZNP-K1-build.git
BUILD_REPOSITORY_COMMIT=9e04b40
LINUXFAMILY=rockchip64
ARCH=arm64
IMAGE_TYPE=user-built
BOARD_TYPE=conf
INITRD_ARCH=arm64
KERNEL_IMAGE_TYPE=Image
FORCE_BOOTSCRIPT_UPDATE=
VENDOR="Armbian-unofficial"
VENDORDOCS="https://docs.armbian.com/"
VENDORURL="https://duckduckgo.com/"
VENDORSUPPORT="https://community.armbian.com/"
VENDORBUGS="https://armbian.atlassian.net/"
BOOTSCRIPT_FORCE_UPDATE="no"
BOOTSCRIPT_DST="boot.cmd"
VERSION=24.2.0-trunk
REVISION=24.2.0-trunk
BRANCH=current
mks@znp-k1:~$ uname -a
Linux znp-k1 6.1.67-current-rockchip64 #2 SMP PREEMPT Mon Dec 11 09:41:01 GMT 2023 aarch64 GNU/Linux
mks@znp-k1:~$ sudo lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Armbian-unofficial 24.2.0-trunk bookworm
Release:	12
Codename:	bookworm

Additional information

mainsail-crew/mainsail#1021

@halfmanbear halfmanbear added GH - Bug Something isn't working GH - Evaluation Needed This topic needs to be discussed to evaluate it's aspects and feasability labels Mar 5, 2024
@pedrolamas
Copy link
Member

Hi @halfmanbear, thank you for opening this ticket.

Can you please check and confirm what version of Moonraker you have installed? It should be visible on Fluidd Settings, in the Updates section.

@pedrolamas pedrolamas added the GH - Waiting for Feedback Waiting for the issuer to reply label Mar 5, 2024
@halfmanbear
Copy link
Author

Yes I'm currently running v0.8.0-319-gf0ae5179

@pedrolamas
Copy link
Member

Yes I'm currently running v0.8.0-319-gf0ae5179

By any chance are you running a fork of Moonraker? I can't find that version on the official Moonraker repository...

I would expect the suffix gf0ae5179 to match a commit in https://github.com/Arksine/moonraker, but I can't see any matching

@matmen
Copy link
Member

matmen commented Mar 5, 2024

I think the g isn't supposed to be there (it's not a hex character) - I assume it's the latest commit Arksine/moonraker@f0ae5179

@pedrolamas pedrolamas removed GH - Waiting for Feedback Waiting for the issuer to reply GH - Evaluation Needed This topic needs to be discussed to evaluate it's aspects and feasability labels Mar 5, 2024
@pedrolamas
Copy link
Member

This is indeed a bug, though one could argue that the problem is actually upstream in Moonraker, or even on with the distro dependency that it then uses to retrieve the distribution values.

Having said that, I too experienced this issue in the past with the BTT CB1 and even submitted a fix to distro repository that was merged and released in version 1.9.0 - at the time this fix was enough for the CB1, but might not be the case with other Armbian releases!

@pedrolamas
Copy link
Member

FWIW, I've just pushed a pull request upstream to the distro repository to have this fixed directly there.

@pedrolamas pedrolamas added the Upstream - Moonraker For everything API related label Mar 6, 2024
@pedrolamas pedrolamas self-assigned this Mar 13, 2024
@pedrolamas pedrolamas added this to the 1.28.2 milestone Mar 13, 2024
@pedrolamas
Copy link
Member

I don't see this getting fixed upstream any time soon, so I've now pushed a fix on our side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GH - Bug Something isn't working Upstream - Moonraker For everything API related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants