Alon Bar-Lev has posted comments on this change.

Change subject: engine: Import single certificate
......................................................................


Patch Set 21:

(1 comment)

http://gerrit.ovirt.org/#/c/35485/21/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/CertificateMapper.java
File 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/CertificateMapper.java:

Line 23:     @Mapping(from = CertificateInfo.class, to = Certificate.class)
Line 24:     public static Certificate map(CertificateInfo entity, Certificate 
template) {
Line 25:         Certificate model = template != null ? template : new 
Certificate();
Line 26:         model.setContent(entity.getPayload());
Line 27:         model.setSubject(entity.getDescription());
> So please define for now what description should be.
description should be:

Certificate with subject name %s issued by %s with %s sha1 fingerprint.

I can write the code.
Line 28:         return model;
Line 29:     }


-- 
To view, visit http://gerrit.ovirt.org/35485
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic9adb21ded6e6d9fb09fc68331872c1cd88f88a9
Gerrit-PatchSet: 21
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gsher...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to