-
Notifications
You must be signed in to change notification settings - Fork 4
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
sonar #16
Comments
Olhando na receita default do cookbook sonarqube [1], vi que era preciso criar primeiro o banco de dados do sonar e daí informar, por meio dos atributos do nó, as credenciais de acesso à receita sonarqube. Ou seja, não adiantava executar sonarqube::default antes de radar::sonar [2]. Daí eu removi o sonarqube::default do bootstrap e o coloquei como um passo no meio da receita radar::sonar, logo após a configuração do banco sonar (no postgresql). Daí, vendo os atributos de nó usados em sonarqube::default, tive que setar mais alguns desses atributos no node.json [3]. Agora o problema, estranho, é que ao se executar a receita sonarqube::default, temos o seguinte erro:
que acontece na linha
da receita sonarqube::default. [1] https://github.com/radar-parlamentar/implantacao/blob/master/cookbooks/sonarqube/recipes/default.rb |
Bom, já q esse último problema parece um problema estranho do cookbook, uma boa saída pode ser tentar esse outro cookbook aqui pra instalar o sonar: https://github.com/ctrabold/chef-sonar |
Opa, o último erro aconteceu só pq faltou declarar a dependência do cookbook sonarqube no cookbook radar. Fiz isso e consegui executar a receita radar::sonar sem erros, mas ainda não consegui acessar o sonar =S |
Executei de novo e consegui acessar o Sonar via navegador!!!
|
humm... o jenkins não executa o build no repo que o sonar usa... então não foi culpa do jenkins q o log ficou pertencente ao root. Além disso, arquivos em exportadores/dados tb ficaram pertencente ao root! |
Com as alterações de 352fadb a coisa tá funcionando... |
@leonardofl que tal colocar isto para rodar em uma ferramenta externa? O CodeClimate está com beta aberta para Python https://codeclimate.com/ |
Parece uma boa! Por outro lado seria legal explorar o Sonar, q é bastante conhecido e usado por aí, além de ser open source... mas acho q exige um esforço maior, neh. Outra plataforma de métricas é a Mezuro, do Paulo Meiirelles. Mas eles ainda tão trabalhando para o suporte a Python. Talvez a gente possa ir pro Climate e no futuro migrar pro Mezuro, hehe. |
Eh o workaround q eu tentei fazer pra contornar o problema da instalação do sonar não funcionou muito bem.. deu um outro problema =T |
Ih, parece que o codeclimate não suporta python : / |
Leonardo... http://ricardoduarte.net/posts/como-configurar-um-projeto-django-no-jenkins.html Ih, parece que o codeclimate não suporta python : / — |
Então, o code cliamte tem algum suporte a Python, mas não pra medir a cobertura de testes, o que considero ser o principal. Quanto a usar o próprio Jenkins pra fazer a análise, gostei da ideia! |
A partir de agora, acho q vou adotar a seguinte estratégia... tirar a parte do sonar do projeto, tentar por tudo isso pra valer em produção, e depois q estiver funcionando, acrescentar a parte da análise estática de código via Jenkins. |
Esse não serve?
https://coveralls.io/
(Tem service para github)
|
Bom... Pra cobertura rola o coveralls!
|
Vou tentar aqui : ) |
Aqui eles usam vários desses services, pode servir de exemplo:
https://github.com/divio/django-cms/blob/develop
|
Vixe, loguei lá, mas o radar não apareceu na minha lista de repos :S |
Tem que deixar sua participação na orgainzação (radar-parlamentar) pública
e depois resincronizar no coveralls.
Link 1: https://github.com/orgs/radar-parlamentar/people (tornar sua
participação pública)
Link 2: https://coveralls.io/refresh (resincronizar)
|
Legal, acho que consegui! :D |
Ops, funcionou na 1a tentativa na qual eu rodei apenas os testes da app analises. Mas quando rodei de novo com todos os testes de unidade, o coveralls no after_success do travis deu erro, e daí o coveralls não foi atualizado : ( O erro que deu pode ser visto em https://travis-ci.org/radar-parlamentar/radar/builds/62319120, e é:
|
Isso pode ajudar:
TheKevJames/coveralls-python#64
É um erro de encoding/decode. Mas ainda não achei como resolver....
|
Tem esse link aqui tbm:
z4r/python-coveralls#1
|
Parece que sonar não fica de pé, o que deveria ser feito pela receita radarqueube.
Pau que dá na receita radar::sonar:
The text was updated successfully, but these errors were encountered: