Skip to content
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

Release 5.12.0 #142

Merged
merged 23 commits into from
Sep 19, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
e6db1fc
EPMRPP-87592 || Migrate from springfox-swagger to SpringDoc for api s…
APiankouski Dec 6, 2023
07a43d6
EPMRPP-84831 || Update Gradle build scrip
APiankouski Jan 31, 2024
6b4a77c
Merge pull request #125 from reportportal/EPMRPP-84831
APiankouski Feb 1, 2024
721345d
EPMRPP-82701 || Update Search configs and naming
APiankouski Feb 12, 2024
0e5aa0e
EPMRPP-82701 || Update Search configs and naming
APiankouski Feb 19, 2024
d5a9193
rc/5.11.0 || Upd commit
Mar 3, 2024
345eaf6
Merge branch 'develop' of https://github.com/reportportal/service-job…
Mar 13, 2024
0d169cb
Merge pull request #129 from reportportal/rc/5.11.0
pbortnik Mar 13, 2024
e2cda26
EPMRPP-90182 || Update JDBC Driver to the latest
APiankouski Apr 9, 2024
0f75b62
Merge pull request #132 from reportportal/EPMRPP-90182
APiankouski Apr 9, 2024
838dce7
Merge pull request #134 from reportportal/master
IvanKustau Apr 22, 2024
5ead2ef
Add jira versions sync workflow
raikbitters Apr 22, 2024
e52b2fa
EPMRPP-90918 || Add skip of important launches
IvanKustau May 23, 2024
71fd27e
EPMRPP-90918 || Add skip attachments
IvanKustau May 24, 2024
749c8f2
Merge pull request #136 from reportportal/EPMRPP-90918-jobs-should-sk…
IvanKustau May 28, 2024
ca8085a
EPMRPP-88736 implement send analytics job (#138)
grabsefx Jul 3, 2024
3b15164
EPMRPP-92477 rename variable (#139)
grabsefx Jul 8, 2024
e99ab6e
EPMRPP-92477 set default GA variables (#140)
grabsefx Jul 10, 2024
614f643
Update dependencies (#141)
APiankouski Sep 3, 2024
722be38
Release 5.12.0
APiankouski Sep 3, 2024
dbc7fe6
Update gradle scripts version
APiankouski Sep 3, 2024
230167d
EPMRPP-95322 || Job 'CleanLaunchcron' doesn't clean the attachments f…
APiankouski Sep 18, 2024
2ac782d
EPMRPP-95331 fix analyzed items count (#145)
grabsefx Sep 19, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions .github/workflows/sync-jira-versions.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
name: Add GitHub release version to Jira issues

on:
pull_request:
types: [opened, synchronize, reopened]

jobs:
call-jira-sync:
name: Call Jira versions update
uses: reportportal/.github/.github/workflows/update-jira-versions.yaml@main
with:
jira-server: ${{ vars.JIRA_SERVER }}
secrets: inherit
4 changes: 2 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM --platform=$BUILDPLATFORM gradle:8.4.0-jdk21 AS build
FROM --platform=$BUILDPLATFORM gradle:8.10.0-jdk21 AS build
ARG RELEASE_MODE
ARG APP_VERSION
WORKDIR /usr/app
Expand All @@ -10,7 +10,7 @@ RUN if [ "${RELEASE_MODE}" = true ]; then \
else gradle build --exclude-task test -Dorg.gradle.project.version=${APP_VERSION}; fi

# For ARM build use flag: `--platform linux/arm64`
FROM --platform=$BUILDPLATFORM amazoncorretto:21.0.1
FROM amazoncorretto:21.0.4
LABEL version=${APP_VERSION} description="EPAM Report portal. Jobs Service" maintainer="Andrei Varabyeu <andrei_varabyeu@epam.com>, Hleb Kanonik <hleb_kanonik@epam.com>"
ARG APP_VERSION=${APP_VERSION}
ENV APP_DIR=/usr/app
Expand Down
43 changes: 0 additions & 43 deletions Jenkinsfile

This file was deleted.

57 changes: 0 additions & 57 deletions Jenkinsfile-candidate

This file was deleted.

13 changes: 7 additions & 6 deletions build.gradle
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
plugins {
id 'org.springframework.boot' version '2.7.17'
id 'io.spring.dependency-management' version '1.0.11.RELEASE'
id 'org.springframework.boot' version '2.7.18'
id 'io.spring.dependency-management' version '1.1.4'
id 'java'
}

Expand All @@ -11,9 +11,8 @@ project.ext {

ext['junit-jupiter.version'] = '5.10.0'

def scriptsUrl = 'https://raw.githubusercontent.com/reportportal/gradle-scripts/' + (releaseMode ? '5.11.0' : 'EPMRPP-85756')
def scriptsUrl = 'https://raw.githubusercontent.com/reportportal/gradle-scripts/' + (releaseMode ? '5.12.0' : 'develop')

apply from: "$scriptsUrl/build-docker.gradle"
apply from: "$scriptsUrl/build-commons.gradle"
apply from: "$scriptsUrl/build-info.gradle"
//apply from: "$scriptsUrl/build-quality.gradle"
Expand All @@ -26,7 +25,7 @@ tasks.withType(JavaCompile).configureEach {
}

wrapper {
gradleVersion = '8.4'
gradleVersion = '8.10'
}

bootJar {
Expand Down Expand Up @@ -54,7 +53,8 @@ ext['log4j2.version'] = '2.21.1'
ext['log4j-to-slf4j.version'] = '2.21.1'
//https://nvd.nist.gov/vuln/detail/CVE-2022-26520
ext['postgresql.version'] = '42.6.1'
ext['snakeyaml.version'] = '1.33'
ext['snakeyaml.version'] = '2.2'
ext['spring-boot.version'] = '2.7.18'
//

dependencies {
Expand Down Expand Up @@ -91,6 +91,7 @@ dependencies {
implementation 'org.springframework.boot:spring-boot-starter-jdbc'
implementation 'org.springframework.boot:spring-boot-starter-web'
implementation 'org.springframework.boot:spring-boot-starter-amqp'
implementation 'org.springframework:spring-jdbc:6.1.5'
implementation 'org.apache.jclouds.api:s3:2.5.0'
implementation 'org.apache.jclouds.provider:aws-s3:2.5.0'
implementation 'org.apache.jclouds.api:filesystem:2.5.0'
Expand Down
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
version=5.11.2
version=5.12.0
description=EPAM Report portal. Service jobs
dockerServerUrl=unix:///var/run/docker.sock
dockerPrepareEnvironment=
Expand Down
Binary file modified gradle/wrapper/gradle-wrapper.jar
Binary file not shown.
2 changes: 1 addition & 1 deletion gradle/wrapper/gradle-wrapper.properties
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-8.4-bin.zip
distributionUrl=https\://services.gradle.org/distributions/gradle-8.10-bin.zip
networkTimeout=10000
validateDistributionUrl=true
zipStoreBase=GRADLE_USER_HOME
Expand Down
7 changes: 5 additions & 2 deletions gradlew
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.
#
# SPDX-License-Identifier: Apache-2.0
#

##############################################################################
#
Expand Down Expand Up @@ -55,7 +57,7 @@
# Darwin, MinGW, and NonStop.
#
# (3) This script is generated from the Groovy template
# https://github.com/gradle/gradle/blob/HEAD/subprojects/plugins/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
# https://github.com/gradle/gradle/blob/HEAD/platforms/jvm/plugins-application/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
# within the Gradle project.
#
# You can find Gradle at https://github.com/gradle/gradle/.
Expand Down Expand Up @@ -84,7 +86,8 @@ done
# shellcheck disable=SC2034
APP_BASE_NAME=${0##*/}
# Discard cd standard output in case $CDPATH is set (https://github.com/gradle/gradle/issues/25036)
APP_HOME=$( cd "${APP_HOME:-./}" > /dev/null && pwd -P ) || exit
APP_HOME=$( cd -P "${APP_HOME:-./}" > /dev/null && printf '%s
' "$PWD" ) || exit

# Use the maximum available, or set MAX_FD != -1 to use that value.
MAX_FD=maximum
Expand Down
22 changes: 12 additions & 10 deletions gradlew.bat
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@
@rem See the License for the specific language governing permissions and
@rem limitations under the License.
@rem
@rem SPDX-License-Identifier: Apache-2.0
@rem

@if "%DEBUG%"=="" @echo off
@rem ##########################################################################
Expand Down Expand Up @@ -43,11 +45,11 @@ set JAVA_EXE=java.exe
%JAVA_EXE% -version >NUL 2>&1
if %ERRORLEVEL% equ 0 goto execute

echo.
echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
echo.
echo Please set the JAVA_HOME variable in your environment to match the
echo location of your Java installation.
echo. 1>&2
echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. 1>&2
echo. 1>&2
echo Please set the JAVA_HOME variable in your environment to match the 1>&2
echo location of your Java installation. 1>&2

goto fail

Expand All @@ -57,11 +59,11 @@ set JAVA_EXE=%JAVA_HOME%/bin/java.exe

if exist "%JAVA_EXE%" goto execute

echo.
echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME%
echo.
echo Please set the JAVA_HOME variable in your environment to match the
echo location of your Java installation.
echo. 1>&2
echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME% 1>&2
echo. 1>&2
echo Please set the JAVA_HOME variable in your environment to match the 1>&2
echo location of your Java installation. 1>&2

goto fail

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
/*
* Copyright 2024 EPAM Systems
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

package com.epam.reportportal.config;

import com.fasterxml.jackson.databind.DeserializationFeature;
import com.fasterxml.jackson.databind.MapperFeature;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.fasterxml.jackson.databind.SerializationFeature;
import com.fasterxml.jackson.databind.introspect.JacksonAnnotationIntrospector;
import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;

/**
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> reported by reviewdog 🐶
Summary javadoc is missing.

* @author Siarhei Hrabko
*/
@Configuration
public class JacksonConfiguration {

/**
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> reported by reviewdog 🐶
Summary javadoc is missing.

* @return Configured object mapper
*/
@Bean(name = "objectMapper")
public ObjectMapper objectMapper() {
ObjectMapper om = new ObjectMapper();
om.setAnnotationIntrospector(new JacksonAnnotationIntrospector());
om.configure(MapperFeature.DEFAULT_VIEW_INCLUSION, true);
om.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
om.configure(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS, false);
om.registerModule(new JavaTimeModule());
return om;
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
* @author <a href="mailto:maksim_antonov@epam.com">Maksim Antonov</a>
*/
@Configuration
@ConditionalOnProperty(prefix = "rp.elasticsearch", name = "host")
@ConditionalOnProperty(prefix = "rp.searchengine", name = "host")
public class BackgroundProcessingConfiguration {

public static final String LOG_MESSAGE_SAVING_QUEUE_NAME = "log_message_saving";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,12 @@
import org.springframework.stereotype.Service;

/**
* Empty client to work with Elasticsearch.
* Empty client to work with Search engine.
*
* @author <a href="mailto:maksim_antonov@epam.com">Maksim Antonov</a>
*/
@Service
public class EmptyElasticSearchClient implements ElasticSearchClient {
public class EmptySearchEngineClient implements SearchEngineClient {

@Override
public void save(List<LogMessage> logMessageList) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@
import java.util.List;

/**
* Client interface to work with Elasticsearch.
* Client interface to work with Search engine.
*
* @author <a href="mailto:maksim_antonov@epam.com">Maksim Antonov</a>
*/
public interface ElasticSearchClient {
public interface SearchEngineClient {

void save(List<LogMessage> logMessageList);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,23 +19,23 @@
import org.springframework.web.client.RestTemplate;

/**
* Simple client to work with Elasticsearch.
* Simple client to work with Search engine.
*
* @author <a href="mailto:maksim_antonov@epam.com">Maksim Antonov</a>
*/
@Primary
@Service
@ConditionalOnProperty(prefix = "rp.elasticsearch", name = "host")
public class SimpleElasticSearchClient implements ElasticSearchClient {
@ConditionalOnProperty(prefix = "rp.searchengine", name = "host")
public class SimpleSearchEngineClient implements SearchEngineClient {

protected final Logger LOGGER = LoggerFactory.getLogger(SimpleElasticSearchClient.class);
protected final Logger LOGGER = LoggerFactory.getLogger(SimpleSearchEngineClient.class);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> reported by reviewdog 🐶
Abbreviation in name 'LOGGER' must contain no more than '1' consecutive capital letters.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.MemberNameCheck> reported by reviewdog 🐶
Member name 'LOGGER' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.


private final String host;
private final RestTemplate restTemplate;

public SimpleElasticSearchClient(@Value("${rp.elasticsearch.host}") String host,
@Value("${rp.elasticsearch.username:}") String username,
@Value("${rp.elasticsearch.password:}") String password) {
public SimpleSearchEngineClient(@Value("${rp.searchengine.host}") String host,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck> reported by reviewdog 🐶
Missing a Javadoc comment.

@Value("${rp.searchengine.username:}") String username,
@Value("${rp.searchengine.password:}") String password) {
restTemplate = new RestTemplate();

if (!username.isEmpty() && !password.isEmpty()) {
Expand Down
Loading
Loading