Labelproblem in linbo

dorian · 24. Dezember 2020 um 00:20

Konnte es soeben unter LMN7 mit den neusten stable Paketen reproduzieren.

dorian · 24. Dezember 2020 um 00:25

Hmm… du meinst quasi eine Art Zusammenfassung der GitHub Issues auf eine Art und Weise, die auch für nicht - Entwickler gut einsehbar ist?

Klar, das ist sinnvoll, solange das Problem untersucht wird. Sobald aber klar ist, was das Problem ist, sollte es meiner Meinung nach trotzdem in einer GitHub Issue festgehalten werden, weil so verlieren es die Entwickler nicht aus dem Blick.
~~Ich glaube das ist auch der Grund, dass bei deiner Issue #136 nix mehr passiert ist: Du hast sie nicht wieder aufgemacht, also ging Thomas vermutlich davon aus, dass das Problem behoben ist.~~ (Will mich da nicht einmischen, sorry)

VG
Dorian

dorian · 24. Dezember 2020 um 00:27

Probiere ich gleich aus

EDIT:
Läuft problemlos durch.

thoschi · 24. Dezember 2020 um 00:39

Genau - für die Teilprojekte ist Github ja perfekt (sowohl für eine Entwicklerdoku als auch die Issues). Aber ich glaube, es fehlt da auf Projektebene oft und vielen der Überblick.

Also eine Art Projekt-Github (gibt es ja eigentlich schon) oder ein Ticketsystem mit Versionierung (redmine, …) für das Projekt. Wo ich einen guten Überblick über den Stand bekomme und was ausgehend von den ganzen Diskussionen gepflegt wird.

dorian · 24. Dezember 2020 um 00:39

Das wäre auf jeden Fall ein guter Anfang

dorian · 24. Dezember 2020 um 00:41

Ich hab jetzt mal eine Issue auf gemacht, auch wenn das #136 Problem noch da ist, ist es meiner Meinung nach ein anderes Thema

thoschi · 24. Dezember 2020 um 00:43

Hast Du bei Dir auf dem Client in /usr/bin/linbo_cmd die dd-Zeile:

dd if=/dev/zero of=$disk bs=512 count=1 || RC="1"

Dann wäre das Problem ja behoben (und eben einfach nicht mehr zur 6.2 runtergewandert).

dorian · 24. Dezember 2020 um 00:46

Jap, die habe ich.

thoschi · 24. Dezember 2020 um 00:48

Danke! Wäre dieser Teil geklärt

baumhof · 24. Dezember 2020 um 10:21

Hallo Thomas,

Dann wäre das Problem ja behoben (und eben einfach nicht mehr zur 6.2
runtergewandert).

das stimmt so nicht. Thomas hat die gleichen Versionen von linbo 2.3 für
die lmn7 und die lmn6.2 gemacht.
Bei der 6.2 ist es aber aus dem testing noch nicht raus, weil einfach
keine Leute das getestet haben (oder halt nicht zurückgemeldet haben).
Die 6.2 user würden sich ja auch nicht freuen, wenn die Pakete in der
6.2 ungetestet veröffentlicht werden mit dem HInweis: in der 7 läuft es.
Also: Testen, Rückmelden, und dann wandert das auch dort in stable.
Ich bin Thomas total Dankbar, dass er sich die Arbeit macht für die 6.2.
Normalerweise hab ich das halt getestet: aber ich hab jetzt halt seit
1,5 Jahren die 7 in der Schule und nicht die 6.2: also gibt es ein
„Testerloch“.

LG

Holger

dorian · 24. Dezember 2020 um 10:50

Hi zusammen,

Ich hab die Stellen gefunden, an denen die start.conf verändert wird:
Hier wird es angestoßen

github.com

linuxmuster/linuxmuster-linbo/blob/d5576d6f58d98804cb1b208725525ba802a723e6/linbofs/init.sh#L200


  local cachedev="$(printcache)"
  if [ -b "$cachedev" ]; then
    trycopyfromcache "$cachedev" "$1" && return 0
  fi
  # iterate through partitions
  local device=""
  ls -l /dev/disk/by-uuid/ | grep ^l | awk -F\/ '{ print $3 }' | sort -u | while read device; do
    [ -b "/dev/$device" ] || continue
    if trycopyfromcache "/dev/$device" "$1"; then
      if [ "$1" = "start.conf" ]; then
        # start.conf correction due to partition labels
        grep -qi ^label /start.conf && linbo_cmd update_devices
      fi
      return 0
    fi
  done
  return 1
}
# modify cache entry in start.conf
modify_cache(){

Und hier wird es dann durchgeführt:

github.com

linuxmuster/linuxmuster-linbo/blob/d5576d6f58d98804cb1b208725525ba802a723e6/linbofs/usr/bin/linbo_cmd#L468


    fi
  done
  # issue 85
  [ -n "$UDEVADM" ] && "$UDEVADM" trigger
  [ -e /dev/disk/by-label ] || echo "Keine Partitions-Label gefunden!" | tee -a /tmp/linbo.log
  sendlog
  return "$RC"
}
# update partition devices in start.conf according to labels
update_devices(){
  local line
  local dev
  local devnew
  local label
  local disk
  local disknew
  grep -i ^dev /start.conf | sort -u | while read line; do
    dev=""; devnew=""; label=""; disk=""; disknew=""
    dev="$(echo "$line" | awk -F\= '{ print $2 }' | awk '{ print $1 }')"
    label="$(print_partlabel "$dev")"

Ich verstehe aber immer noch nicht, WARUM das gemacht wird

VG
Dorian

thoschi · 24. Dezember 2020 um 13:22

Hallo Holger,

Dann werde ich das gerne mal testen (wobei ich lieber heute als morgen zur v7 wechseln würde.
Wobei ich vielleicht abwarte, bis das neue Issue, das Dorian erstellt hat, geschlossen wird. Da scheint es ja gewisse Überschneidungen bei den Problemen zu geben.

Viele Grüße
Thomas

dorian · 24. Dezember 2020 um 13:28

Hallo @thoschi

Thomas hat die einschlägigen Stellen entfernt, die Issue ist gelöst

VG
Dorian

thoschi · 24. Dezember 2020 um 13:30

Danke @dorian,

dann werde ich das nach den Feiertagen mal testen (lassen)

Viele Grüße
Thomas