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

Check for files in rpm-ostree install script #2692

Merged
merged 1 commit into from
Feb 21, 2024
Merged

Conversation

KwisatzHaderach
Copy link
Collaborator

@KwisatzHaderach KwisatzHaderach commented Feb 19, 2024

When installing files (like /usr/bin/flock) installation check fails only with 'rpm -q' and on rpm-ostree system it might be a problem as it changes image digest (caused a failed upgrade test on our env)
Also installation will be faster with this check

Pull Request Checklist

  • implement the feature
  • extend the test coverage

@KwisatzHaderach KwisatzHaderach added bug Something isn't working no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. trivial A simple patch - a couple of lines, an easy-to-understand change, a typo fix. labels Feb 19, 2024
@KwisatzHaderach KwisatzHaderach added this to the 1.32 milestone Feb 19, 2024
@KwisatzHaderach KwisatzHaderach added the full test Pull request is ready for the full test execution label Feb 19, 2024
tmt/steps/prepare/install.py Outdated Show resolved Hide resolved
@psss psss added the prepare Prepare step label Feb 20, 2024
@psss psss self-assigned this Feb 20, 2024
Copy link
Collaborator

@psss psss left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for improving this! In e1f8aec I just propose to include a simple test coverage as well.

@KwisatzHaderach
Copy link
Collaborator Author

That is great, thanks, didn't realise we already have some tests for ostree envs.

When installing files (like /usr/bin/flock) installation check
fails only with 'rpm -q' and on rpm-ostree system it might
be a problem as it changes image digest
Also installation will be faster with this check
@psss psss changed the title Check for files in rpm-ostree install script Check for files in rpm-ostree install script Feb 20, 2024
@psss psss merged commit 3999dfa into main Feb 21, 2024
22 checks passed
@psss psss deleted the install_check_files branch February 21, 2024 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working full test Pull request is ready for the full test execution no functional change "No Functional Change" intended. Patch should not change tmt's behavior in any way. prepare Prepare step trivial A simple patch - a couple of lines, an easy-to-understand change, a typo fix.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants