From Dan Kenigsberg <[email protected]>: Dan Kenigsberg has submitted this change and it was merged.
Change subject: vm: tests: add test for initial vcpu limit setup ...................................................................... vm: tests: add test for initial vcpu limit setup We update the vm vcpu limit both on startup - if SLA QOS policy is given and after policy updates received from clients (e.g. Engine). We had tests for the latter, not for the former cases. This patch adds the missing test. For the sake of the practicality, we take one shortcut and we test a private Vm method. This is bad, but not having tests it is even worse, as the related BZ demonstrates. Change-Id: I7ae776e4f2abee2d4716ec260d00954fdd0b4eac Related-To: https://bugzilla.redhat.com/1426727 Backport-To: 4.1 Signed-off-by: Francesco Romani <[email protected]> --- M tests/vm_test.py 1 file changed, 20 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/73198 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7ae776e4f2abee2d4716ec260d00954fdd0b4eac Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[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: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
