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

CASMPET-6819: Create Goss Test Suite for SPBS #565

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

aasha-hpe
Copy link
Contributor

Summary and Scope

This is goss test suite written newly for iscsi based boot content projection solution.

Testing

Tested on Mug and Noname systems. Will test on CSM 1.5 once it is installed.

Tested on:

Tested on Mug and Noname.

Test description:

The new test cases written for iscsi based boot content projection and added to CSM goss testing framework This get invoked as part of CSM health checks post OS and CSM services are installed.

  • Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)? yet to run in upgrade scenario
  • Were continuous integration tests run? If not, why?
  • Was upgrade tested? If not, why?- TBD when system is available.
  • Was downgrade tested? If not, why?
  • Were new tests (or test issues/Jiras) created for this change?

Risks and Mitigations

None.

Pull Request Checklist

  • Version number(s) incremented, if applicable
  • Copyrights updated
  • License file intact
  • Target branch correct
  • CHANGELOG.md updated
  • Testing is appropriate and complete, if applicable
  • HPC Product Announcement prepared, if applicable

@aasha-hpe aasha-hpe force-pushed the sbps_goss_CASMPET-6819 branch from 28ae2b0 to a82c346 Compare January 23, 2024 07:45
@aasha-hpe aasha-hpe force-pushed the sbps_goss_CASMPET-6819 branch from a82c346 to e4c1701 Compare February 7, 2024 18:23
@aasha-hpe aasha-hpe marked this pull request as ready for review February 7, 2024 18:25
@aasha-hpe aasha-hpe requested a review from a team as a code owner February 7, 2024 18:25
@aasha-hpe aasha-hpe force-pushed the sbps_goss_CASMPET-6819 branch from e4c1701 to ebafd32 Compare February 8, 2024 09:26
@jeremy-duckworth
Copy link
Contributor

@aasha-hpe is this ready for review? Thanks.

Copy link
Contributor

@jeremy-duckworth jeremy-duckworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the shell-check warnings and follow up on the K8S worker labels.

@aasha-hpe
Copy link
Contributor Author

@aasha-hpe is this ready for review? Thanks.

@jeremy-duckworth Yes. This was ready for review.

@aasha-hpe
Copy link
Contributor Author

aasha-hpe commented Mar 5, 2024

Please address the shell-check warnings and follow up on the K8S worker labels.

There were some shell check errors which I fixed, thought we can ignore the warnings. Can we ? We have done this way of implementation in Mercury, we never faced shell-check errors nor any issues with the code.
@jeremy-duckworth, I have taken care of shellcheck warnings. Pls take a look.

@aasha-hpe aasha-hpe force-pushed the sbps_goss_CASMPET-6819 branch from ebafd32 to a53f8b5 Compare April 4, 2024 12:05
@aasha-hpe
Copy link
Contributor Author

Please address the shell-check warnings and follow up on the K8S worker labels.

@jeremy-duckworth, Have taken care of shell-check warnings and K8s worker labels. Now the label is 'iscsi=sbps'. Pls take a look.

@aasha-hpe
Copy link
Contributor Author

Please address the shell-check warnings and follow up on the K8S worker labels.

There were some shell check errors which I fixed, thought we can ignore the warnings. Can we ? We have done this way of implementation in Mercury, we never faced shell-check errors nor any issues with the code.

@jeremy-duckworth, I have taken care of shellcheck warnings. Pls take a look.

Copy link
Contributor

@jeremy-duckworth jeremy-duckworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with suggestions

@aasha-hpe aasha-hpe requested a review from dborman-hpe May 8, 2024 07:51
@aasha-hpe
Copy link
Contributor Author

@dborman-hpe, Thanks a lot for your review and comments. I have addressed them. Please take a look.

@aasha-hpe
Copy link
Contributor Author

I incorporated the changes and tested. Its working fine.

@dborman-hpe
Copy link

Please fix the shellcheck warnings:

