Skip to content

Allow Passing in Full API URL to Rancher Driver#958

Closed
mmellison wants to merge 1 commit intoapache:trunkfrom
arroyonetworks:rancher-fixes
Closed

Allow Passing in Full API URL to Rancher Driver#958
mmellison wants to merge 1 commit intoapache:trunkfrom
arroyonetworks:rancher-fixes

Conversation

@mmellison
Copy link
Contributor

Rancher Driver: Allow Passing in Full API URL

Description

The scheme (secure) and port no longer need to be explicitly specified, allowing a user to simply copy in the string provided to them from Rancher.

Status

Done, ready for review

Checklist (tick everything that applies)

  • Code linting (required, can be done after the PR checks)
  • Documentation
  • Tests
  • ICLA (required for bigger changes)

Ping @InAnimaTe

The scheme (secure) and port no longer need to be explicitly specified.
@tonybaloney
Copy link
Contributor

Changes LGTM. @InAnimaTe ?

@InAnimaTe
Copy link
Contributor

LGTM!

@tonybaloney
Copy link
Contributor

great. merging. ☃️

@asfgit asfgit closed this in 88a5954 Dec 27, 2016
asfgit pushed a commit that referenced this pull request Dec 27, 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.

3 participants