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.

Sign up for GitHub

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

Jump to bottom

Add TriagerX workflows #19745

Merged
merged 103 commits into from
Jul 17, 2024
Merged

Add TriagerX workflows #19745

merged 103 commits into from
Jul 17, 2024

Conversation

Belami02
Copy link
Contributor

Related to: #19552

LongyuZhang
LongyuZhang reviewed Jun 26, 2024
View reviewed changes
Copy link
Contributor

@LongyuZhang LongyuZhang left a comment

Choose a reason for hiding this comment

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

Also, please update the title to Add TriagerX workflows. Thanks.

.github/workflows/TriageClosedIssue.yml Outdated Show resolved Hide resolved
.github/workflows/TriageOpenIssue.yml Show resolved Hide resolved
.github/workflows/TriageClosedIssue.yml Outdated Show resolved Hide resolved
@Belami02 Belami02 changed the title Added the TriagerX workflows Add TriagerX workflows Jun 27, 2024
Belami02 and others added 16 commits June 27, 2024 10:21
@Belami02
Set the github token under specific action
60e93d4
@Belami02
Set the github token under specific action
0e1ee59
@diyorbekibragimov
Merge branch 'eclipse-openj9:master' into master
c88a67c
@diyorbekibragimov
added the workflow for the triagerx model to also run when the commen…
bbf5e13 
…t starts with 'triagerx' keyword
@diyorbekibragimov
added the workflow for the triagerx model to also run when the commen…
952359d 
…t starts with 'triagerx' keyword
@diyorbekibragimov
removed checkout repository step
872ad2c
@diyorbekibragimov
fixed a spelling issue
f525b8a
@diyorbekibragimov
broke the url for graceful failure test
ab8e791
@diyorbekibragimov
fixed an error with the issueNumber
49363ec
@diyorbekibragimov
fixed an error with the issueNumber
a894bd9
@diyorbekibragimov
Merge branch 'master' of https://github.com/Tigers-X/openj9
7e70f58
@diyorbekibragimov
fixed an error with the issueNumber
f4ef216
@diyorbekibragimov
added a feature to collect a list of PR authors
ea99439
@Belami02
Debugging the feature added (PR)
d37fea5
@Belami02
Changed the sandbox number and the rest.issues (script@v7)
0aed6fc
@Belami02
Fixed the wrong name
d120048
diyorbekibragimov and others added 23 commits July 7, 2024 17:43
@diyorbekibragimov
Update triggerTriagerx.yml
11731a4
@diyorbekibragimov
Removed .rest property
312f90c
@diyorbekibragimov
added the issue number to the environment
816b136
@diyorbekibragimov
Added the issue number
7cdd4ef
@diyorbekibragimov
Update recommendation.js
cab4c89
@diyorbekibragimov
Update triggerTriagerx.yml
6d490df
@diyorbekibragimov
Update triggerTriagerx.yml
d3db4b9
@diyorbekibragimov
added octokit for GITHUB API
af4ec9b
@diyorbekibragimov
added Github token to the environment
2f85c6c
@diyorbekibragimov
Fixed the code for external .js integration
9aa7db5
@diyorbekibragimov
Removed payload for issue number
e0fb33a
@diyorbekibragimov
Update TriageOpenIssue.yml
a3220f8
@diyorbekibragimov
transformed issue number to string issue number
a119032
@ftm-2005
Delete .github/workflows/callSlack.yml
98b6c0a
@ftm-2005
deleted trailing white spaces
8ff69f7
@ftm-2005
deleted trailing white spaces
c9ca3f9
@diyorbekibragimov
Update recommendation.js
a528ffe
@diyorbekibragimov
Update TriageOpenIssue.yml
00c5710
@diyorbekibragimov
Update TriageOpenIssue.yml
ef0261d
@diyorbekibragimov
Update recommendation.js
8bb41b6
@diyorbekibragimov
Added id to the step
19ecce6
@diyorbekibragimov
Removed console.logs
d38e817
@Belami02
Resolve the issue of trailing whitespaces and corrected the sandbox info
29bba29
@Belami02 Belami02 requested a review from LongyuZhang July 9, 2024 13:46
LongyuZhang
LongyuZhang approved these changes Jul 9, 2024
View reviewed changes
Copy link
Contributor

@LongyuZhang LongyuZhang left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM.
This workflow currently will only comment in the sandbox issue #19673, without linking to other issues so far. The prediction results in the sandbox issue will be used to evaluate the machine-learning based TriagerX model then.
@llxia @smlambert Could you also review it? Thanks.

llxia
llxia approved these changes Jul 11, 2024
View reviewed changes
Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

LGTM

@llxia llxia requested a review from smlambert July 11, 2024 15:37
@llxia
Copy link
Contributor

llxia commented Jul 16, 2024

@smlambert could you review the PR? Thanks

@smlambert smlambert merged commit 0e944e7 into eclipse-openj9:master Jul 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LongyuZhang LongyuZhang LongyuZhang approved these changes

@llxia llxia llxia approved these changes

@smlambert smlambert smlambert approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@Belami02 @llxia @smlambert @LongyuZhang @GioChkhaidze @diyorbekibragimov @coolujain @ftm-2005