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

[improve][build] Support git worktree working directory while building docker images #22851

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jun 5, 2024

Motivation

git supports sharing a single local directory across multiple working directories as explained in https://pulsar.apache.org/contribute/setup-git/#checking-out-each-pulsar-maintenance-branch-in-a-separate-working-directory . This makes it easier to maintain multiple Pulsar branches since each maintenance branch will be checked out in it's own working directory and could be open simultaneously in IntelliJ.

However, building docker images has been broken when using git worktree working directory. This is fixed with this PR.

Modifications

  • fix the check for git repository existence

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@lhotari lhotari self-assigned this Jun 5, 2024
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jun 5, 2024
@merlimat merlimat merged commit c23e677 into apache:master Jun 5, 2024
54 of 56 checks passed
lhotari added a commit that referenced this pull request Jun 6, 2024
lhotari added a commit that referenced this pull request Jun 6, 2024
lhotari added a commit that referenced this pull request Jun 6, 2024
lhotari added a commit that referenced this pull request Jun 6, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Jun 7, 2024
…g docker images (apache#22851)

(cherry picked from commit c23e677)
(cherry picked from commit 9d09daf)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Jun 7, 2024
…g docker images (apache#22851)

(cherry picked from commit c23e677)
(cherry picked from commit 9d09daf)
lhotari added a commit to apache/pulsar-site that referenced this pull request Jun 9, 2024
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.

2 participants