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
Completely remove Kubeops #5256
Conversation
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
✅ Deploy Preview for kubeapps-dev canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I've just pointed out a tiny detail it would be nice to fix. Great work!!
dashboard/src/shared/Auth.test.ts
Outdated
@@ -279,7 +279,7 @@ describe("is403FromAuthProxy", () => { | |||
expect( | |||
Auth.is403FromAuthProxy({ | |||
status: 403, | |||
data: 'namespaces is forbidden: User "system:serviceaccount:kubeapps:kubeapps-internal-kubeops" cannot list resource "namespaces" in API group "" at the cluster scope', | |||
data: 'namespaces is forbidden: User "system:serviceaccount:kubeapps:kubeapps-internal-kubeapps-apis" cannot list resource "namespaces" in API group "" at the cluster scope', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems there's a typo in either this or the following editions in this file. Note that we are using kubeapps-apis
(with a hyphen) here, while we're using kubeappsapis
in the rest of the occurrences.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! It is a frontend test that actually checks if data points to a system:serviceaccount
, so what comes after that is not used. But I'll update it to be consistent. Thanks.
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
…pps into 4785-remove-kubeops-4 Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Description of the change
Completely remove Kubeops from the repo.
Only to be done after #5254
Benefits
Kubeapps repo is lighter, and less confusion on Kubeapps architecture.
Possible drawbacks
N/A
Applicable issues