[HPE-C02XJ1WHJGH6] shellcheck iscsi_sbps_sanity.sh 

In iscsi_sbps_sanity.sh line 129:
for s in $(dig -t srv +short _sbps-hsn._tcp."${host}" | sort -k3 | awk '{print $NF;}' | xargs); do printf "** $s **\n"; dig +short $s; done
                                                                                                          ^----------^ SC2059 (info): Don't use variables in the printf format string. Use printf '..%s..' "$foo".
                                                                                                                                   ^-- SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean: 
for s in $(dig -t srv +short _sbps-hsn._tcp."${host}" | sort -k3 | awk '{print $NF;}' | xargs); do printf "** $s **\n"; dig +short "$s"; done


In iscsi_sbps_sanity.sh line 131:
for s in $(dig -t srv +short _sbps-nmn._tcp."${host}" | sort -k3 | awk '{print $NF;}' | xargs); do printf "** $s **\n"; dig +short $s; done
                                                                                                          ^----------^ SC2059 (info): Don't use variables in the printf format string. Use printf '..%s..' "$foo".
                                                                                                                                   ^-- SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean: 
for s in $(dig -t srv +short _sbps-nmn._tcp."${host}" | sort -k3 | awk '{print $NF;}' | xargs); do printf "** $s **\n"; dig +short "$s"; done

For more information:
  https://www.shellcheck.net/wiki/SC2059 -- Don't use variables in the printf...
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...

@aasha-hpe aasha-hpe force-pushed the sbps_goss_CASMPET-6819 branch from 6633be8 to 24cdbfb Compare July 10, 2024 19:04
@aasha-hpe
Copy link
Contributor Author

Please fix the shellcheck warnings:

[HPE-C02XJ1WHJGH6] shellcheck iscsi_sbps_sanity.sh 

In iscsi_sbps_sanity.sh line 129:
for s in $(dig -t srv +short _sbps-hsn._tcp."${host}" | sort -k3 | awk '{print $NF;}' | xargs); do printf "** $s **\n"; dig +short $s; done
                                                                                                          ^----------^ SC2059 (info): Don't use variables in the printf format string. Use printf '..%s..' "$foo".
                                                                                                                                   ^-- SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean: 
for s in $(dig -t srv +short _sbps-hsn._tcp."${host}" | sort -k3 | awk '{print $NF;}' | xargs); do printf "** $s **\n"; dig +short "$s"; done


In iscsi_sbps_sanity.sh line 131:
for s in $(dig -t srv +short _sbps-nmn._tcp."${host}" | sort -k3 | awk '{print $NF;}' | xargs); do printf "** $s **\n"; dig +short $s; done
                                                                                                          ^----------^ SC2059 (info): Don't use variables in the printf format string. Use printf '..%s..' "$foo".
                                                                                                                                   ^-- SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean: 
for s in $(dig -t srv +short _sbps-nmn._tcp."${host}" | sort -k3 | awk '{print $NF;}' | xargs); do printf "** $s **\n"; dig +short "$s"; done

For more information:
  https://www.shellcheck.net/wiki/SC2059 -- Don't use variables in the printf...
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...

Fixed these warnings.

Copy link

@dborman-hpe dborman-hpe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aasha-hpe aasha-hpe force-pushed the sbps_goss_CASMPET-6819 branch from cb5bee7 to 24cdbfb Compare July 11, 2024 15:47
Sanity tests for iSCSI based boot content projection.
@aasha-hpe aasha-hpe force-pushed the sbps_goss_CASMPET-6819 branch from 8043f21 to 673b1f4 Compare July 11, 2024 16:02
@dborman-hpe dborman-hpe merged commit 051f60c into release/1.6 Jul 15, 2024
3 checks passed
@dborman-hpe dborman-hpe deleted the sbps_goss_CASMPET-6819 branch July 15, 2024 16:33
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

Successfully merging this pull request may close these issues.

4 participants