-
Notifications
You must be signed in to change notification settings - Fork 92
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
archiving ceph
directory for the job 'ceph-pr-clang-tidy'
#2276
base: main
Are you sure you want to change the base?
Conversation
The job `ceph-pr-clang-tidy` requires Ceph's build but since the job `ceph-pull-requests` is already building Ceph it is best to archive & copy the directory and work further upon it. It is done to save time and resources required to again build Ceph. Signed-off-by: Suyash Dongre <[email protected]>
Can one of the admins verify this patch? |
|
Yes we must run after every make-check build. |
Well, not "must", it, like everything, is a choice. Let me bring this up to the clt. |
Okay no problem |
Added permission to copy the archived `.tar` file from `ceph-pull-requests` to `ceph-pr-clang-tidy` Signed-off-by: Suyash Dongre <[email protected]>
Hi @dmick! I have added the code to give permission to our job |
@dmick |
does everything look good ? @dmick |
I'm sorry we haven't made more progress with this. You've hit on an area of Ceph development that is not very flexible to new work right now. We're going to continue trying to move the discussion along. |
@dmick |
The job
ceph-pr-clang-tidy
requires Ceph's build but since the jobceph-pull-requests
is already building Ceph it is best to archive & copy the directory and work further upon it. It is done to save time and resources required to again build Ceph.artifact required by: #2269