Skip to content

[build] Replace ipmitool_1.8.19-4+deb12u1 with updated existing version#20791

Closed
wdoekes wants to merge 1 commit intosonic-net:masterfrom
ossobv:fix-ipmitool-newer-version-20789
Closed

[build] Replace ipmitool_1.8.19-4+deb12u1 with updated existing version#20791
wdoekes wants to merge 1 commit intosonic-net:masterfrom
ossobv:fix-ipmitool-newer-version-20789

Conversation

@wdoekes
Copy link
Contributor

@wdoekes wdoekes commented Nov 13, 2024

Why I did it

ipmitool_1.8.19-4+deb12u1 is gone. Now there is
ipmitool_1.8.19-4+deb12u2.

Added -u to dget because Aron Xu's 8160A17841FE key was not in an installed keyring.

Addresses #20789.

How to verify it

make works again.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305
  • 202405

Observed in 202405, so I'd like it fixed there too.

==== Why I did it

ipmitool_1.8.19-4+deb12u1 is gone. Now there is
ipmitool_1.8.19-4+deb12u2.

Added `-u` to dget because Aron Xu's 8160A17841FE key was not in an
installed keyring.

Addresses sonic-net#20789.
@wdoekes
Copy link
Contributor Author

wdoekes commented Nov 18, 2024

Duplicated by #20836 .

Obsoleted by #20349 . Can close once that is merged.

@bingwang-ms
Copy link
Contributor

@wdoekes Can we close this PR since #20349 is merged?

@wdoekes
Copy link
Contributor Author

wdoekes commented Nov 19, 2024

Yes. With ba48f43 in master and 87493db in 202405, this is not needed anymore.

Thanks! Closing.

@wdoekes wdoekes closed this Nov 19, 2024
@wdoekes wdoekes deleted the fix-ipmitool-newer-version-20789 branch November 19, 2024 06:55
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