Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Gerrit integration #1491

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Gerrit integration #1491

wants to merge 10 commits into from

Conversation

SreedharaBalaji
Copy link

No description provided.

@chtompki
Copy link
Contributor

This seems to have conflicts, should this pull request remain open?

@mohdnisart
Copy link

mohdnisart commented Jul 18, 2019

Anyone has used gerrit-scm-collector recently. I am planning to use this collector and I see that it is very old.
Can some one help me with the steps to follow with the latest Hygieia
@tabladrum @SreedharaBalaji

Error form api.jar while saving the gerrit repo with GUI:

[http-nio-8080-exec-3] WARN c.c.d.rest.RestApiExceptionHandler - Bad Request - bind exception:
org.springframework.validation.BindException: org.springframework.validation.BeanPropertyBindingResult: 1 errors
Field error in object 'collectorItemRequest' on field 'collectorId': rejected value [null]; codes [NotNull.collectorItemRequest.collectorId,NotNull.collectorId,NotNull.org.bson.types.ObjectId,NotNull]; arguments [org.springframework.context.support.DefaultMessageSourceResolvable: codes [collectorItemRequest.collectorId,collectorId]; arguments []; default message [collectorId]]; default message [may not be null]
at com.capitalone.dashboard.rest.RestApiExceptionHandler.handleMethodArgumentNotValid(RestApiExceptionHandler.java:60) [api.jar!/:3.0.2-SNAPSHOT]
at org.springframework.web.servlet.mvc.method.annotation.ResponseEntityExceptionHandler.handleException(ResponseEntityExceptionHandler.java:167) [spring-webmvc-4.2.5.RELEASE.jar!/:4.2.5.RELEASE]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:1.8.0_212]

@CLAassistant
Copy link

CLAassistant commented Mar 31, 2020

CLA assistant check
All committers have signed the CLA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants