Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

Fix floating ip add for ibm cloud#3310

Merged
mergify[bot] merged 1 commit intoinstructlab:mainfrom
danmcp:floatingip
Apr 23, 2025
Merged

Fix floating ip add for ibm cloud#3310
mergify[bot] merged 1 commit intoinstructlab:mainfrom
danmcp:floatingip

Conversation

@danmcp
Copy link
Copy Markdown
Contributor

@danmcp danmcp commented Apr 22, 2025

There was an extra INSTANCE_NAME being passed so the command would only work if the instance name and floating ip names were the same.

Credit to @RobotSail for finding the issue.

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

There was an extra INSTANCE_NAME being passed so the command would only work if the instance name and floating ip names were the same.

Co-authored-by: RobotSail  <[email protected]>
Signed-off-by: Dan McPherson <[email protected]>
@danmcp danmcp requested a review from RobotSail April 22, 2025 23:34
@mergify mergify Bot added the one-approval PR has one approval from a maintainer label Apr 23, 2025
@mergify mergify Bot merged commit 9039253 into instructlab:main Apr 23, 2025
7 checks passed
@mergify mergify Bot removed the one-approval PR has one approval from a maintainer label Apr 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants