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

chore(install-dev): block until DB container stabilizes #2378

Merged
merged 13 commits into from
Sep 30, 2024

Conversation

Yaminyam
Copy link
Member

@Yaminyam Yaminyam commented Jul 4, 2024

image
When starting codespace, redis-related settings are not being put into etcd.
Since other etcd put requests were processed normally, it is expected that the problem occurred because the put request was executed before etcd was ready.

So, after performing a health check of the halfstack service, proceed to the next step.

image
When using install-dev.sh in the provisioning stage, such as codespace, script execution may be faster than executing docker daemon.
The script runs after the docker daemon's health check.

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version

Copy link

graphite-app bot commented Jul 4, 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.

@github-actions github-actions bot added comp:installer Related to Installer size:XS ~10 LoC labels Jul 4, 2024
@kyujin-cho kyujin-cho changed the title chore: After passing the health check of halfstack service, proceed to the next step chore(install-dev): block until DB container stabilizes Jul 4, 2024
@kyujin-cho kyujin-cho added this to the 23.09 milestone Jul 4, 2024
@Yaminyam Yaminyam marked this pull request as draft July 5, 2024 02:59
@achimnol achimnol added the type:bug Reports about that are not working label Jul 6, 2024
@github-actions github-actions bot added size:S 10~30 LoC and removed size:XS ~10 LoC labels Jul 12, 2024
@github-actions github-actions bot added size:M 30~100 LoC and removed size:S 10~30 LoC labels Jul 12, 2024
@Yaminyam Yaminyam marked this pull request as ready for review July 12, 2024 04:03
Copy link
Member

@achimnol achimnol left a comment

Choose a reason for hiding this comment

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

Let's refactor the repeated spin-wait codes as a bash function.

@Yaminyam Yaminyam requested a review from achimnol August 16, 2024 05:31
@achimnol achimnol added this pull request to the merge queue Sep 30, 2024
Merged via the queue into main with commit f35ccb9 Sep 30, 2024
20 checks passed
@achimnol achimnol deleted the ci/docker-compose-wait branch September 30, 2024 22:20
lablup-octodog pushed a commit that referenced this pull request Sep 30, 2024
Co-authored-by: Joongi Kim <[email protected]>
Backported-from: main (24.09)
Backported-to: 23.09
Backport-of: 2378
lablup-octodog pushed a commit that referenced this pull request Sep 30, 2024
Co-authored-by: Joongi Kim <[email protected]>
Backported-from: main (24.09)
Backported-to: 24.03
Backport-of: 2378
github-merge-queue bot pushed a commit that referenced this pull request Sep 30, 2024
github-merge-queue bot pushed a commit that referenced this pull request Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:installer Related to Installer size:M 30~100 LoC type:bug Reports about that are not working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants