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

DAOS-6038 build: relax ipmctl version requirement for leap15 in spec … #5551

Merged
merged 2 commits into from
Apr 26, 2021

Conversation

tanabarr
Copy link
Contributor

…(#5491)

To avoid problems if ipmctl version is updated from stock on leap15,
relax the dependency version requirement in RPM specfile to
"ipmctl >= 20.00.00.3733". Versions that exhibit a corruption-related
bug between 3809-3816 will still be blocked due to runtime version
checks in daos_server.

Signed-off-by: Tom Nabarro tom.nabarro@intel.com

…5491)

To avoid problems if ipmctl version is updated from stock on leap15,
relax the dependency version requirement in RPM specfile to
"ipmctl >= 20.00.00.3733". Versions that exhibit a corruption-related
bug between 3809-3816 will still be blocked due to runtime version
checks in daos_server.

Signed-off-by: Tom Nabarro <tom.nabarro@intel.com>
@tanabarr tanabarr requested a review from johannlombardi April 23, 2021 10:10
Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

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

LGTM. No errors found by checkpatch.

@tanabarr
Copy link
Contributor Author

so presume this is too late to make it into 1.2, shall I close?

@tanabarr tanabarr self-assigned this Apr 24, 2021
Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

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

LGTM. No errors found by checkpatch.

@johannlombardi johannlombardi merged commit e7358e5 into release/1.2 Apr 26, 2021
@johannlombardi johannlombardi deleted the tanabarr/build-version-ipmctl3-rel1_2 branch April 26, 2021 12:34
@ashleypittman ashleypittman mentioned this pull request Apr 28, 2021
@ashleypittman ashleypittman mentioned this pull request May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants