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

feat: Implement get_package_version in hosts/base.py #133

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

jakub-vavra-cz
Copy link
Contributor

@jakub-vavra-cz jakub-vavra-cz commented Oct 18, 2024

Find package version and parse it to major, minor, patch, release.

This is needed to detect sssd and ipa-client version to match expected features against versions and put workaround for RHEL 10 gating.

@ikerexxe
Copy link
Contributor

As a heads up, this PR is related to SSSD/sssd#7597 and they should work together to detect features.

While I find this PR generally fine, I have reservations about SSSD/sssd#7597. Rather than creating a separate test case for each feature, we could streamline with a single test and a table indicating feature presence or absence in each distribution version.

Copy link
Contributor

@spoore1 spoore1 left a comment

Choose a reason for hiding this comment

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

My main question is around the static version number format. It may be possible to make it more flexible to handle different types of version numbering.

sssd_test_framework/hosts/base.py Outdated Show resolved Hide resolved
Copy link
Contributor

@spoore1 spoore1 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ikerexxe ikerexxe left a comment

Choose a reason for hiding this comment

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

LGTM!

Find package version and parse it to major, minor, patch,
pre-release, update release.
@jakub-vavra-cz jakub-vavra-cz merged commit 5b872da into SSSD:master Oct 25, 2024
5 checks passed
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.

3 participants