Skip to content

fix(rbac): implement conditional aliases #1847

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

Merged

Conversation

AndrienkoAleksandr
Copy link
Collaborator

@AndrienkoAleksandr AndrienkoAleksandr commented Jun 27, 2024

What doest this pull request do:

Implement conditional aliases

What does this pull request fix:

https://issues.redhat.com/browse/RHIDP-1765

Demo

(https://youtu.be/XyAf6K28ISE)

Copy link

openshift-ci bot commented Jun 27, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from andrienkoaleksandr. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link
Member

@PatAKnight PatAKnight left a comment

Choose a reason for hiding this comment

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

Tested this and it works as expected. Looking pretty good to me so far, can you also add some documentation for this new feature?

@AndrienkoAleksandr AndrienkoAleksandr force-pushed the implement-conditional-aliases branch from 60aee15 to 3dbd7c0 Compare July 31, 2024 09:22
@AndrienkoAleksandr AndrienkoAleksandr force-pushed the implement-conditional-aliases branch 3 times, most recently from 56b1a1e to b4e75ad Compare August 1, 2024 09:01
@AndrienkoAleksandr
Copy link
Collaborator Author

I updated demo to count changes.

@AndrienkoAleksandr AndrienkoAleksandr force-pushed the implement-conditional-aliases branch 2 times, most recently from bbc63d9 to 91ba830 Compare August 2, 2024 10:01
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Copy link

sonarqubecloud bot commented Aug 6, 2024

Copy link
Member

@PatAKnight PatAKnight left a comment

Choose a reason for hiding this comment

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

lgtm

@openshift-ci openshift-ci bot added the lgtm label Aug 6, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit dbc9a0b into janus-idp:main Aug 6, 2024
9 checks passed
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## 1.0.0 (2024-08-06)

### Bug Fixes

* **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## @janus-idp/backstage-plugin-rbac-common [1.8.2](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Bug Fixes

* **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## [1.17.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([#1950](#1950)) ([a1b790a](a1b790a))
* **rbac:** nested condition ([#1814](#1814)) ([228f1a9](228f1a9))

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([#2012](#2012)) ([97c9667](97c9667))
* **acr:** revert downgrade of shared-react and janus-cli ([#2009](#2009)) ([8c28891](8c28891))
* **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## [1.27.1](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([#2012](#2012)) ([97c9667](97c9667))
* **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## [4.7.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([#1950](#1950)) ([a1b790a](a1b790a))
* **rbac:** nested condition ([#1814](#1814)) ([228f1a9](228f1a9))

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([#2012](#2012)) ([97c9667](97c9667))
* **acr:** revert downgrade of shared-react and janus-cli ([#2009](#2009)) ([8c28891](8c28891))
* **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
* fix(rbac): implement conditional aliases

Signed-off-by: Oleksandr Andriienko <[email protected]>

* fix(rbac): fix build

Signed-off-by: Oleksandr Andriienko <[email protected]>

* fix(rbac): rework currentUser alias to cover ownershipEntityRefs

Signed-off-by: Oleksandr Andriienko <[email protected]>

* fix(rbac): add more unit tests

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): make feature working after backstage update to 1.29.2

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): fix compilation and tests

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): Support two aliases: currentUser and ownerRefs

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): add doc about conditional policies aliases

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): clean up unit test

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): clean up

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): clean up

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): fix dependencies

Signed-off-by: Oleksandr Andriienko <[email protected]>

* feat(rbac): update yarn.lock

Signed-off-by: Oleksandr Andriienko <[email protected]>

---------

Signed-off-by: Oleksandr Andriienko <[email protected]>
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
## 1.0.0 (2024-08-06)

### Bug Fixes

* **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
## [1.17.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([janus-idp#1950](janus-idp#1950)) ([a1b790a](janus-idp@a1b790a))
* **rbac:** nested condition ([janus-idp#1814](janus-idp#1814)) ([228f1a9](janus-idp@228f1a9))

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([janus-idp#2012](janus-idp#2012)) ([97c9667](janus-idp@97c9667))
* **acr:** revert downgrade of shared-react and janus-cli ([janus-idp#2009](janus-idp#2009)) ([8c28891](janus-idp@8c28891))
* **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
## [4.7.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([janus-idp#1950](janus-idp#1950)) ([a1b790a](janus-idp@a1b790a))
* **rbac:** nested condition ([janus-idp#1814](janus-idp#1814)) ([228f1a9](janus-idp@228f1a9))

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([janus-idp#2012](janus-idp#2012)) ([97c9667](janus-idp@97c9667))
* **acr:** revert downgrade of shared-react and janus-cli ([janus-idp#2009](janus-idp#2009)) ([8c28891](janus-idp@8c28891))
* **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants