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

ci(docker): increase git commit sha slug to 12 characters #538

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

AtomicFS
Copy link
Collaborator

@AtomicFS AtomicFS commented Jan 30, 2025

  • we are still having some issues with Docker pulling containers (docker fails to pull containers #549 ), maybe increasing the short git commit sha form default 8 characters to 12 will improve things (it will certainly decrease the likelihood of collision)

@AtomicFS AtomicFS requested a review from MDr164 as a code owner January 30, 2025 13:45
@github-actions github-actions bot added github_actions Pull requests that update GitHub Actions code python Pull requests that update Python code labels Jan 30, 2025
@AtomicFS AtomicFS enabled auto-merge January 30, 2025 13:50
@AtomicFS
Copy link
Collaborator Author

Now that I am digging into the docker issue, there might be different reason for this.

@AtomicFS AtomicFS disabled auto-merge January 30, 2025 15:51
@AtomicFS
Copy link
Collaborator Author

AtomicFS commented Jan 30, 2025

Never-mind, the thing I was thinking of seems like red herring.

@AtomicFS AtomicFS force-pushed the ci/increase-sha-slug-length branch 2 times, most recently from 911d4f1 to 110db52 Compare January 30, 2025 16:27
@AtomicFS AtomicFS enabled auto-merge January 30, 2025 16:33
@github-actions github-actions bot added the testing Testing related label Jan 30, 2025
- we are still having some issues with Docker pulling containers, maybe
  increasing the short git commit sha form default 8 characters to 12
  will improve things (it will certainly decrease the likelihood of
  collision)

Signed-off-by: AtomicFS <[email protected]>
@AtomicFS AtomicFS force-pushed the ci/increase-sha-slug-length branch from 110db52 to 39ce883 Compare January 31, 2025 10:50
@AtomicFS AtomicFS added this pull request to the merge queue Jan 31, 2025
Merged via the queue into main with commit b146d3d Jan 31, 2025
41 checks passed
@AtomicFS AtomicFS deleted the ci/increase-sha-slug-length branch January 31, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update GitHub Actions code python Pull requests that update Python code testing Testing related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants