[BUG]: Using Authorization without dependency on jq

[tdawe]

Bug Description

CSM Authorization documentation needs to be updated to show token generation steps without using jq. Remove references to using jq.

Logs

n/a

Screenshots

No response

Additional Environment Information

No response

Steps to Reproduce

Use Authorization proxy server VM without jq installed

Expected Behavior

There shouldn't be a dependency on jq

CSM Driver(s)

n/a

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

n/a

Operating System

RHEL 7.9

[donatwork]

@tdawe Is Python 3 an option? We could use sed which does not have a dependency on much or Python 3, which should be standard now on most distributions.

| python3 -c "import json,sys; print(json.load(sys.stdin)['Token'])"

[tdawe]

@donatwork Checking on my RHEL 7.9 setup, it comes with Python 2.7.5 so we may need to rely on sed.

[donatwork]

Python 2.7.x has been EOLd since 2020-01-01. Even then the same command can work with Python 2.7.x as well as Python 3. Python has been around for a long time. We can use sed but it is ugly. Is that your preference? The command will be this:

| sed -e 's/"Token": //' -e 's/[{}"]//g' -e 's/\n/\n/g'

[atye]

Could we have karavictl generate token output valid yaml instead of requiring the user to parse the response? Perhaps we could have a new flag to support valid yaml output.

[tdawe]

I like that idea @atye , instead of having to rely on other tools to parse the output. A flag such as --yaml could be added so we remain backwards compatible with the current command but provide an option to get the valid yaml.

[sharmilarama]

I will add this as an enhancement in our backlog. We can update the documentation to use sed for now if we do not want Python dependency @tdawe?

[tdawe]

I will add this as an enhancement in our backlog. We can update the documentation to use sed for now if we do not want Python dependency @tdawe?

Yes, doc update with sed for now should be fine.

[donatwork]

Removed dependency on jq in favor of sed which is more portable.