python-hpOneView icon indicating copy to clipboard operation
python-hpOneView copied to clipboard

Published 20 hours ago • verified publisher icon HewlettPackard

Allow connection reuse

Open eamonnotoole opened this issue 7 years ago • 4 comments

We have to use the SDK in environments were the total number of connections is restricted, where repeatedly opening and closing connections can cause instability, and with fairly large latencies in opening a new connection. To aid stability and SDK performance in these environments we allow for the reuse of https connections as returned by get_connection(). Connections will be reused when 'reuse_connection' is set to True or when the ONEVIEWSDK_REUSE_CONNECTION is anything other than an empty string. We've added a unit-test for this new capability.

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve. e.g., Fixes #01]

Check List

  • [ ] New functionality includes testing.
    • [ ] All tests pass for Python 2.7+ & 3.4+($ tox).
  • [ ] New functionality has been documented in the README if applicable.
    • [ ] New functionality has been thoroughly documented in the examples (please include helpful comments).
    • [ ] New endpoints supported are updated in the endpoints-support.md file.
  • [ ] Changes are documented in the CHANGELOG.

eamonnotoole avatar Feb 09 '18 13:02 eamonnotoole

Can we add this flag-reuse_connection in the sample config file i.e examples/config-rename.json as well?

soodpr avatar Feb 14 '18 07:02 soodpr

Thanks Priyanka - change to examples/config-rename.json added

eamonnotoole avatar Feb 14 '18 15:02 eamonnotoole

@soodpr I'm having difficulty pushing up the change - networking issues on our side - so I'll try again tomorrow AM (my time) when hopefully the problems will be resolved.

eamonnotoole avatar Feb 14 '18 17:02 eamonnotoole

@soodpr Change pushed up.

eamonnotoole avatar Feb 15 '18 09:02 eamonnotoole