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

ocm ci: append "\" before shell command output while parsing it #268

Open
venkataanil opened this issue Nov 14, 2022 · 0 comments
Open

ocm ci: append "\" before shell command output while parsing it #268

venkataanil opened this issue Nov 14, 2022 · 0 comments

Comments

@venkataanil
Copy link
Collaborator

OCM CI failing with below errors
{{[2022-11-09, 05:00:23 EST] {subprocess.py:92} INFO - Clean-up existing OSD access keys..
[2022-11-09, 05:00:23 EST] {subprocess.py:92} INFO - -bash: line 20: conditional binary operator expected
[2022-11-09, 05:00:23 EST] {subprocess.py:92} INFO - -bash: line 20: syntax error near 2' [2022-11-09, 05:00:23 EST] {subprocess.py:92} INFO - -bash: line 20: if [[ -eq 2 ]]; then'
[2022-11-09, 05:00:23 EST] {subprocess.py:96} INFO - Command exited with return code 2}}

{{}}

Recently PR #255 introduced some code for identifying last active AWS key which is failing in CI run. Inside run_ocm_benchmark.sh, the script ssh into ocm CI jumphost and execute this code. So we need to add additional "" before parsing/using command output i.e

  • AWS_KEY=$(aws iam list-access-keys --user-name OsdCcsAdmin --output text --query 'AccessKeyMetadata[*].AccessKeyId')
  • LEN_AWS_KEY=echo $AWS_KEY | wc -w
  • if [[ ${LEN_AWS_KEY} -eq 2 ]]; then
  •    aws iam delete-access-key --user-name OsdCcsAdmin --access-key-id `printf ${AWS_KEY[0]}`
    
  • AWS_KEY=$(/usr/bin/aws iam list-access-keys --user-name OsdCcsAdmin --output text --query 'AccessKeyMetadata[*].AccessKeyId')
  • LEN_AWS_KEY=$(echo $AWS_KEY | wc -w)
  • if [[ ${LEN_AWS_KEY} -eq 2 ]]; then
  •   /usr/bin/aws iam delete-access-key --user-name OsdCcsAdmin --access-key-id \$(printf \${AWS_KEY[0]})
    
venkataanil added a commit to venkataanil/airflow-kubernetes that referenced this issue Nov 14, 2022
OCM CI failing with the errors described in
cloud-bulldozer#268

Recently PR cloud-bulldozer#255 introduced some code for identifying last active
AWS key which is causing errors in CI run.
Inside run_ocm_benchmark.sh, the script ssh into ocm CI jumphost
and execute this code. So we need to add additional '\' before
parsing/using command output.
dry923 pushed a commit that referenced this issue Nov 14, 2022
* ocm: append '\' before shell command output

OCM CI failing with the errors described in
#268

Recently PR #255 introduced some code for identifying last active
AWS key which is causing errors in CI run.
Inside run_ocm_benchmark.sh, the script ssh into ocm CI jumphost
and execute this code. So we need to add additional '\' before
parsing/using command output.

* Use ocm-api-load from master instead of branch

Pass rate and duration values for newly added endpoints.
rsevilla87 pushed a commit to rsevilla87/airflow-kubernetes that referenced this issue Jan 12, 2023
* ocm: append '\' before shell command output

OCM CI failing with the errors described in
cloud-bulldozer#268

Recently PR cloud-bulldozer#255 introduced some code for identifying last active
AWS key which is causing errors in CI run.
Inside run_ocm_benchmark.sh, the script ssh into ocm CI jumphost
and execute this code. So we need to add additional '\' before
parsing/using command output.

* Use ocm-api-load from master instead of branch

Pass rate and duration values for newly added endpoints.
rsevilla87 pushed a commit to rsevilla87/airflow-kubernetes that referenced this issue Apr 26, 2023
* ocm: append '\' before shell command output

OCM CI failing with the errors described in
cloud-bulldozer#268

Recently PR cloud-bulldozer#255 introduced some code for identifying last active
AWS key which is causing errors in CI run.
Inside run_ocm_benchmark.sh, the script ssh into ocm CI jumphost
and execute this code. So we need to add additional '\' before
parsing/using command output.

* Use ocm-api-load from master instead of branch

Pass rate and duration values for newly added endpoints.
rsevilla87 pushed a commit to rsevilla87/airflow-kubernetes that referenced this issue May 10, 2023
* ocm: append '\' before shell command output

OCM CI failing with the errors described in
cloud-bulldozer#268

Recently PR cloud-bulldozer#255 introduced some code for identifying last active
AWS key which is causing errors in CI run.
Inside run_ocm_benchmark.sh, the script ssh into ocm CI jumphost
and execute this code. So we need to add additional '\' before
parsing/using command output.

* Use ocm-api-load from master instead of branch

Pass rate and duration values for newly added endpoints.
rsevilla87 pushed a commit to rsevilla87/airflow-kubernetes that referenced this issue Jul 10, 2023
* ocm: append '\' before shell command output

OCM CI failing with the errors described in
cloud-bulldozer#268

Recently PR cloud-bulldozer#255 introduced some code for identifying last active
AWS key which is causing errors in CI run.
Inside run_ocm_benchmark.sh, the script ssh into ocm CI jumphost
and execute this code. So we need to add additional '\' before
parsing/using command output.

* Use ocm-api-load from master instead of branch

Pass rate and duration values for newly added endpoints.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant