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.

Sign up for GitHub

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

Jump to bottom

Upgrade to Google Cloud Service 2.12.0 #1281

Merged
merged 1 commit into from
Sep 19, 2024
Merged

Upgrade to Google Cloud Service 2.12.0 #1281

merged 1 commit into from
Sep 19, 2024

Conversation

loicmathieu
Copy link
Contributor

No description provided.

@quarkus-bot
Copy link

quarkus-bot bot commented Sep 19, 2024

Status for workflow CI Build

This is the status report for running CI Build on commit 0e7b745.

Failing Jobs

Status Name Step Failures Logs Raw logs
✖ Build - JDK 17 Build with Maven Failures Logs Raw logs
✖ Check generated project has been updated Check that generated project is properly updated ⚠️ Check → Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ Build - JDK 17 #

- Failing:

📦 Root project

✖ Failed to execute goal io.quarkus:quarkus-platform-bom-maven-plugin:0.0.111:invoke-platform-project (build-platform-project) on project quarkus-platform-config: Failed to build the platform project, please consult the errors logged above.

📦 generated-platform-project/quarkus-camel/integration-tests/camel-quarkus-integration-test-foundation-grouped

✖ org.apache.camel.quarkus.component.language.it.LanguageTest.testLanguage(String, TestResult)[1] line 131 - More details - Source on GitHub

java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <500>.

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)

📦 generated-platform-project/quarkus-camel/integration-tests/camel-quarkus-integration-test-micrometer

✖ org.apache.camel.quarkus.component.micrometer.it.MicrometerTest.testMicrometerSummary - More details - Source on GitHub

