From Dan Kenigsberg <[email protected]>:

Dan Kenigsberg has posted comments on this change.

Change subject: caps: Report hosted engine deployment status
......................................................................


Patch Set 5:

(1 comment)

https://gerrit.ovirt.org/#/c/67706/5/lib/vdsm/host/caps.py
File lib/vdsm/host/caps.py:

Line 258:     try:
Line 259:         is_deployed = client.is_deployed
Line 260:     except AttributeError:
Line 261:         logging.warning("The installed version of hosted engine 
doesn't "
Line 262:                         "support the checking of deployment status.")
> Yes, we will release fully compatible hosted engine packages in 4.1 so this
which version are these? Can we have spec-side

Conflicts: ovirt-hosted-engine < X.Y.Z

instead of this check?
Line 263:         return False
Line 264: 


-- 
To view, visit https://gerrit.ovirt.org/67706
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iff5baaf6412e6e318ee83e8f6c9d14eec731ecac
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Phillip Bailey <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Phillip Bailey <[email protected]>
Gerrit-Reviewer: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to