Temurin-build: Investigue um gancho de relações públicas para verificar se alguém assinou o OCA e, se não, solicitá-lo

Criado em 30 mar. 2017  ·  5Comentários  ·  Fonte: adoptium/temurin-build

Isso é para que se quiséssemos doar qualquer código funcional para o projeto OpenJDK, poderíamos fazê-lo.

Isso poderia funcionar da mesma forma que o bot do MS PR se conecta a este PR -> https://github.com/Azure/azure-iot-sdks/pull/478

Fizemos alguns trabalhos iniciais em uma encarnação anterior do farm de construção OpenJDK (Betterrev) para puxar uma lista de signatários OCA do site openjdk (raspador de tela HTML) e, portanto, acho que podemos usar isso para verificar o endereço de e-mail do usuário do GitHub (se disponível na API).

Isso deve ser aplicado a todos os openjdk- * repos

Um esforço também deve ser feito para garantir que todas as contribuições de código existentes antes que este gancho de PR seja implementado possam ser doadas ao projeto OpenJDK em uma data posterior, garantindo que os responsáveis ​​pelo commit assinaram o OCA.

Possivelmente estou perdendo alguns casos / esclarecimentos aqui - comentários extras são bem-vindos

help wanted invalid

Todos 5 comentários

A página de signatários do OCA não contém e-mail ou id do github para todos eles - por exemplo, para nossa entrada é apenas meu nome e organização: (exemplo de http://www.oracle.com/technetwork/community/oca- 486395.html # g)

   Goldman Sachs - OpenJDK (Sunny Chan, Jessica Man)

Alguns deles realmente têm uma conta github associada a ele, em formato diferente:

   Adolfo Dos Santos Jr - Jersey - GitHub adolfojunior
   Alessandro Gherardi - Jersey - GH agherardi

Embora alguns tenham o id java.net:

 Mohammad W. Abdo - JDK - java.net - mohammad

Portanto, não tenho certeza de quanto você pode filtrar o recado (bem, eu não vi o código de recorte da tela melhorrev)

Talvez a melhor maneira de fazer isso é que, antes de aceitar a solicitação Pull, exigiremos que o usuário coloque uma linha específica que corresponda à entrada (por exemplo, OCA: Goldman Sachs) e seu script para corresponder ao nome. Se a linha OCA não estiver lá, envie um e-mail.

Eu concordo com @sunnychanwork , haverá uma vantagem limitada em raspar a lista OCA.
Eu sugiro que simplesmente conduzamos uma verificação manual para cada novo contribuidor e deixemos o bot manter nossa própria lista de ids do github que são signatários OCA conhecidos para comparação.

Se estivermos sobrecarregados com muitos colaboradores, então (seria um bom problema de se ter e) poderemos descobrir mais automação, mas não estou esperando que seja um problema imediato.

Este requisito mudou e ainda é obrigatório? E aqueles que não assinaram (ou não podem) o OCA, essas contribuições não deveriam ser permitidas?

openjdk-build e openjdk-infrastructure são uma espécie de distanciamento de estarem diretamente vinculados às mudanças de código do openjdk como eu entendo. Eles deveriam se enquadrar no mesmo requisito?

Estou colocando isso no Icebox porque (no momento) não estamos realmente agindo como um sistema de envio de patch para upstream

Não é necessário, pois os OCAs são assinados para patches upstream.

Esta página foi útil?
0 / 5 - 0 avaliações