Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get a token using Keystone Application Credential #282

Merged
merged 1 commit into from Oct 8, 2018

Conversation

@zioproto
Copy link
Member

@zioproto zioproto commented Sep 6, 2018

What this PR does / why we need it:

This PR makes possibile to login using Keystone Application Credentials.
fixes #232

Special notes for your reviewer:

This PR requires this gophercloud PR to be merged first
gophercloud/gophercloud#1224

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 6, 2018

Build succeeded.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 6, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 6, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 6, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 6, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 6, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 6, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 6, 2018

@zioproto zioproto force-pushed the zioproto:application_credential branch 2 times, most recently from 70574c0 to b59d23d Sep 7, 2018
@zioproto
Copy link
Member Author

@zioproto zioproto commented Sep 7, 2018

@RdL87 make sure you test commit b59d23d . thanks

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

Build succeeded.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

Build failed.

  • cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.11 : RETRY_LIMIT in 9s
@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Sep 7, 2018

Build failed.

@zioproto
Copy link
Member Author

@zioproto zioproto commented Sep 18, 2018

The gophercloud patch was merged upstream:
gophercloud/gophercloud#1224

It is now possible to review ! thanks

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 6, 2018

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 6, 2018

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 6, 2018

@dims
Copy link
Member

@dims dims commented Oct 8, 2018

/test all

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

Build succeeded.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

Build succeeded.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

Build succeeded.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

@dims
Copy link
Member

@dims dims commented Oct 8, 2018

/test pull-cloud-provider-openstack-test

@dims
Copy link
Member

@dims dims commented Oct 8, 2018

/test pull-cloud-provider-openstack-check

@dims
Copy link
Member

@dims dims commented Oct 8, 2018

/test cloud-provider-openstack-unittest

@dims
Copy link
Member

@dims dims commented Oct 8, 2018

/test cloud-provider-openstack-format

@k8s-ci-robot
Copy link
Contributor

@k8s-ci-robot k8s-ci-robot commented Oct 8, 2018

@zioproto: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
pull-cloud-provider-openstack-check 9d7c98d link /test pull-cloud-provider-openstack-check
pull-cloud-provider-openstack-test 9d7c98d link /test pull-cloud-provider-openstack-test

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

Build failed.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

Build succeeded.

@theopenlab-ci
Copy link

@theopenlab-ci theopenlab-ci bot commented Oct 8, 2018

@dims
Copy link
Member

@dims dims commented Oct 8, 2018

/override openlab/cloud-provider-openstack-format
/override openlab/cloud-provider-openstack-unittest
/override pull-cloud-provider-openstack-check
/override pull-cloud-provider-openstack-test

@k8s-ci-robot
Copy link
Contributor

@k8s-ci-robot k8s-ci-robot commented Oct 8, 2018

@dims: Overrode contexts on behalf of dims: openlab/cloud-provider-openstack-format, openlab/cloud-provider-openstack-unittest, pull-cloud-provider-openstack-check, pull-cloud-provider-openstack-test

In response to this:

/override openlab/cloud-provider-openstack-format
/override openlab/cloud-provider-openstack-unittest
/override pull-cloud-provider-openstack-check
/override pull-cloud-provider-openstack-test

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@dims
Copy link
Member

@dims dims commented Oct 8, 2018

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm label Oct 8, 2018
@k8s-ci-robot k8s-ci-robot merged commit 42a9b45 into kubernetes:master Oct 8, 2018
16 checks passed
16 checks passed
cla/linuxfoundation zioproto authorized
Details
openlab/cloud-provider-openstack-acceptance-test-csi-cinder cloud-provider-openstack-acceptance-test-csi-cinder status: success
Details
openlab/cloud-provider-openstack-acceptance-test-e2e-conformance cloud-provider-openstack-acceptance-test-e2e-conformance status: success
Details
openlab/cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.10 cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.10 status: success
Details
openlab/cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.11 cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.11 status: success
Details
openlab/cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.12 cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.12 status: success
Details
openlab/cloud-provider-openstack-acceptance-test-flexvolume-cinder cloud-provider-openstack-acceptance-test-flexvolume-cinder status: success
Details
openlab/cloud-provider-openstack-acceptance-test-k8s-cinder cloud-provider-openstack-acceptance-test-k8s-cinder status: success
Details
openlab/cloud-provider-openstack-acceptance-test-keystone-authentication-authorization cloud-provider-openstack-acceptance-test-keystone-authentication-authorization status: success
Details
openlab/cloud-provider-openstack-acceptance-test-lb-octavia cloud-provider-openstack-acceptance-test-lb-octavia status: success
Details
openlab/cloud-provider-openstack-acceptance-test-standalone-cinder cloud-provider-openstack-acceptance-test-standalone-cinder status: success
Details
openlab/cloud-provider-openstack-format Overridden by dims
Details
openlab/cloud-provider-openstack-unittest Overridden by dims
Details
pull-cloud-provider-openstack-check Overridden by dims
Details
pull-cloud-provider-openstack-test Overridden by dims
Details
tide In merge pool.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

5 participants
You can’t perform that action at this time.