12107 Commits

Author SHA1 Message Date
zlyqqq
d0adf7d1d3 Reorganize api-ref: v3-ext federation auth
Change-Id: I1b904be87377669e5e725d093c0a329c34b8e4ea
2017-10-12 01:42:48 +08:00
Jenkins
fe87d28422 Merge "Confusing log messages in project hierarchy checking" 2017-09-05 01:58:54 +00:00
zlyqqq
b5c3dec60f Confusing log messages in project hierarchy checking
When is_domain is false, the project must contain a valid
domain_id and parent_id. We check the domain_id specified
is consistent with the domain of its parent.
The log messages is not very clear, this patch optimize it.

Change-Id: Ie83d295f036f94d36c9dfa88ae03e074cdfd9810
2017-09-04 08:36:13 +00:00
Jenkins
e0a8780b63 Merge "Copy specific distro pages for install guide" 2017-08-30 02:39:50 +00:00
Gage Hugo
04515333f4 Copy specific distro pages for install guide
This change breaks the get-started-identity, keystone-users,
and keystone-openrc pages into multiple copies for each distribution
release in the keystone install guides. Before this change, sphinx
would not render the links correctly and these pages would be skipped
in both the RDO and Ubuntu guides.

To avoid unneccessary duplication of the same text, the files were
moved into common/ and are simply included into the newly created
files for each release.

Change-Id: I7b478504f1e68034ea993c0214705045da86bb6e
Closes-Bug: #1712335
2017-08-29 16:27:52 -05:00
Jenkins
746a2ee2fa Merge "Delete redundant code" 2017-08-28 21:27:48 +00:00
Jenkins
888574b85e Merge "Log format error" 2017-08-28 17:12:15 +00:00
OpenStack Proposal Bot
785d8fe405 Imported Translations from Zanata
For more information about this automatic import see:
https://docs.openstack.org/i18n/latest/reviewing-translation-import.html

Change-Id: I8810af7b31edfb67d3b0b92c4bef5bb5fa64248d
2017-08-26 08:09:42 +00:00
Jenkins
83d3d53268 Merge "Clarify documentation for release notes" 2017-08-25 18:57:31 +00:00
Jenkins
6b6e1a7322 Merge "Call methods with kwargs instead of positionals" 2017-08-25 16:56:09 +00:00
chenaidong1
296429fe50 Log format error
%(rule)r  should be %(type)r

Change-Id: I2da1e44c7a278d4db56e840cca9132c6df107fcf
2017-08-25 06:49:07 +00:00
OpenStack Proposal Bot
a1f19c73b6 Updated from global requirements
Change-Id: I1e451ebff88a14928fe7d4014a837e11dfb91f6d
2017-08-24 05:50:44 +00:00
Lance Bragstad
2373cfbbf9 Ignore release notes for pike and master
These notes were modified [0] and reverted [1] during the Pike
cycle. This commit ensures they don't get rendered for the Pike
release:

[0] https://review.openstack.org/#/c/493479/
[1] https://review.openstack.org/#/c/496367/

Change-Id: I04395b469bff4ca27825d589f1b85637fa09e69f
2017-08-23 21:45:53 +00:00
Lance Bragstad
428cec4a20 Clarify documentation for release notes
This was discussed in today's keystone meeting. It was decided that
there should be some more clarification around release notes and how
to update them.

Change-Id: I4a06fda76dc8589a755438ccc12bc0bf2f575dcb
2017-08-23 13:39:45 +00:00
Jenkins
4950c24bdd Merge "Revert "Fix wrong links"" 2017-08-23 00:03:23 +00:00
Jenkins
3e8f16dec4 Merge "Remove missing release note from previous revert" 2017-08-22 20:59:29 +00:00
Lance Bragstad
6a20aa8587 Revert "Fix wrong links"
This reverts commit 77500b3615ae94ea45837f3fc0d503c8aadcc462.

Change-Id: I44a3f47329b06d4b85fa0bb944ce3bc8084fffa3
2017-08-22 18:54:25 +00:00
Gage Hugo
94e3e9803d Remove missing release note from previous revert
This change removes a release note that was added as part of [0] but
the change was later reverted [1], however the note was missed
as part of the revert.

[0] https://review.openstack.org/#/c/438035/
[1] https://review.openstack.org/#/c/464009/

Change-Id: I4a9041ad1c1d12a328f8c5951b5a7df8c0d7e390
2017-08-22 12:56:28 -05:00
Lance Bragstad
d1562fbccb Include a link in release note for bug 1698900
This was merged without a link to the bug that it fixed. This should
be backported to stable/pike so that it's rendered properly in the
release notes.

Change-Id: I7230b48ba232f3f6807689f82efda4a010924d1c
2017-08-22 17:14:23 +00:00
chenaidong1
faec97fb2b Delete redundant code
_trustor_trustee_only has been called in function
TrustV3::get_trust. It is not necessary to be called again.

Change-Id: Ice93b826fef4616801fd02bcf8175b7c8dc11839
2017-08-22 02:22:44 +00:00
Lance Bragstad
c025cb3c16 Call methods with kwargs instead of positionals
If a method is defined with kwargs it should be called with
kwargs. Calling a method that has specified kwargs with positional
arguments is fragile and should be avoided. It leaves the caller
open to breakages if the method signature of the callee ever changes.
If the callee's signature changes, it requires the callers to be
updated, even if they aren't using the new functionality or if the
argument doesn't apply to them.

This change applies that practice to parts of the assignment API.
This will make it easier to reuse the existing assignment driver to
implement global role assignments.

bp global-roles

Change-Id: I0a228ec25cd88f456cacd5824082f1b322e0262d
2017-08-17 13:49:40 +00:00
Jenkins
682cfa5c6d Merge "Remove duplicate roles from federated auth" 2017-08-16 21:04:36 +00:00
Jenkins
d20a3e971f Merge "Resource backend is SQL only now" 2017-08-16 19:55:53 +00:00
Jenkins
c9a4a46754 Merge "Add the step to create a domain" 2017-08-16 16:35:44 +00:00
Lance Bragstad
058a23c087 Remove duplicate roles from federated auth
We were using a one-liner to prune duplicate role references from a
list of roles, but it didn't work in all cases. This reworks the
logic to pass the existing test case. I also added a comment
explaining why the logic we used previously doesn't work so we can
hopefully avoid the pattern in the future.

Change-Id: Id786d6463364ad8f4f02c22bb83221baac4b83d0
Closes-Bug: 1701324
2017-08-16 15:20:58 +00:00
zhiguo.li
df03cb25d3 Add the step to create a domain
Two reasons for adding this step:
1. The title of this chapter named “Create a domain, projects, users,
   and roles”,but there is no step to introduce how to create a
   domain.
2. Explain to users where the domain 'default' comes from because the
   default domain is used in later steps.

Change-Id: I5b627ce15f94f18fd538866a32b84e540090bfff
Closes-Bug: 1707130
2017-08-16 12:21:26 +00:00
Jenkins
538b9a2063 Merge "Add int storage of datetime for password created/expires" 2017-08-16 04:30:25 +00:00
Jenkins
7e9645b811 Merge "Remove deprecation of domain_config_upload" 2017-08-16 01:06:45 +00:00
Morgan Fainberg
38974af24c Add int storage of datetime for password created/expires
Due to MySQL (in some versions) not storing datetime resolution below
one second, keystone occasionally ends up with weird behavior such as
a New password not being valid. The password created at and expires at
columns now store both datetime (for rolling upgrades) and integers.

Keystone from Pike and beyond leans on the new created_at_int column
and expires_at_int column.

Change-Id: I2c219b4b9b353f1e2cce6088849a773196f0e443
Closes-Bug: #1702211
2017-08-15 16:29:18 +00:00
Jenkins
2164d0550c Merge "Imported Translations from Zanata" 2017-08-14 21:57:52 +00:00
Morgan Fainberg
3d46c8a5d9 Resource backend is SQL only now
This change has been implemented to avoid the need to backport
signficantly impactful Foreign Key dropping backports.

Resource is highly relational data and it makes sense to allow the
use of FKs from other subsystems to project/domains.

Change-Id: Ic3831d1c7ae41fe4d406d60a013770cc1258584f
2017-08-14 19:01:02 +00:00
yfzhao
77500b3615 Fix wrong links
Some docs links have changed. We should update the wrong links in our codes.

Change-Id: I54587d1ca9a3b1628fc5437ca49b468a4e4107bc
Closes-Bug: #1710572
2017-08-14 16:26:42 +08:00
OpenStack Proposal Bot
5fbe54054a Imported Translations from Zanata
For more information about this automatic import see:
http://docs.openstack.org/developer/i18n/reviewing-translation-import.html

Change-Id: Ia1144e7677067832875cc46b8b27216d90d3813a
2017-08-12 07:47:36 +00:00
Jenkins
99b3641fa0 Merge "Fix typo in index documentation" 2017-08-11 20:14:35 +00:00
Jenkins
fe20e5a9a7 Merge "Update reno for stable/pike" 2017-08-11 18:29:05 +00:00
Jenkins
6621c78107 Merge "Updated URLs in docs" 2017-08-11 15:49:12 +00:00
Jenkins
833f1047ab Merge "Update docs: fernet is the default provider" 2017-08-11 15:41:02 +00:00
Lance Bragstad
ad094a6184 Remove deprecation of domain_config_upload
This was the outcome of a discussion on the mailing list:

  http://lists.openstack.org/pipermail/openstack-dev/2017-June/119090.html

Change-Id: I835e4a722855756035725fa7d4524c1fe641ffcf
2017-08-11 02:58:26 +00:00
f57a318e5c Update reno for stable/pike
Change-Id: I0f18c1e309dbde610a13b1f4299337a9ae7b94ae
2017-08-11 02:50:55 +00:00
Jenkins
e45e771703 Merge "Unset project ids for all identity backends" 2017-08-11 01:52:33 +00:00
Jenkins
6a1d5b0e4d Merge "Cache list projects and domains for user" 2017-08-10 21:01:22 +00:00
Lance Bragstad
d0ad287df3 Unset project ids for all identity backends
Previously, the default behavior for the callback that unset
default project ids was to only call the method for the default
domain's identity driver. This meant that when a project was deleted,
only the default identity backend would have references to that
project removed. This means it would be possible for other identity
backends to still have references to a project that doesn't exist
because the callback wasn't invoked for that specific backend.

This commit ensures each backend clears project id from a user's
default_project_id attribute when a project is deleted.

Change-Id: Ibb5396f20101a3956fa91d6ff68155d4c00ab0f9
Closes-Bug: 1705072
2017-08-10 19:35:51 +00:00
Samuel de Medeiros Queiroz
455a21e63d Update docs: fernet is the default provider
Co-Authored-By: Samriddhi Jain <j.samriddhi13@gmail.com>
Change-Id: Ia595170b5850be9f0ca9cf8dbbae6d5c2dca78c4
2017-08-10 21:01:59 +05:30
Jenkins
785c114e7a Merge "Add description for relationship links in api-ref" 2017-08-10 10:49:31 +00:00
Jenkins
d5ed4a7c04 Merge "Remove unused hints from assignment APIs" 2017-08-10 02:28:45 +00:00
Gage Hugo
6c8ea57210 Add description for relationship links in api-ref
This adds a section within the index file that describes what a
relationship link is and what it is used for in terms of each
operation within keystone. There will be a relationships section
in both v3 and v3-ext.

This should help clarify any confusion that may arise when a user is
viewing the api-ref about what the relationship links are.

Change-Id: I9c6b7959ed6ea682c565c515af0cf509b6a64e5d
Closes-Bug: #1674676
2017-08-09 20:59:55 +00:00
Samriddhi Jain
82785553dd Updated URLs in docs
With the docs migration and re-arrangement, some docs
have been shifted to new places, however the referenced
links are still old. Some of them give 404 error or
just point to the home page. This patch fixes those URLs.

Change-Id: Ie6b18ab3d4aa346dac8436dd426277fee4f07fcd
2017-08-09 23:46:33 +05:30
Lance Bragstad
63124f703a Cache list projects and domains for user
Listing projects and domains for a user based on their role
assignments was noted as being really slow, especially when users
have a lot of assignments. This commit implements caching to mitigate
the issue while we continue to investigate ways to speed up the
assignment API.

Change-Id: I72e398c65f01aa4f9a37f817d184a13ed01089ce
Closes-Bug: 1700852
2017-08-09 14:45:58 +00:00
Jenkins
6167850d12 Merge "Consolidate certificate docs to admin-guide" 2017-08-09 14:12:09 +00:00
Lance Bragstad
9fccd38d1b Remove unused hints from assignment APIs
The controller is responsible for listing user projects based on role
assignments and would build a hints objects and pass it to the
assignment manager. This is a common pattern used across keystone's
APIs. But, the assignment API never actually passed the hints objects
to the backend implementation.

This commit removes the hints from being passed to the manager for
list_projects_for_user and list_domains_for_user because those
APIs never use the hints object. This should allow us to implement
caching to speed up those calls later.

Change-Id: I9b1c8c30ca6a78dd6e78add7de278e467ceea046
Related-Bug: 1700852
2017-08-09 14:10:45 +00:00