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
added upload artifact in workflow #66
Conversation
Signed-off-by: Atul-source <atulprajapati6031@gmail.com> fixing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> fixing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> added name field Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing work flow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> testing workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> test workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> test workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> test workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> test workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> test workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> test workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> test workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com> test workflow Signed-off-by: Atul-source <atulprajapati6031@gmail.com>
1f01124
to
7574d03
Compare
copy D:/a/eBPF-Package-Repository/eBPF-Package-Repository/xdp-root/build/xdp_root_kern.o upload_artifact | ||
copy D:\a\eBPF-Package-Repository\eBPF-Package-Repository\build\xdp-root\Release\xdp_root.exe upload_artifact |
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.
seems odd to mix /
and \
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.
This paths I got from github actions log
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.
Should use the same delimiter in both lines.
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.
Done
copy D:/a/eBPF-Package-Repository/eBPF-Package-Repository/ratelimiting/build/ratelimiting_kern.o upload_artifact | ||
copy D:\a\eBPF-Package-Repository\eBPF-Package-Repository\build\ratelimiting\Release\ratelimiting.exe upload_artifact |
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.
Seems odd to mix /
and \
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.
Done
@@ -92,3 +98,18 @@ jobs: | |||
run: | | |||
cmake -B build | |||
cmake --build build -j $(nproc) --config ${{env.BUILD_CONFIGURATION}} | |||
|
|||
- name: Create artifact directory | |||
if: matrix.configurations == 'Release' |
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.
Why only for Release? I think artifacts should be uploaded for Debug too, which is what we do in ebpf-for-windows
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.
Yes for debug case files are created at different path
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.
Do we need for both cases ?
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.
I would say yes.
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.
Done
Signed-off-by: Atul-source <atulprajapati6031@gmail.com>
Signed-off-by: Atul-source <atulprajapati6031@gmail.com>
In this PR I have added upload artifact workflow for each eBPF program. This PR addressed issue #64 .