-
Notifications
You must be signed in to change notification settings - Fork 22
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
ci: change to use GitHub actions #86
Conversation
c566e5c
to
a07861f
Compare
Hi @derekbit, may you help to check the error test case? It looks like we didn't have CI for this repository. |
iSCSI service is not installed. |
Found found the Dockerfile.dapper relise ubuntu:16.04 base image. @FrankYang0529 Can you help create a ticket for migrating it to BCI image? Thank you. |
Thank you. It looks like the error message change after I add |
kernel module is missing. Try |
d54b319
to
58e1301
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to fix the failed PR check.
Running tests
modprobe: FATAL: Module iscsi_tcp not found in directory /lib/modules/6.5.0-1018-azure
Failed to load iscsi_tcp module
time="2024-04-30T06:37:02Z" level=fatal msg="exit status 1"
make: *** [Makefile:11: ci] Error 1
d727aea
to
11e5542
Compare
@FrankYang0529 Is it ready to review? |
This one is like go-spdk-helper and longhorn-spdk-engine. It need system modules like iSCSI daemon. Do you know how to start it in container? The testing error message is:
|
It seems kernel module is not found. |
We can't use |
You can modprobe it in the hostname space instead. |
40ee0e3
to
5b2d615
Compare
You are right. I just realized we never run CI here. |
Signed-off-by: PoAn Yang <[email protected]>
Longhorn 8039 Signed-off-by: Derek Su <[email protected]>
Longhorn 8039 Signed-off-by: Derek Su <[email protected]>
Longhorn 8039 Signed-off-by: Derek Su <[email protected]>
@FrankYang0529 @innobead
I think we can disable CI test as before and fix it later. WDYT? |
Agree. The go-iscsi-helper works well. We just need to find a way to test it in the future. Thank you. |
Which issue(s) this PR fixes:
longhorn/longhorn#8039
What this PR does / why we need it:
Special notes for your reviewer:
Additional documentation or context