Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: Handle container start error in separate try-catch context #2316

Merged

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Jun 20, 2024

Fix

  • Separate the exception context of container.create() and container.start() to handle them granularly.

Refactor

  • Scratch clean codes to clean_scratch()

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version

Copy link

graphite-app bot commented Jun 20, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “flow:merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “flow:hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

Copy link
Member Author

fregataa commented Jun 20, 2024

@fregataa fregataa added this to the 24.09 milestone Jun 20, 2024
@fregataa fregataa modified the milestones: 24.09, 24.03 Jun 20, 2024
@fregataa fregataa changed the title fix: handle container start error in separate try-catch context fix: Handle container start error in separate try-catch context Jun 20, 2024
@fregataa fregataa force-pushed the topic/06-20-fix_smelly_error_handling_in_kernel_creation branch 2 times, most recently from 3446b86 to 62fd88d Compare June 21, 2024 05:46
@github-actions github-actions bot added size:L 100~500 LoC and removed size:M 30~100 LoC labels Jun 21, 2024
@fregataa fregataa force-pushed the topic/05-23-fix_enhanced_kernel_termination_handling branch 2 times, most recently from bd7d1e4 to 25e70f3 Compare June 24, 2024 07:05
@fregataa fregataa force-pushed the topic/06-20-fix_smelly_error_handling_in_kernel_creation branch 2 times, most recently from 4866bb1 to 6af3d2c Compare June 25, 2024 15:27
@fregataa fregataa changed the base branch from topic/05-23-fix_enhanced_kernel_termination_handling to topic/06-26-feature_enable_to_configure_sync_container_lifecycle_task June 25, 2024 15:27
@fregataa fregataa changed the title fix: Handle container start error in separate try-catch context refactor: Handle container start error in separate try-catch context Jun 26, 2024
@fregataa fregataa force-pushed the topic/06-26-feature_enable_to_configure_sync_container_lifecycle_task branch from 35be346 to 1a75de9 Compare July 4, 2024 14:41
@fregataa fregataa force-pushed the topic/06-20-fix_smelly_error_handling_in_kernel_creation branch from 6af3d2c to 37fe73f Compare July 4, 2024 14:41
@fregataa fregataa force-pushed the topic/06-26-feature_enable_to_configure_sync_container_lifecycle_task branch from 1a75de9 to b6d53ae Compare July 11, 2024 12:11
@fregataa fregataa force-pushed the topic/06-20-fix_smelly_error_handling_in_kernel_creation branch from 37fe73f to 0165d2e Compare July 11, 2024 12:11
Base automatically changed from topic/06-26-feature_enable_to_configure_sync_container_lifecycle_task to main July 11, 2024 12:32
@fregataa fregataa force-pushed the topic/06-20-fix_smelly_error_handling_in_kernel_creation branch from 0165d2e to 566a43c Compare July 11, 2024 12:35
@fregataa fregataa marked this pull request as ready for review July 11, 2024 12:40
Copy link

graphite-app bot commented Jul 11, 2024

Merge activity

…2316)

### Fix
- Separate the exception context of `container.create()` and `container.start()` to handle them granularly.

### Refactor
- Scratch clean codes to `clean_scratch()`

**Checklist:** (if applicable)

- [x] Milestone metadata specifying the target backport version
@fregataa fregataa force-pushed the topic/06-20-fix_smelly_error_handling_in_kernel_creation branch from 566a43c to e7268d5 Compare July 11, 2024 12:46
@graphite-app graphite-app bot merged commit e7268d5 into main Jul 11, 2024
25 of 26 checks passed
@graphite-app graphite-app bot deleted the topic/06-20-fix_smelly_error_handling_in_kernel_creation branch July 11, 2024 12:49
lablup-octodog pushed a commit that referenced this pull request Jul 11, 2024
…2316)

### Fix
- Separate the exception context of `container.create()` and `container.start()` to handle them granularly.

### Refactor
- Scratch clean codes to `clean_scratch()`

**Checklist:** (if applicable)

- [x] Milestone metadata specifying the target backport version

Backported-from: main (24.09)
Backported-to: 24.03
Backport-of: 2316
jopemachine pushed a commit that referenced this pull request Jul 14, 2024
…2316)

### Fix
- Separate the exception context of `container.create()` and `container.start()` to handle them granularly.

### Refactor
- Scratch clean codes to `clean_scratch()`

**Checklist:** (if applicable)

- [x] Milestone metadata specifying the target backport version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:agent Related to Agent component size:L 100~500 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant