fix(rasdaemon): update to 0.8.1 #18324
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,57 @@ | ||
| # rasdaemon — RAS (Reliability, Availability and Serviceability) error logging tool. | ||
| # | ||
| # Azure Linux carries rasdaemon 0.8.1, which is newer than the version Fedora | ||
|
There was a problem hiding this comment. question: Is there a specific reason you're recommending 0.8.1? If we are considering an upgrade -- that still appears to be ~2 years old.
Contributor
Author
There was a problem hiding this comment. I picked the version that contains the fix needed for our site, happy to bump to the latest version. |
||
| # dist-git currently ships (0.8.0). Because there is no upstream commit to pin to | ||
| # yet, the newer version is applied as an overlay on top of the Fedora 0.8.0 spec | ||
| # plus the matching 0.8.1 source tarball. Drop this overlay and the source-files | ||
| # entry once Fedora dist-git packages 0.8.1 and the default snapshot picks it up. | ||
| [components.rasdaemon] | ||
|
|
||
| [[components.rasdaemon.overlays]] | ||
| description = "Bump rasdaemon to 0.8.1 (ahead of Fedora, which still ships 0.8.0)" | ||
| type = "spec-update-tag" | ||
| tag = "Version" | ||
| value = "0.8.1" | ||
|
|
||
| [components.rasdaemon.overlays.metadata] | ||
| category = "azl-temp-workaround" | ||
| upstream-status = "upstreamable" | ||
|
|
||
| [[components.rasdaemon.overlays]] | ||
| description = "Point the project URL at the maintainer's current GitHub repository" | ||
| type = "spec-update-tag" | ||
| tag = "URL" | ||
| value = "https://github.com/mchehab/rasdaemon" | ||
| [components.rasdaemon.overlays.metadata] | ||
| category = "azl-branding-policy" | ||
| upstream-status = "inapplicable" | ||
|
|
||
| [[components.rasdaemon.overlays]] | ||
| description = "Fetch the release tarball from the maintainer's GitHub releases instead of infradead" | ||
| type = "spec-update-tag" | ||
| tag = "Source0" | ||
| value = "https://github.com/mchehab/rasdaemon/releases/download/v%{version}/%{name}-%{version}.tar.bz2" | ||
| [components.rasdaemon.overlays.metadata] | ||
| category = "azl-branding-policy" | ||
| upstream-status = "inapplicable" | ||
|
|
||
| [[components.rasdaemon.overlays]] | ||
| description = "Package the mc_event/mem_fail RAS trigger scripts newly shipped in 0.8.1" | ||
| type = "spec-append-lines" | ||
| section = "%files" | ||
| lines = [ | ||
| "%dir %{_sysconfdir}/ras/triggers", | ||
| "%{_sysconfdir}/ras/triggers/mc_event_trigger", | ||
| "%{_sysconfdir}/ras/triggers/mem_fail_trigger", | ||
| ] | ||
| [components.rasdaemon.overlays.metadata] | ||
| category = "azl-temp-workaround" | ||
| upstream-status = "upstreamable" | ||
|
|
||
| # Pristine upstream 0.8.1 release tarball from the maintainer's GitHub releases | ||
| # (matches the spec's %{name}-%{version} Source0). Replaces the pruned 0.8.0 | ||
| # tarball once the Version overlay renders the new filename. | ||
| [[components.rasdaemon.source-files]] | ||
| filename = "rasdaemon-0.8.1.tar.bz2" | ||
| hash = "7c227b6ec0b73bd0cf727910f2adc49fc3f5167c41cd4b41d3d7d17b84a4a25681def89c82a275efe5f445049d6b0db8bd35f863384aca9a848b228402917d4d" | ||
| hash-type = "SHA512" | ||
| origin = { type = "download", uri = "https://github.com/mchehab/rasdaemon/releases/download/v0.8.1/rasdaemon-0.8.1.tar.bz2" } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1,2 @@ | ||
| SHA512 (rasdaemon-0.8.0.tar.bz2) = 3f69bf41ae1efe11c153ffdf736116a78eee7d34dac1148f938e183e5523bc7c2da60af6376a1fc29c113dfad2c4b9798ba3b700f0eca7454f213d86df31c7a9 | ||
| SHA512 (rasdaemon-0.8.1.tar.bz2) = 7c227b6ec0b73bd0cf727910f2adc49fc3f5167c41cd4b41d3d7d17b84a4a25681def89c82a275efe5f445049d6b0db8bd35f863384aca9a848b228402917d4d |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
question(blocking): While we're open to package-level upgrades on a case-by-case basis, our default is presently to stick with what's in Fedora 43. The PR title says "fix"; is there a specific set of issues or use cases that you're looking to resolve with this upgrade? Some additional explanation/justification in the PR will better help us understand the trade-off.
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.
0.8.1 comes with a fix that allows for memory controller bank type decoding on dual-socket AMD systems.