Sandro Bonazzola has posted comments on this change.

Change subject: bin: add the host to a specific cluster
......................................................................


Patch Set 13:

(2 comments)

http://gerrit.ovirt.org/#/c/25456/13/src/plugins/ovirt-hosted-engine-setup/engine/add_host.py
File src/plugins/ovirt-hosted-engine-setup/engine/add_host.py:

Line 448:             )
Line 449:         except ovirtsdk.infrastructure.errors.RequestError as e:
Line 450:             self.logger.debug(
Line 451:                 'Cannot add the host to cluster {cluster}'.format(
Line 452:                     cluster_name),
> Minor
Thanks!
Line 453:                 exc_info=True,
Line 454:             )
Line 455:             self.logger.error(
Line 456:                 _(


Line 488:                 cluster.update()
Line 489:             except ovirtsdk.infrastructure.errors.RequestError as e:
Line 490:                 self.logger.debug(
Line 491:                     'Cannot set CPU level of cluster {cluster}',
Line 492:                     'cluster'.format(cluster_name),
> cluster=cluster_name
Thanks!
Line 493:                     exc_info=True,
Line 494:                 )
Line 495:                 self.logger.error(
Line 496:                     _(


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I00167e70296645e5cf732b64d87efa1cf66ca783
Gerrit-PatchSet: 13
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Meital Bourvine <mbour...@redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knese...@redhat.com>
Gerrit-Reviewer: Lev Veyde <lve...@gmail.com>
Gerrit-Reviewer: Meital bourvine <meitalbourv...@gmail.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stira...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@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