Skip to content

Commit bc67ee2

Browse files
committed
Merge branch 'main' of github.com:jaydeluca/opentelemetry-java-instrumentation into spring-actuator-update
2 parents c3c86ec + 9c678a9 commit bc67ee2

File tree

6 files changed

+23
-44
lines changed

6 files changed

+23
-44
lines changed
Lines changed: 14 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,20 @@
11
name: Auto license report
22
on:
3-
pull_request:
4-
types:
5-
- opened
6-
- synchronize
3+
push:
4+
branches:
5+
- 'renovate/**/*'
76

87
concurrency:
9-
group: ${{ github.workflow }}-${{ github.event.pull_request.number }}
8+
group: ${{ github.workflow }}-${{ github.ref }}
109
cancel-in-progress: true
1110

1211
permissions:
1312
contents: read
1413

1514
jobs:
16-
check:
17-
if: ${{ github.event.pull_request.draft == false }}
15+
auto-license-report:
16+
permissions:
17+
contents: write
1818
runs-on: ubuntu-latest
1919
steps:
2020
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
@@ -33,26 +33,16 @@ jobs:
3333
with:
3434
cache-read-only: true
3535

36-
- name: Check out PR branch
37-
env:
38-
GH_TOKEN: ${{ github.token }}
39-
run: gh pr checkout ${{ github.event.pull_request.number }}
36+
- name: Use CLA approved bot
37+
run: .github/scripts/use-cla-approved-bot.sh
4038

4139
- name: Update license report
4240
run: ./gradlew generateLicenseReport --no-build-cache
4341

44-
- id: create-patch
45-
name: Create patch file
42+
- name: Commit and push if there are changes
4643
run: |
47-
git add -N --ignore-removal licenses
48-
git diff > patch
49-
if [ -s patch ]; then
50-
echo "exists=true" >> "$GITHUB_OUTPUT"
44+
git add licenses
45+
if ! git diff --cached --quiet; then
46+
git commit -m "Update license report"
47+
git push
5148
fi
52-
53-
- name: Upload patch file
54-
if: steps.create-patch.outputs.exists == 'true'
55-
uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4 # v5.0.0
56-
with:
57-
path: patch
58-
name: patch

instrumentation/spring/spring-jms/spring-jms-6.0/javaagent/build.gradle.kts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,6 @@ dependencies {
2727

2828
testLibrary("org.springframework.boot:spring-boot-starter-test:3.0.0")
2929
testLibrary("org.springframework.boot:spring-boot-starter:3.0.0")
30-
31-
// tests don't work with spring boot 4 yet
32-
latestDepTestLibrary("org.springframework.boot:spring-boot-starter-test:3.+") // documented limitation
33-
latestDepTestLibrary("org.springframework.boot:spring-boot-starter:3.+") // documented limitation
3430
}
3531

3632
// spring 6 requires java 17

instrumentation/spring/spring-jms/spring-jms-6.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/spring/jms/v6_0/AbstractSpringJmsListenerTest.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -45,12 +45,12 @@ abstract class AbstractSpringJmsListenerTest {
4545
@BeforeAll
4646
static void setUp() {
4747
broker =
48-
new GenericContainer<>("quay.io/artemiscloud/activemq-artemis-broker:artemis.2.27.0")
49-
.withEnv("AMQ_USER", "test")
50-
.withEnv("AMQ_PASSWORD", "test")
48+
new GenericContainer<>("apache/activemq-artemis:2.44.0")
49+
.withEnv("ARTEMIS_USER", "test")
50+
.withEnv("ARTEMIS_PASSWORD", "test")
5151
.withEnv("JAVA_TOOL_OPTIONS", "-Dbrokerconfig.maxDiskUsage=-1")
5252
.withExposedPorts(61616, 8161)
53-
.waitingFor(Wait.forLogMessage(".*Server is now live.*", 1))
53+
.waitingFor(Wait.forLogMessage(".*Server is now active.*", 1))
5454
.withStartupTimeout(Duration.ofMinutes(2))
5555
.withLogConsumer(new Slf4jLogConsumer(logger));
5656
broker.start();

instrumentation/spring/spring-rabbit-1.0/javaagent/build.gradle.kts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ muzzle {
1212
}
1313
}
1414

15+
val latestDepTest = findProperty("testLatestDeps") as Boolean
16+
1517
dependencies {
1618
library("org.springframework.amqp:spring-rabbit:1.0.0.RELEASE")
1719

@@ -24,9 +26,9 @@ dependencies {
2426
// spring-retry is required by org.springframework.amqp:spring-rabbit:4.0.0
2527
testLibrary("org.springframework.retry:spring-retry")
2628

27-
// tests don't work with spring boot 4 yet
28-
latestDepTestLibrary("org.springframework.boot:spring-boot-starter:3.+") // documented limitation
29-
latestDepTestLibrary("org.springframework.boot:spring-boot-starter-test:3.+") // documented limitation
29+
if (latestDepTest) {
30+
testLibrary("org.springframework.boot:spring-boot-starter-amqp:latest.release")
31+
}
3032
}
3133

3234
tasks {
@@ -36,8 +38,6 @@ tasks {
3638
}
3739
}
3840

39-
val latestDepTest = findProperty("testLatestDeps") as Boolean
40-
4141
// spring 6 requires java 17
4242
if (latestDepTest) {
4343
otelJava {

instrumentation/spring/spring-security-config-6.0/javaagent/build.gradle.kts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,6 @@ dependencies {
3131
// testLibrary("jakarta.servlet:jakarta.servlet-api:6.0.0")
3232
testImplementation("jakarta.servlet:jakarta.servlet-api:6.0.0")
3333
latestDepTestLibrary("jakarta.servlet:jakarta.servlet-api:6.1.0") // documented limitation
34-
// remove after 7.0 is released for spring security
35-
// spring-test 7 requires spring-context 7
36-
latestDepTestLibrary("org.springframework:spring-context:latest.release")
37-
latestDepTestLibrary("org.springframework:spring-web:latest.release")
3834
}
3935

4036
otelJava {

instrumentation/spring/spring-security-config-6.0/library/build.gradle.kts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,6 @@ dependencies {
2020
testImplementation(platform("org.junit:junit-bom:6.0.1"))
2121

2222
testLibrary("org.springframework:spring-test:6.0.0")
23-
// remove after 7.0 is released for spring security
24-
// spring-test 7 requires spring-context 7
25-
latestDepTestLibrary("org.springframework:spring-context:latest.release")
2623
}
2724

2825
otelJava {

0 commit comments

Comments
 (0)