Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Some powershell commands in the "How to Run the OSM Manual Demo" dont work #66

Closed
apoutney opened this issue Mar 1, 2021 · 7 comments
Closed

Comments

@apoutney
Copy link

apoutney commented Mar 1, 2021

TBH: I'm not sure if its the documentation that is wrong or if its something in my environment or if there is an issue with the commands.

I have been following the "How to Run the OSM Manual Demo" and have encountered some issues

First, in the section "Create the Bookstore Application Namespaces" the following script is supplied

for i in bookstore bookbuyer bookthief bookwarehouse; do kubectl create ns $i; done

However this results in the following error:

Missing opening '(' after keyword 'for'.

I managed to get around this by creating the namespaces individually.

But then in the "Verify the Traffic Policy Mode" the following command is given:

kubectl get configmap -n osm-system osm-config -o json | jq -r '.data["permissive_traffic_policy_mode"]'

This produces the following error:

jq: error: permissive_traffic_policy_mode/0 is not defined at , line 1:
.data[permissive_traffic_policy_mode]
jq: 1 compile error

I again managed to get around this, by using the kubectl describe command on the config map to check the permissive_traffic_policy_mode value.

Then, in the section "Permissive Traffic Policy Mode" the following command is given to update the config map:

kubectl patch ConfigMap osm-config -n osm-system -p '{"data":{"permissive_traffic_policy_mode":"true"}}' --type=merge

But again this doesnt work and responds with the following error:

Error from server: Invalid JSON Patch

Is it the documentation that is wrong? Or have I missed something?

@shashankram
Copy link
Member

@apoutney , the commands that are failing for you are to be run on a Unix (or similar) shell. These aren't powershell commands, so running them on powershell are going to fail.

@michelleN
Copy link

We should update the demo to be powershell friendly.

@apoutney
Copy link
Author

apoutney commented Mar 3, 2021

@shashankram, thanks for the clarification :). I hadnt seen anywhere in the documentation that suggested they Unix shell commands. Since kubectl is just the cli I assumed they would work in powershell

@adithyaakrishna
Copy link

adithyaakrishna commented Apr 28, 2021

@shashankram, thanks for the clarification :). I hadnt seen anywhere in the documentation that suggested they Unix shell commands. Since kubectl is just the cli I assumed they would work in powershell

@shashankram Does the doc need to be updated with a title UNIX as suggested above?
If so, Please assign me this :)

@shashankram
Copy link
Member

@shashankram, thanks for the clarification :). I hadnt seen anywhere in the documentation that suggested they Unix shell commands. Since kubectl is just the cli I assumed they would work in powershell

@shashankram Does the doc need to be updated with a title UNIX as suggested above?
If so, Please assign me this :)

The prerequisites mentions this:
a workstation capable of executing Bash scripts

Does that suffice?

@adithyaakrishna
Copy link

I have windows capable of running bash scripts
image

@bridgetkromhout bridgetkromhout transferred this issue from openservicemesh/osm May 20, 2021
@bridgetkromhout bridgetkromhout added this to In Progress in Documentation May 24, 2021
@bridgetkromhout bridgetkromhout moved this from In Progress to TODO in Documentation Jun 8, 2021
@shashankram
Copy link
Member

#66 (comment)

Documentation automation moved this from TODO to Done Sep 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Development

No branches or pull requests

4 participants