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

Fix ci e2e running failed issue. #410

Merged
merged 1 commit into from
Jun 9, 2023
Merged

Conversation

jaypume
Copy link
Member

@jaypume jaypume commented Jun 9, 2023

Fix ci e2e running failed issue.

The github action runner image ubuntu22.04 has upgrade kind version from v0.18.0 to v0.19.0:
https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md
image

but this will lead sedna all-in-one.sh running failed in github action, for the incompatible interface.

image

@kubeedge-bot kubeedge-bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 9, 2023
@jaypume jaypume force-pushed the fix-ci branch 6 times, most recently from 4e8cc70 to d80b0a5 Compare June 9, 2023 09:04
@kubeedge-bot kubeedge-bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jun 9, 2023
@jaypume jaypume force-pushed the fix-ci branch 2 times, most recently from 630b200 to 402adc0 Compare June 9, 2023 09:08
@kubeedge-bot kubeedge-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jun 9, 2023
@jaypume jaypume force-pushed the fix-ci branch 2 times, most recently from 41c0ab5 to 5bae85b Compare June 9, 2023 09:39
@kubeedge-bot kubeedge-bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jun 9, 2023
@kubeedge-bot kubeedge-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jun 9, 2023
Signed-off-by: Jie Pu <i@jaypu.com>
@Poorunga
Copy link
Member

Poorunga commented Jun 9, 2023

/lgtm

@kubeedge-bot kubeedge-bot added the lgtm Indicates that a PR is ready to be merged. label Jun 9, 2023
@jaypume
Copy link
Member Author

jaypume commented Jun 9, 2023

/approve

@kubeedge-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jaypume

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubeedge-bot kubeedge-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 9, 2023
@kubeedge-bot kubeedge-bot merged commit dad8203 into kubeedge:main Jun 9, 2023
11 checks passed
@seveirbian
Copy link

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants