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

[SPARK-34994][BUILD][2.4] Fix git error when pushing the tag after release script succeeds #32100

Closed
wants to merge 1 commit into from

Conversation

viirya
Copy link
Member

@viirya viirya commented Apr 8, 2021

What changes were proposed in this pull request?

This patch proposes to fix an error when running release-script in 2.4 branch.

Why are the changes needed?

When I ran release script for cutting 2.4.8 RC1, either in dry-run or normal run at the last step "push the tag after success", I encounter the following error:

fatal: Not a git repository (or any parent up to mount parent ....)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

Does this PR introduce any user-facing change?

No, dev only.

How was this patch tested?

Manual test.

@viirya
Copy link
Member Author

viirya commented Apr 8, 2021

cc @dongjoon-hyun @HyukjinKwon

@viirya viirya changed the title [SPARK-34994][BUILD] Fix git error when pushing the tag after release script succeeds [SPARK-34994][BUILD][2.4] Fix git error when pushing the tag after release script succeeds Apr 8, 2021
@SparkQA
Copy link

SparkQA commented Apr 8, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41674/

@SparkQA
Copy link

SparkQA commented Apr 8, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41674/

@dongjoon-hyun
Copy link
Member

cc @holdenk

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

If this is tested by @viirya , I'm good.
Just curious if something is change from @holdenk 's release.

@viirya
Copy link
Member Author

viirya commented Apr 8, 2021

I thought it is only under dry-run. But when I ran with normal run, I also encountered this issue. I checked the working directory at the point, it was wrong and needs cd spark to enter cloned spark repo.

@SparkQA
Copy link

SparkQA commented Apr 8, 2021

Test build #137096 has finished for PR 32100 at commit 4613672.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun
Copy link
Member

dongjoon-hyun commented Apr 8, 2021

Ya. Feel free to merge please, @viirya . Release manager can do everything on the release branch to unblock the release.

@viirya
Copy link
Member Author

viirya commented Apr 8, 2021

Thank you @dongjoon-hyun. Merging to branch-2.4.

viirya added a commit that referenced this pull request Apr 8, 2021
…lease script succeeds

### What changes were proposed in this pull request?

This patch proposes to fix an error when running release-script in 2.4 branch.

### Why are the changes needed?

When I ran release script for cutting 2.4.8 RC1, either in dry-run or normal run at the last step "push the tag after success", I encounter the following error:

```
fatal: Not a git repository (or any parent up to mount parent ....)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
```

### Does this PR introduce _any_ user-facing change?

No, dev only.

### How was this patch tested?

Manual test.

Closes #32100 from viirya/fix-release-script-2.

Authored-by: Liang-Chi Hsieh <viirya@gmail.com>
Signed-off-by: Liang-Chi Hsieh <viirya@gmail.com>
@viirya viirya closed this Apr 8, 2021
@viirya viirya deleted the fix-release-script-2 branch April 8, 2021 23:27
@HyukjinKwon
Copy link
Member

LGTM2

@viirya
Copy link
Member Author

viirya commented Apr 9, 2021

Thank you @HyukjinKwon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants