Change in vdsm[master]: BZ#856167 - Validate downloaded CA certificate

2012-09-19 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: BZ#856167 - Validate downloaded CA certificate .. Patch Set 4: (4 inline comments) Commit Message Line 3:

Change in vdsm[master]: BZ#856167 - Validate downloaded CA certificate

2012-09-18 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: BZ#856167 - Validate downloaded CA certificate .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/8021 To unsubscribe, visit

Change in vdsm[master]: BZ#856167 - Validate downloaded CA certificate

2012-09-18 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: BZ#856167 - Validate downloaded CA certificate .. Patch Set 4: Juan, See[1]. In any update of production file, I prefer to to write it with $FILE.suffix, then

Change in vdsm[master]: BZ#856167 - Validate downloaded CA certificate

2012-09-18 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: BZ#856167 - Validate downloaded CA certificate .. Patch Set 4: I understand, but this patch doesn't change how the file is written, so I think that

Change in vdsm[master]: BZ#856167 - Validate downloaded CA certificate

2012-09-18 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: BZ#856167 - Validate downloaded CA certificate .. Patch Set 4: Looks good to me, but someone else must approve Fair enough, although if you wrote a file, the

Change in vdsm[master]: BZ#856167 - Validate downloaded CA certificate

2012-09-18 Thread dougsland
Douglas Schilling Landgraf has posted comments on this change. Change subject: BZ#856167 - Validate downloaded CA certificate .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit

Change in vdsm[master]: BZ#856167 - Validate downloaded CA certificate

2012-09-18 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: BZ#856167 - Validate downloaded CA certificate .. Patch Set 4: (4 inline comments) minor comments, and one annoying assertion about fingerprints.

Change in vdsm[master]: BZ#856167 - Validate downloaded CA certificate

2012-09-18 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: BZ#856167 - Validate downloaded CA certificate .. Patch Set 4: (1 inline comment) Commit Message Line 3: