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

Add a 1 second delay before accessing the boostrap disk. #2242

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

gwright83
Copy link
Contributor

On my FreeBSD CI system, on a relatively old server, I was seeing "disk controller not ready" errors when trying the format the bootstrap disk pack when building ITS with simh. This error occurred consistently when running on CI server, but did not happen when entering the commands manually.

The fix is to add a 1 second delay before trying to format the bootstrap pack. This fixes the problem seen in CI and has no effect when building on may (much faster Apple M1) laptop.

@gwright83 gwright83 temporarily deployed to Deploy October 11, 2023 22:45 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 11, 2023 22:45 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 11, 2023 22:45 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 11, 2023 22:45 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 11, 2023 22:45 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 12, 2023 07:04 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 12, 2023 07:04 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 12, 2023 07:04 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 12, 2023 07:04 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 12, 2023 07:04 — with GitHub Actions Inactive
@gwright83 gwright83 temporarily deployed to Deploy October 12, 2023 07:04 — with GitHub Actions Inactive
Copy link
Member

@larsbrinkhoff larsbrinkhoff left a comment

Choose a reason for hiding this comment

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

I was considering something like this myself, because the same thing happens with GitHub Actions. Thanks!

@larsbrinkhoff larsbrinkhoff merged commit 2f5f1c4 into PDP-10:master Oct 12, 2023
8 checks passed
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.

2 participants