openshift_connect reuse kubernetes_connect #7
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We've been keeping 2 nearly-same functions in sync, especially painful after repo split:
@zakiva @moolitayer Please review carefully.
options[:path]
, previously was forcing/oapi
.connect_client
still work right? It sets:service
which chooseskubernetes_connect
/openshift_connect
, but it also provides:path
,:version
which AFAICT covers both distinctions between them. I suspect you previously passed:service
here to work around forced/oapi
...api_version
method distinction from parent'skubernetes_version
(whichkubernetes/container_manager.rb
also aliases asapi_version
)? Both are de-factov1
so this is effectively dead code. Anyway that's for another PR.I'd like to see this backported, so that later changes touching only
kubernetes_connect
will be backportable without remembering to also changeopenshift_connect
...But my hunch is we don't want to backport right now (not a blocker).