Skip to content

Commit

Permalink
ci: enable issue comments for our TFaGA
Browse files Browse the repository at this point in the history
  • Loading branch information
jamacku committed Sep 23, 2024
1 parent 57ab8f8 commit 7888fc9
Show file tree
Hide file tree
Showing 13 changed files with 19 additions and 15 deletions.
1 change: 1 addition & 0 deletions .github/workflows/base-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,3 +25,4 @@ jobs:
tmt_plan_regex: "smoke_plan"
pull_request_status_name: "Smoke test"
update_pull_request_status: "true"
create_issue_comment: "true"
1 change: 1 addition & 0 deletions .github/workflows/secrets_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,4 @@ jobs:
pull_request_status_name: "Secrets test"
secrets: "SOME_DUMMY_SECRET=FOO_BAR_SECRET;OS_SECRET=DUMMY_SECRET_OS"
update_pull_request_status: "true"
create_issue_comment: "true"
1 change: 1 addition & 0 deletions .github/workflows/timeout-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ jobs:
pull_request_status_name: "Timeout test"
update_pull_request_status: "true"
timeout: 60
create_issue_comment: "true"

- name: Check if testing farm test reached past the timeout limit
if: ${{ !cancelled() }}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/variables_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,4 @@ jobs:
pull_request_status_name: "Variables test"
variables: "SOME_DUMMY_VARIABLE=FOO_BAR_VARIABLE;OS_VARIABLE=DUMMY_OS"
update_pull_request_status: "true"
create_issue_comment: "true"
4 changes: 2 additions & 2 deletions dist/action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/action.js.map

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions dist/schema/testing-farm-api.d.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/schema/testing-farm-api.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/schema/testing-farm-api.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/action.ts
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ async function action(pr: PullRequest): Promise<void> {

summary.updateData({
...summary.newData,
runTime: tfResult.run_time,
runTime: tfResult.run_time || 0,
updated: tfResult.updated,
status: tfResult.state,
});
Expand Down Expand Up @@ -288,7 +288,7 @@ async function action(pr: PullRequest): Promise<void> {

summary.updateData({
...summary.newData,
runTime: tfResult.run_time,
runTime: tfResult.run_time || 0,
updated: tfResult.updated,
status: state,
outcome: result,
Expand Down
2 changes: 1 addition & 1 deletion src/schema/testing-farm-api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ export const requestDetailsSchema = z.object({
overall: z.string(),
})
.nullable(),
run_time: z.number(),
run_time: z.number().nullable(),
created: z.string(),
updated: z.string(),
});
Expand Down

0 comments on commit 7888fc9

Please sign in to comment.