Skip to content

ipmi: switch to lanplus#19

Open
halfcrazy wants to merge 1 commit into
kubevirtbmc:mainfrom
halfcrazy:oss-lanplus
Open

ipmi: switch to lanplus#19
halfcrazy wants to merge 1 commit into
kubevirtbmc:mainfrom
halfcrazy:oss-lanplus

Conversation

@halfcrazy

Copy link
Copy Markdown
Collaborator

@starbops starbops left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you also include a note stating that IPMI v1.5 will be no longer supported? Thank you.

starbops
starbops previously approved these changes Jul 8, 2026

@starbops starbops left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Thank you.

Please hold the merge until we publish 0.9.0.

@starbops starbops added the pr/do-not-merge Do not merge the PR yet even approved by all the reviewers label Jul 8, 2026
@halfcrazy

Copy link
Copy Markdown
Collaborator Author

Good news, bougou/go-ipmi#72 was merged. Update to the latest go-ipmi will support dual-stack.

@anishbista60

Copy link
Copy Markdown
Member

Good news, bougou/go-ipmi#72 was merged. Update to the latest go-ipmi will support dual-stack.

Do we have the latest version in our code base ?

@halfcrazy

Copy link
Copy Markdown
Collaborator Author

Good news, bougou/go-ipmi#72 was merged. Update to the latest go-ipmi will support dual-stack.

Do we have the latest version in our code base ?

no,until kubevirtbmc/kubevirtbmc#220 gets merged

@anishbista60

Copy link
Copy Markdown
Member

Cool. Thanks !

Comment thread docs/ipmi-guide.md Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/do-not-merge Do not merge the PR yet even approved by all the reviewers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants