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-32245][INFRA][FOLLOWUP] Reenable Github Actions on commit #29076

Closed
wants to merge 1 commit into from
Closed

[SPARK-32245][INFRA][FOLLOWUP] Reenable Github Actions on commit #29076

wants to merge 1 commit into from

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Jul 12, 2020

What changes were proposed in this pull request?

This PR reenables GitHub Action on every commit as a next step.

Why are the changes needed?

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at #29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on master branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Manual.

@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Jul 12, 2020

Hi, @HyukjinKwon , @gengliangwang , @viirya . As a next step, I want to enable this back and see the traffic.

Comment on lines +4 to +6
push:
branches:
- master
Copy link
Member

Choose a reason for hiding this comment

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

For Github Actions running for push event, where I can look at the action logs? Still in "Check" tab of PRs? When will it be triggered? Merging?

Copy link
Member Author

Choose a reason for hiding this comment

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

We can see at two places.

  1. At commit log. At every commit log, the green check mark is attached.
  2. At commit info,

Currently, it's disabled yesterday. So, recent commits do not have that.

@SparkQA
Copy link

SparkQA commented Jul 12, 2020

Test build #125721 has finished for PR 29076 at commit ac96756.

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

Copy link
Member

@viirya viirya left a comment

Choose a reason for hiding this comment

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

I think we need to merge this to observe the traffic, right? It looks okay to me.

@dongjoon-hyun
Copy link
Member Author

Yes. Right. Thanks, @viirya .
Merged to master.

@dongjoon-hyun dongjoon-hyun deleted the reenable_gha_commit branch July 12, 2020 21:51
Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

Yes, looks good. Let's see.

HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 18, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 19, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 19, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 19, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 19, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 19, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 19, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 19, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
HyukjinKwon pushed a commit to HyukjinKwon/spark that referenced this pull request Aug 19, 2020
This PR reenables GitHub Action on every commit as a next step.

We carefully enabled GitHub Action on every PRs, and it looks good so far.

As we saw at apache#29072, GitHub Action is already triggered at every commits on every PRs. Enabling GitHub Action on `master` branch commit doesn't make a big difference. And, we need to start to test at every commit as a next step.

No.

Manual.

Closes apache#29076 from dongjoon-hyun/reenable_gha_commit.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants