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

[BUG]: CSM Authorization - karavictl generate token should output valid yaml #767

Closed
atye opened this issue Apr 19, 2023 · 1 comment
Closed
Assignees
Labels
area/csm-authorization Issue pertains to the CSM Authorization module type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@atye
Copy link
Contributor

atye commented Apr 19, 2023

Bug Description

karavictl generate token should output yaml that can be directly applied as a k8s secret instead of requiring jq/sed for processing.

Logs

karavictl generate token --tenant Finance --insecure --addr <server>

{
  "Token": "\napiVersion: v1\nkind: Secret\nmetadata:\n  name: proxy-authz-tokens\ntype: Opaque\ndata:\n  access: ZXlKaGJHY2lPaUpJVXpJMU5pSXNJblI1Y0NJNklrcFhWQ0o5LmV5SmhkV1FpT2lKcllYSmhkbWtpTENKbGVIQWlPakUyTlRNek1qUXhPRFlzSW1keWIzVndJam9pWm05dklpd2lhWE56SWpvaVkyOXRMbVJsYkd3dWEyRnlZWFpwSWl3aWNtOXNaWE1pT2lKaVlYSWlMQ0p6ZFdJaU9pSnJZWEpoZG1rdGRHVnVZVzUwSW4wLmJIODN1TldmaHoxc1FVaDcweVlfMlF3N1NTVnEyRzRKeGlyVHFMWVlEMkU=\n  refresh: ZXlKaGJHY2lPaUpJVXpJMU5pSXNJblI1Y0NJNklrcFhWQ0o5LmV5SmhkV1FpT2lKcllYSmhkbWtpTENKbGVIQWlPakUyTlRVNU1UWXhNallzSW1keWIzVndJam9pWm05dklpd2lhWE56SWpvaVkyOXRMbVJsYkd3dWEyRnlZWFpwSWl3aWNtOXNaWE1pT2lKaVlYSWlMQ0p6ZFdJaU9pSnJZWEpoZG1rdGRHVnVZVzUwSW4wLkxNbWVUSkZlX2dveXR0V0lUUDc5QWVaTy1kdmN5SHAwNUwyNXAtUm9ZZnM=\n"
}

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

Generate a token with karavictl

Expected Behavior

Output that can be applied to k8s like so:

karavictl generate token --tenant Finance --addr <server> | kubectl -n <driver-namespace> apply -f -

CSM Driver(s)

Authorization supported drivers

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

All supported k8s versions

Operating System

NA

@atye atye added needs-triage Issue requires triage. type/bug Something isn't working. This is the default label associated with a bug issue. labels Apr 19, 2023
@csmbot
Copy link
Collaborator

csmbot commented Apr 19, 2023

@atye: Thank you for submitting this issue!

The issue is currently awaiting triage. Please make sure you have given us as much context as possible.

If the maintainers determine this is a relevant issue, they will remove the needs-triage label and assign an appropriate priority label.


We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at [email protected].

@atye atye self-assigned this Apr 19, 2023
@atye atye added area/csm-authorization Issue pertains to the CSM Authorization module and removed needs-triage Issue requires triage. labels Apr 19, 2023
@shaynafinocchiaro shaynafinocchiaro added this to the v1.7.0 milestone Apr 21, 2023
@atye atye closed this as completed Apr 24, 2023
@shaynafinocchiaro shaynafinocchiaro changed the title [BUG]: karavictl generate token should output valid yaml [BUG]: CSM Authorization - karavictl generate token should output valid yaml Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/csm-authorization Issue pertains to the CSM Authorization module type/bug Something isn't working. This is the default label associated with a bug issue.
Projects
None yet
Development

No branches or pull requests

3 participants