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.

Sign up for GitHub

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

Jump to bottom

Add CSM for Authorization v1 to v2 migration guide #1265

Merged

alikdell merged 5 commits into main from pvt/auth-migration

Oct 4, 2024

Contributor

alikdell commented Oct 1, 2024 •

edited

Loading

Description

CSM for Authorization v2 is not backward compatible. This steps will guide existing CSM for Authorization v1 users how to migrate to v2.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
#dell/csm#1281

Checklist:

Have you run a grammar and spell checks against your submission?
Have you tested the changes locally?
Have you tested whether the hyperlinks are working properly?
Did you add the examples wherever applicable?
Have you added high-resolution images?


          Add CSM for Authorization v1 to v2 migration guide

b39d4b3

alikdell requested review from gallacher, mjsdell, prablr79, rajkumar-palani, AronAtDell and sharmilarama as code owners

October 1, 2024 15:59

github-actions bot commented Oct 1, 2024 •

edited

Loading

Test Results

77 tests ±0 77 ✅ ±0 3s ⏱️ ±0s
3 suites ±0 0 💤 ±0
1 files ±0 0 ❌ ±0

Results for commit 2aa5a42. ± Comparison against base commit bc1a64f.

♻️ This comment has been updated with latest results.

EvgenyUglov approved these changes

View reviewed changes

shaynafinocchiaro reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved


          pr review edits

c2503dc

shaynafinocchiaro reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Show resolved Hide resolved

content/docs/authorization/Migration guide from v1 to v2/_index.md Show resolved Hide resolved


          pr edits

9d2c4e6

shaynafinocchiaro approved these changes

View reviewed changes

atye approved these changes

View reviewed changes

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

atye reviewed

View reviewed changes

content/docs/authorization/Migration guide from v1 to v2/_index.md Outdated Show resolved Hide resolved

alikdell added 2 commits

October 3, 2024 15:03


          pr review edits

944518c


          pr review edits

2aa5a42

gallacher approved these changes

View reviewed changes

rajkumar-palani approved these changes

View reviewed changes

Collaborator

rajkumar-palani left a comment

lgtm

alikdell merged commit 2803643 into main

7 checks passed

alikdell deleted the pvt/auth-migration branch

October 4, 2024 13:01

coulof pushed a commit that referenced this pull request


          Add CSM for Authorization v1 to v2 migration guide (#1265)

cff2acc

* Add CSM for Authorization v1 to v2 migration guide

* pr review edits

* pr edits

* pr review edits

* pr review edits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

atye atye approved these changes

gallacher gallacher approved these changes

rajkumar-palani rajkumar-palani approved these changes

EvgenyUglov EvgenyUglov approved these changes

shaynafinocchiaro shaynafinocchiaro approved these changes

mjsdell Awaiting requested review from mjsdell mjsdell is a code owner

prablr79 Awaiting requested review from prablr79 prablr79 is a code owner

AronAtDell Awaiting requested review from AronAtDell AronAtDell is a code owner

sharmilarama Awaiting requested review from sharmilarama sharmilarama is a code owner

Labels

None yet