Add wrapper script #63

2021-11-13T20:04:23Z

akatch commented

2021-11-13 20:04:23 +00:00

(Migrated from github.com)

This PR adds a wrapper script that makes it a little easier to run the flavor you want with the correct options.

Usage: ./run-openqa-tests.sh [FLAVOR] where FLAVOR is one of universal, dvd-iso, boot-iso, or minimal-iso.

This PR adds a wrapper script that makes it a little easier to run the flavor you want with the correct options. Usage: `./run-openqa-tests.sh [FLAVOR]` where FLAVOR is one of `universal`, `dvd-iso`, `boot-iso`, or `minimal-iso`.

lumarel (Migrated from github.com) reviewed 2021-11-13 20:04:23 +00:00

StackKorora (Migrated from github.com) reviewed 2021-11-13 20:04:23 +00:00

tcooper (Migrated from github.com) approved these changes 2021-11-15 00:46:37 +00:00

tcooper (Migrated from github.com) left a comment

Looks good.

tcooper commented

2021-11-16 06:50:29 +00:00

(Migrated from github.com)

For the proposed report script a minor change will fix the format...

[rocky@openqa-dev rocky]$ diff -Naur ~/build-report.sh ~/build-report.sh.new
--- /home/rocky/build-report.sh 2021-11-15 22:37:11.181100264 -0800
+++ /home/rocky/build-report.sh.new     2021-11-15 22:36:30.278606190 -0800
@@ -3,10 +3,11 @@
 
 BUILD_NAME=$1
 
-printf '# %s\n' "$BUILD_NAME"
+printf '# Build%s\n' "$BUILD_NAME"
 printf "| Test | Result | Failure Reason | Effort to Fix | Notes |\n"
-printf "| ---- | ------ | -------------- | ------------ | ----- |\n"
+printf "| ---- | ------ | -------------- | ------------- | ----- |\n"
 
 openqa-cli api -X GET jobs build="$BUILD_NAME" | \
     jq -r '.jobs[] | {name,result} | join(" | ") | split("-") | last' | \
-    sort
+    sort | \
+    sed 's,^,| ,g;s,$, | | | |,g'

This produces output in the correct table format...

[rocky@openqa-dev rocky]$ ~/build-report.sh.new "20211114.151023.0"
# Build20211114.151023.0
| Test | Result | Failure Reason | Effort to Fix | Notes |
| ---- | ------ | -------------- | ------------- | ----- |
| install_minimal@64bit | passed | | | |
| install_minimal@64bit | user_cancelled | | | |
| install_minimal_upload@64bit | passed | | | |
| install_minimal_upload@64bit | user_restarted | | | |

[rocky@openqa-dev rocky]$ ~/build-report.sh.new "20211114.151023.0"

Build20211114.151023.0

Test	Result	Failure Reason	Effort to Fix	Notes
install_minimal@64bit	passed
install_minimal@64bit	user_cancelled
install_minimal_upload@64bit	passed
install_minimal_upload@64bit	user_restarted

Also, for a future update we'll want to think about...

Including a header with other Build metadata.
pre-filtering the jobs so that the latest value is shown.

We can discuss in the meeting. This is a great start!

For the proposed report script a minor change will fix the format... ```diff [rocky@openqa-dev rocky]$ diff -Naur ~/build-report.sh ~/build-report.sh.new --- /home/rocky/build-report.sh 2021-11-15 22:37:11.181100264 -0800 +++ /home/rocky/build-report.sh.new 2021-11-15 22:36:30.278606190 -0800 @@ -3,10 +3,11 @@ BUILD_NAME=$1 -printf '# %s\n' "$BUILD_NAME" +printf '# Build%s\n' "$BUILD_NAME" printf "| Test | Result | Failure Reason | Effort to Fix | Notes |\n" -printf "| ---- | ------ | -------------- | ------------ | ----- |\n" +printf "| ---- | ------ | -------------- | ------------- | ----- |\n" openqa-cli api -X GET jobs build="$BUILD_NAME" | \ jq -r '.jobs[] | {name,result} | join(" | ") | split("-") | last' | \ - sort + sort | \ + sed 's,^,| ,g;s,$, | | | |,g' ``` This produces output in the correct table format... ```bash [rocky@openqa-dev rocky]$ ~/build-report.sh.new "20211114.151023.0" # Build20211114.151023.0 | Test | Result | Failure Reason | Effort to Fix | Notes | | ---- | ------ | -------------- | ------------- | ----- | | install_minimal@64bit | passed | | | | | install_minimal@64bit | user_cancelled | | | | | install_minimal_upload@64bit | passed | | | | | install_minimal_upload@64bit | user_restarted | | | | ``` [rocky@openqa-dev rocky]$ ~/build-report.sh.new "20211114.151023.0" # Build20211114.151023.0 | Test | Result | Failure Reason | Effort to Fix | Notes | | ---- | ------ | -------------- | ------------- | ----- | | install_minimal@64bit | passed | | | | | install_minimal@64bit | user_cancelled | | | | | install_minimal_upload@64bit | passed | | | | | install_minimal_upload@64bit | user_restarted | | | | Also, for a future update we'll want to think about... - Including a header with other Build metadata. - pre-filtering the jobs so that the latest value is shown. We can discuss in the meeting. This is a great start!

Sign in to join this conversation.

No reviewers

lumarel

StackKorora

tcooper