Skip to content

Conversation

@Frezzle
Copy link
Contributor

@Frezzle Frezzle commented Sep 19, 2016

Signed-off-by: Frezzle frederico.mazzone@citrix.com

…tabase"

Signed-off-by: Frezzle <frederico.mazzone@citrix.com>
@lindig
Copy link
Contributor

lindig commented Sep 20, 2016

Have you tested this? The code looks fine.

@Frezzle
Copy link
Contributor Author

Frezzle commented Sep 20, 2016

I've tested with running the command xe host-list params=uuid,ha-network-peers and for each host it shows the host UUIDs for the HA network they are in; so that seems correct.

I've ran /opt/xensource/debug/quicktest and I get 62/63 tests passed; this is the same for the fresh xapi install from trunk-ring3 as well as with my changes, so I assume my changes haven't caused this. Here is the last part of the message after running the quicktest:

Listing available Storage Repositories
Found 13 SRs
[ Success ]
ISO SR should be able to create VDIs for /opt/xensource/packages/is
Will plug into host xrtuk-02-02
Plugging PBD

Total tests started: 63; total passed: 62 (pass rate 98.41%)
Test neither succeeded nor failed: ISO SR should be able to create VDIs for /opt/xensource/packages/iso
*** Some tests failed ***
INTERNAL_ERROR: Storage_interface.Redirect(_)

@lindig lindig merged commit 15ab3b9 into xapi-project:master Sep 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants