search

SUSE / SAPHanaSR

SAP HANA System Replication Resource Agent for Pacemaker Cluster
GNU General Public License v2.0
26 stars 21 forks source link

SAPHanaSR-angi 1.2.0 #194

Closed fmherschel closed 1 year ago

fmherschel commented 1 year ago

This new version implements

fmherschel commented 1 year ago

@angelabriel: Hmm der shellcheck auf die einzelnen libs hat etwas zu meckern, weil er nicht versteht, dass es libs sind und ggf die Variablen wo anders genutzt oder definiert werden. Da müssten wir zusammen den checker verbessern.

angelabriel commented 1 year ago

@angelabriel: Hmm der shellcheck auf die einzelnen libs hat etwas zu meckern, weil er nicht versteht, dass es libs sind und ggf die Variablen wo anders genutzt oder definiert werden. Da müssten wir zusammen den checker verbessern.

@fmherschel : Bei den letzten Versionen hast Du bei jeder Variable ein 'shellcheck disable=...' davor gesetzt. Wahrscheinlich hast Du das diesmal einfach vergessen? Für 'SC2034' könnten wir es uns einfacher machen und es in den 'Kopf' der Libraries schreiben. Dann wird auch der Code kürzer, weil nicht so viele 'shellcheck disable' drin stehen. Nachteil: Sollte tatsächlich mal eine unbenutzte Variable auftreten, merken wir es nicht, da der Check abgestellt ist. Ist jetzt eine Abwägung. Bei diesem Review gab es 2 Stellen, da waren es tatsächlich unbenutzte bzw. nicht gesetzte Variablen. Das würden wir dann übersehen - zumindest bei so großen/vielen Änderungen.

Aber für 'In ./ra/saphana-controller-common-lib line 415: SC2154' muß noch eine Zeile in den Code eingefügt werden. Merkwürdig, dass das erst jetzt kommt, obwohl Du dort nichts geändert hast.

fmherschel commented 1 year ago

@fmherschel : Bei den letzten Versionen hast Du bei jeder Variable ein 'shellcheck disable=...' davor gesetzt. Wahrscheinlich hast Du das diesmal einfach vergessen? Für 'SC2034' könnten wir es uns einfacher machen und es in den 'Kopf' der Libraries schreiben.

@angelabriel: Ich denke "unbenutzt" ist nicht so schlimm wir "undefined". Ersteres kann halt beim benutzen von libraries geschehen, wenn ein zusätzliches Programm die lib nutzt aber nicht alles braucht.