Fabian Deutsch has posted comments on this change. Change subject: Providing correct path to fingerprint keyes ......................................................................
Patch Set 2: Code-Review+2 (1 comment) https://gerrit.ovirt.org/#/c/44323/2/scripts/ovirt-init-functions.sh.in File scripts/ovirt-init-functions.sh.in: Line 1523: runcon -t unconfined_t "$handler" 2>&1 Line 1524: done Line 1525: fi Line 1526: Line 1527: ovirt_store_config $(ls /etc/ssh/ssh_host*_key*) > There is a loop inside ovirt_store_config So, aftzer investiagting that myself I see that we alreeady have a flow where a list is used in scripts/ovirt-functions.in - so ack from my side. However, my concerns are still there that we have several implementations (bash, py1, py2) implementations for this area (persistence) Line 1528: Line 1529: # FIXME is this still needed? Line 1530: if is_standalone; then Line 1531: return 0 -- To view, visit https://gerrit.ovirt.org/44323 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id8466ccabc143e41b1507d5020664a0a05e5c784 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <[email protected]> Gerrit-Reviewer: Anatoly Litovsky <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Fabian Deutsch <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ node-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/node-patches
