Anatoly Litovsky has posted comments on this change.

Change subject: Providing correct path to fingerprint keyes
......................................................................


Patch Set 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*)
> I agree with Douglas, let's make a loop and then we are good - The issue is
There is a loop inside ovirt_store_config 
that goes on input parameters 
using for p in $@  ......

Why we want one more loop over the parameters,
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

Reply via email to