-
Notifications
You must be signed in to change notification settings - Fork 12
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
test: adjust os name variable in test cases #7877
Conversation
ostree-ami-image.sh
Outdated
;; | ||
*) | ||
echo "unsupported distro: ${ID}-${VERSION_ID}" | ||
exit 1;; | ||
esac | ||
|
||
if [[ "${ID}-${VERSION_ID}" == "rhel-9.5" ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better to add a rhel 9.5 switch case and then set os_name like what you did in other cases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, thanks!
b01e7f7
to
c5f6aef
Compare
ostree-ami-image.sh
Outdated
OSTREE_REF="rhel/9/${ARCH}/edge" | ||
SYSROOT_RO="true" | ||
ANSIBLE_OS_NAME="rhel-edge" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on rhel9.3 and rhel9.4 , the os_name should be redhat?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
others looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thanks!
c5f6aef
to
4195c32
Compare
We have seen in our downstream CI several failed tests due to inconsistency of os name (or ansible os name) variable across distros. Thus, playbook tasks such as find /var /mnt mount points failed.
This PR configures mentioned variable value to
rhel-edge
only when test are perfomed by RHEL-9.5, centos-stream-9. In addition edge-commit will also haverhel-edge
variable value across all distros under test.