Skip to content

Commit 1b38e13

Browse files
committed
removed debug logging
1 parent 81d0d65 commit 1b38e13

File tree

3 files changed

+11
-13
lines changed

3 files changed

+11
-13
lines changed

src/main/java/com/redhat/labs/omp/resource/EngagementResource.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public class EngagementResource {
4141
@Timed(name = "performedCreates", description = "How much time it takes to create an engagement", unit = MetricUnits.MILLISECONDS)
4242
public Response createEngagement(Engagement engagement, @Context UriInfo uriInfo,
4343
@NotBlank @QueryParam("username") String author, @NotBlank @QueryParam("userEmail") String authorEmail) {
44-
LOGGER.debug("engagement from resource {}", engagement);
44+
4545
Project project = engagementService.createEngagement(engagement, author, authorEmail);
4646

4747
UriBuilder builder = uriInfo.getAbsolutePathBuilder();

src/main/java/com/redhat/labs/omp/service/EngagementService.java

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -54,11 +54,11 @@ public class EngagementService {
5454

5555
// create an engagement
5656
public Project createEngagement(Engagement engagement, String author, String authorEmail) {
57-
LOGGER.debug("creating engagement in gitlab with {}", engagement);
57+
5858
// create project structure
5959
Project project = createProjectStucture(engagement);
6060
engagement.setProjectId(project.getId());
61-
LOGGER.debug("engagement after project structure created {}", engagement);
61+
6262
// get all template files
6363
List<File> templateFiles = new ArrayList<>();
6464
templateFiles.add(createEngagmentFile(engagement));
@@ -110,23 +110,21 @@ public List<Engagement> getAllEngagements() {
110110
}
111111

112112
private File createEngagmentFile(Engagement engagement) {
113-
LOGGER.debug("creating engagement file from engagement {}", engagement);
113+
114114
String fileContent = json.toJson(engagement);
115-
LOGGER.debug("file content generated from engagement {}", fileContent);
116115
File file = File.builder().content(fileContent).filePath("engagement.json").build();
117-
LOGGER.debug("created file {}", file);
116+
118117
return file;
119118
}
120119

121120
private Project createProjectStucture(Engagement engagement) {
122-
LOGGER.debug("creating project structure with {}", engagement);
121+
123122
// create group for customer name
124123
Group customerGroup = getOrCreateGroup(engagement.getCustomerName(),
125124
Group.builder().name(engagement.getCustomerName())
126125
.path(GitLabPathUtils.generateValidPath(engagement.getCustomerName()))
127126
.parentId(engagementRepositoryId).build());
128-
LOGGER.debug("engagement after customer group {}", engagement);
129-
LOGGER.debug("customer group {}", customerGroup);
127+
130128
// create group for project name
131129
Group projectGroup = getOrCreateGroup(engagement.getProjectName(),
132130
Group.builder().name(engagement.getProjectName())

src/main/java/com/redhat/labs/omp/service/FileService.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,16 +50,16 @@ public Optional<File> createFile(Integer projectId, String filePath, File file)
5050
public boolean createFiles(Integer projectId, CommitMultiple commit) {
5151

5252
Response response = null;
53-
LOGGER.debug("commit before encoding {}", commit);
53+
5454
// encode actions in commit
5555
commit.encodeActions();
56-
LOGGER.debug("commit after encoding {}", commit);
56+
5757
// call gitlab api to commit
5858
response = gitLabService.commitMultipleFiles(projectId, commit);
59-
LOGGER.debug("commit before decoding {}", commit);
59+
6060
// decode actions in commit
6161
commit.decodeActions();
62-
LOGGER.debug("commit before decoding {}", commit);
62+
6363
// should get a 201 back if commit created
6464
return HttpStatus.SC_CREATED == response.getStatus();
6565
}

0 commit comments

Comments
 (0)