Hi John,
Really sorry about the wait. Unfortunately I think the reality is that
there are more people trying to contribute code than there are people
that can review code right now. Even in the core OSD code we have cases
where Adam, Igor and I all have PRs we've made that we should be
reviewing for each other and they can end up spending months in queue
because none of us have time to properly review and merge them. It's
not personal, just the reality of everyone trying to balance writing
code, reviewing code, and still having some kind of life outside work
(It's easy to be completely consumed and burn out if you let it, I've
been there).
I see Kefu approved your PR, but doesn't want to merge it himself as
he's not a maintainer anymore. That's fair. He used to maintain quite
a bit of the build script stuff but since he left we haven't really had
any specific point person take over afaik. I looked over the PR but I'm
not super familiar with the internals of install-deps.sh or run-make.sh.
I'd really feel better if someone that's worked on those files before
did another review and approved the merge. If that doesn't happen,
maybe we can merge based on Kefu's approval alone.
Please don't hesitate to keep poking us. :)
Mark
On 2/10/23 10:39, John Mulligan wrote:
As part of an effort to build and test ceph in containers I posted a PR in
November:
https://github.com/ceph/ceph/pull/48697
As noted in the PR there are two approvals but no one seems willing or has the
time to shepherd the PR through the merge process.
I'm not sure what my next steps on this PR can be. I had planned on having
this PR serve as an intermediate step towards being able to build from source
and run 'make tests' in containers. This is an effort Enesto started and I've
been working on for a while. I'm currently blocked because of the uncertainty
around this PR.
If anyone has any thoughts or recommendations for me I'd appreciate it. I did
ping ceph/core in the PR as well, but I figured I might get some more attention
here on the list.
Thanks!
_______________________________________________
Dev mailing list -- dev@xxxxxxx
To unsubscribe send an email to dev-leave@xxxxxxx
_______________________________________________
Dev mailing list -- dev@xxxxxxx
To unsubscribe send an email to dev-leave@xxxxxxx
