Skip to content

Add explicit instructions to not include ./ in .dockerignore#3515

Merged
copybara-service[bot] merged 1 commit intomainfrom
bvandermoon-patch-1
Mar 31, 2026
Merged

Add explicit instructions to not include ./ in .dockerignore#3515
copybara-service[bot] merged 1 commit intomainfrom
bvandermoon-patch-1

Conversation

@bvandermoon
Copy link
Copy Markdown
Collaborator

Description

Clearing up confusion to not include the leading ./ in the .dockerignore file when facing docker upload issues

Tests

upload_maxtext_docker_image CLOUD_IMAGE_NAME=${CLOUD_IMAGE_NAME?}

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

Comment thread docs/build_maxtext.md Outdated
@bvandermoon bvandermoon force-pushed the bvandermoon-patch-1 branch from 0469610 to b699bbd Compare March 27, 2026 23:38
@copybara-service copybara-service Bot merged commit 87b1861 into main Mar 31, 2026
53 of 54 checks passed
@copybara-service copybara-service Bot deleted the bvandermoon-patch-1 branch March 31, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants