Project

General

Profile

Actions

Bug #5374

closed

Avoid relying on keystone's admin token

Added by Soren Hansen almost 11 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Normal
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The current Keystone integration requires knowledge of the keystone admin token. The keystone admin token is for Keystone's own use and for administrative use to create the initial, genuine Keystone users.

The correct way to integrate with Keystone is to create a user for Ceph to allow it to authenticate tokens. When starting up, the username/tenant/password is used to get a token from Keystone. This token is the used instead of the admin token.

Actions #1

Updated by Christophe Courtaut almost 11 years ago

A pull request has been made for this issue https://github.com/ceph/ceph/pull/392
Might need testing.

Actions #2

Updated by Christophe Courtaut almost 11 years ago

The patch above needs review.
Please change ticket status.

Actions #3

Updated by Christophe Courtaut almost 11 years ago

This issue need update has a fix has been merged.

Actions #4

Updated by Yehuda Sadeh almost 11 years ago

actually, it's not merged yet. I still need to complete the review on it, hopefully I can get that done soon.

Actions #5

Updated by Ian Colle over 10 years ago

wip-5374

Actions #6

Updated by Ian Colle over 10 years ago

  • Status changed from New to Fix Under Review
  • Assignee set to Yehuda Sadeh
Actions #7

Updated by Yehuda Sadeh over 10 years ago

I rebased wip-5374 again, went over it. Almost there but not quite yet, will get there soon.

Actions #8

Updated by Sage Weil over 10 years ago

  • Assignee changed from Yehuda Sadeh to Christophe Courtaut
Actions #9

Updated by Christophe Courtaut over 10 years ago

What is missing to close this issue?

Actions #10

Updated by Yehuda Sadeh over 10 years ago

Mainly needs to be brought up to date with current code base.

Actions #11

Updated by Yehuda Sadeh over 10 years ago

  • Status changed from Fix Under Review to Resolved

Merged, commit:b20d1bf33bdf6ed25ef9bb37afc3890282ece6d4

Actions

Also available in: Atom PDF