Skip to content

Rerun the failed tests by comments#2309

Merged
eolivelli merged 2 commits intoapache:masterfrom
zymap:rerun-failure-job
May 4, 2020
Merged

Rerun the failed tests by comments#2309
eolivelli merged 2 commits intoapache:masterfrom
zymap:rerun-failure-job

Conversation

@zymap
Copy link
Copy Markdown
Member

@zymap zymap commented Apr 15, 2020


Motivation

Currently, non-committers cannot rerun the failure tests, we need to
provide the ability to let non-committer to rerun the failed tests.
I wrote a Github Actions that can rerun the failed test by comments
rerun failure tests and add it to the bookkeeper ci process.

---

*Motivation*

Currently, non-committers cannot rerun the failure tests, we need to
provide the ability to let non-committer to rerun the failed tests.
I wrote a Github Actions that can rerun the failed test by comments
`rerun failure tests` and add it to the bookkeeper ci process.
@zymap
Copy link
Copy Markdown
Member Author

zymap commented Apr 15, 2020

Before merging this pull requests. We need to add a secrets named BOT. Using the default token can not trigger the another action. Can anyone help me?
https://github.community/t5/GitHub-Actions/Push-from-Action-does-not-trigger-subsequent-action/td-p/37872

@eolivelli
Copy link
Copy Markdown
Contributor

@sijie do you know how to help?
Shall we ask INFRA

@sijie
Copy link
Copy Markdown
Member

sijie commented Apr 20, 2020

@eolivelli we need to ask INFRA for doing that. You can search the INFRA jiras to find my JIRA ticket created for Pulsar.

@eolivelli
Copy link
Copy Markdown
Contributor

@sijie I will try to do it

@sijie sijie added this to the 4.11.0 milestone Apr 21, 2020
@eolivelli
Copy link
Copy Markdown
Contributor

eolivelli commented Apr 22, 2020

I have created a ticket to INFRA (INFRA-20170)
the name of the secret will be BKBOT_TOKEN

@eolivelli
Copy link
Copy Markdown
Contributor

@zymap which token should I give to INFRA ?
one of my personal tokens ?
what permissions should I give to it ?

@eolivelli
Copy link
Copy Markdown
Contributor

According to the guide you posted we need a
Please create a personal access token in repo setting(write permission accordingly)

@sijie shall I ask INFRA to create a "repository token" ?

@zymap
Copy link
Copy Markdown
Member Author

zymap commented Apr 24, 2020

@zymap which token should I give to INFRA ?

@eolivelli I am not sure about this. @sijie Could you help with this?

@sijie
Copy link
Copy Markdown
Member

sijie commented Apr 27, 2020

@eolivelli

create a "repository token" ?

I am not sure how to do it. How does "repository token" work?

@eolivelli
Copy link
Copy Markdown
Contributor

@zymap the token is ready.
Can you please check and test it?
Or should I merge this patch now?

@eolivelli
Copy link
Copy Markdown
Contributor

@zymap committed to master branch

can you please test it ?

Ghatage pushed a commit to Ghatage/bookkeeper that referenced this pull request Jun 19, 2020
---

*Motivation*

Currently, non-committers cannot rerun the failure tests, we need to
provide the ability to let non-committer to rerun the failed tests.
I wrote a Github Actions that can rerun the failed test by comments
`rerun failure tests` and add it to the bookkeeper ci process.



Reviewers: Enrico Olivelli <eolivelli@gmail.com>

This closes apache#2309 from zymap/rerun-failure-job
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