Skip to content

feat(infra): add cloud-instance list command; ignore shutting-down instances#3198

Merged
mergify[bot] merged 2 commits intoinstructlab:mainfrom
booxter:cloud-infra-list-terminating
Feb 27, 2025
Merged

feat(infra): add cloud-instance list command; ignore shutting-down instances#3198
mergify[bot] merged 2 commits intoinstructlab:mainfrom
booxter:cloud-infra-list-terminating

Conversation

@booxter
Copy link
Contributor

@booxter booxter commented Feb 27, 2025

Some improvements to the cloud-instance.sh script:

  • feat(infra): add cloud-instance.sh list command
  • feat(infra): ignore shutting-down machines when calculating aws ids

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 booxter requested a review from danmcp February 27, 2025 15:54
@mergify mergify bot added the ci-failure PR has at least one CI failure label Feb 27, 2025
@booxter booxter force-pushed the cloud-infra-list-terminating branch from 922b962 to f757d38 Compare February 27, 2025 15:56
@mergify mergify bot removed the ci-failure PR has at least one CI failure label Feb 27, 2025
The command lists all matching ids.

Signed-off-by: Ihar Hrachyshka <ihar.hrachyshka@gmail.com>
shutting-down means `The instance is preparing to be terminated.` as
per:

https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ec2-instance-lifecycle.html

Signed-off-by: Ihar Hrachyshka <ihar.hrachyshka@gmail.com>
@booxter booxter force-pushed the cloud-infra-list-terminating branch from 3e76e49 to e48c5ea Compare February 27, 2025 17:01
@mergify mergify bot added the one-approval PR has one approval from a maintainer label Feb 27, 2025
Copy link
Contributor

@courtneypacheco courtneypacheco left a comment

Choose a reason for hiding this comment

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

This is a good change. 👍

@mergify mergify bot merged commit edf0741 into instructlab:main Feb 27, 2025
7 checks passed
@mergify mergify bot removed the one-approval PR has one approval from a maintainer label Feb 27, 2025
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