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

IAM: Provider resource overhaul #2232

Merged
merged 7 commits into from Jul 20, 2023
Merged

IAM: Provider resource overhaul #2232

merged 7 commits into from Jul 20, 2023

Conversation

artem-lifshits
Copy link
Member

Summary of the Pull Request

Implement new resource opentelekomcloud_identity_provider to manage identity providers with SAML/OIDC support

PR Checklist

Acceptance Steps Performed

=== RUN   TestAccIdentityProviderBasic
--- PASS: TestAccIdentityProviderBasic (50.26s)
=== RUN   TestAccIdentityProviderOIDC
--- PASS: TestAccIdentityProviderOIDC (51.19s)
=== RUN   TestAccIdentityOIDCProvider_import
--- PASS: TestAccIdentityOIDCProvider_import (33.64s)
=== RUN   TestAccIdentitySamlProvider_import
--- PASS: TestAccIdentitySamlProvider_import (35.68s)
PASS

Process finished with the exit code 0

@artem-lifshits artem-lifshits marked this pull request as ready for review July 19, 2023 16:04
Copy link
Contributor

@vladimirvshivkov vladimirvshivkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks fine

@artem-lifshits artem-lifshits dismissed vladimirvshivkov’s stale review July 19, 2023 16:10

The merge-base changed after approval.

Polina-Gubina
Polina-Gubina previously approved these changes Jul 19, 2023
@artem-lifshits artem-lifshits dismissed stale reviews from vladimirvshivkov and Polina-Gubina July 19, 2023 16:20

The merge-base changed after approval.

Polina-Gubina
Polina-Gubina previously approved these changes Jul 19, 2023
@artem-lifshits artem-lifshits dismissed Polina-Gubina’s stale review July 19, 2023 16:25

The merge-base changed after approval.

Type: schema.TypeString,
Optional: true,
},
"access_config": {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

conflict with metadata

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

}

func getDefaultConversionOpts() *mappings.CreateOpts {
localRules := []mappings.LocalRuleOpts{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try to make rules editable

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@artem-lifshits artem-lifshits added the gate Merge PR label Jul 20, 2023
@otc-zuul
Copy link

otc-zuul bot commented Jul 20, 2023

Build succeeded.
https://zuul.otc-service.com/t/eco/buildset/918f20ade1ea4147be8ccf681227e99f

✔️ build-otc-releasenotes SUCCESS in 4m 26s
✔️ otc-golangci-lint SUCCESS in 2m 55s
✔️ golang-make-test SUCCESS in 4m 09s
✔️ golang-make-vet SUCCESS in 3m 40s
✔️ tflint SUCCESS in 2m 57s
✔️ goreleaser-build SUCCESS in 7m 38s

@otc-zuul otc-zuul bot merged commit 96b70d0 into devel Jul 20, 2023
3 checks passed
@otc-zuul otc-zuul bot deleted the iam_oidc branch July 20, 2023 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants