Hi Lei,
> On Jun 20, 2015, at 11:21 AM, Lei Yang <yltt1234...@gmail.com> wrote:
I am sorry it has taken me so long to test this workflow.
> Hi,
>
> Kunaal and I have completed a prototype of the first part of our GSoC project
> (implement a new doc toolchain). Our goal is to implement a doc toolchain for
> short how-to docs and lower the barrier for new comers to contribute.
I am not trying to be dense, but I want to make sure I understand the goals
here:
> This is the workflow of the prototype:
> 1. Contributor creates a new pull request on GitHub
A fairly well understand platform with lots of potential users - cool. Github
also has some social cachet.
> 2. A corresponding issue is automatically created on Pagure
> 3. Staff and contributor discuss on Pagure
This part is a bit odd to me, today. I believe your goal is to make sure the
discussion happens in a CentOS controlled space in case github ever becomes an
issue. However, it seems odd that we don't just mirror the whole PR over (see
below for where I read the paragraph where you said this was coming ...).
Right now it is weird to have to visit one site to see the change and another
to discuss it. Furthermore I feel like we are creating an educational
challenge (which could be resolved with some great docs and a bot that posted
in the PR on github) to get people to understand what is going on.
> 4. When the pull request can be accepted, it is marked as “fixed”, and the
> pull request on github is automatically merged
OK
> 5. New change is synced to Pagure (doc website will update according to
> Pagure repo, to be implemented later)
While this makes sense, it seems like we are barely using github and I am not
sure it is helping us in this equation completely. Can you help me understand
what I am missing?
Openstack does something sort of similar, but backwards from our workflow.
They have both a private git (with gerrit) and a github repo. If you open a PR
on github it is closed by a bot with instructions to submit the PR via gerrit
to their private repo. When the PR is resolved and merged in the private repo,
the entire commit is pushed to github awarding the social "status" that comes
from a commit. I am not suggesting we use gerrit.
> We have made a prototype performing the steps above, and is now running ;)
> Next we plan to integrate a CI so the doc can be built in real time and ready
> for staff to review. Could you please give us some suggestions about the
> workflow above and about the toolchain? The original GSoC idea can be found
> here: http://wiki.centos.org/GSoC/2015/Ideas#docs-toolchain
> <http://wiki.centos.org/GSoC/2015/Ideas#docs-toolchain>
I like the CI. I'd love to have us consider including something like emender
in a future release that could do CI on the PR and post a response with any
automated integration failures.
> We use Pagure as a part of the toolchain. Pagure is an excellent git-centered
> forge created by Pierre-Yves Chibon <pin...@pingoured.fr
> <mailto:pin...@pingoured.fr>>. Currently Pagure is in its early stage and the
> APIs and web hooks are changing and improving. We are working closely with
> Pierre to better integrate Pagure into the toolchain. In the future we plan
> to sync complete pull request so that discussion can happen either on github
> or Paugre, and contributor doesn’t need to leave github in the whole process,
> which lowers the barrier of contributing a lot.
Do you have a timeline for the full PR mirroring?
I agree that using github will definitely lower the barrier to contribution. My
experience with sync problems has shown that two-way is problematic. Do you
have a plan for that?
Can we achieve our goals by having a one-way sync from github to pagure? This
would seem to simplify the problem and provide us the protection - unless I am
missing something.
> To test it, you may create a PR on github. A corresponding issue will be
> created automatically on Pagure, so you may go to Pagure.io to check it. If
> you want to merge the PR, change the status of issue from Open to Fixed on
> Pagure (need admin access to the repo, if you want to test this part, please
> email me or Kunaal <kunaa...@gmail.com <mailto:kunaa...@gmail.com>> so we can
> add you as admin). Once the issue is marked as Fixed, the PR on github will
> be merged.
Feel free to merge or reject my PR as you see fit :)
> Also, we are not very sure whether running a Pagure instance will add much
> workload to sys admin team. What’s your idea?
I can't speak to the overhead, but I can say that I believe having Pagure
running is a good idea.
regards,
bex
_______________________________________________
CentOS-docs mailing list
CentOS-docs@centos.org
http://lists.centos.org/mailman/listinfo/centos-docs