java.lang.RuntimeException: 
java.lang.RuntimeException: io.quarkus.builder.BuildException: Build failure: Build failed due to errors
	[error]: Build step io.quarkus.arc.deployment.ArcProcessor#validate threw an exception: jakarta.enterprise.inject.spi.DeploymentException: jakarta.enterprise.inject.UnsatisfiedResolutionException: Unsatisfied dependency for type io.micrometer.prometheus.PrometheusMeterRegistry and qualifiers [@Default]
	- injection target: org.apache.camel.quarkus.component.micrometer.it.MicrometerResource#prometheusMeterRegistry
	- declared on CLASS bean [types=[java.lang.Object, org.apache.camel.quarkus.component.micrometer.it.MicrometerResource], qualifiers=[@Default, @Any], target=org.apache.camel.quarkus.component.micrometer.it.MicrometerResource]
	at io.quarkus.arc.processor.BeanDeployment.processErrors(BeanDeployment.java:1551)
	at io.quarkus.arc.processor.BeanDeployment.init(BeanDeployment.java:338)
	at io.quarkus.arc.processor.BeanProcessor.initialize(BeanPr...

✖ org.apache.camel.quarkus.component.micrometer.it.NoInstrumentedThreadPoolMicrometerTest.testInstrumentedThreadPoolFactory - More details - Source on GitHub

java.lang.RuntimeException: 
java.lang.RuntimeException: io.quarkus.builder.BuildException: Build failure: Build failed due to errors
	[error]: Build step io.quarkus.arc.deployment.ArcProcessor#validate threw an exception: jakarta.enterprise.inject.spi.DeploymentException: jakarta.enterprise.inject.UnsatisfiedResolutionException: Unsatisfied dependency for type io.micrometer.prometheus.PrometheusMeterRegistry and qualifiers [@Default]
	- injection target: org.apache.camel.quarkus.component.micrometer.it.MicrometerResource#prometheusMeterRegistry
	- declared on CLASS bean [types=[java.lang.Object, org.apache.camel.quarkus.component.micrometer.it.MicrometerResource], qualifiers=[@Default, @Any], target=org.apache.camel.quarkus.component.micrometer.it.MicrometerResource]
	at io.quarkus.arc.processor.BeanDeployment.processErrors(BeanDeployment.java:1551)
	at io.quarkus.arc.processor.BeanDeployment.init(BeanDeployment.java:338)
	at io.quarkus.arc.processor.BeanProcessor.initialize(BeanPr...

✖ org.apache.camel.quarkus.component.micrometer.it.NoMessageHistoryMicrometerTest.testNoHistory - More details - Source on GitHub

java.lang.RuntimeException: 
java.lang.RuntimeException: io.quarkus.builder.BuildException: Build failure: Build failed due to errors
	[error]: Build step io.quarkus.arc.deployment.ArcProcessor#validate threw an exception: jakarta.enterprise.inject.spi.DeploymentException: jakarta.enterprise.inject.UnsatisfiedResolutionException: Unsatisfied dependency for type io.micrometer.prometheus.PrometheusMeterRegistry and qualifiers [@Default]
	- injection target: org.apache.camel.quarkus.component.micrometer.it.MicrometerResource#prometheusMeterRegistry
	- declared on CLASS bean [types=[java.lang.Object, org.apache.camel.quarkus.component.micrometer.it.MicrometerResource], qualifiers=[@Default, @Any], target=org.apache.camel.quarkus.component.micrometer.it.MicrometerResource]
	at io.quarkus.arc.processor.BeanDeployment.processErrors(BeanDeployment.java:1551)
	at io.quarkus.arc.processor.BeanDeployment.init(BeanDeployment.java:338)
	at io.quarkus.arc.processor.BeanProcessor.initialize(BeanPr...

📦 generated-platform-project/quarkus-cxf/integration-tests/quarkus-cxf-integration-test-hc5

✖ io.quarkiverse.cxf.hc5.it.Hc5Test.add(String)[4] line 55 - More details - Source on GitHub

java.lang.AssertionError: 

Expecting actual not to be null
	at io.quarkiverse.cxf.hc5.it.Hc5Test.add(Hc5Test.java:55)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.junit.QuarkusTestExtension.runExtensionMethod(QuarkusTestExtension.java:973)
	at io.quarkus.test.junit.QuarkusTestExtension.runExtensionMethod(QuarkusTestExtension.java:903)
	at io.quarkus.test.junit.QuarkusTestExtension.interceptTestTemplateMethod(QuarkusTestExtension.java:863)

@aloubyansky
Copy link
Member

The LanguageTest seems to be a new failure in the CQ testsuite. FYI @jamesnetherton

@jamesnetherton
Copy link
Collaborator

The LanguageTest seems to be a new failure in the CQ testsuite. FYI @jamesnetherton

It also showed up on #1279. I already tested the platform with CQ 3.15.0 and that issue was not present.

@aloubyansky aloubyansky merged commit 9a323e8 into quarkusio:main Sep 19, 2024
2 of 4 checks passed
@aloubyansky
Copy link
Member

@loicmathieu this is targeting 3.16. Do you also want this in 3.15 (LTS) as well? If so, please open a PR for the 3.15 branch as well. Thanks.

@gemmellr
Copy link
Contributor

gemmellr commented Sep 19, 2024 •
edited
Loading

This appears to have broken the main build. (I also also see generated changes, as per #1282)

If you run ./mvnw -Dsync it completes but does show a warning:

[WARNING] The POM for io.quarkiverse.googlecloudservices:quarkus-google-cloud-services-main-it:pom:2.12.0 is missing, no dependency information available

However if you run ./mvnw clean install -DskipTests it actually fails, not being able to resolve the related deps.

This is also visible in the CI runs from main, e.g snippet from CI logs:

2024-09-19T17:21:30.6229966Z [ERROR] Failed to execute goal on project quarkus-google-cloud-services-main-it: Could not resolve dependencies for project io.quarkus.platform:quarkus-google-cloud-services-main-it:jar:999-SNAPSHOT: The following artifacts could not be resolved: io.quarkiverse.googlecloudservices:quarkus-google-cloud-services-main-it:jar:2.12.0 (absent), io.quarkiverse.googlecloudservices:quarkus-google-cloud-services-main-it:jar:tests:2.12.0 (absent): Could not find artifact io.quarkiverse.googlecloudservices:quarkus-google-cloud-services-main-it:jar:2.12.0 in google-maven-central (https://maven-central.storage-download.googleapis.com/maven2/) -> [Help 2]

@gemmellr gemmellr mentioned this pull request Sep 19, 2024
Merged
@aloubyansky
Copy link
Member

@loicmathieu Robbie's right https://repo.maven.apache.org/maven2/io/quarkiverse/googlecloudservices/quarkus-google-cloud-services-main-it/
quarkiverse/quarkus-google-cloud-services@e3fbd51 was not a good idea. Could you please add the test jars back?

@loicmathieu
Copy link
Contributor Author

@aloubyansky will be done by quarkiverse/quarkus-google-cloud-services#675

Should I release a new version and upgrade the platform?

@loicmathieu
Copy link
Contributor Author

@loicmathieu this is targeting 3.16. Do you also want this in 3.15 (LTS) as well? If so, please open a PR for the 3.15 branch as well. Thanks.

I thought main was 3.15 as 3.15 was released a few days ago, yes it must target also 3.15, I'll open a PR on the 3.15 branch.

@aloubyansky
Copy link
Member

@aloubyansky will be done by quarkiverse/quarkus-google-cloud-services#675

Should I release a new version and upgrade the platform?

That would be great, so we can run proper tests for GCS in the platform.

@loicmathieu loicmathieu deleted the gcp-2-12-0 branch September 20, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aloubyansky aloubyansky aloubyansky left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@loicmathieu @aloubyansky @jamesnetherton @gemmellr