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

feat(infra): add -i option for instance id to cloud-instance.sh#3199

Merged
mergify[bot] merged 1 commit intoinstructlab:mainfrom
booxter:add-i-option
Feb 28, 2025
Merged

feat(infra): add -i option for instance id to cloud-instance.sh#3199
mergify[bot] merged 1 commit intoinstructlab:mainfrom
booxter:add-i-option

Conversation

@booxter
Copy link
Copy Markdown
Contributor

@booxter booxter commented Feb 27, 2025

  • feat(infra): allow to pass instance id with -i in cloud-instance.sh

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.

@booxter
Copy link
Copy Markdown
Contributor Author

booxter commented Feb 27, 2025

Will open for review once #3198 is merged and I rebase.

It was possible to set INSTANCE_ID= envvar; but it was awkward. An
option seems to be better UX to me.

Signed-off-by: Ihar Hrachyshka <[email protected]>
@booxter booxter marked this pull request as ready for review February 27, 2025 22:41
@booxter booxter requested a review from danmcp February 27, 2025 22:41
@mergify mergify Bot added the one-approval PR has one approval from a maintainer label Feb 28, 2025
@mergify mergify Bot merged commit 9c682f2 into instructlab:main Feb 28, 2025
@mergify mergify Bot removed the one-approval PR has one approval from a maintainer label Feb 28, 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