Skip to content

Conversation

@koushikcgit
Copy link

The login_with_password XenAPI call takes an "originator"
string as its fourth parameter.

If a client does this, then it gets its own pool of xapi sessions.
Moreover it will not have its xapi sessions destroyed prematurely
as a result of some other misbehaving client that keeps creating
sessions and not logging out of them.

This patch adds the "originator" to all invokes of
login_with_password.

Signed-off-by: Koushik Chakravarty koushik.chakravarty@citrix.com

@xen-git
Copy link
Contributor

xen-git commented Jun 11, 2015

Can one of the admins verify this patch for testing?

@koushikcgit
Copy link
Author

This PR depends on xapi-project/xen-api-client#29

@thomassa
Copy link

I think you can add the version string from the Version module which is created by the Makefile:
~originator:("xcp-networkd v" ^ Version.version)

@thomassa
Copy link

@xen-git Test this please

The login_with_password XenAPI call takes an "originator"
string as its fourth parameter.

If a client does this, then it gets its own pool of xapi sessions.
Moreover it will not have its xapi sessions destroyed prematurely
as a result of some other misbehaving client that keeps creating
sessions and not logging out of them.

This patch adds the "originator" to all invokes of
login_with_password.

Signed-off-by: Koushik Chakravarty <koushik.chakravarty@citrix.com>
@thomassa
Copy link

In it goes!

thomassa added a commit that referenced this pull request Jun 16, 2015
CP-9895: Added originator for xapi login_with_password
@thomassa thomassa merged commit bf8ccd9 into xapi-project:master Jun 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants