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

run-checker-daily.yml: If the openssl app is not built do not run it #22443

Closed
wants to merge 1 commit into from
Closed

run-checker-daily.yml: If the openssl app is not built do not run it #22443

wants to merge 1 commit into from

Conversation

t8m
Copy link
Member

@t8m t8m commented Oct 19, 2023 •
edited

This fixes the current runchecker daily failures.

@t8m t8m added branch: master Merge to master branch approval: review pending This pull request needs review by a committer approval: otc review pending This pull request needs review by an OTC member triaged: bug The issue/pr is/fixes a bug tests: exempted The PR is exempt from requirements for testing labels Oct 19, 2023
@t8m t8m marked this pull request as ready for review October 19, 2023 15:34
tmshort
tmshort approved these changes Oct 22, 2023
View reviewed changes
Copy link
Contributor

@tmshort tmshort left a comment

Choose a reason for hiding this comment

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

LGTM

@tmshort tmshort removed the approval: review pending This pull request needs review by a committer label Oct 22, 2023
@levitte levitte added approval: done This pull request has the required number of approvals and removed approval: otc review pending This pull request needs review by an OTC member labels Oct 22, 2023
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Oct 23, 2023
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@mattcaswell
Copy link
Member

Pushed. Thanks.

openssl-machine pushed a commit that referenced this pull request Oct 23, 2023
@t8m @mattcaswell
run-checker-daily.yml: If the openssl app is not built do not run it
bde5411 
Reviewed-by: Todd Short <todd.short@me.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #22443)
wanghao75 pushed a commit to openeuler-mirror/openssl that referenced this pull request Oct 26, 2023
@t8m
run-checker-daily.yml: If the openssl app is not built do not run it
77dbf7d 
Reviewed-by: Todd Short <todd.short@me.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from openssl/openssl#22443)

Signed-off-by: fly2x <fly2x@hitls.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@levitte levitte levitte approved these changes

@tmshort tmshort tmshort approved these changes

Assignees
No one assigned
Labels
approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Merge to master branch tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@t8m @openssl-machine @mattcaswell @levitte @tmshort